[5.10] SimplifyBeginCOWMutation: don't eliminate end_cow_mutation instructions and keep the [keep_unique] flag

Explanation:  The [keep_unique] flag for the end_cow_mutation is important for following optimizations. Some simplifications dropped the flag or removed the whole end_cow_mutation instruction which has this flag set. This can result in miscompiles because following optimizations (like the ObjectOutliner) must not run if the flag is set for a reference.

Issue: rdar://119178823

Risk: Low. The fixes are small changes which make the affected simplifications more conservative.

Reviewer: @meg-gupta

Main branch PR: https://github.com/apple/swift/pull/70233

rdar://119178823

@swift-ci test


Set error messages in helix for state transition failues

It can help access the errors from the controller UI without searching server logs for segment errors or broker/server instances go into error status.

can you please elaborate "set error messages in helix"

If I understand what @fx19880617  is saying, it is to provide an API in the controller to retrieve Helix state transition error messages and perhaps history as well. I concur that it is a useful feature to have. Once we have the servers and the error messages, we can look into the server for a specific cause.

Yes, right now we only know that segment in error state from external view. The goal is to also set the stacktrace for segment loading and easy for users to retrieve it.

Got it. One thing to keep in mind is that state transition error messages get cleaned up automatically by Helix. we should probably do something like this

each server stores the error message/segment status in segmentDataManager
exposes an api to get the status along with message
controller can invoke the server api on demand

Please don't save the error stacktrace in Helix

I think Helix preserves the error stack/messages for a few days? State stored in segmentDataManager can go away on restart. I think there is value in retrieving helix messages (if they exist) as well.
On a recent issue in Linkedin, it turned out that the state transitions did not succeed and caused a performance hit on pinot servers. The errors went away upon restart. I can totally imagine an admin restarting the servers to get the system up & running.
It may make sense to have an API gather all debug information available and output it as a a blob of text. We can enhance this as we feel like, without regarding to backward compat, etc.
curl controller:port/tables/{tableName}/debugState 1> saved 2>&1
can be done before any restart is attempted. This can also retrieve states from each of the servers that have segments for the table.

Hi, from experience, this will be a really useful feature to me. I would love to give it a try if nobody is working on it.


migrate to nnbd

blocker https://github.com/mponkin/fading_edge_scrollview/issues/11

Is forking the repo and using the fork a possible solution?


quote response status codes to make them json compatible

swagger supports json, but yaml is much more flexible than json. It allows numeric keys, unlike json.
I'm actually not 100% sure whether this explanation is correct, or whether it is actually an implementation issue, but it definitely fails in python with pyyaml and jsonschema validation.
repro using bravado-core:
import yaml
from bravado_core.spec import Spec
Spec.from_dict(yaml.load(open('/path/to/zipkin-api.yaml'))) # throws jsonschema exception

wfm!


Cassandra 2.1.x compatibility

Datastax community edition now providing Cassandra 2.1.6
For titan 0.9 GA current documentation says 0.9 is compatible with Cassandra 2.0.x.
Can the Cassandra compatibility version be bumped to either 2.x.x or 2.1.x ?
http://planetcassandra.org/cassandra/

reworded title to avoid the verb "Certify", which is a bit loaded

Upgrading from 2.0.8 to 2.1.8 turned out to touch way more parts of Titan than I had anticipated.  The commit log for e5c3421c60934d3fc58b45f69b35884da29eaf60 lists specifics.
The titan-cassandra tests passed when I ran them against 2.1.8 earlier.  I then saw that 2.1.9 was released today.  I committed a patchlevel increment from 2.1.8 -> 2.1.9 as c3bd6b3c53dc6a1194735db03512467baba8f1be.  I'd be surprised if that patchlevel increment broke anything; I only bothered to make sure everything still compiled before pushing the increment commit.


iOS 10 Swift 3 support

I have a project which I have just updated to swift 3 and iOS 10. I use the Windows.AzureMessaging.Framework in this project, but it seems the tutorial site (https://azure.microsoft.com/en-us/documentation/articles/notification-hubs-ios-apple-push-notification-apns-get-started/) does not point to an updated version for swift 3. Is there an updated version and if so where can I find it ?

I found a solution and I post here all the steps to integrate the Azure Notification Hub with Swift:

On your project root, create a new Folder called "Headers"
Inside that Folder create a New File > Header File > Call it "HubInfo.h"
Copy this code inside your HubInfo.h file (remember to replace the values of HUBNAME and HUBLISTENACCESS with your project's hubName and hubConnString):
`
#ifndef HubInfo_h
#define HubInfo_h

#import <WindowsAzureMessaging/WindowsAzureMessaging.h>
#define HUBNAME @""
#define HUBLISTENACCESS @""
#endif /* HubInfo_h */
`
4) Select your project under "Targets" and then go to "Build Settings"
5) Search for "bridging header" in the search field
6) In "Objective-C Bridging Header" add your Header File: Headers/HubInfo.h
7) In your AppDelegate add "import UserNotifications" on the top
Then build the project and follow the tutorial here (starting from the step 8) using Swift instead of Objective-C: https://docs.microsoft.com/en-us/azure/notification-hubs/notification-hubs-ios-apple-push-notification-apns-get-started#connect-your-ios-app-to-notification-hubs

Seems like @elvirberisha has found a solution. I'm going to close/lock this issue as resolved.


xminorlocator=1 or yminorlocator=1 doesn't work

I think this line creates a problem when we define yminorlocator=1 or xminorlocator=1:
https://github.com/lukelbd/proplot/blob/e1a603729305abe791ed23d84cfeff11e5a4cea6/proplot/axes/cartesian.py#L1101
Example
import proplot as plot
import numpy as np

fig, ax = plot.subplots(aspect=4, axwidth=5)

ax.format(
    xlim=(0, 200), xminorlocator=10, xlocator=30,
    ylim=(0, 4), yminorlocator=1, ylocator=2
)

Raise the warning:
proplot/axes/cartesian.py:1102: ProPlotWarning: You passed yminorticks=1, but this argument is used to specify
tick *locations*. If you just want to *toggle* minor ticks on and off, please use ytickminor=True or
ytickminor=False.

A way to fix this is to pass minorlocator=1.000001 for example, I guess the problem is that in Python 1 is equivalent to True.
Proplot version
0.6.4

Thanks! One-line fix: a5d0c44


refactor: UI tweaks for required, readonly fields

Adds indicator for required fields
Fixes title for channel update modal
Makes Token, User ID fields read-only

Looks like some nice QoL UI improvements @brandonroberts!  Thank you for the PR.


Test Rails 6 with 2.4.x branch

Looks like the 2.4 release is pinned to Rails < 6.0.  I need a way to run rails 6 on the 2.4.x branch.
Bump the rails version requirement, or if rails 6 is still considered unstable maybe we need a 2.x development branch?

Why can't you upgrade to 3.0.0?
The 2.4 release will not support Rails 6.


RDS headers not found

Hello,
I have successfully created an example connecting to S3, so I figured I would try to pull some data from RDS. I added the following includes
#include <aws/rds/RDSClient.h>
#include <aws/rds/model/DescribeReservedDBInstancesResult.h>

and added the code to use these however, when I run make on the command line (not using xcode on mac) I am getting the following error:
main.cpp:5:10: fatal error: 'aws/rds/RDSClient.h' file not found
#include <aws/rds/RDSClient.h>
         ^
1 error generated.
make[2]: *** [CMakeFiles/AWS_TEST.dir/main.cpp.o] Error 1
make[1]: *** [CMakeFiles/AWS_TEST.dir/all] Error 2
make: *** [all] Error 2

Question is why would it find
#include <aws/core/Aws.h>
#include <aws/s3/S3Client.h>
#include <aws/s3/model/Bucket.h>

but not these
#include <aws/rds/RDSClient.h>
#include <aws/rds/model/DescribeReservedDBInstancesResult.h>

Ok, I have gotten a bit further. After re-reading over and over about how to configure the CMakeList.txt, I realized why the s3 header files were found. I needed to add the rds to link libraries. target_link_libraries(AWS_TEST aws-cpp-sdk-s3 **aws-cpp-sdk-rds**)
However, I am not getting
main.cpp:10:10: error: no type named 'SDKOptions' in namespace 'Aws'     Aws::SDKOptions options;     ~~~~~^ error: no type named 'InitAPI' in namespace 'Aws'     Aws::InitAPI(options);     ~~~~~^ ...
Any ideas? I will keep poking around until I find out something.

That last comment was due to a missing
#include <aws/core/Aws.h>
This must have been deleted in troubleshooting.


Use scoped refspec

Use scoped refspec for PRs to work around a performance issue with newer git implementations, especially on Windows

Fixes https://github.com/dotnet/core-eng/issues/1701


tmp102: Initial add of tmp102 driver.

This is an implementation of a driver for the Texas Instruments TMP102 sensor. It's compatible with TMP112, and TMP75 as well.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 79.29293% with 41 lines in your changes missing coverage. Please review.

Project coverage is 46.8%. Comparing base (2cf8fa1) to head (8c83f68).

Files with missing lines
Patch %
Lines

tmp102/tmp102.go
79.3%
26 Missing and 15 partials :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##            main     #76     +/-   ##
=======================================
+ Coverage   46.4%   46.8%   +0.4%
=======================================
  Files         82      83      +1
  Lines      10031    8119   -1912
=======================================
- Hits        4658    3803    -855
+ Misses      5211    4137   -1074
- Partials     162     179     +17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks!


Setting up project infrastructure

@turnerm - I've updated for the request changes.  I've not tested the github actions yet, as you say it is not available until in main.

I realized that we forgot to properly implement the extras because there is no import check. However Mike's changes in HAPI 215 mean we won't have to have the extras anymore so I think that's fine.


[ISSUE] Helm Chart Installation Errors with SNAPSHOTS & Alpha Branch

Describe the issue:

Each of the nightly test runs of the cross-component test suite failed last night due to helm chart installation errors.
Actual behavior:

Failed test runs can be seen here:
Test Run 1
Test Run 2
Test Run 3
Expected behavior:

The helm chart installation should be successful.
How to reproduce:

Using the workflows of the test runs linked above.
Logs:

