the historical sanifix3.py python script [feature request]

Hello,
In the rdkit-users mailing list, this script was mentioned as a way for people to
correct molecules so that they will pass Chem.SanitizeMol without throwing
an exception.
It would be nice if someone can find out the corresponding script
and place a copy in the Contrib directory.
Alternatively, we could also have Chem.SanitizeMol having a flag
saying that if SanitizeMol fails, rdkit will try to salvage the molecule...
There could be a list of flags saying which operations the user allows
while trying to salvage a molecule.
Regards,
F.

Interestingly, it seems that Chem.Sanitize on an RWMol fails more often than
on a molecule constructed from the SMILES gotten out of the same RWMol ?!
So, it seems that rdkit SMILES conversion did some repairs on some molecules.

Interestingly, it seems that Chem.Sanitize on an RWMol fails more often than
on a molecule constructed from the SMILES gotten out of the same RWMol ?!
So, it seems that rdkit SMILES conversion did some repairs on some molecules.

I can ask on the mailing-list if I really want that script.


Sidekiq5 UI menu missing

Using this lib in two projects, with sidekiq4 everything is well, but for sidekiq5 UI's it doesn't add any UI elements.

Hi @sveredyuk
Which versions are you using?
I tested it with sidekiq-scheduler v2.1.7, and v2.1.10, sidekiq v5.0.4 and v5.0.5.
Here's config.ru file:
require 'sidekiq/web'
require 'sidekiq-scheduler/web'

run Sidekiq::Web

Hi, @snmgian. Already found an issue on project side. Thx for attention and sorry for disturb.


Optimize Snapshot persistence in KafkaFlow Snapshot

LGTM

LGTM

@dfahritdinov I have added some random comments, but feel free to merge. It looks good to me too.

@dfahritdinov I have added some random comments, but feel free to merge. It looks good to me too.


Add new schemas from DMa local repo: store, times, hours

What are the schemas that are affected by the issue
new schemas as below
-store
-times
-hours
What are examples of products that are impacted by the issue
platform xdm definition

Close this. @cmathis @cdegroot-adobe


Image Not Display in Center

Summary
I have displayed a video with an epub file in landscape and portrait mode. But not display image proper(in center) in both the mode. I am attaching below snap shot

What platform(s) does this occur on?
Android, iOS
Environment
Getting same issue in for Environment
Minimal reproducible example
I have displayed a video with an epub file in landscape and portrait mode. But not display image proper(in center) in both the mode. I am attaching below snap shot

We are using atEnd and atStart property for disable the next, previous button. but we are not getting proper value. so please check and resolve asap.

Hello! You can provide this epub file?

We have attached a zip file (Epub is not supported). Please check the attached epub file in landscape mode with a 50% width.

EBOOK_19a428d3-73b3-4260-af96-a7fb19862367.zip


[Bug]

Describe the Bug
when going to a channel with the discord buttons feature, it gives the "well this is awkward" error
To Reproduce
go to any channel with a button thing
Expected Behavior
no crashing
Screenshots

Discord Version
Stable
Additional Context

closing issue, since i just found out that its caused by betterbottags


Jcobian J_f(x), page 432, before 13.23

Why did we use the transpose of x in the denominator?

It's explained here:
https://en.wikipedia.org/wiki/Matrix_calculus#:~:text=Layout conventions[edit]
However I don't use this consistently, so will remove it.


Add Create, Update, Delete functionality for HBase

Create: It's possible to create tables and insert rows.
Update: Insert and update are the same in HBase.
Delete: It's possible to drop a table and delete rows based on the rowKey.

I like what this PR does, but it seems a bit one-off to me. It's nice to see us supporting on-the-fly column definitions during insert, but the same principle could also apply for other connectors we have, such as the ones for CouchDB, MongoDB, ElasticSearch and more. So since the approach is a bit bespoke for HBase, and we don't know if there will be some consolidation later if we end up implementing something a bit more generic for these types of datastores, I think for now we should add the @InterfaceStability.Evolving annotation to the HBaseDataContext. What do you think about that?

However, the Travis build seems to have failed :-)

Oh I think that's a glitch. I've seen that test fail on rare occasions before.


[TEVA-1639] Remove trigger on staging branch

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1639
Changes in this PR:

Removed GitHub actions workflow trigger on pushes to the staging branch, as the deploy workflow also deploys to the staging environment as preparation for deploying to production.

Looks good - can you also change the docs here though:
https://github.com/DFE-Digital/teaching-vacancies/blob/550e875dda49986bd82d4c01421c80d90d45e414/documentation/hosting.md#deploy-to-dev-or-staging-via-a-commit-to-github
Might be worth us documenting more explicitly in that doc file what the purpose of dev and staging are as well?


Add instructions to publish to NPM

Follow this template: https://github.com/dgraph-io/dgraph4j/blob/master/PUBLISHING.md

Updated in master.


issue while opening the cdm

C:\Temp\covid-vaccine-booking-main>python src\covid-vaccine-slot-booking.py
Traceback (most recent call last):
File "C:\Temp\covid-vaccine-booking-main\src\covid-vaccine-slot-booking.py", line 5, in
import requests, sys, argparse, os, datetime
ModuleNotFoundError: No module named 'requests'

You forgot to download requirements module
Pls run pip install -r requirements.txt
If it shows errors like pip not found then you have to reinstall python with add to path check box ticked
Or
Run pip install requests

cowin now blocking mobile nos  for 24 hours and may block the mobile no permanently as per cowin helpdesk

@dbn1972 Blocking on doing what?


python311Packages.sensor-state-data: 2.17.1 -> 2.18.0

Diff: https://github.com/Bluetooth-Devices/sensor-state-data/compare/refs/tags/v2.17.1...v2.18.0
Changelog: https://github.com/Bluetooth-Devices/sensor-state-data/releases/tag/v2.18.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.


Revert "e2fsprogs: build fuse2fs on darwin"

Description of changes
This change, while fine in isolation, breaks evaluation in combination with https://github.com/NixOS/nixpkgs/pull/329721, as xar depends on e2fsprogs which now depends on macfuse-stubs which depends on xar. This broke staging-next.
A couple possible solutions are to disable the e2fsprogs dependency in the version of xar used for the bootstrap, or to build macfuse-stubs from source to avoid the xar dependency.
This reverts commit 0dfc820f4a915a979ccb4ba420a8fcad48345170.
cc @KoHcoJlb @7c6f434c @tie @reckenrode
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It seems like that PR hasn’t actually made it to staging-next, so even though this will cause issues, we don’t yet know what the report of an infinite recursion error that looked very much like this is about yet.

Okay, the commit landed in staging-next now and broke as expected.

Or xar could use e2fsprogs with fuse disabled
macfuse went closed-source awhile back, so It's not possible to build it from source

No, the relevant part of macFUSE is LGPL, as noted in our package definition for the stubs; here’s the repository. If it was non‐free the PR would have been a bigger issue as it would have made e2fsprogs non‐free by default :)
xar overriding e2fsprogs to disable FUSE like that seems reasonable, since it presumably only uses the library. @reckenrode pointed out that macFUSE could also just use a xar without ext2 support to unpack the .pkg and that would also break the loop enough to bring your PR back in. I’m not sure if we’d want to pull the macFUSE stubs into the stdenv bootstrap, though. If we do, we should probably build it from source, at least.
Sorry about the quick revert, but we needed to fix eval on staging-next rather than getting blocked on this. I’m happy to help get your PR back in if we can agree on what the best way to handle this is.

Regardless of the macFUSE issue, the stdenv bootstrap probably shouldn’t include e2fsprogs. It’s fine for now, but I’ll probably include in the Darwin refactor using a stripped down xar for cctools and ld64.
One of my goals for the bootstrap is reducing the amount of unnecessary stuff that’s built (e.g., 24.05 builds Python five times during the bootstrap, but the refactor builds it only once). Using a xarMinimal is in line with that. I wonder if xarMinimal be a top-level package?

We’ve circled back around to https://github.com/NixOS/nixpkgs/pull/329721#discussion_r1712951994 here. I agree that we should just gate the problematic dependencies behind a single feature flag and use that for the compiler tools.

Did you agree on preferred solution?

Is there anything I can help with? Just wondering how I can get my PR back)

So the problem is that because of the xar change, e2fsprogs is now an indirect dependency of ld64. Any change to e2fsprogs would rebuild the entire world on Darwin, and therefore needs to go to staging. @reckenrode plans to make ld64 use a stripped‐down xar in his upcoming SDK rework. That will happen for the next staging cycle in a couple of weeks, so unfortunately there’s nothing we can really do to speed that up.
The only thing I can think that we could do in the interim is to add a default‐on flag to e2fsprogs conditionalizing the FUSE stuff, and make sure it’s turned off by the xar in the stdenv bootstrap. If done carefully this will ensure that there are no mass rebuilds, but it will be somewhat ugly and get blown away by the next staging cycle.
Work that would be good for the future but wouldn’t immediately solve this would be to build the macFUSE stubs from source, perhaps even reusing the same derivation that is used for libfuse on Linux, to remove the xar dependency.
Sorry about this unfortunate collision of changes…

Ah, I see now, because stdenv now depends on xar/e2fsprogs any changes to them would cause mass rebuild thus need to go to staging.
So after stdenv is decoupled from e2fsprogs should I resubmit my original PR?
Also, if I'm not missing anything, original loop would remain (e2fsprogs -> macfuse-stubs -> xar -> e2fsprogs) which would need to be fixed separately by using xarMinimal in macfuse-stubs or building them from source.
Is this something taken care of as part of xarMinimal refactor or should I submit PR for this as well?
In the meantime I'll try to look into building macfuse-stubs from source, maybe I will submit PR for that too.
By the way will changes to macfuse-stubs cause mass rebuild?

Yes, it should be fine to revert the revert after the xarMinimal stuff goes in. macfuse-stubs should be able to use xarMinimal, since it’s just unpacking a .xar archive; that can go in the revert‐revert PR. A from‐source build would be ideal, of course, but shouldn’t be required.
I think that we don’t want macfuse-stubs to be part of the bootstrap, so hopefully it won’t cause mass rebuilds after the SDK rework.

Status update on the xarMinimal stuff. I’m working to get the draft PR for the Darwin rework up sometime tonight. It will include xarMinimal along with an update of xar to 501, which adds a new API (and then immediately deprecates it).


Feature/1985/modify wes credential passing

Ticket: https://ucsc-cgl.atlassian.net/browse/DOCK-1985
Currently targeted at branch in this PR: https://github.com/dockstore/dockstore-cli/pull/98, but is probably easiest to re-target to develop after #98  is merged.
Allows the Dockstore CLI to accept AWS profiles for credentials, and modifies the way some credentials are handled. If you wanted to run a command entirely from the command line, it would look something like:
dockstore workflow wes launch \
--wes-url <url> \
--wes-auth <type> <authorization> \
--aws-region <region> \
--entry myEntry \
--json myJson

