Null-coalescing operator with a "throw" expression messes with dart2js ineterceptors

Reduced test case (see on Dartpad: https://dartpad.dartlang.org/35b1478eec218cf4275a588ff7e679c8)
main() {
  var mayBeNull = new DateTime.now().millisecondsSinceEpoch != 1 ? 'foo bar baz' : null;
  var nullCoalescedString = mayBeNull ?? (throw new Error());
  print(nullCoalescedString.runtimeType); // prints 'String'
  print(nullCoalescedString.indexOf('bar')); // prints `null` unexpectedly
}

Replacing (throw new Error()) with other expressions results in correct behavior, with the indexOf line printing 4 instead of null. Expressions I tried:

a number literal (this is a strong mode error, so it's not completely relevant)
a call to a void function (this is a strong mode error, so it's not completely relevant)
a string literal
null

Interestingly enough, it also happens in this simplified case... (thanks @georgelesica-wf)
main() {
  var nullCoalescedString = 'foo bar baz' ?? (throw new Error());
  print(nullCoalescedString.runtimeType); // prints 'String'
  print(nullCoalescedString.indexOf('bar')); // prints `null` unexpectedly
}

On DartPad today the second example prints 4 instead of null so it looks like this was fixed.


flyctl: 0.0.402 -> 0.0.403

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/flyctl/versions.
meta.description for flyctl is: "Command line tools for fly.io services"
meta.homepage for flyctl is: "https://fly.io/"
meta.changelog for flyctl is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.403 with grep in /nix/store/qzivhxk44p1hj9a1z4nff4ggjnznlzf1-flyctl-0.0.403
found 0.0.403 in filename of file in /nix/store/qzivhxk44p1hj9a1z4nff4ggjnznlzf1-flyctl-0.0.403

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
flyctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qzivhxk44p1hj9a1z4nff4ggjnznlzf1-flyctl-0.0.403 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A flyctl https://github.com/r-ryantm/nixpkgs/archive/00d53d212b4cfa317287f3670fd215bdbb5891d3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qzivhxk44p1hj9a1z4nff4ggjnznlzf1-flyctl-0.0.403
ls -la /nix/store/qzivhxk44p1hj9a1z4nff4ggjnznlzf1-flyctl-0.0.403/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

flyctl

Maintainer pings
cc @aaronjanse @jsierles @techknowlogick @viraptor for testing.

Result of nixpkgs-review pr 194186 run on x86_64-darwin 1

1 package built:

flyctl

$ results/flyctl/bin/flyctl version
flyctl v0.0.403 darwin/amd64 Commit: v0.0.403 BuildDate: 1970-01-01T00:00:00Z


Status still pending after agreement

URL to the linked Repo/Org:
Steps to reproduce the problem:

What is the expected result?
What happens instead?

Could you please retrigger the CLA check by using this API as mentioned here - https://github.com/cla-assistant/cla-assistant/issues/520#issuecomment-571590565 ?

Hi,
I am not sure what to fill in for the organization name, since I’m not a member of any…
Kindest,
Linda
Van: Akshay Iyyadurai Balasundaram<EMAIL_ADDRESS>Verzonden: dinsdag 16 februari 2021 16:28
Aan: cla-assistant/cla-assistant<EMAIL_ADDRESS>CC: L. Bakker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [cla-assistant/cla-assistant] Status still pending after agreement (#662)
Could you please retrigger the CLA check by using this API as mentioned here - #520 (comment) https://github.com/cla-assistant/cla-assistant/issues/520#issuecomment-571590565  ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cla-assistant/cla-assistant/issues/662#issuecomment-779910858 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARARYOSEWNASLRMK6CUQC3LS7KFGNANCNFSM4XWUOIBQ .  https://github.com/notifications/beacon/ARARYOWV6LFFBHDAMIWDJBTS7KFGNA5CNFSM4XWUOIB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFZ6H5SQ.gif

It seems that I’ve agreed the CLA with another user name (‘LBakkerTDS’) where it should have been ‘lmbakker’. I have changed my user name to ‘lmbakker’.
Could you please send the CLA link again, so that I can resign it with the proper user name? Thanks a lot!
Van: Akshay Iyyadurai Balasundaram<EMAIL_ADDRESS>Verzonden: dinsdag 16 februari 2021 16:28
Aan: cla-assistant/cla-assistant<EMAIL_ADDRESS>CC: L. Bakker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [cla-assistant/cla-assistant] Status still pending after agreement (#662)
Could you please retrigger the CLA check by using this API as mentioned here - #520 (comment) https://github.com/cla-assistant/cla-assistant/issues/520#issuecomment-571590565  ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cla-assistant/cla-assistant/issues/662#issuecomment-779910858 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARARYOSEWNASLRMK6CUQC3LS7KFGNANCNFSM4XWUOIBQ .  https://github.com/notifications/beacon/ARARYOWV6LFFBHDAMIWDJBTS7KFGNA5CNFSM4XWUOIB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFZ6H5SQ.gif

Could you please send the link to the repository where you are facing issue with CLA assistant ?

https://github.com/rdkcentral/Lightning-SDK
Van: Akshay Iyyadurai Balasundaram<EMAIL_ADDRESS>Verzonden: dinsdag 16 februari 2021 19:29
Aan: cla-assistant/cla-assistant<EMAIL_ADDRESS>CC: lmbakker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [cla-assistant/cla-assistant] Status still pending after agreement (#662)
Could you please send the link to the repository where you are facing issue with CLA assistant ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cla-assistant/cla-assistant/issues/662#issuecomment-780032934 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARARYOUIXA6DP5N5CN7NS7TS7K2PVANCNFSM4XWUOIBQ .  https://github.com/notifications/beacon/ARARYOQCD7JSA33HLLZP7VDS7K2PVA5CNFSM4XWUOIB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFZ7FXJQ.gif

Hi, there is no need to take action on this issue anymore. The problem was an inconsistent user name and email address.
Kind regards,
Linda
Van: Akshay Iyyadurai Balasundaram<EMAIL_ADDRESS>Verzonden: dinsdag 16 februari 2021 19:29
Aan: cla-assistant/cla-assistant<EMAIL_ADDRESS>CC: lmbakker<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [cla-assistant/cla-assistant] Status still pending after agreement (#662)
Could you please send the link to the repository where you are facing issue with CLA assistant ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cla-assistant/cla-assistant/issues/662#issuecomment-780032934 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARARYOUIXA6DP5N5CN7NS7TS7K2PVANCNFSM4XWUOIBQ .  https://github.com/notifications/beacon/ARARYOQCD7JSA33HLLZP7VDS7K2PVA5CNFSM4XWUOIB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFZ7FXJQ.gif

Thanks so much for informing us. Then I will close this issue.


.NET CORE3

Preview5 has a problem ，EMIT，Entity mapping error.Bad IL format

Old version is good.

The target version is preview5.
:octocat: From gitme iOS

Please create minimal repro that is actionable.
Once you have that, feel free to reopen.


Adding free format form & research from Error

Adding embed form and links

@pezholio
This is the Error Creative stuff getting ready to go up tomorrow
When adding in the report (as a blogpost and 2 images) my local instance starts up ok but none of the site will display. Error message below. Can you have a quick look?

undefined method `delete_if' for nil:NilClass
Rails.root: /Users/70a/Documents/Open Addresses/oa site/theodolite
Application Trace | Framework Trace | Full Trace
Request
Parameters:
{"path"=>"main"}

Hmmmm... That's odd. Seems to work fine for me. Can you try restarting it? (ctrl + c on the terminal and then rails server again)

Did that, twice, just in case ;-) Will reboot.

No joy. Here's the log from Terminal:
https://gist.github.com/peterkwells/6e6215fa6ee087247a47

That's really odd. Seems related to the Jekyll-esque thing we have. I don't know too much about it, but @Floppy may be able to help. You haven't made any uncommitted changes have you?

Found it: https://github.com/OpenAddressesUK/jekylly/issues/2

Ah ha! Weird that it doesn't do it for me

Random unnamed folders don't sync!

Oooooooooooh I seeeeeeeeeeeee!

Merging


Coolify Redis Connection Failure: Temporary Name Resolution Error, I can't enter in coolify

Description
Encountered a persistent issue with Coolify where the application fails to connect to Redis, resulting in the error: "php_network_getaddresses: getaddrinfo for coolify-redis failed: Temporary failure in name resolution." This problem arises after deploying the app with Docker and using Coolify for orchestration. Additionally, the Coolify container enters a restart loop following a server reboot, exacerbating the connectivity issue and rendering the application inaccessible.
Minimal Reproduction (if possible, example repository)
create a docker compose on coolify and put docker build -t q080scg-laravel:latest . on custome build commande
Exception or Error
When trying to access Coolify befor restarting, the following error message is displayed:
`Something is not okay, are you okay?
There has been an error, we are working on it.
Error: php_network_getaddresses: getaddrinfo for coolify-redis failed: Temporary failure in name resolution`
After rebooting the server, the Coolify container enters a continuous restart loop. Here are the relevant logs:
`--------------------------------------------------------------------

/ |  ___ _ ____   _____ _ __  / |() | | ___  | | | | __
_ \ / _ \  \ \ / / _ \  | _ | |/ ` |/ _ \ | | | |  _
) |  __/ |   \ V /  __/ |     ) | | (| |  __/ | || | |) |
|/ _||    _/ _||    |/||_,|_|  _/| .__/
||
Brought to you by serversideup.net
To support Server Side Up projects visit:
https://serversideup.net/sponsor
GID/UID
User uid:    9999
User gid:    9999
🔓 SSL_MODE has been DISABLED, setting the web server to work in HTTP only...
🏃♂ Checking for Laravel automations...
🔐 Linking the storage...
ERROR  The [public/storage] link already exists.
[30-May-2024 20:31:47] NOTICE: fpm is running, pid 100
[30-May-2024 20:31:47] NOTICE: ready to handle connections
[30-May-2024 20:31:47] NOTICE: systemd monitor interval set to 10000ms
In PhpRedisConnector.php line 159:
php_network_getaddresses: getaddrinfo for coolify-redis failed: Temporary f
ailure in name resolution
In PhpRedisConnector.php line 159:
Redis::connect(): php_network_getaddresses: getaddrinfo for coolify-redis f
ailed: Temporary failure in name resolution
s6-rc: warning: unable to start service db-migration: command exited 1
/run/s6/basedir/scripts/rc.init: warning: s6-rc failed to properly bring all the services up! Check your logs (in /run/uncaught-logs/current if you have in-container logging) for more information.
prog: fatal: stopping the container.
[30-May-2024 20:31:49] NOTICE: Terminating ...
[30-May-2024 20:31:49] NOTICE: exiting, bye-bye!
`
Version
coolify:4.0.0-beta.291

As this is quite old I assume it has probably been fixed if not please provide an example repository as I am unable to reproduce this issue. Also please upgrade to the latest version of Coolify v4 and try again, and if this is still an issue, feel free to reopen this issue or create a new one.


Issue18 - Publishing updated docker image on merge-to-master

Plus script to emulate the behavior we expect travis does continuously. Feedback would be appreciated (it took me very long efforts, especially because of travis-docker images are ugly :( )

See also the updated project description in docker hub: https://hub.docker.com/r/dachad/tcpgoon/

This should cover https://github.com/dachad/tcpgoon/issues/18 completetly

i've done a first reading and just small comments, nothing blocking from my side
but I will do a new review to test the scripts and get deep into the bash scripts
Note: The history of commits makes it a bit difficult to follow

Comments replied. Found one minor bug while testing some of the checks you were mentioning. New commit added. And... why do you care so much of the history behind this branch? I will squash+merge when this is integrated into master, and I don't know why you look at previous commits :( (imho, just review the latest version of the diff/PR)


🚑 [fix] restore network list functionality

This commit fixes a MAJOR error which was an incidental deletion of the network chain URL causing the CLI to error out instead of returning the list of manuscripts when using the init command.
Description
An oversight caused code to be pushed that knocked out the network list fundamentally breaking the CLI init command.
This emergency fix, fixes basic CLI functionality by restoring the endpoint.
Type of Change

[X] Bug fix

After looking at the git diff before merging, it appears - the issue existed only on my local branch.


Loading extension by full path only?

According to Third-party Extensions — MathJax 2.6 documentation, it is said that

Usually, third-party extensions have to be specified with their full paths (and matching loadComplete calls); this limits portability. To simplify this process, the MathJax configuration can include (possibly multiple) third-party locations for easier reference.

So, 3rd party extension can be loaded from full paths only?
And what does it mean by "possibly multiple"? Does it only mean different extensions can have different locations only? Or does it mean I can specify multiple locations for the same third-party extensions as a fallback option?
Fallback for 3rd party extensions
In my site I have fallback options (by fallback.js) to load CSSs and JSs. But in MathJax, to load a third party extension, one need to write (using the example from the quoted link above):
<script type="text/x-mathjax-config">
  MathJax.Ajax.config.path["Contrib"] = "//cdn.mathjax.org/mathjax/contrib";
</script>

And I'm unsure how one can load this with a fallback option.
Can we load 3rd party extensions with Relative Path?
If MathJax allow a relative path for the extensions, then a fallback can be done easily. e.g. if MathJax is loaded from https://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS_CHTML-full, then loading the extension from mathjax/contrib/ means cdn.mathjax.org/mathjax/contrib.
In this way, if the Fallback loads a fallback path, say, locally, then mathjax/contrib can follow the domain to the local 3rd party extension.
There are other advantages to allow relative path too. e.g. when building a site locally in development, it can load the local copy instead of the uploaded copy.
Sorry it was quite messy up there, I will try to summaries it here:

question:

Are there any way to load the 3rd party extension with a fallback in the current version of MathJax?
Is there a way to load the 3rd party extension with relative path but not full path?

feature request:

Allow 3rd party extension to load with relative path.

Please let me know if I misunderstand anything and if there's already ways to handle what I'm asking. Thanks.

 Although the documentation mentioned that it has to be a full path, but I just tried and a relative path is ok for specifying the MathJax.Ajax.config.path["Contrib"] =  path. I should have tested before I ask.
Anything, may be this can be considered to a suggestion to a change in the documentation.

Sorry for the silence. Glad to see you worked it out yourself!


Crash upon server launch

servercrashreportApotheosis.txt
I see a listener triggered, but unsure of what the index 3 is identifying:
[01:02:52] [main/ERROR] [ne.mi.fm.ja.FMLModContainer/]: Exception caught during firing event: dev/shadowsoffire/placebo/reload/TypeKeyed
        Index: 3
        Listeners:
                0: NORMAL
                1: ASM: dev.shadowsoffire.apotheosis.Apotheosis$ModularDeferredHelper@40ea0ad5 register(Lnet/minecraftforge/registries/RegisterEvent;)V
                2: ASM: dev.shadowsoffire.apotheosis.ench.EnchModule@3488f950 miscRegistration(Lnet/minecraftforge/registries/RegisterEvent;)V
                3: ASM: dev.shadowsoffire.apotheosis.Apotheosis$ModularDeferredHelper@aaa6391 register(Lnet/minecraftforge/registries/RegisterEvent;)V
                4: ASM: dev.shadowsoffire.apotheosis.adventure.AdventureModule@12b171d0 miscRegistration(Lnet/minecraftforge/registries/RegisterEvent;)V

Should mention this is a server running 1.20.1 with most recent 1.20.1 versions for Apotheosis, ApothicAttributes, Patchouli, and Placebo

duplicate of #1077


Git Pre-Commit with huski

Pre-commit

only commit code that passes ci locally
format code with prettier


Setting User Preferences is called synchronously on page load multiple times and causes page load delays

Setup
2.7.0
Describe the bug
We spend nearly 500ms on every page load and route change synchronously setting the "setLastVisited" user preference and in fact every time we mutate user preferences which is often multiple times per page load. Here's a network timeline filtered to just calls to "userpreferences". Part of the problem is that there's no watcher to subscribe to for userpreferences so we've got to make a full GET request to get the latest preferences for the user prior to any mutation to make sure we're not backing out any change made in another browser or another tab (average looks like 250ms) but then we still have to make our PUT call which average 250ms as well (the two calls you see that stretch into the +500ms range are actually 250ish but they've stalled out waiting for other requests to finish).

To Reproduce
Open developer tools, select the network tab and filter to userpreferences. Login and navigate to any page, notice that userpreferences is called multiple times per page at a time cost of nearly 250ms every time. There's unfortunately no way to know that these calls are blocking other parts of the UI from loading without examining code because we also make several other blocking API calls.
Result
Expected Result
When fixed, there should be anywhere from 0.5 seconds to 1.5 seconds average page load reduction depending on which page, which other requests are being fired simultaneously, network conditions, and machine specs but there should be a measurable if slight (potentially more than slight) reduction in page load times.
Screenshots

Neil has reduce the frequency we need to save preferences. We should also try to optimise the save process. For example can we avoid fetching preferences and instead handle conflict errors on save?

However, this may have been addressed via other performance work so we should do an assessment first to see what the current timing is like, then figure out what else to do.

Low priority, can be bumped.
Possible solution, local cache that wouldn't block and is synced independently.
Accessed via ui new api, some could be preferences in local storage, some rancher userpreferences


Add examples for grants and other submodules

I see there are methods in w.grants but they're not documented in the examples path of the repo.
Also would be nice to add a README.md in the examples folder (or add it to the root README) stating that the structure of the SDK is the same as the URLs of the REST API explorer. That would make it easier to know and navigate the complete functionality of the SDK

added more examples


The symbol is packaged unexpectedly, we will fix it in the next release, you can add the packagingOptions to your build.gradle file to temporarily remove it at this time

The symbol is packaged unexpectedly, we will fix it in the next release, you can add the packagingOptions to your build.gradle file to temporarily remove it at this time
packagingOptions {
      exclude "**/libAgoraRtcWrapperSymbol.so"
 }

Originally posted by @littleGnAl in https://github.com/AgoraIO/Agora-Flutter-SDK/issues/631#issuecomment-1074780686
The agora package in flutter is increasing my application size by 30 mb which is absolutely ridiculous. Excluding this part of the package did nothing to the size. Can you please help me out?

@littleGnAl

Such a small issue still not fixed, disappointed.

The symbol had been removed, you can try agora_rtc_engine: ^5.2.0.
I think the config should be working fine, you can view what caused the increasing size on Android Studio of your APK.


ci: use k8s pod template agents

What does this PR do?
PoC with K8s CI Workers
Checklist

[ ] This is an enhancement of existing features, or a new feature in existing plugins

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] I have made corresponding changes to the documentation

[ ] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

[ ] This is a new plugin

[ ] I have updated CHANGELOG.asciidoc
[ ] My code follows the style guidelines of this project
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have updated supported-technologies.asciidoc
[ ] Added an API method or config option? Document in which version this will be introduced
[ ] Added an instrumentation plugin? Describe how you made sure that old, non-supported versions are not instrumented by accident.

[x] This is something else

[ ] I have updated CHANGELOG.asciidoc

Why

Delegate the Agents configuration to the consumers
Faster provisioning.

For instance, the agent is provisioned almost immediately, compared to the 2 minutes average per CI worker

Actions

[ ] Cached workspace to speed up builds?
[ ] Matrix support for difference openjdk versions

:broken_heart: Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-12-05T15:12:28.410+0000

Duration: 3 min 22 sec

Commit: 725700f9a1834358978fe1ff0eba850ab9232e07

Steps errors
Expand to view the steps failures

mvn install

Took 0 min 18 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 5 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 5 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 5 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 5 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

:grey_exclamation: Flaky test report
No test was executed to be analysed.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergifyio refresh

@Mergifyio update

@v1v do we want to proceed with this, or can I close this PR?

Closing this issue as we are moving away from Jenkins


Unbreak snappy and tensorflow Haskell packages

Description of changes
This unbreaks the Haskell TensorFlow bindings, and - as it's a dependency - the snappy package.
This obsoletes #119411.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] 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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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.

Thx, for fixing this outdated package!
PR looks good and compiles.
For future reference: Feel free to run maintainers/scripts/haskell/regenerate-hackage-packages.sh as part of your PR. That makes it easier to test. I have done it for you this time and pushed that to the haskell-updates branch.

@maralorn Thanks! (And will do next time.)


Rebuild and reload typedefs in dev server on change

This PR adds a watcher for any file in the /type-definitions/ folder and rebuilds the docs on changes. This makes it easier to work on docs and seeing the results without constantly having to rebuild manually.

This looks good to me but Travis is failing for some apparently-flawed reason. Let me try rerunning Travis and if that's good then let's accept this.

@lacker, is this PR good to be merged?

Most excellent.


[modify][cms] rescue from all exceptions on generating pages/nodes

Coverage decreased (-9.5%) to 70.544% when pulling 86d4fadd772eb078ab63be96f37021962c40be65 on sunny4381:features/rescue-with-all into f000a668f4419e8d7cec12f747173117a011fee4 on shirasagi:master.


Fix Broken Firmware Link

It appears that the firmware link was broken, likely due to the release of V4. This pull request proposes to correct the URL.

Thank you for updating!


Fixed missing flags + missing _epi

Small fix regarding compress_store on AVX512

@JPietrzykTUD As a reminder, after I sent you the compile errors i ran into, you told me to add those flags, because they are missing, which fixed my errors. I just pushed those changes.
I reproduced the error. Obviously those flags are not needed. Just the epi was missing.


Update paper.md

Trying to use the suggestion feature. Let's see if this works.

This did not work.


User pics

Description
Profile photo upload working on Employer
browse seekers profile pic rendering correctly
browse jobs profile pic rendering correctly
get matched jobs route populates employer
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

TODO: there should only be one ProfilePicContainer component file (is it a container or a component?) but I didn't want to change a bunch of file names. Another reason to sort out the whole /containers fiasco


luarocks built with Lua 5.3 can't upload rocks

My attempts to upload end:
Sending /home/rrt/.local/var/repo/lrexlib/lrexlib-pcre-2.8.0-1.src.rock ...

Error: API Failed: version_id must be an integer

This is with luarocks 2.2.0. Running an otherwise identical build but with lua 5.2, the upload works.

I believe this is fixed in the latest code from Git. Could you please verify? Thanks!

Installing git luarocks to use my setup is a bit complicated; I'm happy to take your word for it and/or leave this bug open until the next time I upgrade!

Closing this since the latest code was released. If the issue comes up again, please reopen! Thanks!

Thanks, just uploaded my first rock with Lua 5.3!


Updates

feel free to contribute things which you deem useful for other colleagues

Yeah these are mostly some convenience updates I've created for filing the results more easily (e.g., "German" data-format + buy+sell values of the currencies as these are requested in the Anlage) - there were no actual bugs. I can clean that up once more + push for review.
I also added parsing the transactions from a sheet living on google drive, but accessing that is a pita and I might need to clean this up still / make it more solid (see one of the commits above)


Update image-version.md

command was wrong. was referncing managed image instead of VMs

@sandeepraichura
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Remove VS warning

1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrApplicationWindow\CtrlrDocumentPanel.cpp(19,56):
warning C4305: 'argument': truncation from 'double' to 'float'
 setBackgroundColour((Colours::lightgrey).darker(0.2));
becomes:
 setBackgroundColour((Colours::lightgrey).darker(0.2f));
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(357,35): warning C4458: declaration of 'canvasHeight' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(82,16): message : see declaration of 'CtrlrPanelEditor::canvasHeight' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(358,34): warning C4458: declaration of 'canvasWidth' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(83,16): message : see declaration of 'CtrlrPanelEditor::canvasWidth' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(359,40): warning C4458: declaration of 'canvasAspectRatio' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(84,16): message : see declaration of 'CtrlrPanelEditor::canvasAspectRatio' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
            double canvasHeight = getCanvas()->getHeight();
            double canvasWidth = getCanvas()->getWidth();
            double canvasAspectRatio = double(canvasWidth) / double(canvasHeight);

becomes:
            canvasHeight = getCanvas()->getHeight();
            canvasWidth = getCanvas()->getWidth();
            canvasAspectRatio = canvasWidth / canvasHeight;

because they are already declared as double in CtrlrPanelEditor.h
!!! To get confirmation from Damien if those are not local variables rather than class variables (then do not change but use other variable names)
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.cpp(92,20): warning C4458: declaration of 'constrainer' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.h(169,30): message : see declaration of 'CtrlrEditor::constrainer' (compiling source file ....\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.cpp)
            `if (auto* constrainer = getConstrainer())`

becomes
           ` if (constrainer = getConstrainer())`

because they are already declared in CtrlrEditor.h
!!! To get confirmation from Damien if those are not local variables rather than class variables (then do not change but use other variable names)
Why auto*?

FIXED:  CtrlrDocumentPanel.cpp
CtrlrDocumentPanel::CtrlrDocumentPanel (CtrlrManager &_owner)
    : ctrlrEditor(0), owner(_owner)
{
	/* Full screen mode is not completely implemented in JUCE 6
	   we get some assertions when adding the first Tab to an invisble
	   panel, as it wants to grep the focus, which is not possible at this stage.
	*/
	// useFullscreenWhenOneDocument (true);
    setSize (600, 400);
    setBackgroundColour((Colours::lightgrey).darker(0.2f)); // Added v.6.30. Updated v5.6.31 for (0.2f). Sets background colour behind main window by default on grey to please everyone :)
}

FIXED: CtrlrPanelEditor.cpp
 else if (property == Ids::uiPanelCanvasRectangle)
        {
            getCanvas()->setBounds(VAR2RECT(getProperty(property))); // update canvas size if values in the property field are changed
            canvasHeight = getCanvas()->getHeight(); // Updated v5.6.31 by GoodWeather. Removed type double(canvasHeight)
            canvasWidth = getCanvas()->getWidth(); // Updated v5.6.31 by GoodWeather. Removed type double(canvasWidth)
            canvasAspectRatio = canvasWidth / canvasHeight; // Updated v5.6.31 by GoodWeather. Removed type double(canvasAspectRatio) = double(canvasWidth) / double(canvasHeight)
            setProperty(Ids::uiViewPortFixedAspectRatio, canvasAspectRatio); // update canvas aspect ratio if canvas is resized
            resized();
        }

FIXED :
auto* var = getFunction() //  auto* stresses better the intent that var is a pointer.
Both should work  though it can be
auto var = getFunction()
 if (auto constrainer = getConstrainer())
Needs to be checked with VS if the warning goes away with that mod.
if (!JUCEApplication::isStandaloneApp() && owner.getInstanceMode() == InstanceSingleRestriced)
            {
                setResizable(vpResizable, true);

                if (auto constrainer = getConstrainer()) // According to GoodWeather, it *auto returns type warning in VS
                {
                    if (vpEnableFixedAspectRatio == true)
                    {
                        constrainer->setFixedAspectRatio(vpFixedAspectRatio);

                        if (vpEnableResizableLimits == true)
                        {
                            if (vpMinWidth != 0 && vpMaxWidth != 0)
                            {
                                setResizeLimits(vpMinWidth, round(vpMinWidth/vpFixedAspectRatio), vpMaxWidth, round(vpMaxWidth/vpFixedAspectRatio));
                            }
                            else if (vpMinWidth != 0 && vpMinHeight != 0 && vpMaxWidth != 0 && vpMaxHeight != 0)
                            {
                                setResizeLimits(vpMinWidth, vpMinHeight, vpMaxWidth, vpMaxHeight);
                            }
                        }
                        else
                        {
                            constrainer->setMinimumSize(editorRect.getWidth(), editorRect.getHeight());
                        }
                    }
                    else if (vpEnableResizableLimits == true && vpMinWidth != 0 && vpMinHeight != 0 && vpMaxWidth != 0 && vpMaxHeight != 0)
                    {
                        setResizeLimits(vpMinWidth, vpMinHeight, vpMaxWidth, vpMaxHeight);
                    }
                }
            }

Some other ones to fix. The issue comes that there is one declaration in the header file (at class level) then another one in the .cpp file (my interpretation, I'm not a C++ specialist).
CtrlrPanelEditor
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(408,35): warning C4458: declaration of 'canvasHeight' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(109,16): message : see declaration of 'CtrlrPanelEditor::canvasHeight' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(409,34): warning C4458: declaration of 'canvasWidth' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(110,16): message : see declaration of 'CtrlrPanelEditor::canvasWidth' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp(410,40): warning C4458: declaration of 'canvasAspectRatio' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.h(111,16): message : see declaration of 'CtrlrPanelEditor::canvasAspectRatio' (compiling source file ....\Source\UIComponents\CtrlrPanel\CtrlrPanelEditor.cpp)
CtrlrEditor:
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.cpp(92,20): warning C4458: declaration of 'constrainer' hides class member
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.h(169,30): message : see declaration of 'CtrlrEditor::constrainer' (compiling source file ....\Source\UIComponents\CtrlrApplicationWindow\CtrlrEditor.cpp)
CtrlrStandaloneWindow.cpp
1>G:\2. Source code for build\CtrlrX <IP_ADDRESS>\Source\Core\StandaloneWrapper\CtrlrStandaloneWindow.cpp(118,12): warning C4458: declaration of 'constrainer' hides class member


mandibular hyperostosis

mandibular hyperostosis
DEF: hyperostosis (bony overgrowth) of the mandible
SOURCE: HP:0004472 (include synonyms below)
has_broad_synonym: excessive growth of mandibular bone; thick lower jaw bone; overgrowth of mandibular bone; thick mandibular bone
has_exact_synonym: hyperostosis of lower jaw; increased ossification of mandibular bone; hyperostosis of mandibular bone
has_related_synonym: increased ossification of lower jaw; increased ossification of mandible
Annotations in J:168526 (MGI:4889220), J:186384 (MGI:5432491), J:67105 (MGI:3852640)

mandibular hyperostosis MP:0030223


Add more information and refine the information on the about, home, and contact pages

The home page needs some more information, as well as some graphics and images, so it is not simple plain text.

All of the pages are pretty boring with just plain-text so making UI more appealing is the goal.


Update concepts-maintenance.md

Add important notice of avoid server operation during maintenance

@code-sidd
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Refactor TextInputField to use new Input component

See Slack thread

This uses an older input field, which I think someone updated to kinda look like the new one, but isn't really. Renaming this issue to be a refactor task to use our new component.


Can not show.py

I try show.py.
But I can not.
$ python3 show.py
name= cityscapes
train size: 2975
val size: 500
Traceback (most recent call last):
  File "show.py", line 358, in <module>
    show_cityscapes_model()
  File "show.py", line 337, in show_cityscapes_model
    show(model,val_loader,device,show_cityscapes_mask,num_images=num_images,skip=skip,images_per_line=images_per_line)
  File "show.py", line 134, in show
    outputs = model(images)
  File "/home/sounansu/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/sounansu/RegSeg/model.py", line 76, in forward
    x=self.stem(x)
  File "/home/sounansu/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/sounansu/RegSeg/blocks.py", line 22, in forward
    x = self.conv(x)
  File "/home/sounansu/.local/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/sounansu/.local/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 446, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/sounansu/.local/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 442, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

Can you git pull and try again? I made some changes.

Thank You!  RolandGao.
I can do show.py, and pop up this windows.


Responding with a DirectMessage

Hello! Thank you for a great platform!
I am currently building a bot, and I'd like to be able to respond to users in a direct message. I'm a bit new with C#, so I couldn't find the required information straight away, so I thought to contact here.
For examples sake, lets say I was using the !ping listener, how would I begin forming the message as a DM, or does this function not yet exist?
Thanks!

Going off of the example bot from the README, responding to DM's actually already works. In the example, we are grabbing the channel that the message was created in as an ITextChannel. ITextChannel represents the common functionality between DM channels and guild text channels.
Of course you can also limit the code to only respond to DM's. One way would be to check ITextChannel.ChannelType before responding to see if it's equal to DiscordChannelType.DirectMessage.
Another option is casting the result from the cache to a DiscordDMChannel, for example:
DiscordDMChannel dm = shard.Cache.Channels.Get(message.ChannelId) as DiscordDMChannel;
if (dm != null)
{
    dm.CreateMessage("Hello!");
}

Hope this helps! Also if you haven't seen it, we do have a wiki over here.

Sure opens it up a bit :) Thank you, I'm currently reading through your docs.
Actually I went off explaining the wrong way, sorry for that.
What if I wanted to answer !command in a chat channel as a DM to the user that invoked the !command?
Sorry for making such trouble.

So what you're trying to do (and correct me if I'm wrong) is receive a message via a guild channel, and respond to the user who created that message through a DM?
Currently the only way to do this is through an HTTP call. Assuming the message object is named msg and your DiscordWebSocketApplication is named app you would want to do:
DiscordDMChannel dm = await app.HttpApi.Users.CreateDM(msg.Author.Id);

That DiscordDMChannel will be between your bot and the user who created the message.
I should probably make a wiki entry for this :)

Thank you so much! I will try this out today. :) :+1:


Background mode: Remove deleted files

Would be a useful feature for the background mode. I´m not too sure how to do this as I just started coding ABAP a couple weeks ago, but given the size of some repositories checking if every file still exists in the repo sounds horrible. Maybe theres some Event fired when a file is deleted which we could intercept?

Hi,
abapGit always checks all files, as there is no events to catch(assuming the SAP standard code is not changed)

merged, closing


ci: unit_tests: merge coverage with HIL tests

tests/unit_tests/test.sh is very verbose, while it is possible to do most
of it with single ctest command.
Use Ninja generator to build using multiple threads automatically.
Install both gcovr and ninja from pypi, so a single tool (pip) is used.
Move unit_tests job from lint_build_unit_test.yml to hil_tests.yml,
so that coverage data can be easily combined as part of single workflow.
Also rename lint_build_unit_test.yml to lint_build.yml, as there are no
more unit tests as part of this workflow.

There's a lot of little improvements here that have been bugging me for a while! I wasn't a fan of calling out to another shell script for testing, I wonder if we can just delete that whole script now? I also wonder if we should rename hil_tests.yml to just test.yml since we have a few tests in there now which don't run on hardware.

Good suggestions, I'll create followup PRs with those changes!


Babel Macros - not working

Hi, it would seem I cannot import the macro version of styled-components:
https://www.styled-components.com/docs/tooling#babel-macro
import styled from '@xstyled/styled-components/macro';

Failed to compile.
Error: Cannot find module '@xstyled/styled-components/macro'

Thought trying also importing from @xstyled/macro but no luck.

Importing directly from styled-components does work well:
import styled from 'styled-components/macro';

Background:
We have an app based on create-react-app, not ejected, and not using rewire, because CRA supports babel macros, and I wish to utilize that ability to be able to use the babel-plugin-styled-components plugin.
I need this so my styled-components will be generated with a real display name, to be able to access components in enzyme unit tests.
Related thread

Hello, yes it is not yet supported. If someone want to work on it, feel free. Else I will probably do it in september.

I've implemented xstyled in dozens of files in a large app thinking I could use the Babel macro later (for testing) but was surprised it did not work.
Sadly, I do not have the time to look into this, because I am working full time, plus have my own Github open-source projects to maintain, with plenty of open issues to address
Enjoy your summer holiday!
I highly appreciate your major contribution to the styled-component comunity.

@neoziro Shall I work on this?

@jmlweb yes you can!

@neoziro My idea is to create a Higher Order Functions over the current logic so that you can pass the utils from the styled-components version required (standalone or macro). Something like this:
export const buildCss = scCss => {
  return function css(...rawArgs) {
    const scCssArgs = scCss(...rawArgs)
    const flattenedArgs = flattenStrings(scCssArgs)
    return flattenedArgs.map(transform)
  }
}

and then, you use it like this:
import { css as scCss } from 'styled-components'
import { buildCss } from './_internal'

export const css = buildCss(scCss)

or
import { css as scCss } from 'styled-components/macro'
import { buildCss } from './_internal'

export const css = buildCss(scCss)

Does it looks good, or do you have a better implementation idea?

It should work yes. No I don't have better idea.

It seems it is not that simple...
It seems we need to export the enhanced version of the Styled Components macro, but I still had no luck.
We'll try to learn a bit about macros before taking this task.

Hi, anyone knows if there is active development on this?

I took a look, but I don't have any idea about babel macros, so I got no success... Feel free to take it :)

@jmlweb - I know even less 😝
Are we stuck with nobody that knows and can help?
Are we doomed refreshing this issue once a month to see no activity?

I am afraid I am the only one involved who knows how to do it. Unfortunately it is not a required feature for me, so I can't give you a date for that feature.

Hey @neoziro
I started to work on this issue today, but actually i think there is some issues between babel-plugin-macro and rollup.
The umd version don't even build at all :D
cf:
most recent one : related issue
oldest: oldest issues
And it doesnt seem to be solved...
My work around on this was to remove the name from the rollup config of the package styled-components  to build only the cjs and esm version.

And still there is some issues
when files are builded instead of having the import as:
... = require('styled-components/macro')
// we have
... = require('styled-components')

so i used the plugin babel-plugin-transform-rename-import, but still when i test it with a yarn link it throw an error so i am like 🤯! the error is probably related to this issue

@Wizyma you have to create a specific folder inside the styled-components folder with a package.json and create a specific rollup build for that. So the macro will not have umd. It is like a package inside another.

@neoziro yeah i thinked about that but i didnt knew how much liberty i could take to do what i need for that :D
And then we should expose a @xstyled/styled-components/macro i think

Any progress in this? This is mega important because I cannot eject my React app and I need automatic classnames for my styled-components for Selenium E2E tests.. Thanks.. This package is deeply integrated in our app in hundred+ files..

Looks like the styled-components issue has been fixed in CRA:
https://github.com/styled-components/styled-components/issues/2713#issuecomment-557802907
I would suggest roll your own babel macro if the problem persists.

Still an issue for v2.

Still an issue for v2.

I know that people would like the macro to work, but for reference in the meantime, here is the formula for using the plugin with xstyled and CRA

Install rescripts (replacement for rewire) and the plugin

yarn add -D @rescripts/cli @rescripts/rescripts-env babel-plugin-styled-components

Apply this change to package.json

--- a/package.json
+++ b/package.json
   "scripts": {
-    "start": "react-scripts start",
-    "build": "react-scripts build",
-    "test": "react-scripts test",
-    "eject": "react-scripts eject"
+    "start": "rescripts start",
+    "build": "rescripts build",
+    "test": "rescripts test"
+  },
+  "rescripts": [
+    "env"
+  ],
+  "babel": {
+    "presets": [
+      "react-app"
+    ],
+    "plugins": [
+      [
+        "babel-plugin-styled-components",
+        {
+          "topLevelImportPaths": [
+            "@xstyled/styled-components",
+            "@xstyled/styled-components/no-tags",
+            "@xstyled/styled-components/native",
+            "@xstyled/styled-components/primitives"
+          ]
+        }
+      ]
+    ]
   },

If you don't already import React at the top of your components, you'll get a build error, so you'll need to add to each one:

import React from 'react'

and then you'll get nice debuggable classes in your dev tools.
@gregberge It would be super cool if the classes generated by <x.div ...> also had classes for debugging, e.g. Filename__LineNo but I haven't looked at what would be involved for that. This kind of debuggability is the biggest concern that I hear from devs working on our projects internally.

@agriffis it is not related to Babel Macro, but I agree with you! https://github.com/gregberge/xstyled/issues/190

There are two things preventing the macro from working:

It is hard-coded to 'styled-components', see https://github.com/styled-components/styled-components/blob/master/packages/styled-components/src/macro/index.js#L18-L21
This is the patch against upstream that fixes it:

diff --git a/packages/styled-components/src/macro/index.js b/packages/styled-components/src/macro/index.js
index b6285a7f..74c3be92 100644
--- a/packages/styled-components/src/macro/index.js
+++ b/packages/styled-components/src/macro/index.js
@@ -4,7 +4,12 @@ import traverse from '@babel/traverse';
 import { createMacro } from 'babel-plugin-macros';
 import babelPlugin from 'babel-plugin-styled-components';

-function styledComponentsMacro({ references, state, babel: { types: t }, config = {} }) {
+function styledComponentsMacro({
+  references,
+  state,
+  babel: { types: t },
+  config: { importModuleName = 'styled-components', ...config } = {},
+}) {
   const program = state.file.path;

   // FIRST STEP : replace `styled-components/macro` by `styled-components
@@ -15,10 +20,10 @@ function styledComponentsMacro({ references, state, babel: { types: t }, config
     // generate new identifier
     let id;
     if (refName === 'default') {
-      id = addDefault(program, 'styled-components', { nameHint: 'styled' });
+      id = addDefault(program, importModuleName, { nameHint: 'styled' });
       customImportName = id;
     } else {
-      id = addNamed(program, refName, 'styled-components', { nameHint: refName });
+      id = addNamed(program, refName, importModuleName, { nameHint: refName });
     }

     // update references with the new identifiers
@@ -29,7 +34,14 @@ function styledComponentsMacro({ references, state, babel: { types: t }, config
   });

   // SECOND STEP : apply babel-plugin-styled-components to the file
-  const stateWithOpts = { ...state, opts: config, customImportName };
+  const stateWithOpts = {
+    ...state,
+    opts: {
+      ...config,
+      topLevelImportPaths: (config.topLevelImportPaths || []).concat(importModuleName),
+    },
+    customImportName,
+  };
   traverse(program.parent, babelPlugin({ types: t }).visitor, undefined, stateWithOpts);
 }

With that patch, you can configure in package.json:
  "babelMacros": {
    "styledComponents": {
      "importModuleName": "@xstyled/styled-components"
    }
  }

The second problem is that the upstream macro bundles an older version of the plugin, before topLevelImportPaths was supported. Even if you have the newer version installed, or if you use "resolutions" to try to force it, it won't work. The upstream repo needs to be rebuilt and released with the updated plugin.

Anyway, all of this needs to become an issue and PR upstream, but I have to work on other things at the moment. If someone wants to run with it, please go ahead, otherwise I will get to it eventually.

Pull request is in! Unfortunately it also looks like styled-components upstream is on hiatus, so it could take a while.
Also, the PR is against v6 in development, and then it would need to be backported to the legacy-v5 branch. I have a lot of uncertainty about how quickly this will move forward.
If you want to give it a thumbs up, head over to https://github.com/styled-components/styled-components/pull/3422
If you need something earlier, you can yarn add -D @scampersand/styled-components.macro and then add to your package.json
  "babelMacros": {
    "styledComponents": {
      "importModuleName": "@xstyled/styled-components"
    }
  },

then you can import styled from<EMAIL_ADDRESS>and it should work.

Quick update
This has been merged now to styled-components v6 in development, see https://github.com/styled-components/styled-components/pull/3422
I've also opened the PR to get it on v5 legacy, which is the current stable version, see https://github.com/styled-components/styled-components/pull/3432
That's progress, but unfortunately we're still between releases. There haven't been any v6 development releases, and the last v5 release was in October 2020, so it's hard to say when this will be available for general use. Feel free to give the v5 PR a thumbs-up, it might help.
In the meantime you can use @scampersand/styled-components.macro as mentioned above, it's effectively the same as what I put the in the PR for styled-components

Great news, styled-components 5.2.2 was released with this fix :tada:
https://github.com/styled-components/styled-components/releases/tag/v5.2.2
I think all we need now is an update to xstyled docs and we can close this.


Add serde_path_to_error, Add path in JSONSerialize

Adds path to JSONSerialize and QueryStringSerialize error
Fixes #205

CI approved

Thanks for the quick response, seems like I need to continue to manually approve CI. Running now :)

All green! Thanks for the patience (and for getting a fix upstream!) :tada:


Why is minSDK 15?

Hello There,
can you please tell me why is minSDK set to 15?

The main reason is eventually (someday) support easier Activity binding using ActivityLifecycleCallbacks.
We could lower it to 14, but that would be pointless (since the difference in users between 14 and 15 is nonexistant). But if it would make people happier I can do that.

As far as i can see ActivityLifecycleCallbacks was added in api level 14.
Is there any bug in api level 14 using ActivityLifecycleCallbacks? I've some apps where the userbase have 3-5% api level 14. for now I use the tools:overrideLibrary but i don't see any reason why we would like to stick with api level 15.
I know RXBinding library had a similar issue and it was raised down to 14. https://github.com/JakeWharton/RxBinding/pull/73

Why are you on 14 instead of 15?

Hello dlew, not me personally.
As mentioned in the ticket i attached a popular library is reporting 4.97% of users having 4.0.
If you don't want to go down to api level 14 for any reason i'll respect that. No worries :) I was just wondering why you decided to higher the minSdk to 15 since all the code within this library should be compatible at least with 14. (ActivityLifecycleCallbacks included)
Thank you for your time :+1: and for the great lib

The chart shown does not differentiate between 4.0 (14) and 4.0.3 (15). If you consider that all their 4.0 is actually 4.0.3 then their chart is roughly in line with what Android currently projects: https://developer.android.com/about/dashboards/index.html
That said, I'm going to bump it down to 14 anyways just so that I don't have to keep fielding this question (since I anticipate seeing it a lot otherwise). It won't make any difference regardless.

Hello. What the reason not to set minSdk to 10 or 9? I mean what features/classes need api 14?

Because those are ancient and no one should have to support them.


The option --project-version is now mandatory when interacting with dependency-track --server-url

We went from version 4.x up to version 8.x, and while working well with python and NodeJS projects, the bump of version from version 4 to version 8 did fail.
We finally figured out that --project-version is now mandatory when uploading BOMs.
This is a change in cdxgen (not in dependency-track), because using the old command line with versions up to 4.x does still work.
--project-version seems to have the wrong value and in such case, dependency-track will return an HTTP 400 Error with recent versions of cdxgen. cdxgen does not output a clear message, so, I had to print debug it to find the root cause.
I suspect we might forbid usage of --server-url without --project-version to avoid people looking at why it fails.
I will likely propose a PR for that

@pierresouchay was the 4.x versions using “master” as the default project version? Perhaps I can add the same default again for dependency track submissions?

@prabhu Here is a minimum package project failing:
{
  "name": "cdxgen-test-node",
  "version": "1.0.0",
  "description": "A test package to check that cdxgen works well",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "repository": {
    "type": "git",
    "url": "https://gitlab.redacted.example.com/my-group/my-project/"
  },
  "keywords": [
    "test"
  ],
  "author": "Pierre Souchay",
  "license": "ISC",
  "dependencies": {
    "json5": ">=2.2.3"
  }
}

Running it with:
cdxgen -r -o bom.xml --server-url $DEPTRACK_HOST_URL --api-key $DEPTRACK_TOKEN --project-name MY_PROJECT --fail-on-error
did fail with dependency track in its latest version (v4.7.1).
The version we used to run was the latest published in nodejs: @appthreat/cdxgen@^4 => we replaced with @cyclonedx/cdxgen@^8
And the fix: https://github.com/CycloneDX/cdxgen/pull/257 will make the missing data explicit

@pierresouchay Thank you for sharing. Could I parse the package.json file and default to 1.0.0 for the project version? This way it becomes more automation friendly.

@pierresouchay Thank you for sharing. Could I parse the package.json file and default to 1.0.0 for the project version? This way it becomes more automation friendly.

That might beba reasonable change, but I suspect it would break some existing confs

@pierresouchay Good point. Perhaps we stick with defaulting to master to keep it safe?

Thank you for raising the issue and for sharing a test case. Please try 8.1.6, which includes the fix. Please let me know if we still require your PR.

Thank you for raising the issue and for sharing a test case. Please try 8.1.6, which includes the fix. Please let me know if we still require your PR.

I won't because we already added --project-version master everywhere, but it might help some!


Remove calls to supports and supports back to back

Part of (extracted from):

[ ] https://github.com/ManageIQ/manageiq/pull/22898

No reason to call the same block twice.
This is probably pedantic.
Typically, the value of the first call would be cached for the second call.
But getting away from the duplicate calls is preferred.

update:

rebased (fixed master build so kicking this)


JWT token authentication

This project will be migrating from a session-based authentication mechanism to a token-based system.
A JSON Web Signature should be created both client-side and server-side.  Signatures should be used to sign any data exchanged past the initial authentication post.
The JSON Web Signature should also be used to sign a JSON Web Token used for authentication.  The token should be returned after initial authentication.  Token will not be saved in sessions.
Please feel free to suggest amendments to this solution, as all input regarding the security of the final product is welcome.

Token auth has been implemented, removing the need for a session store.  Passport has been removed for the time being, may be re-implemented later if a need for external authentication methods arises.
Client-endpoint JWS encryption is being withheld for the moment.  Plans to implement do still exist, yet need to be reviewed and better constructed.


Small improvements to website copy

Spelling, punctuation and other small improvements.

This is good stuff, I'm going to look at merging https://github.com/microsoft/TypeScript-Website/pull/2031 first as it's a bit bigger then will come back for this - thanks!


proposal: cmd/go: GOTOOLCHAIN=mod to use exact version of toolchain directive

Proposal Details
Currently, the local Go runtime determines the actual Go version to use by checking the toolchain directive in the go.mod file when GOTOOLCHAIN=auto is set. If the local Go runtime is newer than the version specified in the toolchain directive, it defaults to the local Go version. This behavior aligns with the Go Toolchain documentation.
However, in some cases—such as in production environments—it may be necessary to enforce a specific Go runtime version, even though Go maintains strong forward and backward compatibility. For example, if the toolchain directive specifies 1.22.8 but the Go runtime in the container is 1.23.2, the actual version used would be 1.23.2. This could lead to unexpected issues if the container runtime is updated (even toolchain is never changed), which should sometimes be blocked to prevent unexpected factors.
While setting GOTOOLCHAIN=<version> is a valid solution but usually this GOTOOLCHAIN environment variables might be in disorder in a project; this is against the DRY principle. I believe it would be more convenient to have an option that always respects the version specified in the toolchain directive as the actual runtime version. This would allow developers to control the Go version for an application simply by modifying the version in the toolchain directive.
As a potential solution, how about introducing an option like GOTOOLCHAIN=mod to achieve this behavior?

Related Issues and Documentation

extending Go forward compatibility
extended forwards compatibility for Go #57001 (closed)
cmd/go: toolchain directive in go.mod being updated unnecessarily #65847
Forward Compatibility and Toolchain Management in Go 1.21 > Toolchain Management
Go Toolchains > Go toolchain selection
Go Toolchains > Introduction
cmd/go: implement GOTOOLCHAIN=auto
cmd/go: GOTOOLCHAIN=version+auto doesn't use the named version in go install package@version #69051
Go Modules Reference > go.mod files > toolchain directive
go run not adhering to toolchain of go.mod when specifying package version #69817 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

If you need to use a specified Go toolchain version, you can try GoTV.

If you can enforce GOTOOLCHAIN=mod, then you can also enforce GOTOOLCHAIN=target-version.
That said, from experience, if you need to enforce versions, it's better to do that at the time of providing the toolchain into the build environment, including in providing the container. This avoids any possibility of environment misconfiguration and inefficiencies resulting from needing to download and perform a toolchain switch.

GOTOOLCHAIN=target-version is for all Go projects built on the machine. OP needs a project specific way.

In general we don't want to make it too easy for people to downgrade their go versions.
GOTOOLCHAIN=mod is a forced downgrade eventually, and that's bad.
Avoiding downgrades is the same reason we have minimal version selection.
You can think of GOTOOLCHAIN as being minimal version selection for Go toolchains.
GOTOOLCHAIN=go1.23.4 is like 'go get<EMAIL_ADDRESS>and selects an exact version.
But any non-explicit setting should be some kind of max (minimum value satisfying the input requirements)
that includes the fact that you installed 1.23.6 as a signal that you want to use 1.23.6 or newer.


Tab Group doesn't save over multiple windows

This is the issue. When used over multiple windows the same group has two contents. How should the program "remember" tab groups over multiple windows?

This is how I purposefully designed it. Window IDs change so I didn't want something finnicky to be attached to an ID that changes.


Etcd snapshot restore fails with blank error if token is missing

Environmental Info:
K3s Version:

k3s version v1.22.3+k3s-ef263bd2 (ef263bd2)
go version go1.16.8
Node(s) CPU architecture, OS, and Version:

Ubuntu 20.04, master commit ef263bd2b0ecb29c3404cd4b80fcfcf8b4301912
Cluster Configuration:

1 server and 1 agent node
Describe the bug:

When initializing or reseting cluster, if config file is not present and required config does not gets passed, k3s does not start and throws empty error making it hard to identify what caused the failure. In this case, an etcd snapshot restore was performed without passing in the token flag and the code spit out FATAL without any reason
Steps To Reproduce:

Installed K3s

curl -sfL https://get.k3s.io | INSTALL_K3S_EXEC="--write-kubeconfig-mode 644 --token secret --cluster-init" INSTALL_K3S_COMMIT=ef263bd2b0ecb29c3404cd4b80fcfcf8b4301912 sh -

Took snapshot

k3s etcd-snapshot \
  --s3 \
  --s3-bucket=<BUCKET_NAME> \
  --s3-access-key=<ACCESS_KEY> \
  --s3-secret-key=<SECRET_KEY> \
  --s3-region=<REGION> \
  --s3-timeout=50s

Shutdown k3s

systemctl stop k3s

Perform etcd snapshot restore without passing token flag (no config file was defined)

k3s server \
  --etcd-s3 \
  --etcd-s3-bucket=<BUCKET_NAME> \
  --etcd-s3-access-key=<ACCESS_KEY> \
  --etcd-s3-secret-key=<SECRET_KEY> \
  --etcd-s3-region=<REGION> \
  --cluster-reset \
  --cluster-reset-restore-path=<PATH>

Expected behavior:

Should specify why it failed, like missing token
Actual behavior:

Spits out [FATAL]
Additional context / logs:

Backporting

[x] Needs backporting to older releases

Verified in k3s master with commit d93c82f9585ed9e0ff7ebe95b1506f4587fffc1c on single node cluster
Steps:

Install k3s on 1 server and 1 agent node
Take snapshot of the cluster
Stop k3s server and remove token sudo rm -rf /var/lib/rancher/k3s/server/token
Perform cluster-reset and got
FATA[0000] /var/lib/rancher/k3s/server/token does not exist, please pass --token to complete the restoration
Perform cluster-reset with --token flag
Validated cluster-reset was successful


Unified configuration

This PR implements a unified way of managing the global/thread-local configuration of Chainer. As a first step, it implements the core part of configuration (GlobalConfig and LocalConfig), and moves some easy configurations (debug mode, type check, and backprop mode) to this configuration tool. The main features are:

We can easily add a new entry just by setting an attribute to chainer.global_config.
with chainer.using_config('XXX', yyy): lets us change the thread-local configuration temporarily.
We can diagnose the current configuration with chainer.config.show().

I did not finish #2048 by this PR; train/test mode and use_cudnn flag are not moved to this configuration yet, because both affect a wide range of codes in Chainer and its examples, and I want to separate PRs for them. I will work on them after this PR being merged.

@unnonouno Resolved the comments.

LGTM

Check flake8 result.

@unnonouno fixed relevant codes for flake8.

Please check travis result.

Sorry, I re-fixed it. I think the code format issue is resolved also for Travis.

LGTM!


Explanatory no results on Workflow C.1

https://arax.ncats.io/?source=ARS&id=786c8995-8457-4892-bf01-830738a27d4c

{
  "description": "Success. 0 results found",
  "logs": [
    {
      "code": "KPEmptyResponse",
      "level": "WARNING",
      "message": "Knowledge Provider https://translator.broadinstitute.org/molepro/trapi/v1.1/query returned an empty knowledge graph",
      "timestamp": "2021-07-29T15:39:25"
    }
  ],
  "message": {
    "knowledge_graph": {
      "edges": {},
      "nodes": {}
    },
    "query_graph": {
      "edges": {
        "e00": {
          "object": "n01",
          "predicates": [
            "biolink:related_to"
          ],
          "subject": "n00"
        }
      },
      "nodes": {
        "n00": {
          "categories": [
            "biolink:ChemicalSubstance"
          ]
        },
        "n01": {
          "categories": [
            "biolink:Disease"
          ],
          "ids": [
            "MONDO:0005301"
          ]
        }
      }
    },
    "results": []
  },
  "status": "Success",
  "validation_result": {
    "message": "",
    "n_edges": 0,
    "n_nodes": 0,
    "provenance_summary": {
      "n_sources": 0,
      "predicate_counts": {},
      "provenance_counts": {}
    },
    "size": "1 kB",
    "status": "PASS",
    "version": "1.1.1"
  },
  "araxui_provider": "ARS",
  "araxui_response": "67d7907e-bfc0-40ed-9566-154e2ef79958"
}

Explanatory Agent currently can't handle queries formatted with a known object and an unknown subject within the query graph. We are currently in process of a design update to handle such a query and are expecting it to go live by the start of September Relay.


Serialize layer introduction

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
yes

Tests pass?
yes

Fixed issues

Inspired by https://github.com/yiisoft/yii2-queue/pull/147
'Serialize' abstraction layer introduced, allowing easy support for different serialization methods, which are in demand by data operation libraries like yii2-queue.
Serialization layer applied for SimpleCache.

Good addition. Thank you.


New long table for UMAP plot for biomarkers

Make query to output long table where each row is a cell type and a biomarker. UMAP: each dot will be a cell type.
dataset_id | cell_label | cell_id | organ | tool | b_label | b_mean_expression
Link to existing query: https://github.com/hubmapconsortium/ccf-grlc/blob/main/hra-pop/datasets-with-ct.rq

Query: https://github.com/x-atlas-consortia/hra-pop/blob/main/queries/atlas-ad-hoc/datasets-ct-bm-data.rq
Report: https://github.com/x-atlas-consortia/hra-pop/blob/main/output-data/v0.9.1/reports/atlas-ad-hoc/datasets-ct-bm-data.csv


Add ephemeral storage to cert-manager presubmit

Add ephemeral storage to cert-manager presubmit.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/woof

/hold

/hold

/lgtm-cancel

/lgtm

/unhold
/approve


[TensorAlgebra] Matricized QR tensor decomposition

This defines a function for performing a matricized QR decomposition of a tensor.
The high level API is:
julia> using NDTensors.TensorAlgebra

julia> using LinearAlgebra

julia> a = randn(2, 2, 2, 2);

julia> q, r = qr(a, (:a, :b, :c, :d), (:a, :c), (:b, :d));

julia> q
2×2×4 Array{Float64, 3}:
[:, :, 1] =
 -0.320888   0.370723
 -0.306767  -0.815775

[:, :, 2] =
  0.47314   -0.653369
 -0.515355  -0.289234

[:, :, 3] =
 -0.369393  -0.595887
  0.616975  -0.357504

[:, :, 4] =
 -0.732611  -0.283888
 -0.509553   0.350778

julia> r
4×2×2 Array{Float64, 3}:
[:, :, 1] =
 2.9428  -0.54139
 0.0      2.80448
 0.0      0.0
 0.0      0.0

[:, :, 2] =
 -0.225018  -0.0430216
  1.07535    0.0289476
 -1.58676    0.550383
  0.0        0.0227928

julia> a′ = TensorAlgebra.contract((:a, :b, :c, :d), q, (:a, :c, :qr), r, (:qr, :b, :d));

julia> norm(a - a′)
1.21523787027731e-15

We can also provide a permutation-based API, for example qr(a, (1, 3), (2, 4)) which would be equivalent to the example above.
There is currently a bug in TensorAlgebra.contract which is marked with a broken test so some label combinations can't be properly checked with TensorAlgebra.contract, I'll investigate that in a future PR. But the main point here is to get started with the general code patterns involving matricize and unmatricize for performing matricized tensor decompositions, i.e. these patterns and a lot of code can be shared with SVD and other decompositions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3f1afb8) 85.44% compared to head (a8fd705) 54.70%.

:exclamation: Current head a8fd705 differs from pull request most recent head f9d2e7b. Consider uploading reports for the commit f9d2e7b to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1266       +/-   ##
===========================================
- Coverage   85.44%   54.70%   -30.75%
===========================================
  Files          89       88        -1
  Lines        8402     8349       -53
===========================================
- Hits         7179     4567     -2612
- Misses       1223     3782     +2559

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

Nice – the interface looks really intuitive.


Question: can you use sentry-testkit to extract tracing / performance data?

Not sure if sentry-testkit only provides exception reports or if I just haven't configured Sentry correctly. Seems like the former.
Thank you.

can you pls elaborate more?
specify a use case? what performance data? etc...

can you pls elaborate more?
specify a use case? what performance data? etc...

sentry-testkit indeed provides only exception reports, it doesn't yet support the performance metrics product. I just encountered the same need and will try to work on a PR for that.

@ianwalter I don't know if it's still relevant but I opened https://github.com/wix/sentry-testkit/pull/82

@zivl since the PR was merged and the feature released, can we close this issue?

yep!
thanks @ohana54 & @ianwalter


Automatic PR of restapi_auto_v-anpasi_consumption-patch into master

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2400

See https://github.com/Azure/azure-sdk-for-python/pull/1896


Timestamp and # bugs

Hi, I just installed this theme today and tried to create my site, it is live here: https://icaran.github.io/theBlog/

However, I encountered a few problems, in the markdown file for my article, when i put the date date: "2024-08-03", which is today, the article dissapears both from the featured list and the posts page.
I checked around for what it could be and I notice that for month values 01-07, it works just fine, but for values 08-12, it just dissapears from the site. So for now i just set all 'showdate' flags in the params to false, and set the date to a filler date: "2024", which just reverts back to being 1st jan 2000 by default, but it doesnt show up so i guess its fine ¯_(ツ)_/¯

Also, how and where exactly do I make an About page? It's not written in the docs and I'm very new to HUGO so I don't really know how to go about it, right now clicking on 'about' just takes you to 404...

Thanks for making such an awesome theme however, very useful! I also just installed the latest version with git submodule update --remote which you put out just yesterday.

PS: There's another issue you'll see when you visit my blog, the # work for markdown, but they are also visible in plaintext as well for some reason

Regarding date,

I recommend using the hugo new content command to generate new markdown files, as it will automatically populate the date field.
Hugo does not generate content for markdown files with future dates by default. If you didn’t include the UTC offset (e.g., +05:30 for us Indians), Hugo may have defaulted to UTC+0, and therefore put your file in future.
(or) you can place buildFuture = true in config/_default/hugo.toml.

Regarding About,

Check Hugo Doc

PS: There's another issue you'll see when you visit my blog, the # work for markdown, but they are also visible in plaintext as well for some reason

Sorry, I didn't quite understand. Could you give more info?

Thankyou, the date thing did fix itself as you said yeah, thanks for that!
I'll check the about
I thought it was a bug, but the # does look good on the theme
Thanks a lot :)


ArgumentToEncodingTransformationAttribute should be public

be700729d689f7998110a38889b7962e890e387d changed FileSystemProvider's Encoding parameter to use ArgumentToEncodingTransformationAttribute. However this attibute is marked internal so third-party content cmdlet providers cannot use it.
Please make this attribute public, or provide some way for third-party content cmdlet providers to have an -Encoding parameter with the same functionality as the built-in FileSystemProvider.
Edit: The EncodingConversion class and its constants too.

Please!?

The Engine WG discussed this and agree we should make ArgumentToEncodingTransformationAttribute and ArgumentEncodingCompletionsAttribute public.

Make sense to think about better names.

We discussed that but these are unlikely to conflict with anyone who created their own version, and we can always add a type accelerator later if desired.


Fix import, mobile optimizations

What changed?

Fix bug where importing does not copy data over correctly
Make website a progressive web application (PWA)
Create a lightweight toast library that works independently of React
Optimize MovieMentor website for mobile devices.

How was this accomplished?

No-operation bit for import logic (it's base4 <-> base36 now, not base3)
HTML tag changes for PWA-ness

Testing

Manual integrity checking of data import/export.
Tailwind CSS tweaks tested with browser's responsive-design editor.

Ready to merge. We still need to fix the Stream buttons, but I want you to do so because they are just all sorts of fucky. Consider switching to Flexbox and not using Grid.


Newer binutils version need config.h

Newer version of binutils need config.h or #define PACKAGE "Ctrlr" to be set.
Compiling CtrlrLinux.cpp
In file included from ../../../../Source/Misc/include/libr.h:30:0,
from ../../../../Source/Native/CtrlrLinux.cpp:13:
/usr/include/bfd.h:35:2: erreur : #error config.h must be included before this header
#error config.h must be included before this header
Same with Source/Misc/libr.c and Source/Misc/libr-bfd.c
Work around: add #define PACKAGE "Ctrlr" just after #ifdef LINUX

added


NullPointerException in export to RDF

Suspected that it occurs when a column is not classified.

Hi, @josef-janousek !
Please let us know whether @tomas-knap has checked the changes, as you wanted him to do so and as you told me on Thursday. The latest commit in you branch is four days old and since than I miss any signs of progress.
Also consider naming the branches more descriptively in the future (I would like you to adhere the Successful git branching model, as it is good enough for the size of our team. It has almost no restrictions on the name branches, but "Exports" is just to vague.) and (when done with coding) opening the pull request for further discussions.
Thanks!


How to test rime plugins

I cloned librime-sample to the plugins directory, built it, cd into build/bin, and modified default.yaml:
schema_list:
  - schema: sample

... (omitted)

Then I tried to test the plugin with:
echo "yibaiershisanwansiqianlingwushiliu" | ./rime_api_console

But got an error:
initializing...
message: [0] [deploy] start
message: [0] [deploy] success
ready.
E20211010 10:58:43.545047 77576 engine.cc:346] error creating translator: 'trivial_translator'
schema: sample / 樣例
status: composing
[yibaiershisanwansiqianlingwushiliu]|
page: 1$ (of size 5)
1. [yibaiershisanwansiqianlingwushiliu]

Fixed by setting -DBUILD_MERGED_PLUGINS=ON.


Update kubectl to 1.9.7

https://github.com/rancher/rancher/issues/13554

@superseb I want to make sure that kubectl is not a symlink to the hyperkube binary.  kubectl binary should be about 50m, where as the hyperkube is more like 200mb.  That is why I stuck with the k8s images from 1.6.

Yeah makes sense, it's way too big in hyperkube I see.


useAbsoluteScale in plot.antsImage

I thought useAbsoluteScale should plot the full range of the overlay, but is not coded to do so, and it does not do it.
If this parameter is for something else, I suggest we get a new parameter to easily plot the full range. This is often useful when plotting labels.
If this is the parameter that should do it I can try to fix the code to make it do that.

see ?plot.antsImage  window.* ... the scale is set by the parameter window.overlay ... if you set window.overlay = c(  0, 100 ) then any image overlay will map into that range.

Yes, but rather than having to find the min/max values and set windows.overlay, wouldn't it be easier to tell plot() to avoid truncating intensities. Whether in interactive mode or in scripting, setting windows.overlay adds commands each time plot is called.


/elixir/instrumentation/tagging.html is missing

Found an empty page.
https://docs.appsignal.com/elixir/instrumentation/tagging.html
What content should go here? Or should the page be removed?

It should link to the tagging guide: https://docs.appsignal.com/guides/custom-data/tagging-request.html#elixir

Fixed


bug: waitForTransaction through error multiple times

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
1.4.3
Current Behavior
When calling the waitForTransaction function, it sometimes throws an error multiple times. When I put waitForTransaction inside a try/catch block, it catches the error the first time but then continues to throw it again. In another issue, some users mentioned having this problem with both wagmi and viem (#2530,  #2505). One of your developers responded, saying the error is related to the public RPC. That's a valid point, but why does it throw the error multiple times? I believe this behavior is due to the asynchronous nature of the function. Shouldn't it wait for all asynchronous functions to run and then either throw an error or return a result?
Expected Behavior
My expected behavior is for the function to either return the transaction or throw an error only once.
Steps To Reproduce
link to reproduce ( copy from #2530) Please note that this behavior occurs randomly, so the issue might or might not be reproduced.
Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
https://github.com/wagmi-dev/wagmi/issues/2530#:~:text=https%3A//stackblitz.com/edit/new-wagmi-q5cuif%3Ffile%3Dsrc%252Fcomponents%252FNetworkSwitcher.tsx
Anything else?
No response

Public RPCs have aggressive load balancing which causes some functionality that relies on synchronous behaviour like this to break.
Public RPCs should not be used in production anyway. Please try this again with an RPC like Alchemy, Ankr, QuickNode or Infura.

Hi ! I still got this error with ankr rpc on fantom, is there a way to slower the refresh rate or something to avoid this ?

I still got this error with priced rpc.
I have a question. How to enforce wagmi to use injected wallet( window.ethereum) for read action?
@jxom


auto import cleanup doesn't get triggered in open but not active tabs in VS code when i do save all

i do save all and cleanup only happens in the active tab/file

That is a VSCode feature. they call the organize imports on save. the server does not monitor save events. I would file this under https://github.com/Microsoft/vscode/issues instead.

ok


isahc requires http2 feature for http1 connections

Reproduction:
[dependencies]
isahc = { version = "0.8", default-features = false, features = ["static-curl"] }

Attempting to perform any http call results in curl error "Unsupported protocol" unless http2 feature is added.

Thanks for finding this issue! This is definitely a regression in 0.8. I'll publish a patch for this later today.


Canvas Renderer does not respect texture's frame x and y when tinted

Description
When using the canvas renderer with a tinted Sprite, if you assign a new Rectangle to its texture's frame then the new values for width and height are drawn as expected but the new values for x and y seem to be ignored.
To demonstrate the problem, I merged the spritesheet and tinting demos together.
Minimal example of problem: https://jsfiddle.net/5u8sqzwc/1/
Working behavior in canvas without tinting: https://jsfiddle.net/0wv49ack/
Working behavior in WebGL with tinting: https://jsfiddle.net/3a6rotk4/
PIXI.js version: 4.5.1 (also seen on 4.4.1)
Seen on both Chrome and Firefox, on Window 10 and on Mac.
Background
What I really want is to emulate mipmap for the canvas renderer. To do this, I took an existing spritesheet and appended copies of it scaled down just like the GPU does for the WebGL renderer. I used the same .json file for the spritesheet so that the loader knows nothing about the additional pixels off to the right.
When I change the scale of my stage to zoom out, I calculate the new x, y, width and height for each texture's frame based on where it would be in the mipmap area. This works well for any textures that aren't used by tinted sprites. However, if the sprites have a non-white tint, the canvas renderer seems to only take into account the new width and height.

thanks for sharing @nathantalewis , will take a look and get back to ya 👍

Merged fix! Closing.

Thank you guys!


There is already a swarm listening on port (error)

I don't have reproduction steps for this. This happened when destroying a running torrent (not sure if that's related to the error or not).
events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: There is already a swarm with info hash 3f19b149f53a50e14fc0b79926a391896eabab6f listening on port 62108
    at node_modules/bittorrent-swarm/lib/tcp-pool.js:100:22
    at nextTickCallbackWith0Args (node.js:453:9)
    at process._tickCallback (node.js:382:13)

What version of webtorrent? What version of node.js?
This is really odd, because an 'error' handler is always attached to the swarm. See https://github.com/feross/webtorrent/blob/66e8be7a12c22276bcb94ba81dcc5da3a41ceeda/lib/torrent.js#L216

WebTorrent from Git master (cloned 2 days ago) and Node 5.6.
On Mon, 22 Feb 2016 at 03:47, Feross Aboukhadijeh<EMAIL_ADDRESS>wrote:

What version of webtorrent? What version of node.js?
This is really odd, because an 'error' handler is always attached to the
swarm. See
https://github.com/feross/webtorrent/blob/66e8be7a12c22276bcb94ba81dcc5da3a41ceeda/lib/torrent.js#L216
—
Reply to this email directly or view it on GitHub
https://github.com/feross/webtorrent/issues/635#issuecomment-186976596.


How do i enable this ui?

i want this ui in my game, though with the chat button next to it if that's possible.

{
"FFlagEnableInGameMenuChrome": true,
"FFlagEnableChromePinnedChat": true
}


deps: add @dazaar/payment

eos example gets fixed by https://github.com/bitfinexcom/dazaar-payment-eos/pull/5
depends on a dazaar-payment-eos update

Lgtm!


Initial Commit to add EnRoute OneStep Gateway to Civo Marketplace

Thank you for wanting to submit a Pull Request to the Civo Kubernetes Marketplace repository!
If your pull request is to submit a new application to the marketplace, please answer the following questions:

[ ] Have you followed the guidelines in our Contributing document in setting up the application?
[ ] Have you checked that the application your submission is proposing to add does not yet exist on the Marketplace, or as a pull request to be processed?
[ ] Have you tested your application on the Civo managed Kubernetes service? Please include a screenshot.
[ ] Can you confirm that the software you are submitting licensed for use in a publicly-available Kubernetes marketplace?

If your pull request concerns an existing Marketplace application, please make sure you have:

[ ] Notified the Maintainer of the application in this pull request so that they are aware of your proposal.
[ ] Outlined the changes you are proposing, and the reasons these are required (e.g. stability, compatibility with new versions of Kubernetes implementations, etc).
[ ] Tested that the application works with the proposed updates applied (including a screenshot)

@saiyam1814 - attached logo is a square aspect ratio. Lmk if any other changes are needed. Thanks!

LGTM


/usr/bin/ld: libpointpillars.so: undefined reference to `createNvOnnxParser_INTERNAL'

I try  six times ,alway has this error ,help!!!

try to add "target_link_libraries(pointpillars nvonnxparser)" to CMakelists.txt
@luogantt

尝试将“target_link_libraries(pointpillars nvonnxparser)”添加到 CMakelists.txt @luogantt

thank you very very much!


feat(app): update to support meeting part override

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update Google Guava to 31.1-jre

Update dependency Google Guava to 31.1

Dear guice developers. When do you think this update can be done and delivered?

In https://github.com/google/guice/commit/415a377e3bb7fe4497c3ee9f573d2652c1651038 guava was updated to guava 31.0.1. That's not exactly 31.1 but at least its in the 31 line.

is there a plan for a guava 32 based guice version?

No, there is no plan for a new release with Guava 32. Guice does not shade
the guava dependency, so it should be trivial to exclude the transitive
Guava dependency and instead explicitly include the desired version of
Guava. (Binary incompatible changes are extremely rare in Guava --
particularly in the small subset of Guava that Guice uses -- and
re-releasing Guice whenever Guava releases isn't sustainable.)
sam
On Fri, Jun 2, 2023, 4:25 AM Christian Dietrich @.***>
wrote:

is there a plan for a guava 32 based guice version?
—
Reply to this email directly, view it on GitHub
https://github.com/google/guice/issues/1642#issuecomment-1573352996, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQLWESVRH3ZE4DOYCC7HKLXJGPQVANCNFSM5577X4KQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>