Environment:
Please note: Without the following info, it's hard to resolve the issue and probably it will be closed.

Platform:
Helm CLI version:
Chart version:
Values file:

When investigating the issue I didn't find any logs that jumped up at me. I ran the test later int he day and the installation of the helm charts were completed.
Today @slolatte confirmed the nightly test ran without any helm installation errors. Will close this out.


Any example for number prediction?

Hi @migueldeicaza,
first of all thanks for this great and unique library!
I couldn't find a number prediction example arround here.
Could you give an example how can I predict number with given float/decimal numbers please?
Also I want to predict a number range if it is possible?
Regards!

I have very limited time to build samples, I am afraid I wont be able to do this for you.


Add conversion methods for PresignedRequest and update docs

Motivation and Context

This PR is a result of some papercuts I experienced while investigating aws-sdk-rust#423
Description

update: s3 presigning example with curl command generator
add: impl into for PresignedRequest -> http::request::Builder
sort: cargo deps
Testing

I ran it locally
Checklist

[x] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[x] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@rcoh I removed the examples per @jdisanti's changes. Looking at the example repo, they're behind us a bit and will need to be brought up to speed. I'll make a PR for the example updates to that repo after I talk to our example writer about it.

The related example PR is here, not to be merged until we release the changes in this PR


CSV Reader Configuration not working

The passed CSV Reader Configuration doesn't work.
I am doing a GetDynamicPropertyName in CSV Reader Configuration which doesn't execute.

Thank you for taking this up!


update marked dev and peer dependency