You can also specify your configuration within the Dockstore CLI config file:
[WES]
url: https://1234567.execute-api.us-east-1.amazonaws.com/prod/ga4gh/wes/v1
type: aws
authorization: wes-test-profile
region: us-east-1

Which reduces what you need to pass in on the command line to:
dockstore workflow wes launch \
--entry myEntry \
--json myJson

Additional Notes:

You can specify --aws-config as a path to your AWS credentials file, but we default to ~/.aws/credentials
Originally, the CLI config WES section took authorization: Bearer <token> as a parameter, which is now broken into 2 lines:

type: Bearer
authorization: <token>

Added a note to https://ucsc-cgl.atlassian.net/browse/SEAB-3665 to have the BOM specify CDK and AWS SDK versions.
There's a Snyk finding as well, looks to be from the AWS SDK version. There is a conflict in upgrading the AWS SDK version with a version we get from the webservice, so perhaps it's worthwhile to finish the BOM ticket before this one?

[X] Check that you pass the basic style checks and unit tests by running mvn clean install

Just for some extra details. AWS maintains 2 separate credential files: ~/.aws/config and ~/.aws/credentials. The config file is designated for non-confidential values (region, display settings, etc...) and credentials is for keys and tokens (This isn't a strict requirement, but seems to be the expected practice, the 2 files can overlap in information).
At the time of this comment, the current CLI implementation doesn't allow specifying a custom path to either file, and they are expected to be at  ~/.aws/config and ~/.aws/credentials. The AWS Java SDK handles the pathing for us, so all we need to provide is a profile name. By letting the AWS SDK handling the file location, and not allowing custom pathing, this allows us to remove --aws-config, --aws-credentials, and --aws-region command line and config file parameters. We could re-add this functionality down-the-line, if it seems worthwhile, but for the moment reducing the extra complexity appears reasonable.


In Contact Merge, if Current Address lookup is selected on one contact and Mailing Address fields on another contact, an error is generated

Hub thread: https://powerofus.force.com/0D58000002GMUOX

I originally logged this on hub.  Let me know if you need more info or login access to org where issue occurs to investigate.
thanks

org had two trigger handlers marked Async.  AFFL_Affilations_TDTM on Contact and REL_Relationships_Con_TDTM on Contact.  If I unchecked async REL_Relationships_Con_TDTM, then contact merge worked correctly.
with Async checked on, we hit an exception in runAsync().  Here is the error with NPSP error handling disabled (so we see the underlying error):
Errors
Merge failed. First exception on row 0 with id<PHONE_NUMBER>o1MKWAA2; first error: DELETE_FAILED, npsp.TDTM_Contact: execution of AfterDelete caused by: System.NullPointerException: Attempt to de-reference a null object Class.npsp.TDTM_TriggerHandler.runAsync: line 202, column 1 Class.npsp.TDTM_TriggerHandler.runClass: line 183, column 1 Class.npsp.TDTM_TriggerHandler.run: line 108, column 1 Trigger.npsp.TDTM_Contact: line 34, column 1: []
Failed to merge the contacts. Error: System.DmlException: Merge failed. First exception on row 0 with id<PHONE_NUMBER>o1MKWAA2; first error: DELETE_FAILED, npsp.TDTM_Contact: execution of AfterDelete caused by: System.NullPointerException: Attempt to de-reference a null object Class.npsp.TDTM_TriggerHandler.runAsync: line 202, column 1 Class.npsp.TDTM_TriggerHandler.runClass: line 183, column 1 Class.npsp.TDTM_TriggerHandler.run: line 108, column 1 Trigger.npsp.TDTM_Contact: line 34, column 1: []

Included in beta release version 3.52 (Beta 4) and higher

Included in production release version 3.52


[Bug] Toast and Snackbar not shown on IOS 12.4 (simulator or device)

Description
On IOS 12.4 simulator and device, calling this.DisplayToastAsync and this.DisplaySnackBarAsync does nothing.  Calling (sender as VisualElement).DisplaySnackBarAsync works, and after calling this, all subsequent calls (using this or (sender as VisualElement)) work, but only for the current page.  Open a new page and DisplayToast/Snackbar fail again.
I assume that calling (sender as VisualElement).DisplaySnackbarAsync must change something on the page to make the subsequent calls work
Steps to Reproduce

Create new solution
Add 4 buttons
Button 1 Clicked = this.DisplayToastAsync("Hello");
Button 2 Clicked = (sender as VisualElement).DisplayToastAsync("Hello");
Button 3 Clicked = this.DisplaySnackBarAsync(sbo); //sbi is a SnackBarOptions instance correctly set
Button 4 Clicked = (sender as VisualElement).DisplaySnackBarAsync(sbo);

Expected Behavior
Toast and snackbar to be displayed
Actual Behavior
Press button 1 - No action
Press button 2 - No action
Press button 3 - No action
Press button 4 - Snackbar displayed
Now pressing the other buttons also works.
Same code works without issue on IOS 14.5.
Basic Information

Version with issue: Toolkit 1.2.0 pre2
Last known good version: Couldn't get this to work on any version
IDE: VS2019 - Windows
Platform Target Frameworks:

iOS:   12.4 (works on 14.5)

Nuget Packages:  Essentials 1.6.0; Xamarin Forms <IP_ADDRESS>2
Affected Devices: iphone 5/s and possibly others

@VladislavAntonyuk does this make any sense to you?

I will try to reproduce. maybe some changes with constraints between ios12 and ios14.

This is how it looks in the simulator on ios 12.4, you can see the very bottom of the toast message (I guess because it's a long message).

Does it look correct with the short message?

With a smaller message the toast doesn't display, I guess it's shown, but off screen.  I've added a small video of the issue, as you will see, pressing 'Show toast with anchor' makes the toast work correctly.   This is using the current master branch.
https://user-images.githubusercontent.com/44022687/120606335-00cba400-c447-11eb-8d4c-620eac54742a.mp4

looks like it works correctly with the second click on the button. or it happened after the anchor view is displayed. ok. I will investigate it

Interesting, for the first presses of the button I see a lot happening at the top of the screen, did you notice?

fixed, will create a PR in a minute


Operator '+' is valid only for numeric or string arguments, not for type 'string'.

Running the following:
for c in fmt {
    write(1, "Char: " + c, 1);
}

produces the error:
Operator '+' is valid only for numeric or string arguments, not for type 'string'.

The expected error here would be
Operator '+' is valid only for numeric or string arguments, not for type 'u8'.

Or possibly no error at all since u8 could be considered a numeric and a valid operation.

Error improved in latest commit. Will add some simple facility for coercing a u8 into a string so that concatenation is straightforward.

I think this was fixed so I'm closing the issue.


Unable to install gem on Ruby < 3.0

Using ruby 2.7.4, I'm unable to install github_changelog_generator due to a dependency issue.
Reproducing
With Ruby < 3.0, install the github_changelog_generator gem:
# Ruby 2.7.4
$ gem install github_changelog_generator
ERROR:  Error installing github_changelog_generator:
	There are no versions of io-event (~> 1.0.0) compatible with your Ruby & RubyGems. Maybe try installing an older version of the gem you're looking for?
	io-event requires Ruby version >= 3.0. The current ruby version is <IP_ADDRESS>.

Workaround
Switching to Ruby 3.0, the gem installs fine:
# Ruby 3.0.0
$ gem install github_changelog_generator

Successfully installed github_changelog_generator-1.16.4

As does installing a lower version of async first on Ruby 2.7.4:
# Ruby 2.7.4
$ gem install async -v '~> 1.29'
$ gem install github_changelog_generator

Successfully installed github_changelog_generator-1.16.4

It looks like async bumped their io-event requirement recently, which breaks compatibility with Ruby 2.x: https://github.com/socketry/async/commit/d16f7fad5859dfd8f731388ce98eea2b42f55501
Perhaps the async dependency should be pinned to version 1 to maintain compatibility with Ruby 2?

Desktop

OS: macOS Catalina (10.15.7)
Browser: Chrome 96.0.4664.110
Ruby: <IP_ADDRESS> (installed using rvm)

Workarounds you can do yourself include:

constraining versions of async to be used, using a Gemfile and Bundler. See Bundler's documentation. E.g. https://bundler.io/v2.2/guides/using_bundler_in_applications.html


CodeQL CLI setup docs don't cover warning messages from codeql resolve qlpackscommand

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
Code security/CodeQL CLI/Getting Started/Setting up the CodeQL CLI
What part(s) of the article would you like to see updated?
Under section 3 Launch codeql, the topic Testing the CodeQL CLI configuration tells the reader to run codeql resolve qlpacks.
Then the article says, "ÿou should see output similar to the extract below, if successful:", and shows an example of output if the command succeeds. Following the articles instructions to the letter, I downloaded the codeql-bundle-win64.tar.gz bundle and extracted it, and ran the codeql resolve qlpacks command.
My output contained several messages like this one:

codeql/controlflow is found in 5 same-priority locations, so attempts to resolve it will fail:
C:\dev\New folder\CodeQL\qlpacks\codeql\controlflow\0.1.3
C:\dev\New folder\CodeQL\qlpacks\codeql\csharp-examples\0.0.0.codeql\libraries\codeql\controlflow\0.1.3
C:\dev\New folder\CodeQL\qlpacks\codeql\csharp-queries\0.8.3.codeql\libraries\codeql\controlflow\0.1.3
C:\dev\New folder\CodeQL\qlpacks\codeql\ruby-examples\0.0.0.codeql\libraries\codeql\controlflow\0.1.3
C:\dev\New folder\CodeQL\qlpacks\codeql\ruby-queries\0.8.3.codeql\libraries\codeql\controlflow\0.1.3

I looked at the docs for the codeql resolve qlpacks command and took a lucky guess that because the bundle contains nested folders, I should use the --no-recursive option for the command. Other readers may not be as lucky and not understand why the command given in the instructions does not work as expected.
I expect everyone will get the same attached output using the same CLI bundle, so either the bundle should contain a README explaining how to avoid the warning messages, or your setup docs should contain a link to the docs for the command, with a hint to try the --no-recursive option if they see messages like I did.
resolve-qlpacks-output.txt
Additional information
I am able to reproduce the problem, as its cause lies with the structure of the CLI bundle I used, codeql-bundle-win64.tar.gz. Running the codeql resolve qlpacks command without the --no-recursive option always produces the same output for the same bundle contents.
I am more than happy to submit a PR to add a hint to the CLI setup docs, but I think the problem is bundle dependent, so I'm not sure where to add the advice.

@bradykelly Thanks so much for opening an issue! I'll get this triaged for review ⚡

@bradykelly - thanks for raising this issue and letting us know about this discrepancy between the docs and the bundle. I'm going to check with the engineering team whether the change is intentional or not. Then we can decide on whether or not to update the docs.

It sounds as if there's a newer command that should work better for this case: codeql resolve library-packs.
Would you be able to test that this works better? Then we can convert this into a "help wanted" issue with details of the docs updates needed to fix the problem.

Reopening to give @bradykelly time to respond. Our stale process doesn't respect holidays.


Change measurement page token.

This change is

src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/queries/StreamMeasurements.kt, line 33 at r1 (raw file):
Previously, wangyaopw (Yao Wang) wrote…

"ORDER BY ExternalMeasurementId ASC"

Also order by updateTime. The system API needs to send the measurements in the ascend timestamp order to duchies.
ORDER BY UpdateTime ASC,  ExternalMeasurementId ASC
Note: this was already a bug in the old code.

It wouldn't make sense to order by both since that would mess up the pages when using a page token. So I made order by updatetime default and order by id an option.

src/main/kotlin/org/wfanet/measurement/kingdom/service/api/v2alpha/MeasurementsService.kt, line 293 at r1 (raw file):
Previously, wangyaopw (Yao Wang) wrote…

source.pageSize in 1..MAX_PAGE_SIZE

I guess source.pageSize>0 && source.pageSize <= MAX_PAGE_SIZE is more efficient.

I looked it up and the implementation of the ... actually does that.

src/main/proto/wfa/measurement/internal/kingdom/measurements_service.proto, line 97 at r1 (raw file):
Previously, wangyaopw (Yao Wang) wrote…
Maybe put all pageToken inside cross-media-measurement/src/main/proto/wfa/measurement/api/v2alpha/PageToken.proto

Done.

src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/queries/StreamMeasurements.kt, line 33 at r1 (raw file):
Previously, wangyaopw (Yao Wang) wrote…
We need to order by both for the system API to work.
It is no different from what you did here for the eventGroups https://github.com/world-federation-of-advertisers/cross-media-measurement/blob/main/src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/queries/StreamEventGroups.kt#L29
The query will be
updated_after > @$UPDATED_AFTER  or (updated_after = @$UPDATED_AFTER & externalMeasurementId > @$EXTERNAL_MEASUREMENT_ID)

UpdateTime changes though so it is a bit different. Guess it is fine though, if it matters, it could be changed later to only order by both if update time is specifically set in the public api filter.

src/main/kotlin/org/wfanet/measurement/kingdom/deploy/gcloud/spanner/queries/StreamMeasurements.kt, line 63 at r3 (raw file):
Previously, wangyaopw (Yao Wang) wrote…

 > 0L

Should be != 0L
fixed64 can be negative.

fixed64 can be negative, but external ids aren't allowed to be negative.

src/main/kotlin/org/wfanet/measurement/kingdom/service/api/v2alpha/ProtoConversions.kt, line 91 at r3 (raw file):
Previously, wangyaopw (Yao Wang) wrote…
You are still returning a list.

I can make the caller not a list, but my mapping is from 1 to many in some cases so I have to return a list.

src/main/kotlin/org/wfanet/measurement/kingdom/service/internal/testing/MeasurementsServiceTest.kt, line 770 at r4 (raw file):
Previously, wangyaopw (Yao Wang) wrote…
do you want to delete the if in the beginning?

Done.


[SSoC 2.0] Large Factorial Calculator Ready

Title - Large Factorial Calculator
Issue Number - 403
Closes #403

Demo Video

https://github.com/Vikash-8090-Yadav/Future.WebNet/assets/114330097/a91e9701-be68-4316-84b5-3724af394eec

@Vikash-8090-Yadav @shubhisingh184 please review this pull request

@Avdhesh-Varshney This project is working fine, but one piece of advice for you from my side, if you're working on small projects to learn, then you can create much more attractive UI for your project. Okay !

@Avdhesh-Varshney This project is working fine, but one piece of advice for you from my side, if you're working on small projects to learn, then you can create much more attractive UI for your project. Okay !

@shubhisingh184 ma'am please review the new look of the website

Is it better ?

@Avdhesh-Varshney This project is working fine, but one piece of advice for you from my side, if you're working on small projects to learn, then you can create much more attractive UI for your project. Okay !

@shubhisingh184 ma'am please review the new look of the website

Is it better ?

yeah, now it looks a lot better. Thankyou for your efforts. I really appreciate it.


Added Project.strictMode to fhir.schema.json

Coverage remained the same at 94.411% when pulling 7ee1900418e2b868a4553deb6ec4bac2aee993b0 on cody-add-strict-mode-to-fhir-schema-json into d389bec05d4b6b2f076993ea3c1045383e18e4e0 on main.


Fix babelHelpers.typeof is not a function

We started encountering the error described in #5747 (`babelHelpers.typeof is not a function) after switching from npm to yarn. This comment led to this comment which contains a solution we've been using successfully in our production app.
Maybe I didn't look in the right place but it doesn't seem anyone had actually PR'd this change before (if so and I didn't find it, I apologize).
An alternative solution it seems is to add a different .babelrc, but this seems easier to me.

For other who had this packager error: I was having this issue due to a symlinked module that has it's own .babelrc and is not transformed. The module's babelrc specifies "es2015", "react" presets which causes the packager to choke with the same typeof error
This PR change resolved the issue for me, would love to see this merged!

What does it take to get this PR merged?

Thanks @cmcewen, this problem has been popping up all the time

@cmcewen  Anything we can do to help get this merged?

I'm not sure - looks like @janicduplessis did some work on this file a while back?

@facebook-github-bot shipit


Fix activities storing wrong html

https://trello.com/c/6JF6vzok/215-fix-activities-storing-wrong-html-markdown
Description
When I introduced the decorators it passed in the activities models a decorated instance of investigations which broke the body description as it would have the decorated version of the description instead of the raw version.
Checklist:

[ ] Automated checks are passing locally.
[ ] CHANGELOG updated if change is worth telling users about.

General testing

[ ] Test without javascript
[ ] Test on small screen

Accessibility testing

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable css - does content make sense and appear in a logical order?
[ ] Passes automated checker (automated in build or manual)

Anything we can do for faulty data stored in the interim?

we could right a migration I guess, load and resave the audit. I didn't do it as only adds an extra  and still looks ok

Anything we can do for faulty data stored in the interim?


New setup UI and app UI

You will need to add the setup script to the server_setup.xml page. Add the information in between the  tags

Awesome!


Handle timeout behavior of read functions in NI-DAQmx

What does this Pull Request accomplish?
In Ni-DAQmx functions, whenever a timeout exception has occurred instead of raising a gRPC exception, the error and the partially read data should be sent through the gRPC response.
,
Why should this Pull Request be merged?
This implements Bug 2399915: Partial reads report samps-per-chan-read via DaqReadError but don't update read_array
What testing has been done?
The changes have been tested manually

@Sakthi-SM These system test failures do not appear to be related to this PR.
Windows:
[ RUN      ] NiRFmxWCDMADriverApiTests.ModAccMarkerModeFromExample_FetchData_DataLooksReasonable
D:\a\grpc-device\grpc-device\source\tests\system\nirfmxwcdma_driver_api_tests.cpp(330): error: Expected: (0.0) < (mod_acc_fetch_reference_waveform_response.reference_waveform(0).real()), actual: 0 vs -0.00347697
[  FAILED  ] NiRFmxWCDMADriverApiTests.ModAccMarkerModeFromExample_FetchData_DataLooksReasonable (13146 ms)

Linux:
[ RUN      ] NiFgenDriverApiTest.PerformReset_CompletesSuccessfuly
/home/runner/work/grpc-device/grpc-device/source/tests/system/nifgen_driver_api_tests.cpp:71: Failure
Value of: status.ok()
  Actual: false
Expected: true
/home/runner/work/grpc-device/grpc-device/source/tests/system/nifgen_driver_api_tests.cpp:101: Failure
Value of: status.ok()
  Actual: false
Expected: true
[  FAILED  ] NiFgenDriverApiTest.PerformReset_CompletesSuccessfuly (112 ms)


CRM-91: approach with dynamic parsing service name from resourceId

Approach with dynamic parsing service name from resourceId.
PR for previous solution closed, because considered to be sub-optimal: https://github.com/edekadigital/terraform-azure-modules/pull/3
Solution here should be quite robust, if consider structure convention of Azure resourceId.
Why?

for cases, provider is a part of the resourceId, which should be the case for Azure Web and Function Apps,(MICROSOFT.WEB f.e. in case of Azure Web Apps and Functions App - s. also attached screenshot), service name should reliably be at last index of parsed Azure resourceId
For all another cases, service is set to default "azure" value

In general, this should be enough for our purposes, because this kind of log forwarding (via event hub) is only applicable for Azure Web Apps and Function Apps (I speak mainly about monitorable Azure Compute Services). For VMs and Azure Containers another log shipping approaches are used, like Docker Agent / Fluent Bit Sidecar Container.
By the way, we use same approach in our AWS Lambda Log Forwarder, I took a look there.

I strongly advice not to do it like this. For example in our case, this would leak the stage into the tag making filtering harder than it should be.

Why You think, stage will leak into service tag's value. At least, unit tests are not ondicating such behavioural. Maybe You can explain Your point on Monday.

Ok. Now I understand, what You mean. You mean the "dev" / "test" infix in the service name. It's not "leaking", it's just the name of the Web App in Azure, as we called it. I don't consider to be this a problem. If ever, solution would be to rename the resource in Azure and make it uniquely "cat-app" in all subscrptions.

Additional reason to do it like this for me is the obvious benefit, we bring into the solution in comparison with current Datadog Standard Solution (which is f.e. still used by Asteroids team - they compulsive will still have this problem). Maybe we can move then more teams / people to use this module.

I strongly advice not to do it like this. For example in our case, this would leak the stage into the tag making filtering harder than it should be.

I'd rather prefer removing the stage out of the app name and use resource tags within azure.


Expose cursor to caller

I'm working on an application that needs to be able to scrape large volumes of tweets. This necessitates splitting searches into batches. It would be useful to expose the cursor after completing a query, so that the same cursor could be reused later to resume the search.

The fetchSearchTweets function does return the cursor in the response — it's intended for runtimes that don't support async generators, but I believe it works for this use case as well.

At the moment it only returns the bottom cursor, but for my use-case I need both the bottom and top cursor (scrape historical, but also keep the results updated as new matches come in).


MoveIt 2 interactive marker missing

Environment

Ubuntu 20.04
ROS2 Galactic desktop (osrf docker image)
MoveIt 2
ur_model: UR5e

Issue

I first created a docker and mounted the ur_workspace, follow steps mentioned in the document and run rosdep update rosdep install colcon build.
The workspace works fine and all the example command can be run
Then I try to use MoveIt to remotely plan and execute the ur5e in a simulation environment (The real one is inside the lab, and I would like to do simulation to familiarize the control before working on the real one)
I listed the questions in below to descript the situation I am facing

Question

For simulation, should I use URSim or fake_hardware? I have already setup the URSim docker and installed URCap, also I can ping it in the different docker shell
Assume URSim is the one I should go for, I follow the step to start MoveIt and URSim, however the problem kicks in
The interactive marker doesnt appear after laucnh... I have read the document carefully and ensure all the settings are the same. Seems that it expect to show the marker once launched, so I have no clue how to solve this issue... Am I setting the IP address wrong?

URSim docker IP: <IP_ADDRESS>
External control host IP: <IP_ADDRESS> (actually should I change this to the workspace shell's IP?)
ur_workspace docker IP: <IP_ADDRESS>
So far I could use URSim to manually move the ur5e when launch with fake_hardware
I could provide more information/picture if it is needed. Thanks for everyone help!

Anyone could explain how should I start the Moveit with URSim setup?
So far I can only use URSim with ur_control.launch.py to test with test_joint_trajectory_controller.launch.py and manual control via URSim HMI...

Might be anyone could successfully run MoveIt demo with URSim using ROS2 galactic can share their experience?
I am completely fine for rebuilding the project.

Do I have to use setup assistance to setup the ur5e robot first?? I saw there is a ros-industrial package containing ur5 with moveit, but I don't see it support ROS2, what is the different between that one and this repo? This one is the official right?

yes, this is the official ROS2 driver for UR. The package you mentioned is part of the ROS1 support packages.
To answer a couple of your questions first:

For simulation, should I use URSim or fake_hardware?

Both would work. However, in my opinion using URSim is preferable, since this will be closer to your actual hardware setup. Especially, since fake hardware atm. doesn't support the scaled trajectory controller.

External control host IP: <IP_ADDRESS> (actually should I change this to the workspace shell's IP?)

Yes, this should be the IP under which the URSim container can reach the machine running the ROS driver.

The interactive marker doesnt appear after laucnh...

I assume, you followed the steps in the README? If everything started up correctly (Do you see "You can start planning now" in the MoveIt!-Shell?) you should be able to use the interactive marker (I assume you mean the sphere at the robot's TCP?) to create a plan target. If you are using a locale with , instead of . as a decimal separator, I've had problems in the past. To mitigate this, add a LC_ALL=C in front of the moveit launchfile:
LC_ALL=C ros2 launch ur_moveit_config ur_moveit.launch.py ur_type:=ur5e launch_rviz:=true

I hope, this helps.

Sorry I am a bit busy in recent days, I will update here once tested it out.
Great appreciate for your help @fmauch !!

@fmauch since you are mentioning the decimals here. I've just opened https://github.com/ros-planning/moveit2/issues/1882 describing the problem and suggested fix using locale independent double parsing. This affects MoveIt but also other packages including ros2_control and the UR driver.


Exclude subdirectories?

Is there a way to use -x to exclude subdirectories?  I'm trying to do something like
plato -r -d plato_report server public -x "public/|vendor|js"  -t "My App"
in order to analyze everything in server and public, but exclude everything in public/vendor and public/js.  However, I can't seem to get the exclusion working properly - plato always includes those subdirectories.
I've tried a few different regex patterns, but maybe I'm just not using the right regex.  Or maybe plato can't exclude a subdirectory of a directory it's including?
Could you please let me know whether what I'm trying to achieve is possible, and (a) if so, how, or (b) if not, if this is planned in a future release?
Thank you very much!

@DABH:
Today I had and solved the same issue. For future reference I'll share my solution here.
According to source code I've defined my own function (exclude.test) like this:
    var exclude = {
        test: function (file) {
          return (file.match(regex) !== null);
        }
      },
      files = ['client/**/*.js'],
      outputDir = 'client/report';

    plato.inspect(files, outputDir, { exclude: exclude }, cb);

where regex matches the files (path + name) I want to exclude.


502 error trying to change deployment branch using wiki instructions

Repro steps.

create App Service
create Deployment using Local Git
push code to a branch that is not master, e.g. 'mybranch' (I'm building a Node app)
try to update the branch using the instructions here (the first curl command. Note I had to change the single quotes to doubles in the JSON data) https://github.com/projectkudu/kudu/wiki/Deployment-branch
example: curl -vR<EMAIL_ADDRESS>\     --data '{key: "branch", value: "mybranch" }' \     -H "Content-Type: application/json; charset=UTF-8"
Note: I replaced user:pass@site with the site found on the Overview for the app service on portal.azure.com, and user:pass by going to Deployment Center -> Deployment Credentials -> App Credentials

Project structures.
n/a
The log/error given by the failure.

< HTTP/2 502
< content-length: 260
< content-type: text/html
< etag: "5c162d64-104"
< server: nginx
< set-cookie: ARRAffinity=b4a2564d88fd1d8c2f6d16a087d2a017231cd4562ce17a1f3b764d08bd7a8093;Path=/;HttpOnly;Domain=stctestdeployment.azurewebsites.net
< date: Tue, 15 Jan 2019 18:09:05 GMT
<

        <title> Server Error </title>

Server Error.

        There was an unexpected error in the request processing.

* Connection #0 to host stctestdeployment.azurewebsites.net left intact

Debug your Azure website remotely.
n/a
Mention any other details that might be useful.

Any luck on this? I'm seeing the same HTML block returned from my service.

I don't think I ever got this working. Best of luck!

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Fix path problem in test_download

We had the problem of creating an unexpected towhee folder in tests folder when running test_download.py.
This PR fix this problem by change the cache_path to remove the correct folder after test.

/approve
/lgtm


ci: use git submodules for  Chart Actions

SUMMARY

This adds chart actions submodules to allow chart actions to run.
Added because of implementation for using git submodules for (securely) using third party Github Actions (apache#12709)
Related to https://github.com/apache/superset/pull/14163 that has been accepted.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ X] Has associated issue: https://github.com/apache/superset/pull/14163
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API
@craig-rueda

@amitmiran137 @craig-rueda
See chart linter in action
https://github.com/apache/superset/pull/14211/checks?check_run_id=2368816356
For some reason chart maintainer fails validation, should I change it?

Lint works great now. Should be OK to merge
https://github.com/apache/superset/pull/14211/checks?check_run_id=2368915717

Again , the work your doing here is amazing!!

Apache just approved the action added in #14163 . Closing this in favor of previous PR.

After testing, I think this is a better solution, as it doesn't require additional approvals from Apache infra :)


New package: ReefBiodiversityAccountSetup v0.1.0

Registering package: ReefBiodiversityAccountSetup
Repository: https://github.com/open-AIMS/ReefBiodiversityAccountSetup.jl
Created by: @Rosejoycrocker
Version: v0.1.0
Commit: c83f603d2e8d14ee8f62d7f3d3de8036311a73e1
Reviewed by: @Rosejoycrocker
Reference: https://github.com/open-AIMS/ReefBiodiversityAccountSetup.jl/issues/2#issuecomment-2496823212
Description: Spatial analyses and plots using open access data to inform setup of a reef biodiversity account.

Thank you for submitting your package! Sorry I'm a little late to this, but could you add a little documentation before the registration gets merged? At the very least, that would be a small usage example in the README. An important part of packages in is that any potential user can figure out what the package is about and how to get started with using it. That is really difficult when there is no documentation.

Thanks, the package now contains a detailed README


Feature: Randomization

I think the best way to do this would be to use loot tables. However, the point is that I can make a macro for it and then change it under the hood

[x] Statement Sytnax

@random x in 1..10

[ ] Operation RHS Syntax

x += @random 1..10

Reopening for scope creep


Nested merged

Merging ASAP to get passing tests as a healthy basis. Even if only for py2 for now...


[BUG] Fody ignores an entire assembly unless 'ReactiveObject' is mentioned somewhere in it

I noticed that Fody will ignore an assembly unless certain 'trigger keywords' such as ReactiveObject are mentioned anywhere in the code (which doesn't even have to be used). If you happen to use a derived class such as MyReactiveObject you're out of luck as Fody will silently ignore all your reactive attributes.
There seem to be a variety of triggers, for example using .WhenAny* someplace in the assembly. Just placing an attribute, i.e. [Reactive] on your class will not work.
I attached a very small solution which demonstrates the issue.
FodyBug.zip
Inside is a simple console project which will output PropertyChanged notifications. If you remove a DummyClass.cs, everything will compile and run just fine but you will no longer receive notifications.
If this isn't a bug but rather some form of hidden optimisation I'd recommend we at least mention it somewhere. This bit of documentation:

If a property is annotated with the [Reactive] attribute, the plugin will weave the boilerplate into your output based on the simple auto-property declaration you provide.

Is not entirely true. Fody also expects either of the following, from what I've observed so far:

At least one class in the assembly directly deriving from ReactiveObject
At least some ReactiveUI code used somewhere in the assembly, i.e. WhenAnyValue

What's silly about this issue is that after making a 100 assemblies with Fody that work just fine your 101st one will fail for no reason other than its code not having the magic words 😃.

Hmm. That fody might have a alternative soon. Stand by for a video blog in the next month.

Just ran into this on latest ReactiveUI.Fody (13.2.18).
My setup consists of a library project which defines base ViewModels, and then an application project which extends them (for cross-platform reasons).  The application never directly used "ReactiveObject" as a base class because all of its ViewModels were derived from base ViewModels in the library.  Properties marked [Reactive] in the application's ViewModels did not work as expected until I added a dummy class that derives from ReactiveObject.

The fody is going into maintenance-only mode at the moment while we work on a new project which will replace the functionality in general. Keeping this open to track for the new project.

If possible, please use ReactiveUI.SourceGenerators they work slightly differently but hopefully will fit with your needs.
I updated the example first provided to show that the Source Generators can solve this issue.
FodyBug.zip
As this proves that the issue can be solved by using the ReactiveUI Source Generators, and ReactiveUI.Fody is now in maintenance mode I will close this.


Specifying email address in .env

Hi,
I'm curious why not allow specifying the email address in .env file?
If it is due to multiple email addresses, perhaps can consider the format below
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Thanks!

@bilogic
You can do something in config/sneaker.php
'to' => explode(',', env('SNEAKER_EMAILS')),

and in .env
SNEAKER_EMAILS=a1@b.com,a2<EMAIL_ADDRESS>

@akaamitgupta what I meant was, SNEAKER_SILENT reads from .env by default, why doesn't SNEAKER_EMAIL do the same?

@bilogic
config/sneaker.php is under your control you can update it as you require.

@akaamitgupta i'm merely trying to provide a suggestion for improvement

Thanks @bilogic... really appreciate this. Will look into this in the next release.


FrameworkState has a race condition

I was just looking over FrameworkState.java, and I noticed it's got a race condition. When you getTerminatedTaskInfos(), you check the task statuses (atomic), and then look up each TaskInfo, indirecting through the name of the task. Although each TaskInfo lookup is atomic, they entire method isn't atomic, and so it's possible that a task could appear as running in the result of this method. The race is below, between 2 threads:
T1      T2
---     ---
 X
         Y
 Z

where X is the call to getTaskStatuses(), Y is the replacement of a task (so that it's no longer terminated), and Z is the call to stateStore.fetchTask().

Here's an implementation that avoid this particular race condition: https://github.com/dgrnbrg/dcos-commons/blob/5c537f1915cb61b46297bb6819db8c61f9c2e08a/src/main/java/org/apache/mesos/state/StateStore.java#L111-L134

@dgrnbrg:  Thank you for the bug report.  A PR is here:  https://github.com/mesosphere/dcos-kafka-service/pull/232.  I've included you as a reviewer.


Polyline dblclick issue in Firefox

[x] I've looked at the documentation to make sure the behaviour is documented and expected
[x] I'm sure this is a Leaflet Draw code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

How to reproduce

Leaflet version I'm using: 1.6.0
Leaflet Draw version I'm using: 1.0.4
Browser (with version) I'm using: Firefox 77.0.1
OS/Platform (with version) I'm using: MacOS 10.14.6
Vue.js: 2.6.10
Vue2Leaflet: 2.1.1
leaflet-polylinedecorator: 1.6.0

What behaviour I'm expecting and which behaviour I'm seeing
For creating a polyline by double-clicking on the last point, a leaflet-decorator(custom arrowhead) should be added to this polyline.
But in Firefox, double-clicking on the last point just completes the polyline and my double-click function is ignored. No arrow will be added to the end of the polyline.
Everything works as it should in other browsers but not in Firefox.
Minimal example reproducing the issue
https://codesandbox.io/s/nice-sammet-q0zr3

[x] this example is as simple as possible

I face the problem as well ，but my brower is chrome，firefox is good in drawing polyline.  I fixed the problem by downgrade leaflet to 1.3.1

@bation thanks for your advice but I can't do downgrade from leaflet 1.6.0 to leaflet 1.3.1, that is no solution for me.

@miko866
Try this solution: https://github.com/Leaflet/Leaflet.draw/issues/955#issuecomment-601190466

@miko866
Try this solution: #955 (comment)

Super thank you its works.


Util cleanup and refactoring // Util tests

Coverage increased (+2.19%) to 74.26% when pulling 10290c213347bd9396fb31e56e5a1700ae4fd4ed on stability-improvements-util into d98d8abd91955131fdbad5074c065bd963e615c5 on master.


grouping of zeros

Has there been discussion of whether zeros are grouped?
Some numbers are a little difficult to read on screen e.g. in https://isaacphysics.org/questions/ch_a_p3?board=physicsskills_book_ch_a3
Qu 3.8 shows the number 0.000005638
Can this easily be rendered as "0.000 005 638"?
http://www.npl.co.uk/reference/measurement-units/si-conventions/
Suggests that this is absolutely standard, but of course that doesn't make it trivial to do!

This is a content issue. You are able to control the text in questions via the editor.


Versioning policy?

Hello!
I use fastutil in several projects with the types often appearing as part of published APIs. As fastutil is a healthy project with frequent releases, I often have to update version numbers. Is it safe to assume that fastutil is using semantic versioning? It seems like it is, given how the version numbers have changed over time, but I don't see an explicit policy written down anywhere. With an explicit policy in place, I can use version ranges in dependencies to make it easier to stay up-to-date.

On 22 Feb 2017, at 12:20, io7m<EMAIL_ADDRESS>wrote:
Hello!
I use fastutil in several projects (with the types often appearing as part of a published API). As fastutil is a healthy project with frequent releases, I often have to update version numbers. Is it safe to assume that fastutil is using semantic versioning? It seems like it is, given how the version numbers have changed over time, but I don't see an explicit policy written down anywhere. With an explicit policy in place, I can use version ranges in dependencies to make it easier to stay up-to-date.

Nothing that formal, but yes, more or less I do that :).
Ciao,
				seba

Ok, thanks!


🐛 Bug Report: Unable to start - Illegal instruction (core dumped)

👟 Reproduction steps
I just created a docker-compose stack in Portainer,

The logs are present below,
_pingvin_logs.txt
👍 Expected behavior
It should open the homepage of Pingvin.
👎 Actual Behavior

🌐 Browser
Brave Browser

stonith404/pingvin-share:v0.11.1 is working perfectly fine!
But the latest version isn't working properly, please fix it soon!

That's a strange issue. It might be an issue with the package sharp that was added in the latest release: https://stackoverflow.com/questions/67580821/nodejs-illegal-instruction-core-dumped-error-after-using-sharp-library
Which CPU does your server have?

Intel Core Duo

It's really difficult to debug for me as I can't reproduce this. Could you try the following steps?

Create a fresh installation of Pingvin Share (this will remove your data if you don't back it up)

cd pingvin-share
docker compose down
rm -rf data
docker compose pull
docker compose up -d

Try to install the stand-alone installation and check if this works

It still shows the same error, thanks for helping! Guess I have to stop using your service cause of my shitty CPU. Hope your project touch the peak :)

So the stand alone installation didn't work either? Sorry that I can't help you further here... Thanks for your patience and for your raised issues!

The issue is Sharp library uses some instructions not supported by the Intel Core Duo probably. Since prior versions of Pingvin Share didn't have Sharp for custom logo creation, it worked on that older machine.
You could try testing this by exec'ing into the Pingvin container and trying to downgrade Sharp package like the link above suggested?

Do you use Portainer to manage your Docker containers by any chance? The console button is really handy in these types of situations.

The only other alternative would be if @stonith404 uses an older Sharp version. But regardless, he would still would a) need to know what the last version would support your CPU, and b) if using the older version still allows him to use the features in Pingvin Share.

Yea I use Portainer!
Unfortunately I couldn't even use the console since it shows the same error of logs in a fancy way lol
I'm searching for a mini PC, I really like Pingvin I can't be without it so probably getting an old PC [Dell Optiplex 3070 i3-9100T]

@krintek A Raspberry PI is awesome if you need a mini PC. I'm running Pingvin Share, Nextcloud and many more services on a Raspberry Pi 4 8GB. As long as you don't have much traffic a Raspberry Pi is enough.

Raspberry Pi nowadays costs a kidney lol I live in India, here the Dell Optiplex 3070 i3 costs $185. I think it would work well with 32GB RAM and 2TB Pcie Gen 3 SSD. You finally made me a reason to buy a new mini PC after 13 years lol.

Core Quad 6600 - same error. Bad, very bad.

I'm getting the same error on latest version (0.15.0) on my Synology DS713+ with an INTEL Atom D2700.


Update fsutil.ps1 - Auto Optimize All Drives

Update fsutil.ps1 to auto optimize all drives on a system.

Thank you! 😁

Thank you! 😁


Automatic publication of the NodeJS package

@teolemon I can create a PR for this if nobody else is already working on this? Please let me know when you can.

@PieterT2000 yes, of course, that would be great 👍

Wow, that was fast!
Could you explain the typical release procedure (including branch strategy/Gitflow) or link some info in here? I assume that apart from breaking change releases you also do minor and patch releases?
I can see that there is already a Github action that seems to be handling automated NPM publishing, but it has never been run. (See here) Is this because there hasn't been an official Github release yet?
Moreover, https://github.com/openfoodfacts/openfoodfacts-nodejs/issues/323 mentions about creating alpha and beta NPM releases. Is this still relevant? If yes, would it be an idea to create a beta branch? The new release strategy would then become: release branch -> beta -> develop. Additionally, an alpha release will be created whenever the first release PR is opened on the release-please--branches--* branch.
Alternatively, we can also do just beta releases whenever a release PR is opened, getting rid of the intermediary beta branch. This approach keeps your existing branch strategy.


Create guides and best practices for extension authors

Feature/component description
Lets ensure extension authors have great docs, walkthroughs, and/or other resources to help them build webview extensions that look and feel native to VS Code. This could take the form of Figma files, videos, docs, sample extensions, and more!

Some early thoughts on how we could improve the experience of developing webviews (taken from offline brainstorm w/ @hawkticehurst and @misolori
Hypotheses

There are limited design guidelines and samples that showcase best practices and inspiration for extensions using webviews
The cat coding guide feels intimidating as a first introduction to webviews. It also feels unrelated to a real UI.
Developers aren’t aware that they can leverage a UI toolkit in their webviews

How could we help provide design guidance?

More detailed higher-level patterns (sidebar, editor, both, etc.)
Component-level documentation (do/don't, usage, examples)
Clearly illustrate common API capabilities we see often duplicated (e.g. TreeView)
...

How could we improve the dev docs?

Simplify the core VS Code webview API docs and explain concepts for less experienced developers (e.g. disposing)
Brief summary of toolkit usage in core VS Code docs w/ link to a detailed walkthrough in the toolkit repo
Add hello world starter templates for extension authors to clone in various permutations (vanilla js, frameworks, build systems)
Add generalized guidelines for frameworks in toolkit docs

More to come 🤘


Connection event fires multiple times - no other event would work (using on or emit)

Before I start, I would like to thank you for such a great library, I appreciate all the work put into this.
I am developing a Xamarin Forms application (PCL) as a client that will connect to a node.js server (using express). I have tried using multiple libraries to work with the PCL project but was unsuccessful with it, so I decided to go with a native implementation and use that in portable project using dependencyservices.  (advise me otherwise if necessary)
The issues i'm facing with this library are the following:

The latest stable version did not work at all for me, that is, could not get the client to connect to the server at all, no errors no responses no debug evidence, so I ended up downgrading and finally got the client to connect to the server.
After the client connects using socket = IO.Socket("http://ipaddress:3000"); this statement alone causes the connection to start at the server side and fires the event "connection" which is shown below, the event keeps on firing continuously over and over until I close the application. Neither socket.Disconnect() or socket.Close() would stop this behavior.
Using socket.On and socket.Emit on client side will not fire any events when I emit or listen on server.

Would really appreciate if someone could help me out and guide me through this. I might be missing something or just unaware of how to get things to work properly.
Here are all the relevant versions:
.NETFramework: v4.5
SocketIoClientDotNet: v0.9.13
Engine.IO.Client: v0.9.0
EngineIoClientDotNet: v0.9.22
WebSocket4Net: v0.15.1
Newtonsoft.Json: v9.0.1
SuperSocket.ClientEngine.Core: v0.9.0
System.Collections.Immutable: v1.4.0
Here is the server side code (node.js with express):
const express = require('express');
const socketIO = require('socket.io');
const http = require('http');

var app = express();
var server = http.createServer(app);
var io = socketIO(server);

app.get('/', (request, response) => {
    response.send('Hello Express!');
});

io.on('connection', (socket) => {
  socket.on('onConnected', () => {
    console.log('New Connection Established!');
    socker.close();
  });

  server.getConnections((error, count) => {
      console.log(count);
  });

});

server.listen(3000);

Here is the client side code (xamarin forms - native android code):
 class SocketHelper : ISocketHelper
{
        public void OpenConnection()
        {
            Console.WriteLine("Connecting to server...");
            var socket = IO.Socket("http://ipaddress:3000"); //note that I put the actual IP address
            socket.Emit("onConnected");
            //socket.On("connect",  () => {
            //    Console.WriteLine("Connected to server!");
            //});
        }
   }

Here is how I call the dependency from the pcl code:
private void button_Clicked(object sender, EventArgs e)
        {
           DependencyService.Get<ISocketHelper>().OpenConnection();
         }

This is the output from the node.js console as soon as I click the button (the 1 that is appearing in the output is the number of connections):
[nodemon] starting `node server.js`
1
1
1
.
.
.

I resolved my problem by socket option
socket option by client side
=> AutoConnect = false;

I'm seeing this on Xamarin.Android too.  The AutoConnect = false; doesn't help.

I'm pretty sure this is the same issue as https://github.com/kerryjiang/SuperSocket.ClientEngine/issues/72

I'm not using this package, but maybe it could help someone. I was doing this in my middlewares:
p2p.use(async (socket, next) => {
  let req = socket.request
  let res = req.res || {}

  someMiddleware()(req, res, next)
  anotherMiddleware({extended: false})(req, res, next)
  anotherMiddleware()(req, res, next)
  anotherMiddleware(req, res, next)
  anotherMiddleware()(req, res, next)
  anotherMiddleware()(req, res, next)
})

Every time I was calling a middleware, the next function was getting called and on('connection') was being called too. What I got to do was overwrite the next function after the first middleware, so my code get like so:
  someMiddleware()(req, res, next)
  next = () => {} // NOW its not going to annoy anymore
  anotherMiddleware({extended: false})(req, res, next)
  anotherMiddleware()(req, res, next)
  anotherMiddleware(req, res, next)
  anotherMiddleware()(req, res, next)
  anotherMiddleware()(req, res, next)


[bug]

不能与reacrt一起使用,因为react也会更新视图,请问这个有解决办法吗？

It cannot be used with reacrt because react will also update the view. Is there a solution?


Eloquent orm strong dependency

Laravel Version: 5.3
PHP Version: n/a
Database Driver & Version: n/a

Description:
Is there a plan to make Laravel orm independant like thrid party packages can be used with any orm, like doctrine ?
Thanks

There is already a package for running Doctrine with Laravel. Are you asking about this, or how to use Eloquent in other projects? http://www.laraveldoctrine.org/

I think the question was: Is possible to use Laravel ORM outside of Laravel. If that was the question, that is already possible. Checkout the following video in Laracast:
https://laracasts.com/lessons/how-to-use-eloquent-outside-of-laravel

@nunomaduro No.
@sisve Yes, this package is the origin of my question. I would like to use laraveldoctrine, but I'm afraid that many Laravel's packages won't work, like Passeport, because of orm strong dependency.

It is possible to use it outside. I do.

@GrahamCampbell I believe you answered the question that wasn't asked.

You can already use other ORMs in laravel. Just because eloquent is included in the core, it doesn 't mean you have to use it.

@GrahamCampbell Not shure ... I can read that when using Laraveldoctrine other Lavarel third party packages will use Eloquent nor LaravelDoctrine that can be a source of problems.


[auto-merge] branch-21.08 to branch-21.10 [skip ci] [bot]

auto-merge triggered by github actions on branch-21.08 to create a PR keeping branch-21.10 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

FAILURE - Unable to auto-merge. Manual operation is required.
{'message': 'Pull Request is not mergeable', 'documentation_url': 'https://docs.github.com/rest/reference/pulls#merge-a-pull-request'}

I will take care of this after https://github.com/NVIDIA/spark-rapids/pull/3146 and changelog PR get merged later


[GSB] Improve handling of layout constraints

This PR addresses TODOs from #8241.

It supports merging for layout constraints, e.g., if both a _Trivial constraint and a _Trivial(64) constraint appear on a type parameter, we keep only _Trivial(64) as a more specific layout constraint. We do a similar thing for ref-counted/native-ref-counted. The overall idea is to keep the more specific of two compatible layout constraints.
The presence of a superclass constraint implies a layout constraint, e.g., a superclass constraint implies _Class or _NativeClass layout constraint.

@swift-ci please smoke test

@DougGregor Please have a look.

@DougGregor I pushed the updated version. This one initializes all entries in the merge table. Does it look more reliable?

@swift-ci please smoke test

Thank you!


Provenance for sequencing data in PopFreq Annotations

The evidence and provenance CQs here (rows 16-20 specifically) highlight the need to capture metadata on sequencing  data quality, technology, and other aspects of variant detection relevant as provenance in population frequency studies. This ticket will be used to elicit input and feedback on this topic.
Our short term goal is to define a simpler model for a v0 release by end of summer.  Longer term we will want to coordinate with other GA4GH efforts, and drivers/stakeholders/partners like HL7-CG, to develop a richer common model of sequencing studies/metadata, covering broad use cases, which can be re-used/submitted to schema blocks.
We need folks versed in this area to make recommendations about what is really needed and how we might structure it. Specifically, the types of technologies and protocols applied in population sequencing efforts, and specific metrics/scores used to assess sequencing data quality/reliability.  Within our VA group, Irina is knowledgeable here, and Steven Hart has added some insightful comments as well. I also suspect that the HL7 Clinical Genomics group has thought about this issue, so Bob and/or one of his colleagues here could advise as well.  Please note if there are others we might reach out to.

Starting a list of requirements here based on the CQs and data examples -  but fully aware this is naïve and incomplete.  Hoping folks can help round this out (feel free to edit this list/comment directly)
Experimental provenance:

the sequencing method and technology/platform that generated the sequence data
the molecule type/level of the sequenced material (rna, exome, genome)
the evaluant from which sequenced nucleic acids were taken (e.g. cheek swab, WBC, etc)
Whether the variant was directly observed or imputed in the population? (should this be a separate/new type of statement? or perhaps a qualifier on the statement?)

Quality metrics/flags:

gnomAD Genotype Quality Metrics (link)
a. genotype quality
b. depth (median or average across genome for carriers vs non carriers?  histogram by depth bucket?) (is depth same as coverage?)
c. allele balance
gnomAD Site Quality Metrics (link)
a. 13 different scores/metrics here - which are important/relevant?
What is the gnomAD 'Popmax Filtering AF'
A 'minimum frequency threshold' was mentioned on a call

SNH comment suggested that there are quality scores specific to different sequencing technologies and platforms.
Questions/Considerations:

Keep in mind that the metadata we are interested in here applies at the level of the reported population study (not the level of individual members of the population).  So it should reflect metadata relevant to sequencing all population members (e.g. parameters/metrics/tools applied in sequencing of all individuals, or averages calculated across all individuals).
At what level of detail should we capture/structure such metadata?
a. As noted, we may have different short vs long term requirements here.
b. And keep in mind our primary use case as a data exchange standard. We should only exchange data that may be directly used for search/faceting/filtering in APIs and data operations. For complete metadata, it may be best to simply point out an external reference - e.g. to gnomAD directly.


Wasm websys wip

This is a possible implementation, based on @tversteeg code. For the web part to work you need to have an executor and use spawn_local. This implementation does not force any executor, in fact, it does not even care what you use.
I still need to work on an example, and test it. It compiles, and all that really matters on this is to always keep a reference to the window, so the buffer does not go out of scope (which would trigger this: https://github.com/rustwasm/wasm-bindgen/issues/2445 )
Compile using:
cargo build --target=wasm32-unknown-unknown --verbose --no-default-features --features web

Hi, I put some extra work the last couple of days on this:

Cleaned up the dependencies to make sure we only include the ones we need on the web compilation step. Removed any other dependency from the rest of the work.
Squashed the huge list of commits into only one.
Worked on a multi platform demo that can be compiled to desktop AND web: https://github.com/dc740/minifb-async-examples

This is usable now. Getting minifb on the web requires some extra work, and I can't provide a simple example to do "cargo run minifb_web", that's why I had to add the example in other repository.
I kept it as small as possible, and after some research I don't think there is any other way to implement it. There is no way to keep the simple application loop that the desktop uses, but after this PR it's possible to finally run a (mostly canon) minifb application on the web using webassembly and some auto generated javascript glue code as shown in the demo.
I was hoping this PR could be merged so other people can start making improvements on it. Let me know if you think that's possible, or which things you think are blocking this PR.

Merged! Thanks for your work on this :)


Validate results from each evaluation

Currently, CI just builds the evals and tools and then runs all pairs. There is no signal about whether a tool succeeded or failed on a given eval. We want to designate a specific tool as the "golden" reference implementation (the plan is to use PyTorch), and then for each eval, define a way to compare the results of two tools on that eval. This also relates to #97, since we'll want to visualize those validation outputs on the website. Beyond this, we probably also want each eval to define how two runs should be visualized in more detail, beyond just a binary "correct/incorrect" decision.

Validation needs to be part of an eval, since as far I understand it, run.py is just a very generic message broker, that doesn't actually understand anything about the messages it transmits.

On the other hand, all an eval can do is send messages to the tool, so how is it supposed to report validation errors?

@athas run.py can (and I think probably should) validate the protocol itself, but the eval needs to describe how to validate the data it got from the session.
One way to do this could be to add an "end" message to the protocol, so when the eval wants to terminate the session (either successfully or unsuccessfully), it can send a message that run.py doesn't forward to the tool and instead just logs and exits. The eval could accumulate whatever it wants from the session into rich metadata in this final message, and the protocol could define some standard fields to be visualized consistently on the website (e.g. success vs failure, overall performance, etc).

I have also come to the conclusion that making run.py somewhat protocol-aware is necessary. I think this is fine as GradBench is not supposed to be a completely generic testing tool. The most important thing is to make it easy to add new tools, and achievable to add new evals.
I don't think having an "end" message is a problem, but I don't think it's how to do validation. Now I'm considering a design where evaluate messages must have two fields that are known to run.py:

name, which is a human-readable string unique to the particular input.
input, which contains the actual input to the evaluation (currently a JSON object, may become a filename as per #99).

The response must then contain another field known to run.py:

output, which contains the result.

(You will not that input/output are actually the fields already used by GradBench.)
This means that run.py will have enough information to create a directory tree that contains the input and output for every evaluation. (And also the runtimes and such.) Validation can then be done offline. I propose a separate program per eval, e.g., evals/hello/validate.py. This also doesn't have to run within Docker, as presumably validation does not need any fancy code (because it has to be trustworthy). It is of course easy to write some glue code that runs evaluation automatically after benchmarking.

The tool must also respond with the tool name, I realise.

Working on this in #101.

Resolved by #101.


public studies are being shown in the user own studies

This fixes:

https://github.com/biocore/qiita/issues/2069, now the public study is shown in the owner/shared section so the button is automatically added
and the idea that the owner's studies should always be in their own list even if public but for other users it should be in the public list

Note that these changes will make a lot of the tests fail but want to see which are they ...

Coverage decreased (-0.03%) to 91.929% when pulling 4ecff714f7f7911243748c6b39c4de903402937c on antgonza:user-kept-public-study into b5c7fd5e699049c0ccb16b1b3e63d905fdfc75d5 on biocore:master.

This is ready for review.

Coverage decreased (-0.02%) to 91.94% when pulling 219f479b95a18bdb451d93e2650bc87b573888bc on antgonza:user-kept-public-study into b5c7fd5e699049c0ccb16b1b3e63d905fdfc75d5 on biocore:master.


FIX CI

Pull Request Test Coverage Report for Build 649

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.985%

Totals

Change from base Build 646:
0.0%

Covered Lines:
1226

Relevant Lines:
1558

💛  - Coveralls


MyCarePlanner 2.3.0 - odd behavior regarding user data, share and withdraw links

After deploying MyCarePlanner 2.3.0 to our POC environment, I tested its operation for SDS integration.  In this process, I observed some odd behavior, detailed below.
Initial launch against OHSU with Rose Pink Fhir:
After initial launch, MyCarePlanner loads data for Rose Pink Fhir from OHSU's FHIR server, and associated data from the SDS.  Links are presented to Share and Withdraw, as expected.

After connecting to Cerner Sandbox:
I then connected to the Cerner sandbox via third-party endpoint integration, and loaded Wilma Smart's records into the app.  Doing this worked as expected, but an odd side effect was observed, insofar as the Share and Withdraw links disappeared:

After reloading the page:
In an attempt to reacquire the Share and Withdraw links, I refreshed the page using my browser's Reload button.  This reloaded the app, which brought back the Share and Withdraw links.  However, in so doing, Rose Pink Fhir's data was removed, leaving only data from the SDS and from Wilma Smart:

It seems that the Share and Withdraw links shouldn't disappear after connecting to a third-party endpoint.  Nor does it seem that reloading the app should delete original user data from the app.
Please investigate.

Refresh reloads the entire app and is supposed to get rid of all the data and start from scratch. Will make another card so that closing the tab removes all data and refresh would be functionally like closing the tab.
Will fix the bug for this card where "Share" and "Withdraw" links disappear for this card.


allow for users to load user defined modules

Allow for users to load user defined modules. How this could happen and what checks need to be in place is unclear.

This was fixed in #56.

This was fixed in #56.


Not working on 0.29.1

I'm not sure why I'm getting this error.
I have installed v1.9.0 and RN 0.29.1
If I run react-native link react-native-fabric, I get
rnpm-link info Linking react-native-fabric android dependency
rnpm-link ERR! It seems something went wrong while linking. Error: path must be a string
Please file an issue here: https://github.com/rnpm/rnpm/issues
/Users/Guest/code/testapp/app/node_modules/promise/lib/done.js:10
      throw err;
      ^

TypeError: path must be a string
    at TypeError (native)
    at Object.fs.openSync (fs.js:584:18)
    at Object.fs.readFileSync (fs.js:431:33)
    at applyPatch (/Users/Guest/code/testapp/app/node_modules/react-native/local-cli/rnpm/link/src/android/patches/applyPatch.js:6:29)
    at registerNativeAndroidModule (/Users/Guest/code/testapp/app/node_modules/react-native/local-cli/rnpm/link/src/android/registerNativeModule.js:23:3)
    at /Users/Guest/code/testapp/node_modules/react-native/local-cli/rnpm/link/src/link.js:43:5
    at process._tickCallback (node.js:406:9)
    at Function.Module.runMain (module.js:449:11)
    at startup (node.js:141:18)
    at node.js:933:3

If I run rnpm link, it says everything's ok, but MainApplication.java is not modified
The same error with react-native-device-info.
It populates settings.gradle and build.gradle, it's just MainApplication.java that's missing.

see
https://github.com/rnpm/rnpm/issues/184
https://github.com/rnpm/rnpm/issues/185
https://github.com/rnpm/rnpm/issues/186

@Kureev, 0.29.1 was released and this is supposed to be fixed there.
https://github.com/facebook/react-native/releases/tag/v0.29.1
I have RN 0.29.1, so it should work, but it's not happening for me.
Can you please reopen the issue and check it ?

Have you used react-native link? This repo has been merged into react-native and it literally lives it's last days. I don't think we release any more releases on rnpm (but as I said, it's part of the react-native now, so please, check it out!).

yes, that's exactly what I did
$ react-native link react-native-fabric
$ react-native link react-native-device-info

none of them are updating MainApplication.java
I'll try to debug further, but I think there may be a legit issue here.

@jbrodriguez, I am having the same problem - though I believe that line 30 and 35 in reigsterNativeModule.js wasnt adjusted to the change. When adjusting these the MainApplication.java is patched, but the import line fails. As of now I am just manually adding it.

ref: https://github.com/facebook/react-native/pull/8612

hotfix: https://github.com/facebook/react-native/pull/8807

Faced this problem on 0.30 too when linking realm.

Please, use "react-native link" instead.


more variants no weight name

Latest failure: https://batch.hail.populationgenomics.org.au/batches/483622/jobs/2
We think that this line is the issue:

if ((sum(outList$NumUltraRare_GroupVec) + sum(outList$NumRare_GroupVec)) > 0) {

so trying adding all variants back in, but modifying the weight name

running: https://batch.hail.populationgenomics.org.au/batches/483632/jobs/1


flag provided but not defined: -login_server

Problem/Motivation
Hello,
I've got a problem with the tailscale addon, i can't specify my headscale server, because if i do so, then i get the following error message:
flag provided but not defined: -login_server
I think there is a problem, because the login server flag should look like this: --login-server
Expected behavior
Working flag
Actual behavior
False flag
Steps to reproduce
Start the tailscale addon in home assistant and set a custom login server
Proposed changes
Fix the flag
I hope someone can fix this flag issue :)
Greetings

fixed in #184


Ability to run Counterfact without OpenAPI

Counterfact works fine without an OpenAPI description. It just won't generate code for you. You have to create JavaScript* files manually under the paths directory. The easiest way to implement is to supply an empty OpenAPI object if no file is given. In the CLI, a - can stand for the empty OpenAPI object.
npx counterfact - ./api

The - could become the default value for the OpenAPI file path argument. If you're happy for files to be in the default location (the current working directory), it could be run with no arguments at all.
npx counterfact

*Either JavaScript or TypeScript would work, but TypeScript without the automatically generated types would not be fun.

Running without OpenApi would easier with one of the archived recording features:
https://github.com/pmcelhaney/counterfact/issues/451
or
https://github.com/pmcelhaney/counterfact/issues/452
Then you could just turn on proxy, start recording, and have it build out your files (if not your types). Just mentioning here to possibly bump up those archived feature requests for later discussion.

This was implemented in 0.41.0


[WIP] Different base image

What has changed:

base image. So, python has the version 3.7 now;
added some utils which members of the research team always add by hands;
deleted the installation of some packages.

I've decided to use a different base image because of this issue:
https://neuromation.slack.com/archives/CASUYPPK6/p1589448924103700
With the official pytorch image there is no error occurred during the following simple test:

What do you folks think?

Note: I fixed linter for tests and pushed to your branch

I also tried to add nvtop to the base image, but I decided not to do it because of this issue.

Thank you very much for this PR! :tada:

LGTM as it's better to have later versions of python and torch.
I'm afraid many tests on pip deps will fail, will try to fix them.
Is assert cuda.is_available() a guarantee that torch can see GPU?
Also, it's better to test that tensorflow works on GPU

assert cuda.is_available()
I also tried to run an inference with a small model on GPU. It worked and there was no error about drivers version.
But of course, I'll test the image in the proper way a little bit later.

Tests failed on test_wandb because of a incorrect token (

I have tested this image with my current pose estimation project. Everything worked fine, torch==1.5.1 works.
But I still have problems with the time of computations with half-precision.

I also want to mention that WORKSPASE has also been changed. Now WORKSPASE=/workspase. If there could be any problems because of it, just change it or ask me to do it.

I've updated TF version. tensorflow==2.2.0 works file with 10.1 cudatoolkit.

Now WORKSPASE=/workspase

For now, I changed it to / as it is in the older image. Later, we'll change it to WORKDIR /project that's being used by our Template.


Prompted to install Graph extension

I've attached to our trial Cosmos DB Gremlin account from the Azure Databases VS Code (macOS) extension.
I can drill down to the database but when I click on the Graph I'm presented with this message:

Clicking the View Extension button doesn't result in product page appearing in VS Code or browser.  Searching for 'Cosmos DB Graph' in the extensions marketplace isn't returning a match.

Is there any way to request you to unarchive the graph extension? Or incorporate it in to this extension?

Is there any way to request you to unarchive the graph extension? Or incorporate it in to this extension?

We don't currently have plans to unarchive the graph extension, or to incorporate it into this extension.
There is a documentation page about visualization tools: https://learn.microsoft.com/en-us/azure/cosmos-db/gremlin/visualization-partners, and there is a graph explorer on the Azure Portal.

@alexweininger I came accross this github repository just now and when I saw the gremlin demo on the front page, I was instantly sold.
I was then confused to see it not work and am now very disappointed to read this issue.
Please remove the demo from your front page in order not to waste people's time.

Attempting to open the graph explorer will result in an error notification with a link to this page: https://learn.microsoft.com/en-us/azure/cosmos-db/gremlin/visualization-partners


[DUOS-612][risk=no] Fix document sort bug

Addresses
https://broadinstitute.atlassian.net/browse/DUOS-612
Changes
Minor fix for how we're sorting documents by sort date.

Thank you @ahaessly !


Styled components break when using styled method.

I tried to use the styled method of styled-components and add  {styled(
Test
)`font-size: 50px`} to the styled-components example. I get this error:
styled-components.es.js?b1c4:1997 Uncaught Error: Cannot create styled-component for component: [object Object]
    at constructWithOptions (styled-components.es.js?b1c4:1997)
    at styled (styled-components.es.js?b1c4:1932)
    at new _default (App.js?601b:48)
    at eval (createClassProxy.js?2555:95)
    at instantiate (createClassProxy.js?2555:103)
    at _default (eval at proxyClass (createClassProxy.js?2555:290), <anonymous>:4:17)
    at mountIndeterminateComponent (react-dom.development.js?5a91:8038)
    at beginWork (react-dom.development.js?5a91:8229)
    at performUnitOfWork (react-dom.development.js?5a91:10215)
    at workLoop (react-dom.development.js?5a91:10279)
    at HTMLUnknownElement.callCallback (react-dom.development.js?5a91:540)
    at Object.invokeGuardedCallbackDev (react-dom.development.js?5a91:579)
    at invokeGuardedCallback (react-dom.development.js?5a91:436)
    at renderRoot (react-dom.development.js?5a91:10357)
    at performWorkOnRoot (react-dom.development.js?5a91:10963)
    at performWork (react-dom.development.js?5a91:10916)
    at requestWork (react-dom.development.js?5a91:10832)
    at scheduleWorkImpl (react-dom.development.js?5a91:10715)
    at scheduleWork (react-dom.development.js?5a91:10677)
    at scheduleTopLevelUpdate (react-dom.development.js?5a91:11140)
    at Object.updateContainer (react-dom.development.js?5a91:11178)
    at eval (react-dom.development.js?5a91:15190)
    at Object.unbatchedUpdates (react-dom.development.js?5a91:11049)
    at renderSubtreeIntoContainer (react-dom.development.js?5a91:15189)
    at Object.hydrate (react-dom.development.js?5a91:15251)
    at render (index.js?db1f:14)
    at eval (index.js?db1f:23)
    at Object.../../../../src/index.js (app.da3efc30.js:790)
    at __webpack_require__ (app.da3efc30.js:679)
    at fn (app.da3efc30.js:89)
    at Object.0 (app.da3efc30.js:3564)
    at __webpack_require__ (app.da3efc30.js:679)
    at app.da3efc30.js:725
    at app.da3efc30.js:728

index.js?458a:2177 The above error occurred in the <_default> component:
    in _default
    in AppContainer

or i get 240 errors at once and the browser tab freezes. Any ideas on that? Thank you!

Can you post a proper snippet of your code?

This is an improper usage of styled-components, as you are not passing a component, but instead passing JSX.
Try this:
const H1 = styled(props =>
)`font-size: 50px`
// or simply
const H1 = styled.h1`font-size: 50px`
// Then use it in render:

Hello!


Question - programming

Hi
Im am new at this 😅.
is it possible to do any programming of the module ?.
i would need to have the mic recieved from bluetooth  headset, and also the sound output from the board OFF, until a button is activate :).
Case: walkietalkie mic and headset, by bluetooth
The radio have: two phone jacks. One for output, one for mic.
BUT, it is ONE WAY, communication. So i am not able to answer, when another is talking.
so if the output for mic is activated, all communication will be blocked 🙈. So i need to have at button to push when walking.
(Also i will have limited space for the bluetoot device/board).
Is it possible ? Or do i need to sesrch for something else ?
best
Michael
DK

Not sure I understand the scenario you are describing, but the KCX_BT_EMITTER is a transmit-only device. If you were going to use Bluetooth in both directions you might want to look for a module that does that. If I misunderstood please let me know.

Sorry my mistake :).


Line directive in generated code for @typeparam is incorrect

Steps to reproduce

Enable <EmitCompilerGeneratedFiles>true</EmitCompilerGeneratedFiles> in a razor project.
Create a Razor component called "Component.razor" with this content:

@typeparam TSomeVeryLongName where TSomeVeryLongName : class

The generated code contains #line directives, which are not pointing back to the original positions:
    public partial class Component<
#nullable restore
#line 1 "E:\RazorLines\RazorLines\Component.razor"
TSomeVeryLongName

#line default
#line hidden
#nullable disable
    > : global::Microsoft.AspNetCore.Components.ComponentBase
#nullable restore
#line 1 "E:\RazorLines\RazorLines\Component.razor"
where TSomeVeryLongName : class

#line default
#line hidden
#nullable disable

The first line directive for TSomeVeryLongName misses the @typeparam offset in the original file.
The second line directive for where TSomeVeryLongName : class misses the offset for @typeparam TSomeVeryLongName.
Expected behavior
Enhanced line directives are used to point back to the right position in the original file
Actual behavior
The incorrect line directives break GetMappedLineSpan which leads to bugs like https://github.com/SonarSource/sonar-dotnet/issues/8417.

I think this will be fixed with https://github.com/dotnet/razor/pull/10090

Can confirm this is fixed as of #10090.


The pi type trilogy

This issue is reference issue for the upcomming pi type trilogy, dividing the old RFC into 3 parts.

#1931: Core proposal.
#1932: with bounds on pi types.
#1933: Breaking runtime-compile time barrier.

First rfc up: #1931

Thank you very much @ticki for continuing to champion this feature proposal.  I appreciate all your hard work.

@edwinb @david-christiansen

Discussion from the first RFC that's worth noting here:

@rkruppe: As already pointed out in previous discussion, these really aren't dependent types or Pi-types, at least not until #1933 gets accepted. I had to carefully read both RFCs to come to this conclusion, so this should really be clarified to make this RFC precise and self-contained. A good term that has been used before is "const-dependent types".
@withoutboats: In addition to being inaccurate I think "pi types" is not a very explanatory choice of name for most users. Const parameters seems more obvious.
@eddyb: "const generics", "const generic parameters" and "const parameters" are all better (both user-facing, and even in implementation details) than Π IMO.

We don't talk about Σ types at all when we're discussing Rust's type system and I think that talking about Π types is equally silly.

@clarcharr
I fully agree. Only the last proposal is true dependent types.

Well, that's because Rust doesn't have Σ-types either, surely?

@ticki As I read through the RFCs, I continue to be amazed at how well-thought-out they are! Thanks for the effort :smile:

@Twey I'm guessing by Σ-types @clarcharr was refering to Rust enums/tagged-unions.

I'll make time to read these in the next few weeks, but so far my reading beyond the first amounted to ctrl-F "lifetime", but..
I wondered if one could create a value type from a runtime value and lifetime, so that the value acted as basically an implicit parameter, while the lifetime dictates where the proof must resolve.  As an example :
Imagine we've some crate that requires a struct Params with configuration parameters.  Right now, we have roughly three choices :

We can pass &Params, or Params if small enough, but doing so requires runtime "type" checks for agreement between our different Params structs.  And this agreement might not be simple equality, like say dimension checks in matrix multiplication.

We can pass a &static Paramsthus constraining them to being defined at compile time, so maybe nowdebug_asserts suffice for verifying compatibility between our Params` instances.

We could replace Params with a trait containing various associated constants and types that users must impl for a ZST.  In this way, we do this our checks for agreement of our Params at compile time.  We can even avoid some unsized types in this way, either by defining them as associated types, or by using a network of traits, like what the generic-array crate does.  It's kinda hideous though.

I think 3 will become increasingly popular regardless, but const dependent types, or just a fragment there of, makes doing so clean and idiomatic, as opposed to a morass of trait hacks, and they let you avoid unsized types.
Now can we push this messy trait based Params back into runtime with trait objects?  We actually do not want to anyways because we do not want to erase the type like that.  Also, I'd imagine object safety rules forbid 3 in trait objects, presumably meaning no object safe generic arrays too.
You can nevertheless imagine an implicit parameter being set by a type, and the compiler being asked to prove its correctness, even if under the hood this parameter gets passed at runtime.
Can you return a [u8; n] on the stack where n is runtime?  In principle sure, but the caller must know n to perform the alloca, well n is part of the type of the callee after all.
I think 2 actually provides a useful way to imagine these hypothetical "lifetime dependent types".  There is some lifetime 't inside which we can define some 't types that depends only on immutable values from outside 't.  In compilation, the compiler must prove the 't types check, almost as if it were inserting asserts everywhere and running only the value computations that happen inside the 't types.  It's actually more complex than that however.
Apologies for rambling here.

sigma types are a part of rust, due to DSTs. &[T] is equivalent to &(sigma N: usize, [T; N]). Important to note :)

Hmm, I'm not sure I buy that.  (A, B) is equivalent to Σ(_ : A). B, but that doesn't make it a Σ-type either.  More pertinently, there's no reflection of the dependency in the type system — if xs: &[T], you can get xs.len(), but you can't access anything of type [T; xs.len()].

Right. From the last time I was confused about this and researched it, I believe there are at least three superficially similar but different beasts in this space: existentials, weak sums, and strong sums. I think "sigma type" applies to both of the latter(?), but that it's usually used to mean strong sums. The difference is best discernible if you have a local variable or a top-level constant of the type, that is, whose definition is also in scope. If we think of each of the three as a pair where the type of the second component may refer to the 'value' of the first, then with existentials, the first component is always erased, and can only be used as an abstract (unknown) value; with weak sums, the first component gives back the same value you put in, but when accessing the second component, its type again refers to an abstract placeholder value in place of the first component; while with strong sums, when you access the second component, its type also retains full knowledge of the first's value ("dependent elimination"). It's the last of these which is usually referred to as a "dependent pair".
So I believe Rust's trait objects correspond to existentials, and its slices to weak sums.
(Unfortunately I didn't save the links where I read about this, but if anyone's interested, hopefully googling should lead to the same results.)

Closing this issue since the associated RFCs have all been closed.

Thanks again @ticki ! This was epic.

This paper might be relevant:
https://www.google.co.uk/url?sa=t&source=web&rct=j&url=http://drops.dagstuhl.de/opus/volltexte/2018/9218/pdf/LIPIcs-ECOOP-2018-13.pdf&ved=2ahUKEwj27tHC9sveAhUpLsAKHRiwB4UQFjAAegQIBBAB&usg=AOvVaw3cUw88ELS2ROyQu34OJbJw
It proposes adding a subset of dependant types to a mutable Java-like language by tracking aliases and using move semantics.
It can be fully verified at compile time.
Any linear inequality over the integers is a valid constraint.
As such I think it could fit in with the Rust paradigm very well.