marked has a new 0.x minor version (due to https://www.npmjs.com/advisories/1076).
This:

updates the peer dependency to also accept ^0.7.0
updates the dev dependency to ^0.7.0

Ping on this.  Builds fail their yarn checks due to semantic-release being upgraded to the version of marked without the vulnerability, but this package not being updated.
yarn check v1.16.0
...
error<EMAIL_ADDRESS>|| ^0.5.0 || ^0.6.0" doesn't satisfy found match of<EMAIL_ADDRESS>

Thanks! Will do a new release soon


Apply minor formatting fixes

Forever immortalising my failure to git push after Hixie's  review of #10260.

@Hixie
/me buries head in shame


Flutter needs a way to play inline video

I thought we already had a bug on this, but I don't see one.
It is currently possible to play video fullscreen or over-the-top of other Flutter content, and there are various community-authored plugins to make that easy (e.g. https://pub.dartlang.org/packages/video_launcher)
However this bug is about making it possible to have inline video content (e.g. a video inside a scrolling ScrollView widget inside your Flutter app) properly clipped/positioned/composited with other Flutter content.

This is video that is part of the layer tree right? As opposed to a control that is placed by the embedder over the Flutter view. If it is the latter, the existing platform messages API should allow custom embedders to control platform widgets manually. I can take a look at introducing a custom layers type that allows us to support video within the layer tree.

This is video that is part of the layer tree right?

Correct.

Fixed by texture support #12525. A demo plugin for making use of this facility for video playback is available here.

Exciting. :D  Are there bugs tracking the remaining planned work around inline video (e.g. publishing that package, updating it's readme, possibly including an example of video playback in one of our examples, etc.)?

+1 on publishing the plugin
(nit: would it make sense to let the plugin publish a VideoPlayer widget object with a controller rather than a Texture?)

@eseidelGoogle I'll add links here once there are.
@xster Yes, in fact @sigurdm and I just discussed that today.

People following along might also be interested in https://pub.dartlang.org/packages/chewie , a video player with platform aesthetic controls.

Canonical location for the video player plugin is now here: https://github.com/flutter/plugins/tree/master/packages/video_player

does it support HTTP headers, AES, and other encryption as well?

The plugin uses the platform's video playback mechanism. So MediaPlayer on Android and AVPlayer on iOS. This issue was related to using the textures for video feeds generated by the platform during composition of the Flutter layer tree.
I can't answer your question about support for specific features but you might find the answer in the linked documentation.

I see they moved to exoplayer for Android


Update CouchDB link for filtered replication in doc

In the Replication doc, the CouchDB link doesn't work anymore.
Here's the latest link:
http://docs.couchdb.org/en/2.1.1/replication/intro.html#controlling-which-documents-to-replicate

Hi @azri92, thanks for reporting this.  We have plans to update deprecated links and I will add this to our list.

Cool! Looking forward to the updates 😄


Why is there hard dependency on a specific version of ruamel.yaml?

Why is there a specific version lock on ruamel.yaml==0.16.5 release? Are newer versions incompatible for some reason?

I had some trauma with automatic regressions in some node packages in the past. Thus, I took the habit locking dependencies.
Does it cause a problem because deps are shared in Python?
Would you recommend to specify an open range?

In python ruamel.yaml==0.16.5 means only use this version, so if any other dependency (in projects) requires a newer version, your library won't be installable.

Does it cause a problem because deps are shared in Python?
yes
Would you recommend to specify an open range?
ruamel.yaml>=0.16.5 should be a safe choice.

Thanks @dz0ny for the consulting. I open the range for both dependencies.
f6d8902a2733267f9213a47f2cd7ec5b32a8ca54


Add pipe server process name

Great project! thank you guys <3
I think that having the server process name will be super helpful!

Hi @cookpoo78 ,
thank you :)
Identifying the server process that created a named pipe can be tricky, as there isn't a straightforward place where this information is directly logged or stored.
We have the "Client PIDs" which shows who is using this pipe. This is not the best option, but it can give you a hint who created it.
I had an idea, maybe we can check that the timestamp of the handles and see when it was opened for the first time but from a quick check I saw that Windows doesn't natively track the creation timestamps of handles for named pipes.
I will check if there is other option that I am missing.

@g3rzi Thanks for the detailed response, appreciate it :)


Docker build fails due to Golang compile error

Building the docker image using the Dockerfile fails due to a compilation error on the etcd's clientV3/client.go module.
root@ece14a1449e4:/go/src/github.com/coreos/torus# go install -v github.com/coreos/torus/cmd/torusd
github.com/coreos/etcd/clientv3
# github.com/coreos/etcd/clientv3
../etcd/clientv3/client.go:309: oldconn.State undefined (type *grpc.ClientConn has no field or method State)
../etcd/clientv3/client.go:311: oldconn.WaitForStateChange undefined (type *grpc.ClientConn has no field or method WaitForStateChange)
root@ece14a1449e4:/go/src/github.com/coreos/torus#

The etcd/clientV3 module is referenced from coreos/torus/block/etcd.go. Seems like the grpc.ClientConn API has changed.
Or am I missing something?

Having just encountered this with etcdv3, I can say that this problem is not with torus, but with the etcdv3 client library.

This is an etcd issue https://github.com/coreos/etcd/issues/5561, where etcd clientv3 package is dependent on the old version of gRPC. We will fix.
Thanks.

Modifying this issue: Use vendored state for Docker build.


real workflow and removing unused sort

Is basic_sort_UNIQUE_SUFFIX in use at all?

we will have to fix the linting errors by running black, wanted to get the actual workflow in first


Proxy for Swagger object

Preface
This PR is an example how #446 can be solved in more general way. There are more ways how it can be solved. More options can be found in #668  This is only one of the possible solutions to discuss.
Overview
SwaggerProxy class which allows to filter Swagger model entities using provided predicate.
How it works
SwaggerProxy creates a proxy for Swagger model instance using cglib library. Proxy intercepts all method calls, redirects them to origin and get result.
Call result is checked whether is had getVendorExtensions method or not. If yes then vendorExtensions are extracted and predicate is applied to them. Predicate returns Boolean value.
If predicate returns true then result is returned back to the caller.
If predicate returns false then result will be:

null in case of POJO
filtered HashMap in case of Map
filtered ArrayList in case of List
filtered HashSet in case of Set

Pros

SwaggerProxy can be easily used with OpenAPI object (OpenAPI v3) as well. The only thing that should be changed is vendor extensions method name from getVendorExtensions to getExtensions.
Predicate can be applied to any part of object model regardless where it is located in object graph.
Proxies are build on demand only. That means if there are big model but rules uses only part of it then proxies will be built only for used parts of model.

Cons

It uses reflection what can slowdown process a bit. It can be easily solved with caching because Swagger model is immutable from rules perspective and rules do not change the model.

Codecov Report

Merging #692 into master will decrease coverage by 0.56%.
The diff coverage is 59.25%.

@@             Coverage Diff             @@
##             master    #692      +/-   ##
===========================================
- Coverage     83.27%   82.7%   -0.57%
- Complexity      839     843       +4
===========================================
  Files           153     154       +1
  Lines          2242    2296      +54
  Branches        322     331       +9
===========================================
+ Hits           1867    1899      +32
- Misses          229     246      +17
- Partials        146     151       +5

Impacted Files
Coverage Δ
Complexity Δ

...java/de/zalando/zally/rule/model/SwaggerProxy.java
59.25% <59.25%> (ø)
4 <4> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd242a6...4c636a7. Read the comment docs.

Interesting. Thanks for the idea! However, I would be strongly opposed to using something like cglib in Zally. Vanilla Java and Kotlin features really ought to be enough. It would make the project a lot more complicated.
Although I've become a lot more skeptical of the whole proxying approach in general, have you looked at native Java proxies already? https://docs.oracle.com/javase/8/docs/api/java/lang/reflect/Proxy.html

Nice POC!

@mfellner native Java proxies can not work with classes. It can work only with interfaces. Swagger object is a class. To use java proxy you have to define interfaces for all classes in model. That is why cglib is used.
I have not found anything in Kotlin what can solve this issue in more a elegant way. All possible solutions like property/class delegates look even more complicated. If someone can point me that would be nice.
Also, I have used cglib embedded in spring. It can be changed easily to use external cglib library (spring just merged it inside in a special package) but replacing imported packages. I would say spring dependency is optional here.

@vadeg nice idea, are you planning to proceed with this implementation?

@netme yes if it will be accepted. We had a discussion with @mfellner and there was no clear whether it is acceptable to use proxy or not.

@vadeg I'm in the process of preparing a revised PR for OpenAPI 3 support that includes an alternative implementation for a Proxy. https://github.com/zalando/zally/pull/706 I would appreciate your feedback. Maybe we can close this PR in favour of the other one.


Enable ineffassign in CI

Partially fix #578
Lint is the big one (well, it's mostly a matter of documentation missing, which we can tackle once we're out of alpha).

Codecov Report

Merging #1275 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1275   +/-   ##
======================================
  Coverage    23.7%   23.7%
======================================
  Files          40      40
  Lines        4526    4526
======================================
  Hits         1073    1073
  Misses       3251    3251
  Partials      202     202

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb7b41a...d2fa228. Read the comment docs.

ping @stevvooe @AkihiroSuda

LGTM

LGTM (IANAM)

LGTM


Initial set of API changes for Schema Registry

Contributes to https://github.com/Azure/azure-sdk-for-net/issues/24035

API changes have been detected in this PR. You can review API changes here

API changes have been detected in this PR. You can review API changes here


⚠ Bofrost.de has degraded performance

In 0d2dba2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1960 ms

Resolved: Bofrost.de performance has improved in 7fb6322.


Automatic API Declarations

Problem statement
Was moving/converting data from using the old plugin to the new plugin "React Query" and discovered that the manual typing was old and incomplete. The solution is to automatically generate those types from the source code so that when it changes it is automatically updated. This will avoid it becoming old and incomplete in the future. This will prevent bugs and become a failsafe from errors appearing in the future without the Experience team knowing about them.
This is needed to complete the items in issue #6528 .
Acceptance criteria

[ ] The declaration file successfully outputs to the front end project
[ ] Receive approval from API team before merging

Moved directly to in-progress since @jpandersen87 is currently working on it.

Closing per meeting acceptance criteria.
"It exists. It works" - Joseph Andersen
Created a follow-on ticket #8520 for the packaging. That will be completed in a later sprint.


I am using SDK 2.9.4 integrated with CocoaPods.

@Rupert-RR The default flush interval for debug builds is 1 second: https://github.com/mixpanel/mixpanel-iphone/blob/master/Mixpanel/Mixpanel.m#L117-L118
You can set it yourself via the flushInterval property: https://mixpanel.com/site_media/doctyl/uploads/iPhone-spec/Classes/Mixpanel/index.html#//apple_ref/occ/instp/Mixpanel/flushInterval

Ah... that would explain it — thank you!
Isn't it strange that this only takes effect after backgrounding the app then foregrounding it again though?

That is strange indeed. I'm not able to reproduce that behavior here...

Ok, it's perhaps something to do with my setup then. Anyway, not a big issue.

I have the same issue. Guys is it possible at least to disable that particular event in logs?

@YGeorge I don't think there's a way to disable logging for particular events, but you can turn off all logging with [Mixpanel sharedInstance].enableLogging = NO. If you're finding it too spammy I'd recommend only turning it on when you're debugging Mixpanel stuff, and leaving it off otherwise. Another option would be increasing the flush interval though that changes actual behavior; depending on if you're only having issues with this in debug that could be okay though it would make your events show up later.

I have the same issue as @Rupert-RR - Did you know what was causing that?


Summaries error in MSSQL Environment

Package
filament/filament
Package Version
v3.2.62
Laravel Version
v11.2.0
Livewire Version
No response
PHP Version
8.3.6
Problem description
When I am using the summaries feature from table builder in mssql with a complex table name it generate a select query that has a invalid as for the subquery it uses the table name but points are not allowed in the as
The Problem is for examble in Summirzer.php
$query = DB::connection($query->getModel()->getConnectionName())
    ->table($query->toBase(), $query->getModel()->getTable());

Expected behavior
The subquery should generate valid aliases for tables, even in MSSQL environments with complex schema names.
Steps to reproduce
Create a model with a complex schema name
Use table builder with summarization in MSSQL environment.
TextColumn::make('EFK_VERKAUFSWERT_NETTO')
                    ->summarize(Sum::make())

Observe the error due to invalid table aliases in the generated subquery's AS clause.
Reproduction repository
https://github.com/CruxAllez/filament-issue-mssql
Relevant log output
No response

I am not a user of this driver, so I would appreciate you investigating the issue and opening a PR if you can, I don't know if any of the Filament team are going to take active action on this bug report so I don't want you to be left waiting.


Lab 10 - SSMS not able to connect to Azure SQL Database using AAD

Module: 10
Lab/Demo: Improve performance with hybrid tables
Task: Add an order to the database
Step: 8
Description of issue: Not able to connect SSMS to Azure SQL Database using AAD-password
Cannot connect to dp500server-231363.database.windows.net.
AADSTS900021: Requested tenant identifier '00000000-0000-0000-0000-000000000000' is not valid. Tenant identifiers may not be an empty GUID.
Trace ID: 7a1279c8-9913-4828-b843-aaa5aa067500
Correlation ID: d2add445-8581-446e-afd9-1110b25ffefb
Timestamp: 2022-09-09 10:17:29Z (System.Data)

It is possible to connect with the SQLADMIN account.
The solution to this is to change the instructions at step 6

Thanks Wesley and Anthony - this has been fixed!


Add an end-to-end test and run it through CI

We should have a canned end-to-end test which we can easily run to confirm everything is working. Ephemeral errors may occasionally occur with this test, but it will give us supplemental information beyond the unit tests about whether or not things are working as expected. Once we have this test, we should run it using CI.

This is now working. The test with no parameters set is optimized to run on a local system for quick testing with a simple pass/fail interface and a log file for details. It can have a flag passed in to log to the console for CI. It can also have different URLs or recipes provided, which might be useful in the future if we want to test our recipes more extensively.
I didn't have a great way to test the CI job without simply committing it and seeing what happened, but I now have most of the issues worked out. The one minor issue left is that we cannot use pip dependency caching for now. It seems like the Github action that supports pip dependency caching only supports it if the project tracks dependencies in a requirements.txt file. In our case, dependencies are tracked in a setup.cfg file. This doesn't prevent our job from running, but dependency caching would help speed things up.
The other issue is that #23 is still outstanding. I think this project is still in a place where we don't need to move heaven and earth to keep things working at all times. I am tracking when that merge request goes into pytube, and when it does I will upgrade. In the mean time, the CI runs of the test will fail.


Can't pass instance name within connection string

Trying to connect with connection string like
 server=tcp:<server name>\<instance name>,<port>
to existing instance, getting unknown host error.

Worked, thx. I failed with port omg.


Description of basic functionality

We want to add two operators that make using the Open Ephys commutator easy:

A sink node that takes sequence of doubles and turns the commutator accordly via a specified serial port. It can also expose commutator options such as enable/disable, LED status etc.
A transform node that converts Quaternions to rotation values about an axis.

After playing around a bit, I actually don't like this split. I now like the idea of an integrated node that takes a Quaternion and uses it to update the state of the hardware with all logic required to do this internal. Why?

"A transform node that converts Quaternions to rotation values about an axis"

This was never a pure transform because you need two Quaternion measurements to create a rotation. Although it will behave as one if you bake in some state for the first value, the first command will always do nothing and its de-facto already a combinator

The rate at which commands are sent to the hardware should be kept at 10 Hz or so to prevent communication issues. This means that either the Transform node would need to be combinator that downsampled its outputs, the sink node would need act as a combinator for the same reason, or the user would be expected to put sample between the two and give it a reasonable rate.
The idea of hooking up the Quaternion output from a BNO source to a single node with PortName and RotationAxis specified without any intermediate stuff is very appealing to me.

@jonnew the issue with making this a mega-node is that it cuts off composition to the point of making the node unusable for all but the most basic cases.
In practice we have never used the commutator operating by itself with just the BNO. We need options for manual intervention, panic button, GUI, etc, many of which are really experiment-specific.
By closing off all composability from this library I don't see how this could be useful for the cases discussed above, and I feel it will also complicate logging and future upgrades as well.

100% agree with all of this, but the ease of use trumps most of these advantages in my opinion. Why dont we just ake a couple different nodes for various things.
I guess i could write a bunch of small nodes and then create the mega node from the smaller elements as an included .bonsai file.
I could call the meganode something specialized like AutoCommutator or whatever

I could call the meganode something specialized like AutoCommutator

I think this is a great idea. By having both the embedded workflow and the more granular operators it should be possible to assemble all the usage patterns and more efficiently update the package in the future.
I can see now better the complications behind the initial naive proposal and indeed the problem gets complicated by an interaction between the need for rate control and numerical accuracy when computing the relative twist feedback signal.
I will submit a PR that tries to address both of these ideas. I have also created a default main branch so we can retarget different PRs for comparison if useful.


Tutorial 04 - Trainning

When I run this "!ForceBalance optimize.in" to start training, it shows the following.

Calculation started at 2022-02-02 11:40 AM
Warning: importing 'simtk.openmm' is deprecated.  Import 'openmm' instead.
Traceback (most recent call last):
File "/home/ylinb/anaconda3/envs/OpenFFCrystal/bin/ForceBalance.py", line 45, in Run_ForceBalance
optimizer.Run()
File "/home/ylinb/anaconda3/envs/OpenFFCrystal/lib/python3.8/site-packages/forcebalance/optimizer.py", line 322, in Run
xk = self.OptTabself.jobtype
KeyError: 'single'

Is there a specific dependency I need to install?
I follow the install instruction here
https://openff-evaluator.readthedocs.io/en/latest/install.html

@Yu-Tang-Lin it looks like ForceBalance does not understand "single" for the variable jobtype in the optimize.in file. This should be the default option for jobtype, what happens if you try commenting this out in the input file? The alternative (for a single-point calculation) is to set jobtype to optimize and set maxstep to 0.

@jeff231li
Sorry, I am checking "optimize.in", Do you mean that comment out this one? (I found it is the most closed code that related to singe)
(allcap) The job type, defaults to a single-point evaluation of objective function
jobtype optimize

@Yu-Tang-Lin it looks like ForceBalance does not understand "single" for the variable jobtype in the optimize.in file. This should be the default option for jobtype, what happens if you try commenting this out in the input file? The alternative (for a single-point calculation) is to set jobtype to optimize and set maxstep to 0.

I also try maxstep = 0 in optimize.in, but it will show the same error message

6

@Yu-Tang-Lin it looks like ForceBalance does not understand "single" for the variable jobtype in the optimize.in file. This should be the default option for jobtype, what happens if you try commenting this out in the input file? The alternative (for a single-point calculation) is to set jobtype to optimize and set maxstep to 0.

May you tell me what's your version for forcebalance in your environment? My forcebalance version is 1.9.2

I have version 1.9.2 installed in my environment. What does your optimize.in file look like?

Yes, I also have version1.9.2. The optimize.in is generated from code Tutorial 04.
input_file_path = get_data_filename("tutorials/tutorial04/optimize.in")
Here is my optimize.in which was generated from Tutorial 04.
optimize.zip

Hi, @jeff231li
I also try to modify in forcebalance/optimizer.py (Toolkit in forcebalance)
xk = self.OptTab[self.jobtype]()
It seems that it will extract the jobtype optimize in the optimize.in
But it doesn't work.
I manually add self.jobtype = 'optimize' in forcebalance/optimizer.py
The error message becomes to this. I guess there is some issue in the toolkit in forcebalance for recognizing the jobtype
  File "/home/ylinb/anaconda3/envs/OpenFFCrystal/lib/python3.8/site-packages/forcebalance/optimizer.py", line 325, in Run     xk = self.OptTab[self.jobtype]() KeyError: 'optimize'
Here is my optimize.err
optimize.txt
Let me know if you need any other info, we can test it together.

I'm not sure why it's giving you this error. This python script works for me, run the file with this command python run_evaluator.py | tee console_output.log. The problem could be how you installed your conda environment. Can you send the YAML file of your conda environment: conda env export > environment.yaml.

Hi Jeffry,
It's my environment.yaml
Thank you for helping me!
Sincerely,
Yu-Tang
On Tue, Feb 8, 2022 at 1:47 PM Jeffry Setiadi @.***>
wrote:

I'm not sure why it's giving you this error. This python script
https://github.com/openforcefield/openff-evaluator/files/8027341/tutorials.zip
works for me, run the file with this command python run_evaluator.py |
tee console_output.log. The problem could be how you installed your conda
environment. Can you send the YAML file of your conda environment: conda
env export > environment.yaml.
—
Reply to this email directly, view it on GitHub
https://github.com/openforcefield/openff-evaluator/issues/420#issuecomment-1033046035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVJ22KNIZKYY36SK3YDBFSTU2F6LLANCNFSM5NM3IUGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @jeff231li,
I follow #414 in the previous issue and use v0.3.9 to test Turorial. It can work. I guess there is an update between v0.3.9 and master.
Thank you for your help!

I think this has fallen out of date - please do let us know if you run into new issues.


OpenCV Error: Null pointer (Invalid pointer to file storage) in cvWriteInt

Hi, There are some errors when I run the vs2010 demo!
OpenCV Error: Null pointer (Invalid pointer to file storage) in cvWriteInt, file
........\opencv\modules\core\src\persistence.cpp, line 2964

This error occurs when config folder does not exists. Thank you for reporting this issue.

Then how to solve this problem? In VS 2013 also i am facing the same issue. The solution is building but the setup is not running

Hi @murari023 you need to create a config folder in your project path, bgslibrary use it to store the configuration files...

Hello, i have the same problem. But the config exist folder.

I had a very similar issue (I had an existing config folder) and it was solved when I deleted the XML I saved my calibration output information to and re-ran the program.

Hello! Solve the problem create manually the folder config inside the folder build.


Redirect to backup view if given a submission (instead of 404)

Currently: okpy.org/assign/backup/<id> will 404 if <id> is a submission
It would nice to be able to provide a URL for a backup without having to know if it's a submission or not first.
This PR's behavior:

If the route matches the backup's submission status: No change.
If the route is /.../submission/<id> and the backup isn't a submission: redirect to /backup/id
If the route is /.../backup/<id> and the backup is a submission, redirect to /submission/id

Another option is to provide a single endpoint that always works regardless of submission status:

/.../code/<id>

What happens if it's neither a submission nor a backup? (Like if someone makes a typo.)

It would 404 (this PR maintains that).
The behavior listed above assumes that we can find the row in the database and if the user has permission to see it.

On reflection: having a /.../code/<id> that redirects to /.../submit/<id> or ../backup/<id>  might just be better - this can be a stop-gap for now.


List/Tree: introduce commands for "Open to Side" and "Open Pinned"

We have a good set of commands for our list/tree widgets in our system. However, these commands do not cover:

Ctrl/Cmd-click to open a resource in a side-by-side editor (adopted in most places)
Alt-click to force open the resource pinned (for quick open via #30530)

I think there should be new commands besides list.select that handle those cases. It is unclear however how to name such a generic command because after all, the target tree/list decides how to handle them. We could have something like list.selectAlt1 and list.selectAlt2 for the two things and adopt it, though its not easy as a user to understand the impact
The goal would be to make pinning and opening to the side configurable via keybindings editing.
I have a feeling that once we adopt multi-select for the explorer (via https://github.com/Microsoft/vscode/issues/1023) this will be a more requested thing because we will probably have to change Ctrl+click in the explorer to no longer open to the side but to be used for multi select.

Is this on the agenda to implement?


MarketOverview component not working

[Vue warn]: Failed to resolve component: MarketOverView If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

Hi! Would you please make your issue clear and add your code?

I follow the documentation but show this warning and not working.
 [Vue warn]: Failed to resolve component: MarketOverView If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
<template>
  <MarketOverview  />
</template>

<script>
import { MarketOverview } from 'vue-tradingview-widgets';

export default ({
  name: 'Test',
  components: {
   MarketOverview
  },
});
</script>

I checked, and it's working. Please check the demo on Codesandbox.
Notice that you have to use Vue3 and the latest version of vue-tradingview-widgets


Migrate Documentation

Migrate existing documentation from https://github.com/dexidp/dex#documentation into this repo. Organize as per cncf/hugo-netlify-starter.
@celestehorgan @zacharysarah

Closed with PR https://github.com/dexidp/website/pull/8


[Caffe Frontend] adding Reduction op

Reduction op for Caffe frontend, supporting SUM, ASUM, SUMSQ, MEAN methods.

cc @jcf94 @kazum @FrozenGene please help to manage this  PR. Thank you!

@FrozenGene Please take another look :-)

@zotanika please resolve the conflict.


sideMenu not Disappear

I configured this in my my ViewDidLoad:
let storyboard = UIStoryboard(name: "SideMenu", bundle: nil)
        SideMenuManager.default.leftMenuNavigationController = storyboard.instantiateViewController(withIdentifier: "LeftMenuNavigationController") as? UISideMenuNavigationController

for show the sideMenu from IBAction i use this:
@IBAction func didTouchLeftNavigationItem(_ sender: Any) {
        if let leftMenuNavigationController = SideMenuManager.default.leftMenuNavigationController {
            present(leftMenuNavigationController, animated: true, completion: nil)
        }
    }

when it disappears it locks my screen :(
i force the dismiss from this method:
func sideMenuDidDisappear(menu: UISideMenuNavigationController, animated: Bool) {
        print("SideMenu Disappeared! (animated: \(animated))")
        SideMenuManager.default.leftMenuNavigationController?.dismissMenu()
    }

but not working :(
I need help, i'm using the
pod 'SideMenu', '~> 6.0.0'
I would not like to be calling my method that instantiated my UISideMenuNavigationController

Please ensure you’re running the latest version (currently 6.0.4) and reopen this issue if your problem persists.

I also experience this one. Dismiss will sometimes lock my screen. I don't know why.

Happens when viewWillDisappear(_:) is called but 'viewDidDisappear(_:)' not called. I don't know why.

Update to the latest version, ~> 6.0.
On Mon, Jul 29, 2019 at 11:55 PM denniskeithgaso<EMAIL_ADDRESS>wrote:

Happens when viewWillDisappear(:) is called but 'viewDidDisappear(:)'
not called. I don't know why.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jonkykong/SideMenu/issues/505?email_source=notifications&email_token=ABLWL74F7ECXLZNQAZDLZDTQB7QWRA5CNFSM4IE6PTIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3C7MHY#issuecomment-516290079,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLWL74KAF6TID3Z37PWXS3QB7QWRANCNFSM4IE6PTIA
.

--

I noticed my problem is also related to #520 . If I present the menu using swipe gesture, I cannot dismiss it programatically.
The steps are:

Swipe to open the left SideMenu

Dismiss programatically

Swipe to open again

Dismiss programatically and it locks my screen. Does not dismiss. Here, viewWillDisappear(_:) is called but viewDidDisappear(_:) is not called

I am not sure if this is project specific. It seems that the app lost reference to the viewcontroller.
If I present the SideMenu programatically, I have no problem dismissing it either programatically or by swipe. Just when presenting using swipe gesture has problem.
I noticed that when my screen locks and I put the app in background then activate it to foreground, the SideMenu becomes dismissible.

Fixed in 6.1.2.


_auto_add_head () does not compile necessary dependant files

_auto_add_head () does not load necessary core dependent files to the head for use in default and custom addons.  compiler.head_html.php

Problem solved for view class which needed rewriting.


sql: enable clearing user password

PR: https://github.com/cockroachdb/cockroach/pull/43892
From release notes:

Previously, when setting a user's password to enable password authentication for the user, it was not possible to revert this choice. The only way to disable password authentication was to either drop the user or add a specific per-user HBA rule. This has been fixed and the PostgreSQL-compatible ALTER USER WITH PASSWORD NULL statement can now be used to clear the user's password. [#43892][#43892] {% comment %}doc{% endcomment %}

https://github.com/cockroachdb/docs/issues/6605


Consider including the "I need to be contacted using a text phone" pattern from Elements

A user raised on support that we haven't included this pattern from Elements.

According to @StephenGill there are at least a couple of departments who use textphone on their helplines so we should consider including it in the Ask users for telephone numbers guidance.

This is still used at least by DWP at least on Carer's Allowance:


📧 modal for support email to report naughty comments/users

building a report system will take too long. so we'll do it later.
in the meantime, we need a modal display, that tells users to email a support email address, to report naughty comments and users.
the modal should display a copiable text block, which includes the user's nickname, userId, and the commentId, and tells the reporter user to paste this into the support email.

unclear if @zenkyuv or @joshuaolusayo is currently working on this task?
who's working on this frontend ticket to create the report modal?

unclear if @zenkyuv or @joshuaolusayo is currently working on this task?
who's working on this frontend ticket to create the report modal?

I am working on it


Update headers in action without Attributes (runtime)

Is it possible to set/update for example x-frameancestor csp directive in the action kode (MVC) ?
We need to set the frame domain pr session so wondering how to do that ?

No, there's currently no way to do that. There are a few that have asked about dynamic configuration during runtime, so I have it on my radar. I'll have a look at it after the work with ASP.NET Core is done.


All pools in the front-end should be labeled as "TKN/BNT" and not "BNT/TKN"

fixed and on staging. Please open new issue if I have missed it somewhere.


capture full resolution images with v2.1 and Pi3

Hi there, trying with
import RPi.GPIO as GPIO
import picamera
import time
import sys

GPIO.setmode(GPIO.BCM)
GPIO.setup(18, GPIO.IN, pull_up_down=GPIO.PUD_UP)
with picamera.PiCamera() as camera:
	while True:
		input_state = GPIO.input(18)
		if input_state == False:
			camera.framerate = 5
			camera.resolution=(3280, 2464)
			camera.preview_fullscreen=True
			camera.start_preview()
			time.sleep(2)
			camera.capture('01.jpg')
			time.sleep(1)
			camera.stop_preview()

to capture 3280x2464 pixels, which works fine with raspistill from the command line.
All I get is
$ python minimal.py
mmal: mmal_vc_port_enable: failed to enable port vc.ril.video_splitter:in:0(I420): ENOMEM
mmal: mmal_port_enable: failed to enable connected port (vc.ril.video_splitter:in:0(I420))0x5d0c60 (ENOMEM)
mmal: mmal_connection_enable: output port couldn't be enabled
Traceback (most recent call last):
  File "minimal.py", line 24, in <module>
    camera.resolution=(3280, 2464)
  File "/usr/lib/python2.7/dist-packages/picamera/camera.py", line 2283, in _set_resolution
    self._enable_camera()
  File "/usr/lib/python2.7/dist-packages/picamera/camera.py", line 732, in _enable_camera
    prefix="Failed to enable splitter connection")
  File "/usr/lib/python2.7/dist-packages/picamera/exc.py", line 191, in mmal_check
    raise PiCameraMMALError(status, prefix)
picamera.exc.PiCameraMMALError: Failed to enable splitter connection: Out of memory

If I reduce the resolution in the code to 3000x2000, for example, the procedure works.
The code runs on a Pi3. I did rpi-update, rpi-upgrade and apt-get update.
Am I doing something wrong?
Output preview is on a 800x480 7" Display
Thanks in advance
Jochen

Update:
If I understand it correctly, version 1.10, distributed via apt-get does not support the full resolution of v2.1 camera. So I did some
git clone git://github/waveform80/picamera.git
sudo python setup.py build
sudo python setup.py install

and it worked!!!

If I understand it correctly, version 1.10, distributed via apt-get does not support the full resolution of v2.1 camera.

Yup, that's correct - 1.11 will support the full resolution, but it's taking a lot longer than I expected, sorry! Given you've done a manual installation, when 1.11 is released you'll need to uninstall your manual installation because it'll take precedence over apt's installation. So, when 1.11 comes out (and Raspbian gets the new package), you'll need to do:
# (from within your cloned repo)
cd picamera
sudo python setup.py uninstall
sudo apt-get update
sudo apt-get upgrade

But for now, it's a decent stop-gap.


Firebase register token for FCM doesn't work after app transfer

[REQUIRED] Step 2: Describe your environment

Xcode version: 10.1
Firebase SDK version: 5.13.0
Firebase Component: Auth, Core, Database, Messaging, Storage
Swift Version: 4.2

[REQUIRED] Step 3: Describe the problem
Recently transferred my app to a new Apple ID. I created/upload all of the new certificates for push notifications to firebase console (APN Auth Key). Also, set up other IOS certificates ("IOS distribution", "Apple Push Services") and made sure appID was added with push notifications enabled. My problem is the old firebase register tokens already stored in my database don't seem to work anymore. Once I reinstall the app or get the token to refresh the push notifications work. But again if I were to use the old tokens that were created before the app transfer the push notification fails to send.
What would be the best way to create new push notification tokens for users that have the old token? After reading the firebase docs the only way to create a new token would be if the user does one of the three things below.

The app is restored on a new device
The user uninstalls/reinstall
The user clears app data.

Yet all of those things seem like something the user would have to do themselves.
After doing a bit more research I found this similar issue Change/update Firebase notification token or instance id forcefully via code?. There suggestion was to
1) Delete old Token
let instance = InstanceID.instanceID()
instance.deleteID { (error) in
    print(error.debugDescription)
}

2) Request for new token
if let token = InstanceID.instanceID().token() {
    print("Token : \(token)");
} else {
    print(“Error: unable to fetch token");
}
Messaging.messaging().shouldEstablishDirectChannel = true

Which worked, when I call deleteID on the instanceID the MessagingDelegate's "didReceiveRegistrationToken" is called with the new fcmToken. I then update Database with new token. But this new fcmToken does not work, when I try to send a push notification through the firebase cloud messaging console nothing is received on the phones end. Yet if I were to delete the app and reinstall it, the first fcmToken works. The App receives all push notifications until I try to force update the FCMToken.
Again my goal is to update users with old FCMTokens to new ones so they can receive push notifications without having to reinstall the app or get the old FCM token to work.

The old token is associated with your old APNS token, which should be updated automatically once the new apns token is generated by Apple. We will investigate more to it.
There's another issue we want to clarify: So the solution that you call deleteID should always guarantee a new token is associated with the new APNS token, does this solution not work with you neither?

Ok let me try the few things you suggested. Also technically I haven't sent out an update on new Apple ID just transferred apps. Do you think if I sent out an update that may help tokens automatically refresh?

Do you mean that you haven't switch your app certificate on your app yet? What about the console? have you updated the new certificate to the FCM settings?

No, I did switch app certificates and I  updated FCM settings with new APN certificate. But at the time I did not actually submit a new build to the new Apple ID. When transferring apps between Apple IDs I didn't have to submit a new build,  I just transferred the current build that was on the old Apple ID to new  Apple ID. But this past weekend I sent out an update with new Apple ID, and the users that had old FCM token in the database did receive an updated token. Yet the updated token still doesn't seem to work, the only way I got there token to refresh and work is if the user deletes and redownloads the app.

@stshelton, can you open a support ticket and reference this issue? When filling out the form, please share your FCM tokens that failing to receive pushes and we'll be able to look at what's happening to them on the backend.

Hi @stshelton,
Have you anyhow resolved this issue?
I have exactly the same problem. Apps being subscribed for notifications before app transfer don't get notifications after I updated certificate on firebase console.
Thanks for any hint on it!

@chliangGoogle
Im also facing same issue. Apps will not receive push notification after the app transfer even after i have updated the apns token key in firebase console settings, so to make it work i have to uninstall and reinstall app again then apps started receiving push notification.

We are facing the same problem.
We have transferred the IOS app to a new account and now we can only push to our IOS users after the remove and install the app again.
Please, any advice on what to do will be greatly appreciated! Having to communicate to ALL IOS users that they have to reinstall their app is devastating!
Thanks in advance.

@JakobMaaloe Can you check if your APNS token and FCM token have changed in the same app after you make the transition?
Can you share a test APNS token and FCM token with us so we can bring the issue to our team. If you are not comfortable sharing the token, can you open a support ticket and reference this issue?

Thanks for the ultra quick response ChliangGoogle!
We have created a new APNs Auth Key as part of the set up, since the old one belongs to the old account.
This is a test push token id of one of our test users that currently haven't uninstalled his app yet:
f0rtVkM3cRw:APA91bGaT-hhmffRFjt0VAZNjb1aYjuJWxTkq82QyC43DxXdbYvHUUML8-uEoMB-moUtb8DjMVZtZxle0rMCkjgLVMJiRA5ttqKMc2nh7z96TfdBj-dHpvov86hqYs9bRP-Kg7JMxKaU
I'm unsure what an APNs Token is or where to retrieve it...
Thanks a lot for your support!

We've just tried the following:

Reinstall the app on a device
Push message from our server to the FCM token -> The user receives the message
Change APN auth key to an old one
Change APN auth key back to the new one
Delete the FCM token on our server
App requests new FCM token on startup and receives the same FCM token as before it was deleted
Push message from our server to the FCM token -> The user doesn't receive the message

One solution could be if we could force all users FCM token to refresh... Any way that's possible?
I can send you the APNS token from Apple console in a support ticket if you still need it.

Thanks for the detailed info. You can use this Apple API to get the APNS token.
Can you also do the same steps and let us know if your APNS token is updated?

That requires that we update the app which is something we cannot do at this point with the production app.

Support case created with the APNs (from apple console - I hope this can be used)

Same issue!! Any solution?

can you share the information that if you APNS token and FCM token has changed during this transition? And let us know your old and new tokens.

Installing FirebaseCore 6.1.0 (was 5.4.1)
Installing FirebaseDatabase 6.0.0 (was 5.1.1)
Installing FirebaseFirestore 1.4.2 (was 1.2.1)
Installing FirebaseInstanceID 4.2.2 (was 3.8.1)
Installing FirebaseMessaging 4.1.1 (was 3.5.0)
Installing FirebaseStorage 3.4.0 (was 3.1.1)
Installing GTMSessionFetcher 1.2.2 (was 1.2.1)
I am having same Issue with updated firebase versions.


HtmlAgilitypack dependency

Hello,
It would be nice if the html agility pack is not a dependency for this plugin. Since it is only used in the UWP version of the plugin, and we are not developing for UWP, we don't want to have to include the agilitypack and all its dependencies to our project, when we don't need it

@matteobortolazzo
yes you are correct. I've modified the source locally and removed all references to the agility pack, and removed the removehtmltags and ishtml property, so it builds again. However, I'd rather use the actual nuget package because now i have created a local fork in our apps

In the next release I will remove the dependency :)

Nice! Looking forward to it

Pre release of v3


[Compiler] Bitfield Types

Add bitfield types (for now, 64-bit only).

[x] Fixed shifts

[x] Codegen
[x] IR
[x] Front-end (don't really need this)

[x] Add a Type
[x] Support in TypeSystem
[x] Support in deftype
[x] Support casting integers to bitfields
[x] Support accessing bitfields
[ ] Static bitfields (how do these work? stored in memory or inline in code?)
[ ] Set a bitfield. And test these cases

[ ] Lexical
[ ] Global
[ ] Field of type

Pull Request Test Coverage Report for Build 390211135

174 of 252 (69.05%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 81.047%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

goalc/compiler/Val.cpp
20
23
86.96%

goalc/compiler/compilation/Math.cpp
28
33
84.85%

common/type_system/TypeSystem.cpp
18
28
64.29%

common/type_system/Type.cpp
9
28
32.14%

common/type_system/deftype.cpp
48
89
53.93%

Totals

Change from base Build 390083062:
-0.1%

Covered Lines:
17075

Relevant Lines:
21068

💛  - Coveralls

md5 checksums of files, before I mess with the compiler a bunch:
770dee114b4360da85c78da50b831e1b  ./GAME.CGO
a5e40b0360510dbc609ccb9495ed0c85  ./KERNEL.CGO

Scary changes made to set in https://github.com/water111/jak-project/pull/146/commits/27b80ac23ea39fde7c58971ea456fe3ed511a38f
The files have the same checksum, so it should have no effect on existing behavior.


extensions ignored from workspace recommendations can't be retrieved anymore

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.81.1
OS Version: Linux x64 6.4.0-3-amd64

Steps to Reproduce:

have a workspace file open with extension recommendations in it
right-click on some recommended extension and choose "ignore recommendation"
there is no way to retrieve those ignored extensions ever again

/duplicate of #186690
Workarounds in #144324 involve using a SQLite editor tool.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Click not working when you change the current month

When you select a different month, the click event on the calendar is not working.

Fixed with 0.1.2 release


Use DockerUI as private registry frontend?

The containers are visible
but would it be good enough to use as private registry frontend in production atm?

DockerUI talks to a single Docker daemon, it doesn't have any registry frontend features. The most popular registry frontend that I know of is https://github.com/kwk/docker-registry-frontend


Question: long-running WASM + Erlang scheduler

Hey @bhelx  amazing work!
Erlang scheduler generally doesn't like long-running processes.
For native Erlang / Elixir processes, it uses reductions to preempt so that other processes have fair chance of consuming CPU slice.
How would the scheduler react to WASM running through Rustler?
What advise do you have when someone needs to run potentially long-running WASM modules?

Thanks @munjalpatel! Could you link to any specific docs about what you are referring to? I haven't had this problem but maybe I'm misinterpreting your question.

@bhelx here are some references:
https://youtu.be/JvBT4XBdoUE?t=411
Quote from: https://blog.appsignal.com/2024/04/23/deep-diving-into-the-erlang-scheduler.html

To promote fairness among processes, Erlang's preemptive scheduling relies on reductions rather than time slices. If a process exhausts its allocated reductions, it can be preempted, even if its execution isn't complete. This approach prevents a single process from monopolizing the CPU for an extended period, fostering fairness among concurrent processes. By using reductions as the foundation for preemption, Erlang mitigates the risk of processes starving for CPU time. This design ensures that every process, irrespective of its workload, is periodically allowed to execute.

@munjalpatel correct. Given that this library can't predict how long each WASM call will take to execute, it should at the very least make it into a DirtyCpu NIF so that it runs on the dirty schedulers.

Thanks for all this info @munjalpatel and @scrogson ! I hadn't considered that, but it makes sense. Our bindings using rustler are fairly naive.
@munjalpatel were you interested in contributing this? If not i can start by looking into this "DirtyCpu NIF" that @scrogson mentioned.
Another option could be just re-writing the underlying code to use wasmex and getting rid of the custom nif. I have explored. this but not started or done a proof of concept: https://github.com/extism/elixir-sdk/issues/3

@bhelx happy to help -- I am quite familiar with Elixir but have never worked in Rust before. But can certainly figure things out with guidance :)
I would much rather use wasmex instead of going DirtyCpu route for the following reasons:

With DirtyCpu, its an improvement, but we will still be limited by relatively low number of scheduler threads. This may be fine for some use cases, but use cases where we need to run 1000s of user provided WASM, this is kind of no-go.
@tessi has already done hard work of running WASM modules in an OS thread from Rust in wasmex
( https://github.com/tessi/wasmex/pull/7 )
There is also WIP for ideal solution using Rust's async / await ( https://github.com/tessi/wasmex/pull/462 )

So by using wasmex, we will inherit all these + future improvements :)

@munjalpatel that would be awesome! No pressure of course :) @tessi has done a great job on wasmex and is way ahead of my bindings. I only haven't switched over due to lack of time. If you come find me in our Discord in the #elixir-sdk channel. I can give you some real time advice or even do some pair programming if you feel like it would help.

@munjalpatel that would be awesome! No pressure of course :) @tessi has done a great job on wasmex and is way ahead of my bindings. I only haven't switched over due to lack of time. If you come find me in our Discord in the #elixir-sdk channel. I can give you some real time advice or even do some pair programming if you feel like it would help.

A bit busy this week. But let's get some time on our calendars for the next week and we can figure out the plan ahead.
What timezone are you in?

No problem, I can always find a way to squeeze in some time. I'm in the US in Central Time. In terms of UTC, I'm generally active from 11:00 UTC to 01:00 UTC. Just ping me on Discord, same username as Github.

@tessi thanks for the update on the project. No pressure for you to help of course. I think we should be able to mostly do it on our own. I think either way, we're faced with the decision to build all the stuff you have built ourselves, or join forces. The latter is the obvious choice.

Usually good improvements, but it's still work to keep up. :)

Yeah we are downstream of wasmtime for our standard fallback libextism dependency, so we know the project well!

That being said, I'd love to see people migrate to wasmex and am happy to onboard additional contributors to it. together we have more time and dedication than us alone.

Agreed, there's no reason you should need to shoulder all the burden. Happy to contribute to wasmex where we can if we can pull this off. Also perhaps we can attract some more contributors too to help you out. We have a few users of this library who might be able and willing to help out.


xrdb: Use mcpp as the preprocessor (fixes #9480)

By default, xrdb uses GCC as the preprocessor at runtime for X resource files.
However, gcc is a large dependency, so replace it with mcpp, a much smaller
preprocessor (currently under a megabyte on i686).
Arch Linux already does this as well, so this should be relatively safe:
https://projects.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/xorg-xrdb
I tested this by copying https://github.com/solarized/xresources/blob/master/Xresources.dark as ~/.Xresources and lauching a xterm, which came up with the dark theme as expected.

Sounds nice, thanks!

Please note that there are differences between mcpp and cpp regarding #includes.  See https://bugs.archlinux.org/task/46934.

cpp itself isn't large (~800 kB) and it only needs libc for dynamic linking. However, it still retains references to other directories from gcc. Maybe it doesn't really need to call the executables in them, but the references would have to purged at least.


Every feature using delegate and deduplicate is broken on history navigation

Description
Here's what happens:

We create the DOM elements and attach the delegated listeners
The user navigates away
We remove the listeners
The user goes back
init is not run again because it's "deduplicated"

At this point, the elements still exist, but they have no listeners.
How to replicate the issue

From this issue
Click the "Issues" tab
Go back
Click the collapsible-content-button

I'm fixing this feature specifically, but likely all of them are.
Related:

#5869

Extension version
main
Browser(s) used
Chrome

https://github.com/refined-github/refined-github/pull/5873 took care of delegate-only features. Features with "change dom + delegate" in their init are still broken, they will need changes like:

https://github.com/refined-github/refined-github/pull/5872
custom deduplication detection

This is now a sub-issue of #5895, but just for features that use delegate:

https://github.com/refined-github/refined-github/issues/5895

Starter:
rg --files-without-match "deduplicate: false" (rg -l "delegate\(" source/features/*.tsx) | sort -R | xargs code

Finds every delegate feature  without deduplicate:false and opens it in vscode in random order.
I'll start on some

Not many features left to close this issue ☺
❯ rg -l deduplicate $(rg -l delegate source/features/*.tsx) | sort -R
source/features/copy-file.tsx
source/features/one-click-diff-options.tsx
source/features/one-click-review-submission.tsx
source/features/convert-release-to-draft.tsx
source/features/open-all-conversations.tsx
source/features/quick-mention.tsx
source/features/rgh-sponsor-button.tsx
source/features/rgh-welcome-issue.tsx
source/features/hide-low-quality-comments.tsx
source/features/deep-reblame.tsx
source/features/esc-to-cancel.tsx
source/features/comment-fields-keyboard-shortcuts.tsx
source/features/new-repo-disable-projects-and-wikis.tsx
source/features/pr-filters.tsx


Example Piece: On-chain light client

I just got back from PBA3 where I gave students an assignment to write a smart contract that works as on on-chain light client for another chain. Basically a btc-relay like thing. We were using solidity and ink! so of course the solutions were all account based. Got me thinking about how to implement the same thing in UTXOs. I think each foreign block header could be a separate utxo.

For some details about what the project would require, see: https://github.com/JoshOrndorff/archive-bridge-assignment


Implement BeirMultifieldCollection

WIP; will update when ready for review.

This is ready for review now.

BeirMultifieldCollection indexes title and text in separate fields.
New option in SearchCollection allows user to specify searching across fields with weights.
Working regressions for arguana and climate-fever.
Associated test cases, etc.


Update README.MD

Please review

This PR has no meaningful code and introduces junk files.

This was a demo PR to show DTSE the usage of PR in GitHub


Bump gem.add_dependency for activemodel && activesupport to < 7.2

@Fivell  Please release new gem reflecting new changes as well.  Thanks in advance!


implementation wrong

There are some implementation wrong.
Point out that the place to put the condition is different.
Add condition to output after convolution such like,
Unofficial(this repository)

https://github.com/crcrpar/pytorch.sngan_projection/blob/master/models/discriminators/snresnet.py#L92
https://github.com/crcrpar/pytorch.sngan_projection/blob/master/models/discriminators/snresnet64.py#L92

Official

https://github.com/pfnet-research/sngan_projection/blob/master/dis_models/snresnet_32.py#L29
https://github.com/pfnet-research/sngan_projection/blob/master/dis_models/snresnet_256.py#L36

thanks.

Thanks for reporting again :)

Ah, #24 would report the almost same thing.
So I labeled this duplicate.


Fixes #2369 - Increase default --task_launch_confirm_timeout

You still need to change the documentation. command-line-flags.md


What is seeds?

Hi there :-)
in your examples I noticed you used seeds sometimes:
// p5.brush variables
let seeds = [];

// setup
// p5.brush settings
for (let i = 0; i < 150; i++) seeds.push(random());

// draw
randomSeed(33213 * seeds[67]);
brush.circle(155, 140, 50);

Can you explain a little what this is and what it does?
Thank you for this great library!
Nahuel
edit: I am especially curious how you picked the numbers, eg. 150, 33213, 67. Is there some logic behind this?

Hi @NahuelGerthVK
You can read about an equivalent seed function here: https://p5js.org/reference/#/p5/randomSeed


Add workaround for tox errors during env setup

Summary of changes
This is a workaround for #4031.

Add a cap for pyproject-api<1.6.0' in Github Actions.

Closes
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in newsfragments/.
(See documentation for details)

This might no longer be necessary with tox==4.11.


Command prefix

Second part of PR #21 .
This introduces a [linter]_command_prefix to the various linters. I rewrote the npmBinRun method to instead use prefixes, to allow for overriding that prefix if people need to for whatever reason.
I didn't add any tests for this yet because I want to validate that this is the right approach. I also don't have most of these linters installed, so it'd be an investment I want to wait for (or maybe get help from someone else to do).
Let me know what you think!

@samuelmeuli did you get a chance to look at this?

@samuelmeuli I've made (I think) all changes as discussed. Please take another look and see if you can figure out how to get the tests to pass :)

Nice work @dorner, thanks a lot! :)
I've simplified the way the NPM/Yarn command prefix (yarn run --silent / npx --no-install) is retrieved so it doesn't depend on any user input. This also fixes the tests.

Awesome - thanks!


Support RAML

https://github.com/raml-org/raml-spec

example
#%RAML 1.0
title: Hello world # required title

/greeting: # optional resource
  get: # HTTP method declaration
    responses: # declare a response
      200: # HTTP status code
        body: # declare content of response
          application/json: # media type
            # structural definition of a response (schema or type)
            type: object
            properties:
              message: string
            example: # example how a response looks like
              message: "Hello world"

Test

Spec


iOS swipe-to-go-back

Issue #64 and Issue #44 have been closed as "wontfix", with the stated reason being that the problem is within flutter (https://github.com/flutter/flutter/issues/47441).
However, the flutter issue actually does describe two different workarounds. It seems that one of these could be applied to this package to give it the requested functionality. Any chance of that happening?

Thank you for adding that @kalismeras61 !!
I tested with v2.0.8 and swipe-back works as described with the rightToLeft and fade animations. :)
But there appears to be a slight additional problem with this new feature... When isIos=true, the fade animation doesn't do a fade. Instead it does a rightToLeft.

Is it possible to make this also work with PageTransitionType.scale?

@kalismeras61 I tested PageTransitionType.scale in 2.0.9 with isIos=true. It works, but not in the way I would expect. It has the same issue as fade.
When isIos=false, fade and scale transitions occur directly into the currently displayed pane.
When isIos=true, fade and scale transitions cause a visible right-to-left transition of a new pane (same as you get when configured for right-to-left). The fade/scale occur into this new pane.
So while the fade and scale transitions work (i.e. they function and don't break anything), they behave oddly. I would expect them to occur "in place" without seeing the underlying right-to-left transition.
If possible, I would prefer to not see the right-to-left transition that occurs "underneath". If that's not possible, the new isIos feature is still useful, but the transitions are much less visually appealing than when isIos=false.

I have used isIos=true with rightToLeft animation and it is working fine but not working when I add WillPopScope it not calling onWillPop method.

I have used isIos=true with rightToLeft animation and it is working fine but not working when I add WillPopScope it not calling onWillPop method.

I think this an actively debated flutter issue: Read more here https://github.com/flutter/flutter/issues/14203

is there any possibility to have the iOS swipe back with leftToRight?


Changed inputDigits in First_base_is_one test in AllYourBaseTest

Changed inputDigits in First_base_is_one test to be an array with values, not an empty array. Since the test for throwing an exception when inputDigits is empty is before this one, this test is redundant when inputDigits is empty. Assuming the intention is to test that an exception is thrown when the first base is one, this should fix that so that the correct thing is tested.

Once more, fantastic work! Thanks a lot. :tada:

@exercism/csharp xjava is struggling with this same test case right now. The change you made here seems like it only shifts ambiguity regarding input validation ordering, rather than removing that ambiguity, since
const int inputBase = 1;
var inputDigits = new[] { 1, 0, 1, 0, 1, 0 };

could fail with either

an invalid base message (the error expected by the test); or
an invalid positive digit message (since the largest digit in the input digits is equal to the base).

In fact the only valid input digits for input base 1 would appear to be {0}, which you could argue is no longer an erroneous pairing of inputs (if you treat this as a representation of zero).
Anything I'm missing? My inclination at this time is to reorder the tests in x-common so that validation occurs in parameter order (input base -> input digits -> output digits seems to be the natural ordering here). What would you all think of that as a change?

@stkent I think re-ordering would be the best option, and I quite like the ordering you suggest.


Add support for typedef

Adds support for typedef in zap, supporting things like defining a VideoStreamID as a uint16, see the camera spec: https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/10004
See also Matter SDK template changes @ https://github.com/project-chip/connectedhomeip/pull/36124
This should also eliminate the need for many of the things in chip-types.xml, which can now be simply defined in the XML files as a simple typedef
See also discussion with @bzbarsky-apple: https://github.com/project-chip/connectedhomeip/pull/35773#discussion_r1775652745

@brdandu or @tecimovic could I get one of you to review? Thank you!

I am wondering if this is very similar to base type implementation in https://github.com/project-chip/zap/pull/1472 . Do you think that can work the same way as your implementation?

I am wondering if this is very similar to base type implementation in #1472 . Do you think that can work the same way as your implementation?

This seems to be a superset of that - in fact, this new typedef should probably supercede many of the base types - for example, in src/app/zap-templates/zcl/data-model/chip/chip-types.xml, cluster_id, field_id could instead use  the typedef here
Not only does this PR allow for block helpers, it also allows for dynamic specifying of simple types alongside structs/etc. instead of having to have a definition in chip-types.xml as well as the code definitions - it's a lot more elegant

The atomic identifier won't be associated with these types. Is that fine:

I think so? I think we'd want to move some of the types currently defined in chip-types.xml to just be simple typedefs, so that it's not split between XML + code; but I may not be understanding what the detriment would be of not being associated with the atomic identifier

So when type def comes in then does it also mean it will be a typedef and something else?

Assuming I understand your question: typedef is treated exactly like enum & struct, i.e. the name is added to its own SQL table & it is checked for uniqueness against all types

do our existing helpers need to take that into account?

Yes, although in general there are shared functions - for example, when we lookup the type to output in code, we just do a redirect to the underlying type

For reference, corresponding chip repo PR: https://github.com/project-chip/connectedhomeip/pull/36124

I think so? I think we'd want to move some of the types currently defined in chip-types.xml to just be simple typedefs, so that it's not split between XML + code; but I may not be understanding what the detriment would be of not being associated with the atomic identifier
I could be wrong here but I believe we pass the atomic identifier in our zcl read and write attribute calls if I am not wrong. It is worth checking that our generated code continues to extract the right atomic identifier here when the typedef is introduced to the attribute types and this continues to work the same way:
Eg for Test: Turn attribute type to a type def in xml. Then in you built application try to read and write this attribute to make sure this is working.

Assuming I understand your question: typedef is treated exactly like enum & struct, i.e. the name is added to its own SQL table & it is checked for uniqueness against all types
In our templates we very frequently pass the attribute type to a bunch of helpers for processing. What happens to the those helpers when the typedef is passed?
Eg for test: Turn attribute type to a type def in xml. Now in the .zapt templates look for helpers like as_underlying_zcl_type/asUnderlyingZclType or as_underlying_type/asUnderlyingType which takes in the attribute type and gives a certain type for it. Check if this is behaving appropriately for typedefs

Yes, although in general there are shared functions - for example, when we lookup the type to output in code, we just do a redirect to the underlying type
Can this be verified by adding more unit tests here based on the above example?

I added tests such that they are 1:1 with the enum & struct tests
Unfortunately, the tests are somewhat split between here & the matter SDK repo
Here, the tests just check for the basic functionality: i.e. if we find a typedef in a template, does it generate the right output?
Those tests here are updated to handle typedefs & pass
Then, in the Matter SDK repo, there are numerous other tests around the zap templates there (including the compiler itself - i.e. if the generated files compile)
Those tests are not yet complete, but are more testing the templates rather than ZAP itself (I am working on those now, though some are complete)


feat: delete old videos Directory

Description

Delete all the videos and folders of the old app

fix: LEARNER-9950

Test Cases are still failing ...

Test cases are failing due to context, the issue is already addressed in PR-308. I'll update and merge it once PR-308 is merged.


adding greedy algorithms in python (Kruskals,Prims and Dijkstra)

Is your feature request related to a problem? Please describe.
Hey @rish-singhal I would like to add all major Greedy algorithms like Kruskals Algorithm, Prims Algorithm, Dijkstra's Algorithm, Fractional Knapsack, Huffman Coding in python language
Do you want to work on it
Yes
I will implement this in python language
Kindly Assign me this under GSSOC 21
Thank you

hey @rish-singhal   assign  me this issue if its available


Feature/issue 636 fix rtl issue

AC:

[ ] when importing locale actions use import * as actions
[ ] import actions directly from actions.ts - do not use import/export
[ ] Remove helper functions from locale reducer
[ ] Failure instead of Error - be consistent in the naming
[ ] “Current” is redundant -
[ ] LoadCurrentLocaleResult not needed
[ ] Use term Save instead of Set
[ ] Tests - remove messages, loading variables from actions.ts - the messages are alwayys empty and loading flag is always false
[ ] Remove loading property - always false
[ ] Use linter


Failed to upload iOS release

Hi, i'm using github action to upload iOS and Android to Firebase app distribution and got this error

Source code:
- name: Upload build file
      uses: nickwph/firebase-app-distribution-action@v1
      with:
        app: ${{ secrets.FB_APP_ID }}
        credentials: ${{ secrets.CREDENTIAL_FILE_CONTENT }}
        groups: subwallet-team
        file: output.ipa

Output:

Pls, help me solving this, if you need further information, just ask me. Many thanks!

It seems more like an issue with you IPA file. Maybe you could set debug: true to see what's the detailed error message first? Also you could try to make sure you can upload using firebase-tools.

Thanks! Turns out i was distributed my app with android app id. Silly me :(

Hi @nickwph, my app was uploaded perfectly from my last comment until yesterday.
I think there is a issue with Firebase or your library is out of date with Firebase. I haven't change any configuration from last success build, here is the workflow code https://github.com/Koniverse/SubWallet-Mobile/actions/runs/7126526446/workflow
This is error log

Hello @nguyenduythuc, seems like firebase tools updated that it needs NodeJS version 18 or above. I believe your project is using older version of NodeJS. I suggest you update your project to switch to this version before running the distirbution script: https://github.com/marketplace/actions/setup-node-js-environment
- uses: actions/setup-node@v4
  with:
    node-version: '18.x'

At the mean time let me see what i could do to do the same from my script.

Thanks @nickwph! It worked.


Make incubator notice optional

Resolves: https://github.com/apache/incubator-pekko-sbt-paradox/issues/115
Trying this with pekko core, this is what it looks like with this change (there should be no difference in rendered text)

And when setting pekkoParadoxIncubatorNotice to None

This is what it looks like now which is a lot better

Merging

This is what it looks like now which is a lot better

I also agree with this approach.


negative tests

#5462
#closes 5462
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[ ] C++ Guidelines
[ ] Doxygen docs
[ ] Unit tests
[ ] No unwanted commits/changes
[ ] Descriptive title/description

[ ] PR is single purpose
[ ] Related issue listed

[ ] Comments in source
[ ] No typos
[ ] Update changelog
[ ] Not work-in-progress
[ ] External references or docs updated
[ ] Self review of PR done
[ ] Any breaking changes?

/azp run cpp - tables

/azp run cpp - tables

/azp run cpp - tables - ci


All organizations are visible when adding manufacturer / publisher

In GitLab by @jasonpagetas on Jul 15, 2024, 22:54
When creating a new manufacturer / publisher all organisations are displayed in the organisation dropdown

added 1m of time spent

mentioned in commit 7c62309a31fe2d7b91d584dc06b2a590d3074d95

In GitLab by @jasonpagetas on Jul 15, 2024, 22:54
marked this issue as related to #74


Upload player projections

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

👷 Deploy Preview for betsweatwin processing.

Name
Link

🔨 Latest commit
2612278029eeb6eae9e47306817bd57ec7cdb171

🔍 Latest deploy log
https://app.netlify.com/sites/betsweatwin/deploys/6557a574ccfaab0009253a63


Fix followers and following tabs in profile

Fixes #10195
Screenshot:

Codecov Report

Merging #10202 into master will increase coverage by 0.21%.
The diff coverage is 4%.

@@            Coverage Diff             @@
##           master   #10202      +/-   ##
==========================================
+ Coverage   43.42%   43.63%   +0.21%
==========================================
  Files         576      576
  Lines       79650    79682      +32
==========================================
+ Hits        34586    34768     +182
+ Misses      40788    40624     -164
- Partials     4276     4290      +14

Impacted Files
Coverage Δ

routers/user/profile.go
50.69% <0%> (-0.97%)
:arrow_down:

routers/routes/routes.go
86.47% <100%> (-0.05%)
:arrow_down:

models/unit.go
37.03% <0%> (-2.47%)
:arrow_down:

services/pull/pull.go
32.43% <0%> (-1.51%)
:arrow_down:

models/issue_label.go
63.79% <0%> (-0.69%)
:arrow_down:

models/gpg_key.go
54.81% <0%> (-0.56%)
:arrow_down:

routers/api/v1/repo/label.go
84% <0%> (+84%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74a4a1e...042db92. Read the comment docs.

@lafriks why is this a fix and not a normal enhancement?
for me it looks like a ui restructure

I would make a backport fix -> witch simply add followers and following to reservedUsernames in user.go

No need to add them to reserved usernames and that would not help anyway as problem is repository name

I can of course backport this to 1.11 if needed


Sampler Binding

What do you think about having the sampler as an optional parameter to Texture.use(...)?
Currently:
sampler.use(location=0)
texture.use(location=0)

Sampler parameter:
texture.use(location=0, sampler=sampler)

It may give the impression that a sampler belongs to the bound texture and not the texture unit as a whole? .. or we assume people know the sampler will stick to the unit?
@cprogrammer1994 response:
Is it common to create a few samplers and use with multiple textures?
What do you think, should samplers be created like vao's? (Vaos store a program object. When you call vao.render it will bind the program object that vao internally knows. Sampler.use could bind the texture to a texture unit)
I think it could be more convenient to have samplers as the only option to bind a texture to a texture unit (in the next major). Textures would store the texture's data only. Samplers would be responsible for handling the sampler parameters and binding the texture to its unit when requested.
This could ensure not to have invalid sampler, texture pairs bound.
This could also ensure that build_mipmaps cannot affect the filter parameters.
This would force the user to create similar samplers for different textures.

I will have to think about this a bit more, but it doesn't sound bad if the goal is the ensure samplers do not mess up for the user.
# Create sampler for a texture
sampler = ctx.sampler(texture, ... sampler params ...)

# Bind the sampler and texture
sampler.use(location=N)

An advantage is also that we can check the texture type and properties to ensure no invalid sampler parameters are set causing incompleteness issues. https://www.khronos.org/opengl/wiki/Texture#Texture_completeness
# Would not be allowed if the texture don't have mimpaps
sampler = ctx.sampler(texture, filter=(moderngl.LINEAR_MIPMAP_LINEAR, moderngl.LINEAR)))

# Allowed
texture.build_mipmaps()
sampler = ctx.sampler(texture, filter=(moderngl.LINEAR_MIPMAP_LINEAR, moderngl.LINEAR))

I would at least make the texture property (no setter) available from the sampler instance since passing around two instances is quite the burden.
sampler = ctx.sampler(texture, ... params ...)
sampler.texture.build_mipmaps()
sampler.filter = (moderngl.LINEAR_MIPMAP_LINEAR, moderngl.LINEAR)

This also means we can remove all the sampler parameters from the texture classes.

It is hard for me to make a decision on this texture.use(location=0, sampler=sampler) call.
using texture at location 0 with sampler is clearly understandable.
sampler=None would mean to unset the sampler.
sampler.use will not be needed in that case.
this would break the original idea:
sampler.use()
texture.use() # the default parameters would be location=0, sampler=None

I think you can freely decide to add or not to add this parameter to texture.use. I personally prefer to try them out and base the final decision on them later.

I think just keeping them separate for now is fine. Then we can play with them more and see what makes more sense.

After some more pondering..
We should not forget that this is an important use case for samplers:
sampler_repeat = ctx.sampler(repeat_x=True, repeat_y=True)
sampler_no_repeat = ctx.sampler(repeat_x=False, repeat_y=False)
texture = ctx.texture(...)

sampler_repeat.use(location=0)
texture.bind(location=0)

sampler_no_repeat.use(location=1)
texture.bind(location=1)

Having the ability to bind the same texture to multiple texture units with different samplers is the key. It's used in some rendering techniques.

I think the best would be to use samplers similar to vaos and textures similar to buffers.
Textures only hold data. Samplers hold texture references.

Got some simple code explaining what you mean here?
I assume you mean something like:
texture = ctx.texture(...)
sampler = ctx.sampler(texture, *sampler_params)

# Will bind sampler and texture
sampler.bind(location=0)

# What happens with the previously bound sampler here? I'm guessing the binding clears?
texture.bind(location=1)

... so that a sampler will only affect the texture it references. When needing multiple ways to filter a texture we make multiple samplers referencing the same texture.
It means samplers are not really something you set on a texture channel. I guess this is actually more user friendly (and more performant reducing calls) and you don't end up with the awkward Context.clear_samplers(..). In the vast majority of cases the samplers used during rendering will be more or less static, bit it could be a bit more cumbersome in certain instances as well.. but probably worth the sacrifice if it means using samplers is much more user friendly by not occupying the entire texture channel globally.

Here is a simple example:
https://github.com/cprogrammer1994/ModernGL/blob/master/examples/experimental/crate.py#L58-L65
this example adds a sampler to a scope, but it could be bound manually with sampler.use(location=0)

Here is the implementation of sampler.use:
https://github.com/cprogrammer1994/ModernGL/blob/501fa6421ff8d5b1c7485a87992764f53c897b63/moderngl/experimental/mgl/sampler.cpp#L41-L48
https://github.com/cprogrammer1994/ModernGL/blob/501fa6421ff8d5b1c7485a87992764f53c897b63/moderngl/experimental/mgl/context.cpp#L274-L282

What happens with the previously bound sampler here? I'm guessing the binding clears?

both the texture and the sampler on location=0 will be replaced with the new sampler and its texture

Looks great 👍
Had to browse around looking at scope, contex and whatnot to see that whole picture. I really like how this all works together with the vao scope.
Close this issue I guess?


[OPT] Add SPV_KHR_ray_tracing_position_fetch to allow lists

Fixes https://github.com/microsoft/DirectXShaderCompiler/issues/6844

No tests to make sure optimizations are done?

I've stopped adding tests when we add something new to the allow lists. We are not testing new code paths, just new data.
I don't find those tests useful.


kh2 save anywhere does not work

i don't know if it works differently now since the textures update, but before when i would do L2+R2+R1 my hp would go down to 1, and the save button for triangle would pop up. i have all my scripts correctly installed (including the io_packages) and it is no longer working now.

Closing duplicate