feat(highlights): allow overriding highlights

Adding default = true to the highlight group options to allow users to override these.

It looks good,Thank you for your contributions.


When obfuscated with moonsec, I can't save or overwrite configs, "Failed to Encode Data"

this didn't happen before

try using a different obfuscator thats better like luraph

try using a different obfuscator thats better like luraph

Found out when obfuscated Fluent Emojis mess up and then I can't save configs
also im broke i can't use luraph lol


Build failure: yubikey-manager

Steps To Reproduce
Steps to reproduce the behavior:

nix flake update
build yubikey-manager

Build log
error: builder for '/nix/store/1gp0759l8412j54b15gw1f29y6zxwxpz-python3.10-yubikey-manager-5.1.1.drv' failed with exit code 1;
       last 10 log lines:
       > Finished creating a wheel...
       > Finished executing pipBuildPhase
       > installing
       > Executing pipInstallPhase
       > /build/source/dist /build/source
       > Processing ./yubikey_manager-5.1.1-py3-none-any.whl
       > Requirement already satisfied: click<9.0,>=8.0 in /nix/store/scpriyvc9pn5zkccgds8ad53kv6d91l8-python3.10-click-8.1.6/lib/python3.10/site-packages (from yubikey-manager==5.1.1) (8.1.6)
       > ERROR: Could not find a version that satisfies the requirement keyring<24.0,>=23.4 (from yubikey-manager) (from versions: none)
       > ERROR: No matching distribution found for keyring<24.0,>=23.4
       >
       For full logs, run 'nix log /nix/store/1gp0759l8412j54b15gw1f29y6zxwxpz-python3.10-yubikey-manager-5.1.1.drv'.

Notify maintainers
@benley @lassulus @pinpox
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.4.7-xanmod1, NixOS, 23.11 (Tapir), 23.11.20230805.5faab29`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.1`
 - channels(root): `"nixos, nur"`
 - nixpkgs: `/nix/store/hff7x2v1bpgklcyigawncql8p0jhmjpw-source````

duplicate https://github.com/NixOS/nixpkgs/issues/248180


test if EDITOR set

I just saw https://github.com/pimterry/notes/pull/3 after making this PE but editor doesn't exist on mac so I think it is better to just error out.

Great, thanks! Good work on the tests especially 😀. I've actually merged both this and #3, since they're both good ideas and they combine really nicely.
With both of these together, notes now uses $EDITOR if possible, falls back to editor if not, or throws your nice error if neither exists.
Great work all round, thanks so much for contributing 👍


Issues with build.gradle

Hi,
I found an issue while trying to build a RN project with this package installed.
Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.0.1/userguide/command_line_interface.html#sec:command_line_warnings

FAILURE: Build failed with an exception.

* Where:
Build file '/home/martin/work/app/node_modules/react-native-android-wake-screen/android/build.gradle' line: 98

* What went wrong:
A problem occurred configuring project ':react-native-android-wake-screen'.
> No such property: username for class: java.lang.String

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s

The issue seems that the afterEvaluate hook in build.gradle is called even when the "host" app is build, not only when this package is published. I forked this repo and remove the afterEvaluate hook, shall we do the same here?
Thanks,
Martin

Hello there. Sorry I was unable to look into this issue earlier. There seems to be an issue with gradle upgrades bricking this plugin. I'll look into it if I get time later.


#323 RqMultipart.Fake expecting Request as ctor. input

RqMultipart.Fake  accepting Request as ctor. input type. String type version is removed.

@erimerturk Thanks for the pull request, let me find a reviewer..

@longtimeago please review, thanks

@erimerturk please, mention task number in PR description. Thank you!

@erimerturk Thank you! Please, see 2 remarks above

@longtimeago it is ready review, thanks for comments

@erimerturk thank you!

@rultor merge

@rultor merge

@longtimeago Thanks for your request. @yegor256 Please confirm this.

@rultor try to merge

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor try to merge

@yegor256 Oops, I failed. You can see the full log here (spent 12min)

Results :

Failed tests:
  BkParallelTest.requestsAreParallel:127
Expected: <0L>
     but: was <2L>

Tests run: 226, Failures: 1, Errors: 0, Skipped: 5

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:58 min
[INFO] Finished at: 2015-06-02T21:58:07+00:00
[INFO] Final Memory: 29M/450M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test) on project takes: There are test failures.
[ERROR]
[ERROR] Please refer to /home/r/repo/target/surefire-reports for the individual test results.
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.17:test (default-test) on project takes: There are test failures.

Please refer to /home/r/repo/target/surefire-reports for the individual test results.
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:212)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:116)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:80)
	at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
	at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:120)
	at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:347)
	at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:154)
	at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
	at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
	at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
	at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.apache.maven.plugin.MojoFailureException: There are test failures.

Please refer to /home/r/repo/target/surefire-reports for the individual test results.
	at org.apache.maven.plugin.surefire.SurefireHelper.reportExecution(SurefireHelper.java:82)
	at org.apache.maven.plugin.surefire.SurefirePlugin.handleSummary(SurefirePlugin.java:195)
	at org.apache.maven.plugin.surefire.AbstractSurefireMojo.executeAfterPreconditionsChecked(AbstractSurefireMojo.java:861)
	at org.apache.maven.plugin.surefire.AbstractSurefireMojo.execute(AbstractSurefireMojo.java:729)
	at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:132)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208)
	... 19 more
[ERROR]
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

@yegor256 updated from master, i think it will success now

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 13min)

@ypshenychka please, review this task for compliance with our quality rules

@longtimeago
According to our QA Rules:

The code reviewer found at least three problems in the code.

Two issues were found during code review. Try to find at least three problems while future reviews.
Please confirm that everything is clear and you have no questions regarding this approach.

@rultor deploy

@ypshenychka I confirm

@longtimeago Thank you

@davvd Quality is acceptable.

@rultor deploy

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy

@davvd Done! FYI, the full log is here (took me 7min)

@davvd Quality is acceptable.

@ypshenychka thanks, got it. everybody, please try to make it good next time

@longtimeago 10 mins was added to the account of @ypshenychka (for QA review), in transaction 58652336
thank you, added 20 mins to your acc, payment num is 58652355, 47 hours and 36 mins spent in total to complete
extra minutes for review comments (c=5)
+20 added to your rating, at the moment it is: +3867


Use Rc instead of Arc for storing rustflags

What does this PR try to resolve?
There's no reason that these reference counted pointers (which I introduced in #13900) need to be atomic, so let's use non-atomic pointers.
Additional information
First noticed by @weihanglo here.
r? @weihanglo

Thanks!
If we need it Sync in the future, we can always change later.
@bors r+

:pushpin: Commit 22c6a7db72da6c16fcfac8b77929a9a8c223949c has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 22c6a7db72da6c16fcfac8b77929a9a8c223949c with merge 631b8774e512a69402a55367b4eb9c195220e404...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 631b8774e512a69402a55367b4eb9c195220e404 to master...


[BUG] - showIn and hideIn not working at breakpoint

Describe the bug
When using two text components, one with hideIn and one with showIn, neither is rendered when the screen is at the breakpoint width (in this case, 650).
<Text h1 hideIn='xs'> Big header </Text>
<Text h2 showIn='xs'> Small header </Text>

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Create two text components using hideIn and showIn with the same breakpoint. Resize the screen to the exact value of the breakpoint. Neither text will render.
Expected behavior
As a user, I expect either one or the other to render at the breakpoint. Probably the one with showIn.
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

Hey @ToMeHarris  sorry for the delay please upgrade to V2.  Use TailwindCSS responsive classes.


CalDAV Task Import

It'd be nice to have. I actually don't expect you to do any work related to this or care, but I wanted to create it here just so I could assign it to myself, because that would be awesome and give me some more Rails stuff to work on.

I'll look into this a bit, but would definitely appreciate it if you took this on!


Invalid/expired user token

No matter what I try, I'm always getting an invalid/expired user token in my widget.
I've executed the fitbit_authorizer.rb and can successfully generate a PIN every time, but it always returns as invalid/expired in the widget when I access http://localhost:3030/sampletv

Clean installation:
Rails 4.1.6
Ruby 2.1.2p95 (2014-05-08 revision 45877) [x86_64-darwin13.0]
Listening on <IP_ADDRESS>:3030, CTRL+C to stop
scheduler caught exception:
undefined method []' for nil:NilClass /Users/fulvio/Projects/dashing-fitbit/lib/fitbit.rb:23:in device'
/Users/fulvio/Projects/dashing-fitbit/jobs/fitbit.rb:13:in block in <top (required)>' /Users/fulvio/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rufus-scheduler-2.0.24/lib/rufus/sc/jobs.rb:230:in call'
/Users/fulvio/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rufus-scheduler-2.0.24/lib/rufus/sc/jobs.rb:230:in trigger_block' /Users/fulvio/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rufus-scheduler-2.0.24/lib/rufus/sc/jobs.rb:204:in block in trigger'
/Users/fulvio/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rufus-scheduler-2.0.24/lib/rufus/sc/scheduler.rb:430:in call' /Users/fulvio/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rufus-scheduler-2.0.24/lib/rufus/sc/scheduler.rb:430:in block in trigger_job'

This is because Fitbit has switched entirely from OAuth to OAuth2. The gem used to authenticate with Fitbit, Fitgem is incompatible with Oauth2, so this widget will no longer function.
You can check out my fork - which adds OAuth2 support - if you're still interested.


pkg/actor(ticdc): generic actor message

What problem does this PR solve?
Issue Number: close #4972
What is changed and how it works?
Replace a concrete typed field with a generic typed field in actor message.
Need to merge #4971 first.
Check List
Tests

Unit test
Integration test

Code changes

Has exported function/method change
Has exported variable/fields change

Release note
None

[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-169)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-170)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-171)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-172)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-173)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-174)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-175)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-176)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-177)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-178)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-179)[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-180)[2022-03-22T06:10:01.970Z] pkg/actor/system.go:162:1: receiver-naming: receiver name rd should be consistent with previous receiver name p for invalid-type (revive)

[2022-03-22T06:10:01.970Z] func (rd *ready[T]) prepareStop() {

[2022-03-22T06:10:01.970Z] 	rd.Lock()

[2022-03-22T06:10:01.970Z] 	rd.state = readyStateStopping

[2022-03-22T06:10:01.970Z] 	rd.Unlock()

[2022-03-22T06:10:01.970Z] }

[2022-03-22T06:10:01.970Z] pkg/actor/system.go:168:1: receiver-naming: receiver name rd should be consistent with previous receiver name p for invalid-type (revive)

[2022-03-22T06:10:01.970Z] func (rd *ready[T]) stop() {

[2022-03-22T06:10:01.970Z] 	rd.Lock()

[2022-03-22T06:10:01.970Z] 	rd.state = readyStateStopped

[2022-03-22T06:10:01.970Z] 	rd.Unlock()

[2022-03-22T06:10:01.970Z] 	rd.cond.Broadcast()

[2022-03-22T06:10:01.970Z] }

some of the lint got disabled after upgrade to 1.8, may have some side effects ?
[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] gosimple is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-163)."

[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] staticcheck is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-164)."

[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] structcheck is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-165)."

[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] stylecheck is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-166)."

[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] unparam is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by following the https://github.com/golangci/golangci-lint/issues/2649[](https://ci.pingcap.net/blue/organizations/jenkins/lint-check/detail/lint-check/3605/pipeline/#step-55-log-167)."

[2022-03-22T06:10:01.970Z] level=warning msg="[linters context] unused is disabled because of go1.18. If you are not using go1.18, you can set `go: go1.17` in the `run` section. You can track the evolution of the go1.18 support by

some of the lint got disabled after upgrade to 1.8, may have some side effects ?

Due to upstream x/tools/go/ssa does not support generic yet, we have to disable them, otherwise it panics, see more https://github.com/golangci/golangci-lint/issues/2649

/run-all-tests

I mean some of the lints got disabled may cause some issue supposed to found but not

some of the lint got disabled after upgrade to 1.8, may have some side effects ?

Due to upstream x/tools/go/ssa does not support generic yet, we have to disable them, otherwise it panics, see more golangci/golangci-lint#2649

I mean some of the lints got disabled may cause some issue supposed to found but not

[2022-03-22T06:10:01.970Z] pkg/actor/system.go:168:1: receiver-naming: receiver name rd should be consistent with previous receiver name p for invalid-type (revive)

I think it's a false alarm.

/run-kafka-integration-test
/run-leak-test
/run-verify

/hold

/unhold

/merge

/run-kafka-integration-test

/run-kafka-integration-test

/run-kafka-integration-test

/run-dm-integration-test
/run-kafka-integration-test

/run-kafka-integration-test

fyi https://planetscale.com/blog/generics-can-make-your-go-code-slower


Monaco resource window cursor position resetting and not remembered

Describe the bug
This MIGHT be an enhancement but it seems to work differently than pre v5 versions, so I'm thinking bug over feature.
I did not notice this behavior before the Monaco update, so I'm thinking Monaco might have something to do with this.  When you are working in resource windows (looking at the YAML configs) and have multiple tabs open, the cursor position always seems to reset to line 1 position 1 whenever the tab loses focus and then gains focus again.  This makes it very difficult to compare two configurations or copy/paste sections of configurations.
Before Monaco, I knew I had to actually click in the YAML where I wanted the cursor to "stick", but, once I actually clicked on a line, switching back and forth between tabs always behaved by remembering the cursor position in each tab.  Now, it seems to reset to line 1 position 1 each time switching between tabs.
To Reproduce

Open multiple edit windows for resources (create multiple tabs).  This is better shown if you open YAML configs that are very long.
In "window 1", scroll to a position in one window that is far below the top and click on a particular line
Switch to "window 2".  Note that you are at the top position of the YAML.
Scroll to a position below the top and click in the YAML.
Flip back to "window 1" and notice that you are "reset" to line 1 position 1 (there is a slight lag almost like the window is completely reloading the configuration in case anything changed).
Switch to "window 2" and notice that you are at line 1 position 1 (again with the same lag).

Expected behavior
I get it can be argued either way that the windows reload in case something has changed since the last time you viewed the window, but that's not exactly what I would expect.  I would expect the YAML to stay in memory as it was when it was loaded into the tab.  I would particularly expect that the position of the cursor be remembered so flipping back and forth between tabs would not necessitate me having to find my position over and over.
Environment (please complete the following information):

Lens Version:  Lens: 5.2.6-latest.20211104.1
OS: MacOS Big Sur 11.6.1
Installation method (e.g. snap or AppImage in Linux): Direct DMG install from https://k8slens.dev/

This was fixed by #4225 and is part of 5.3.0-alpha.9


Add an annotation to specify release image

Signed-off-by: zhujian<EMAIL_ADDRESS>
Description of the change(s):

Add an annotation hypershiftdeployment.cluster.open-cluster-management.io/release-image for users to specify the release image.

Why do we need this PR:

This is good for testing.  the hypershift deployment controller always gets the latest openshift release image at present, but sometimes using the latest version will fail to provision nodepools, we can use this annotation to specify an ocp version.

Issue reference:

Test API/Unit - Success
?       github.com/stolostron/hypershift-deployment-controller/api/v1alpha1     [no test files]
?       github.com/stolostron/hypershift-deployment-controller/pkg      [no test files]
?       github.com/stolostron/hypershift-deployment-controller/pkg/constant     [no test files]
ok      github.com/stolostron/hypershift-deployment-controller/pkg/controllers  0.908s  coverage: 77.1% of statements
ok      github.com/stolostron/hypershift-deployment-controller/pkg/controllers/autoimport       0.056s  coverage: 61.5% of statements
?       github.com/stolostron/hypershift-deployment-controller/pkg/helper       [no test files]
ok      github.com/stolostron/hypershift-deployment-controller/test/integration 22.171s coverage: [no statements]

I don't think we need this now that 4.11-fc.0 works.

If a user wants to customize the image, they would specify the images in the release.image stanza.


Flux and secrets

Hi, I'm following the guide in your README.md, you use aws, how can I test your project locally?

Hmm, it's not really meant to be used without AWS...
The whole cluster runs locally, but I'm using AWS Route53 to allow cert-manager to create DNS entries to prove domain ownership so that it can issue certificates. You could try to replace Route53 with a more generic provider, but because I'm using the DNS01 challenge, you're going to need some external provider.

Thanks, are you by any chance Italian? I use cloudflare as a DNS challenge provider. How can I adapt it?

Thanks, are you by any chance Italian?

Indeed I am 🇮🇹🇺🇸

I use cloudflare as a DNS challenge provider. How can I adapt it?

See the cert-manager docs, it supports Cloudflare. You'll need to NOT create the AWS secrets and instead create a secret for Cloudflare and then update the issuers.yaml file to look like the example in the docs. Apparently Cloudflare has API Tokens and also API Keys, you'll need to find the difference and decide which is right for you.

Thanks, I'll try tomorrow.


Introduce java annotation @OptionalAttribute and javadoc tag @ff.optional

Optional overrides any mandatory setting.
Example use case:
AbstractPipe.setName() will get a @ff.mandatory annotation
IWrapperPipe.setName() and IValidator.setName() will get a @Optional annotation, to avoid having to set a name for wrappers and validators

In the use case you explain Java annotation @Optional. What is the meaning of JavaDoc tag @ff.optional? Do they do the same or is there a difference?

They are meant to be the same

Oh, I understand. I should have read issue #69 before writing...

Please also consider PR https://github.com/ibissource/iaf/pull/2808.


"THIS WAS NOT SUPPOSED TO HAPPEN!" error message when opening Bank

Describe the bug
I'm not sure what the bug is, but you've coded the mod to shout at me when something breaks.
[23:31:44] [Render thread/INFO]: [CHAT] §c§lTHIS WAS NOT SUPPOSED TO HAPPEN!
§rPlease make a bug report
§rdescribing what caused this message to appear.
§rhttps://github.com/Natanaelel/BankStorage/issues
§rMath.Random PlayerEntity.bankstorage$getSyncedRandom() returned null
If this error persists, try rejoining the server/world.

To Reproduce
I'm not sure, but here's what I did recently:

Flew to the Stronghold and went to the End (already defeated the Dragon)
When I got there realized I was out of food so I came back through the portal
Got food, repeat step 1.
Mined about 2k obsidian out of one of the towers, which got picked up in on of my banks.
Went back through the portal
Opened my bank by looking at the floor at a normal construction block. Got the error message.
Shift-clicked obsidian out then deposited in ME and repeated a few times, getting the error message immediately upon opening the bank each time.

Other notes:

I have 3 banks in my inventory
The issue happens with each bank right now
The issue only happens when I look at a block and open it. If I open the bank in the air, it does not happen.
After disconnecting from the server and reconnecting, the issue is not happening anymore. I play on a locally hosted server.

Expected behavior
No big chat message!
Screenshots

Logs
If applicable, add logs found in minecraft-instance/logs/latest.log and minecraft-instance/crash-reports/<crashlog>.txt
Additional context
Add any other context about the problem here.

Update to 1.0.2


Recipe mappings

Please mark as ready for review if this is ready.

No other changes I can see coming up, so there you do. @modmuss50

@liach IMO Json de/serialisation is most definitely an IO operation. You will rarely need to do it when you're not reading json data from a file (or loading from the /data folder in this case).
Writing to a packet buffer, too, is for IO.

json = input
packetbuffer = input/output

@liach IMO Json de/serialisation is most definitely an IO operation. You will rarely need to do it when you're not reading json data from a file (or loading from the /data folder in this case).
Writing to a packet buffer, too, is for IO.

This guideline is set up by @sfPlayer1. Debate against him if you disagree with all these from/to stuff.

Rebased and updated. I also removed some of the "problematic" renames.
I am open to changing the ofX methods in Ingredient to just of (assuming there are no conflicts).

Updated and rebased @haykam821

What's the plan for this one?

Rebased and updated to 21w13a. I dropped some of my changes that are no longer required.

@Sollace You should update your parameter documentation to match the javadoc conventions for Yarn. CONVENTIONS.md says:

Write quick descriptions for parameter javadocs as well as @return tags, with no uppercase or period.

(highlighting mine)

For juuz and haykam, feel free to speak out if there is other problems besides that handler field.

:tada: It was just 20 days till the first :birthday: of this pull. A long way but you made it, thanks so much sollace!

@liach 💝


Simplify UserDatabase/UserDatabaseQuery/UserDatabaseTransaction declarations

During review of #448, we considered simplifying several interfaces/types related to UserDatabase.
export interface UserDatabase {
  transaction<R, T extends unknown[]>(transactionFunction: UserDatabaseTransaction<R, T>, config: TransactionConfig, ...args: T): Promise<R>;
  queryFunction<C extends UserDatabaseClient, R, T extends unknown[]>(queryFunction: UserDatabaseQuery<C, R, T>, ...params: T): Promise<R>;
  query<R, T extends unknown[]>(sql: string, ...params: T): Promise<R[]>;
  queryWithClient<R, T extends unknown[] = unknown[]>(client: UserDatabaseClient, sql: string, ...params: T): Promise<R[]>;
  // other members omitted for clarity
}

type UserDatabaseQuery<C extends UserDatabaseClient, R, T extends unknown[]> = (ctxt: C, ...args: T) => Promise<R>;
type UserDatabaseTransaction<R, T extends unknown[]> = (ctxt: UserDatabaseClient, ...args: T) => Promise<R>;

Having a type argument T extending unknown[] doesn't provide much value. I think this type argument could be removed, simplifying the definitions. The new definitions would look something like this:
export interface UserDatabase {
  transaction<R>(transactionFunction: UserDatabaseTransaction<R, T>, config: TransactionConfig, ...args: unknown[]): Promise<R>;
  queryFunction<C extends UserDatabaseClient, R>(queryFunction: UserDatabaseQuery<C, R, T>, ...params: unknown[]): Promise<R>;
  query<R>(sql: string, ...params: unknown[]): Promise<R[]>;
  queryWithClient<R>(client: UserDatabaseClient, sql: string, ...params: unknown[]): Promise<R[]>;
  // other members omitted for clarity
}

type UserDatabaseQuery<C extends UserDatabaseClient, R> = (ctxt: C, ...args: unknown[]) => Promise<R>;
type UserDatabaseTransaction<R> = (ctxt: UserDatabaseClient, ...args: unknown[]) => Promise<R>;

This should probably be closed or assigned a label for v2 as it will change the signature of the related functions.


Initial version of beautifying code editors

[x] Ready to merge

Here's the initial version of the code beautifier. Beautifies HTML, CSS, & JS - but none of their preprocessor cousins. ( From issue #341 )
It functions, but lacks two important features:

Hot-keys - I'm probably just being stupid as to why it doesn't work
Beautify settings - Wasn't sure how you wanted to store them with the rest of the settings and how to display that to people

Should probably also minify the beautifier files.
I tried to copy tern's implementation as much as possible and keep code style conventions.
I'm gonna keep working on (1) but I think (2) might be an implementation detail you guys will want to handle.

Hello. Is the beautify feature implemented? If so, how can I use it? Thanks

@ramtob open the keyboard shortcuts (from the help) and blamo: https://cloudup.com/cqm35U4Z_dk

@remy I tried the keyboard shortcut (Ctrl-Shift-L according to http://jsbin.com/help/keyboard-shortcuts), several times. It did work once, but most of the times it didn't work. Instead, it prints "context reset" on the console panel, and moves the focus to that panel. See https://cloudup.com/cXjQbcokJIK

Anyone?


Dynamic find by cop should observe TargetRailsVersion

Expected behavior
The rails dynamic find_by cop should not flag code if the TargetRailsVersion is < ? (I can't find when this functionality changed).
Actual behavior
The dynamic find_by cop doesn't observe the TargetRailsVersion and flags these issues and even autocorrects them replacing them with invalid code. I realize that I can disable the cop but it makes more sense for TargetRailsVersion to work properly for this cop.
Steps to reproduce the problem
rails version 3.2.16 create a model and use a find_by_attribute call
RuboCop version
0.51.0 (using Parser <IP_ADDRESS>, running on ruby 2.1.0 x86_64-linux)

https://github.com/bbatsov/rubocop/pull/4980#issuecomment-341291779
RuboCop does not support Rails 3.

BTW, we can add a documentation for supported Rails versions probably.

I think we will keep getting this issue for as long as folks run Rubocop on Rails 3 apps. (a.k.a. forever)
Ideally, the Rails cops would be extracted to another gem that depends on Rails 4+. Until then, maybe we should disable all the Rails cops if Rails <4 is detected?
I'm not convinced any documentation is going to prevent folks from being confused/upset initially. They might find the documentation after seeing the offense on their Rails 3 app and not open another issue, but it would be better if we got out ahead of this somehow.

Adding this to the readme would be great for me especially since none of the other Rails cops have given me any trouble. We're running a supported version of ruby and there was no indication that Rails 3 would not be supported. We're also looking to upgrade to rails 4 later this year and I'm particularly motivated because of this issue.
Thanks for the response!


How to build current dev?

Hey all,
How do you build the current DEV version? (on windows)
I've pulled the latest version and have done an npm install   and then npm run build ,  but it gives me 27305 times the following error:
error  Expected linebreaks to be 'LF' but found 'CRLF'  linebreak-style
and no built js file.

@englercj this was what i mentioned yesterday, looks like any windows user will suffer the issue without a git reconfigure.

@whizzkid until that's answered, for now your can either use the latest built pixi directly, guide for that @ https://github.com/pixijs/pixi.js/wiki/FAQs#where-can-i-get-a-build
Or to build locally, open up the package.json , look for the lint npm declaration, and change it's contents to something like 'npm -v'
This stops it performing, and thus failing, the lint process before pixi is actually built

The problem is that when you installed git, you told it to checkout files with windows line endings. So, when you clones pixi it changed all the line endings to CRLF. Then, when you tried to build our linter complains because we don't allow CRLF in the lib.
You should be able to fix this by choosing the "Checkout as-is, commit as-is" option when installing git for windows; or by running the following command post-install:
git config --global core.autocrlf false

I will just go ahead and change the linter though since it seems multiple people will have this issue. But if I get CRLF endings in my lib...I'm going to be so mad...

Since I use source tree, which uses an imbeded git, I had a look to see what they were doing, and according to them they say
"Windows machines convert CRLF to LF when committing and LF to CRLF when checking out"
Thus nothing commit would be the wrong line endings, but whilst on a Windows PC it would fail the linting.
Would adding a .gitattributes file help perhaps?

Added one, would you mind doing a fresh checkout without having core.autocrlf set to false and see if that works?

Yep, that works

Could maybe pinch a relevant preset one from https://github.com/alexkaratarakis/gitattributes to cover all filetypes we may commit

Cool, going to close this for now.
@whizzkid If you try cloning again, it should work for you now (in dev branch). Sorry about that!
@themoonrat I'll take a look and improve ours as necessary, thanks!


Calico and Raven are creating conflicting iptable rules in mangle table

Hi team,
the problem
These are the PREROUTING part of the mangle table in our setup
[6564:716641] -A PREROUTING -m comment --comment "cali:6gwbT8clXdHdC1b1" -j cali-PREROUTING
[1516:531527] -A PREROUTING -m comment --comment "raven traffic rules for mark" -j RAVEN-MARK-CHAIN
[1456029:416042379] -A PREROUTING -m comment --comment "raven traffic rules for mark" -j RAVEN-MARK-CHAIN
[2433306:686274254] -A cali-PREROUTING -m comment --comment "cali:6BJqBjBCd7crtA-7-" -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
[0:0] -A cali-PREROUTING -m comment --comment "cali:KX7AGNd6rMcDUai6" -m mark --mark 0x10000/0x10000 -j ACCEPT
[264207:111043774] -A cali-PREROUTING -m comment --comment "cali:wNH7KsdA3ILKJBsY9" -j cali-from-host-endpoint
[0:0] -A cali-PREROUTING -m comment --comment "cali:Cg96MgVuodPm7UMRo" -m comment --comment "Host endpoint policy accepted packet." -m mark --mark 0x10000/0x10000 -j ACCEPT

Therefore, for requests with ESTABLISHED connection, the cali:6BJqBjBCd7crtA-7- chain will simply short-circuit the raven mark chain by ACCEPTing them directly.
This will cause the raven route table not used when routing supposed raven traffic.
ip rule show
0:	from all lookup local
100:	from all fwmark 0x40 lookup 9027
32766:	from all lookup main
32767:	from all lookup default

potential workaround
Instead of using fwmark/ipset to match requests for raven route rule (9027), I think I could just add the IPs in ipset to the route table itself and remove the fwmark 0x40 from the ip rule 9027 above.
e.g. on a non-gateway edge node, if the ipset has member
<IP_ADDRESS>

Instead of marking requests to it with mangle table rule, just create a route in raven route table like:
# rule
100:	from all lookup 9027
# route
<IP_ADDRESS>/20 via <IP_ADDRESS> dev raven0 src <IP_ADDRESS> onlink mtu 1420
# used to be: default via <IP_ADDRESS> dev raven0 src <IP_ADDRESS> onlink mtu 1420

I guess there should be reasons why we are not doing it, is it because of performance concerns (i.e. a really long route table 9027)? Any other potential solutions?
Thanks in advance!

Also I noticed calico labels our node with:
                    projectcalico.org/IPv4Address: <IP_ADDRESS>/8

Raven is also using 240 as vxlan address prefix, will this cause conflicts? can submit a different issue if we think it's prolematic

Hello @suyuee, thanks for your constructive suggestion.
iptables is executed in order, and the rules of raven we added is before calico.
Just you show the rule:
[1516:531527] -A PREROUTING -m comment --comment "raven traffic rules for mark" -j RAVEN-MARK-CHAIN
[1456029:416042379] -A PREROUTING -m comment --comment "raven traffic rules for mark" -j RAVEN-MARK-CHAIN
[2433306:686274254] -A cali-PREROUTING -m comment --comment "cali:6BJqBjBCd7crtA-7-" -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT

In the PREROUTING chain, raven rule is excuted first, then jump to RAVEN-MARK-CHAIN rule by -j. Wait until the rule RAVEN-MARK-CHAIN is executed before cali-PREROUTING of calico starts. So then are not conflicted.

@luckymrwang Thanks for the quick turnaround!
Actually the cali-prerouting is before the raven:
[6564:716641] -A PREROUTING -m comment --comment "cali:6gwbT8clXdHdC1b1" -j cali-PREROUTING
[1516:531527] -A PREROUTING -m comment --comment "raven traffic rules for mark" -j RAVEN-MARK-CHAIN

cali-PREROUTING is "called" from PREROUTING earlier than the RAVEN-MARK-CHAIN being called. I even tried to put it at top of prerouting chain, but calico quickly put its rule back to top again

set the calico configuration iptablesMangleAllowAction to "return", letting raven rule in prerouting take effect.
https://docs.tigera.io/calico/latest/reference/resources/felixconfig

Works like a charm, thank you!


Implement #896

Fix #896

This is just a simpler idea I came up with long time ago. I used this feature as an excuse to cherry-pick something from the ongoing class overhaul branch

bors r+


test.py ERROR

I made and trained models in the Linux environment.
I created the same environment for Windows.
And I copied the CONFIG file and WORK_DIRS of the model I trained on Linux to Windows.
I only changed TEST, PRETRAINED, LOAD_FROM PATH in the model file I copied.
They copied images that were tested in the Linux environment and changed them to COCODATASET in the Windows environment.
Then I ran test.py('python tools/test.py configs/scar_c/original_step/resnet101_orgs.py work_dirs/resnet101_orgs/latest.pth --show-dir result/'), and the output was shown below.
And it keeps popping up that there is a file in the image folder but there is no specific file. May I know why this is so?
size mismatch for roi_head.bbox_head.0.fc_cls.weight: copying a param with shape torch.Size([2, 1024]) from checkpoint, the shape in current model is torch.Size([81, 1024]).
size mismatch for roi_head.bbox_head.0.fc_cls.bias: copying a param with shape torch.Size([2]) from checkpoint, the shape in current model is torch.Size([81]).
size mismatch for roi_head.bbox_head.1.fc_cls.weight: copying a param with shape torch.Size([2, 1024]) from checkpoint, the shape in current model is torch.Size([81, 1024]).
size mismatch for roi_head.bbox_head.1.fc_cls.bias: copying a param with shape torch.Size([2]) from checkpoint, the shape in current model is torch.Size([81]).
size mismatch for roi_head.bbox_head.2.fc_cls.weight: copying a param with shape torch.Size([2, 1024]) from checkpoint, the shape in current model is torch.Size([81, 1024]).
size mismatch for roi_head.bbox_head.2.fc_cls.bias: copying a param with shape torch.Size([2]) from checkpoint, the shape in current model is torch.Size([81]).
size mismatch for roi_head.mask_head.0.conv_logits.weight: copying a param with shape torch.Size([1, 256, 1, 1]) from checkpoint, the shape in current model is torch.Size([80, 256, 1, 1]).
size mismatch for roi_head.mask_head.0.conv_logits.bias: copying a param with shape torch.Size([1]) from checkpoint, the shape in current model is torch.Size([80]).
size mismatch for roi_head.mask_head.1.conv_logits.weight: copying a param with shape torch.Size([1, 256, 1, 1]) from checkpoint, the shape in current model is torch.Size([80, 256, 1, 1]).
size mismatch for roi_head.mask_head.1.conv_logits.bias: copying a param with shape torch.Size([1]) from checkpoint, the shape in current model is torch.Size([80]).
size mismatch for roi_head.mask_head.2.conv_logits.weight: copying a param with shape torch.Size([1, 256, 1, 1]) from checkpoint, the shape in current model is torch.Size([80, 256, 1, 1]).
size mismatch for roi_head.mask_head.2.conv_logits.bias: copying a param with shape torch.Size([1]) from checkpoint, the shape in current model is torch.Size([80]).

@hyunjung0208 Please use the issue template, thank you.

sorry I don't know there is the issue template


Add redis and ability to save stdout

I just lacked these features to start using this script.
Сcache can use redis out of the box. And usually when people start looking for something for clang-tidy they already have redis setted up for ccache.
I also need the saved error output.

thanks for the PR


failure of Preferences/IPython console/Graphics/Backend/Inline and the solution

Description
Mac OSX. Spyder IPython console initially did inline graphics. But after I checked "Preferences/IPython console/Advanced Settings/Use symbolic math", whenever I launched Spyder anew, plots were in separate window. So, I had to a "% matplotlib inline" if I wanted inline charts. This was the case even though "Preferences/IPython console/Graphics/Backend/Inline" was set. Then, I realized that if I unchecked "Preference/.../Use symbolic math", then Spyder and new IPython console plot inline by default.

What steps will reproduce the problem?

What is the expected output? What do you see instead?

Please provide any additional information below

Version and main components

Spyder Version:  2.3.4
Python Version:  3.4.3
Qt Version    :  4.8.6, PyQt4 (API v2) 4.11.3 on Darwin

Optional dependencies
pyflakes >=0.6.0:  0.8.1 (OK)
pep8 >=0.6      :  1.6.2 (OK)
IPython >=1.0   :  3.0.0 (OK)
zmq >=2.1.11    :  14.5.0 (OK)
pygments >=1.6  :  2.0.2 (OK)
pandas >=0.13.1 :  0.15.2 (OK)
sphinx >=0.6.6  :  1.2.3 (OK)
psutil >=0.3    :  2.2.1 (OK)
rope >=0.9.2    :  0.9.4-1 (OK)
jedi >=0.8.1    :  0.8.1 (OK)
matplotlib >=1.0:  1.4.3 (OK)
sympy >=0.7.3   :  0.7.6 (OK)
pylint >=0.25   :  None (NOK)

Thanks for reporting. I'll take a look at it to see what we can do about it.

I further noticed that if you are in inline plot mode and import sympy and call sympy.init_session() from IPython console, then you loose the inline mode and plots pop up in separate windows.

I further noticed that if you are in inline plot mode and import sympy and call sympy.init_session() from IPython console, then you loose the inline mode and plots pop up in separate windows.
From: Carlos Cordoba<EMAIL_ADDRESS>To: spyder-ide/spyder<EMAIL_ADDRESS>Cc: FredJames32<EMAIL_ADDRESS>Sent: Tuesday, June 2, 2015 6:48 AM
Subject: Re: [spyder] failure of Preferences/IPython console/Graphics/Backend/Inline and the solution (#2463)
Thanks for reporting. I'll take a look at it to see what we can do about it.—
Reply to this email directly or view it on GitHub.


Push notifications not work on Xamarin.Forms Android device (Android 8.0 or higher Version) when app is in foreground

Push notifications not work on Xamarin.Forms Android device (Android 8.0 or higher Version) when app is in foreground.
All working good in lower then Android 8.0 version. but its not work in Android 8.0 or higher Version.

@differenz-system The comments feature of our documentation is intended to enable customers to suggest changes to the docs or ask clarifying questions about the docs.
App Center support is provided directly from within the App Center portal. Any time you need help, just click the blue chat button in the lower-right corner of any page and our dedicated support team will respond. For additional information, see the App Center Help Center.


Creating a haxelib package

Hi there,
I hope this is the right place to ask a question about this project. :)
Is there a way to create the haxelib.json file without having to create it manually? Something like npm init, though the command-line?
Cheers!

For now, you can just copy it from some other project, but I agree this is a nice feature, so let's implement it at some point :)


Error: Incomplete XML Document [char=xx]

Hello there! I'm trying to implement this parser in Garrysmod (A game that uses a... uh, a "custom" implementation of Lua, wich is horrible) and the script throws this error in whatever XML string I pass to the parser:
Stacktrace
[ERROR] addons/compiler/lua/includes/modules/xml2lua.lua:268: Incomplete XML Document [char=44]

  1. error - [C]:-1
   2. errorHandler - addons/compiler/lua/includes/modules/xml2lua.lua:268
    3. _err - addons/compiler/lua/includes/modules/xml2lua.lua:457
     4. parse - addons/compiler/lua/includes/modules/xml2lua.lua:302
      5. unknown - addons/compiler/lua/autorun/test.lua:3

Code
Original code (Moonscript):
require 'xml2lua'

x = xml2lua.parser require 'xmlhandler/tree'

t = x\parse [[

<root>
  <elem>hi</elem>
</root>
]]

print t.root.elem

Tested code (Lua):
require('xml2lua')
local x = xml2lua.parser(require('xmlhandler/tree'))
local t = x:parse([[
<root>
  <elem at="hello">hi</elem>
</root>
]])
print(t.root.elem)

XML Tried
First attempt:
<?xml version="1.0" encoding="UTF-8"?>
<root>
  <elem at="hello">hi</elem>
</root>

Second (Without headers)
<root>
  <elem at="hello">hi</elem>
</root>

Third (Without attributes)
<root>
  <elem>hi</elem>
</root>

Even I've tried adding and removing blank and newline charaters of padding, without success.

Hello @sigmasoldi3r
You have some issues in your code.
You are using the handler module without assigning it to a variable. That variable is the one used to access the elements into your XML.
And you don't need to assign the result of x:parse to a variable t because x is already your parser.
You have just to follow the provided examples.
There also some confusions in your code because your top-level XML element is called root and the table generated after parsing the XML always have an element called root.
This way, in the example below, you have to call handler.root.root.elem[1] to access the first element tag (notice there is the root which is always created by xml2lua and inside it, the root contained in your XML).
require('xml2lua')
local handler = require('xmlhandler/tree')
local parser = xml2lua.parser(handler)
parser:parse([[
<root>
  <elem at="hello">hi</elem>
</root>
]])

--Prints the entire table representing the parsed XML
--xml2lua.printable( handler.root.root)
print("elem 1:", handler.root.root.elem[1])
print("elem 1 at:", handler.root.root.elem._attr.at)


Improvement Request: Whitelist/Blacklist certain unpacked extensions

It would be nice if in the options page I could select which of my unpacked extensions should be reloaded (with, say, a checkmark - or a less user-friendly options that might be easier to implement is to just enter extension IDs)

Thank you for your suggestion.
Do you think that adding a query param with the extension ID to the reload triggering url (e.g., http://reload.extensions?id=EXT_ID) would be enough?


Added config file and some refactoring

Resolves #2
Made a few changes here:

Created categories.json to separate out the sorting logic.
Abstracted sort_folder to take any folder and moved specification of downloads folder into a main() function. So now if you want to import sort_folder somewhere else you can use it to sort any folder you want.
Some minor style changes

Happy to adjust anything if you don't like the style changes.

Thanks for the help! I used your code to create a config file to separate sorting logic which has greatly simplified the script and will be easier to maintain in the future. I didn't merge because I am still working on a very specific implementation of abstracting sort_folder.


FAQ says booster rules will be updated

Where to find the issue

https://www.coronawarn.app/en/faq/#vac_cert_booster_info "How do I become aware of booster vaccinations in the context of Corona?"
https://www.coronawarn.app/de/faq/#vac_cert_booster_info "Wie erfahre ich von Auffrischimpfungen im Kontext von Corona?"

Describe the issue
The FAQ article says:
"How long someone is effectively protected by a COVID-19 vaccination (basic immunization) depends on several factors and can therefore not be answered in general terms according to current knowledge. For this reason, the corresponding CWA rules for vaccination schedules and booster vaccinations are continually reviewed and adapted in coordination with the RKI according to developments and new findings."
The commitment to "continually review and adapt" is not being kept. RKI publishes updated written recommendations, however these have not been translated into CWA rules.

See for instance https://github.com/corona-warn-app/cwa-documentation/issues/814.

Suggested change
If the rules are no longer being updated then the description in the FAQ needs to be changed to say that.

Internal Tracking ID: EXPOSUREAPP-11721

Thanks, @MikeMcC399, we will discuss this topic internally. I created a Jira issue to keep track of it.


New version: Erlang.ErlangOTP version 24.2

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

@Trenly this package is missing the Publisher registry field if you want to go digging :)

@Trenly this package is missing the Publisher registry field if you want to go digging :)

It looks like there's more going on than just the publisher and version. It looks like the previous version entries aren't removed even after update because they use different registry entries for each version

I think I was able to fix both issues at the same time; If the PR I submitted gets rejected, I'll set it to just add the publisher and version and call it good


Magnitude information for field maps?

Hi guys!
We were wondering if it's possible to get a hold of the original magnitude information used to make the field maps. We hope to use it to determine the shape/contour of the subject so we can place them appropriately in the coil. It's difficult to do this with just the field maps/masks as we have them now.
Thanks!
-Ishaan

Will be addressed in https://github.com/shimming-toolbox/b0-fieldmap-realistic-simulation/issues/4 when the magnitude data and the labels will be packaged and published in OpenNeuro


Molecule masses from symbol issue

Describe the bug
Nucleon symbols and values are in the periodictable information, but do not propagate their values into a Molecules mass. For example symbol D or He3 do not correctly review masses of 2 or 3, respectively.
To Reproduce
>>> qcel.models.Molecule(symbols=["He3"], geometry=[0,0,0]).masses
array([4.00260325413])

Expected behavior
The mass of the Helium3 to be 3 and change.
Additional context
The issues comes from reconcile_nucleus where everything is validated off of atomic number rather than the input symbol. This looks like a non trivial fix to that logic.

did you try 3He? that should recognize the isotope. He3 is read as a user-convenience tag of 3 on regular helium.

qcelemental.exceptions.NotAnElementError: 3He
There are unambiguous issues as well with known isotopes:
>>> qcel.models.Molecule(symbols=["D"], geometry=[0,0,0]).masses
array([1.00782503])

ok. it's probably in the handing over of field symbol to molparse -- may need a different field. it shouldn't be deep, though, since this is literally the problem reconcile_nucleus was built to handle. https://github.com/MolSSI/QCElemental/blob/master/qcelemental/molparse/nucleus.py#L381

the below works (though it's not set up to recognize D, T). understandably you'd like to not have so many guts showing to make a simple deuterium.
>>> qcel.models.Molecule(**qcel.molparse.to_schema(qcel.molparse.from_arrays(elbl=["2H"], geom=[0,0,0]), dtype=2)).masses
array([2.01410177812])

The problem comes about pretty much because https://github.com/MolSSI/QCElemental/blob/master/qcelemental/molparse/from_schema.py#L82 . https://github.com/MolSSI/QCElemental/blob/master/qcelemental/molparse/from_arrays.py#L191-L195 I hesitate to flip it without really studying the consequences.

Yea, I don't think this is either a priority or a quick fix. More of a "I was writing a test was and was somewhat surprised". Something to keep in mind I think.

I don't know if it's so much a fix as just the current setup enforces schema field definitions. Below are a couple schema-approved ways to do your original problem. Redefining symbols where as input it's a spec label and as output it's a string is a new kettle of fish. But if that is wanted someday, the underlying tech is ready.
>>> qcel.models.Molecule(symbols=["He"], mass_numbers=[3], geometry=[0,0,0]).masses
array([3.01602932])
>>> qcel.models.Molecule.from_data("""3He 0 0 0""").masses
array([3.01602932])

In this case perhaps symbols=["He3"] should throw an error that information in the symbols is overloaded?

Sounds sensible at first. You have hit on an inconsistency, though. speclabel takes prefix mass number so that post can be label or mass, so 3He. meanwhile, the periodictable takes suffix mass number: He3.
So symbols is being broadly interpreted (6 lines below map to the same Mol) but only to extract the info that is properly contained in symbols. Probably something like an option on to_Z(..., strict=True) that restricts interpretation to element only.
# all same
qcel.models.Molecule(symbols=['He'], geometry=[0,0,0])
qcel.models.Molecule(symbols=['He3'], geometry=[0,0,0])
qcel.models.Molecule(symbols=['He4'], geometry=[0,0,0])
qcel.models.Molecule(symbols=['2'], geometry=[0,0,0])
qcel.models.Molecule(symbols=[2], geometry=[0,0,0])
qcel.models.Molecule(symbols=['helium'], geometry=[0,0,0])
# replies "you fool"
qcel.models.Molecule(symbols=['He300'], geometry=[0,0,0])


Variable missing an 's' and wrong port for WAC

The variable incomingconnections.port is incorrect. It's "incomingconnections.ports"
Add multiple ports:
azcmagent config set incomingconnections.port 22,3516
In the line above the WAC portal Port is 6516 not 3516 by default. Copying this line causes two issues, 1. The variable is incorrect. 2. The port connection will break the WAC in Portal experience if copied as is.
Line should read:
Add multiple ports:
azcmagent config set incomingconnections.ports 22,6516

#sign-off


de.json: translated relay setting strings (now translation from Jack),quick photo capture strings, fixed typos

Thank you for the update!


Update cloud-core minimum dependency

To fix the issue: https://github.com/GoogleCloudPlatform/google-cloud-php/issues/782

I signed it!

Thanks for the contribution, @jackchuka. After discussing with @dwsupplee, we feel it's best to  bump the required version for all components to the latest cloud-core (^1.14) in order to establish a new baseline from which we can be more diligent about future changes to the required version in the various components.
I'm quite interested in hearing your thoughts on this idea, as a user of the library who has encountered an issue which our proposal will effect. Would our proposal negatively impact you in any way?

Hi @jdpedrie, thanks for hearing me out on this.
I totally agree with you two and I'd say go for it. We won't have any negative impact due to this proposal. Besides, creating a new baseline will ensure that the each components are up to date and as a user of the library, I feel more confident to use/update the package as well.

@jackchuka please take a look at #790!

Hi @jdpedrie, just wondering when will this https://github.com/GoogleCloudPlatform/google-cloud-php/pull/790 patch be released?

@jackchuka I'm planning on releasing it in v0.47.0, which should be out later today if all goes as planned.


Input for ColorLight 5A-75B/E, by rewiring pin 1 of buffers to GND and powering with 3.3v?

To use inputs for ColorLight 5A-75B/E, I was thinking what about rewiring a buffer IC's pin 1 (the direction pin) to disconnect it from Vdd and instead connect it to ground?  It would require physically breaking pin 1's connection to the board cause the pin sits on Vdd rail.  Then could carefully solder a wire to it and an exposed ground.
Would also need to power the entire board using 3.3V instead of 5.0V, since the FPGA pins are surely not 5V-tolerant. (The only things the positive rail powers are the buck converters, but according the data sheet the 3.3V buck converters can still work with 3.3V input...the output just can't be higher than the input.)

The readme currently suggests to replace the buffer boards instead, but I wanted to suggest just physically rewiring pin 1 as a simpler solution (which requires less soldering and doesn't require buying a new chip).

I just wanted to note the downside of my suggestion is that once pin 1 has been physically rewired, it would be difficult to go back.  Hypothetically one could instead rewire the pin to a physical switch.  Or could even wire to an output pin from another output-only buffer, to allow that one wire to be selectively reconfigured by writing to that output pin.

Yes, this should work. However as you said the FPGA is certainly not 5V tolerant and i don't know if the entire board will work when powered with 3,3V (not familiar with this buck converter stuff).

It is worth to mention that the buffers (74HC245T) are not 5 V tolerant either when driving, so the peripherals should be also driven by 3.3 Volts in this case. A slightly less intervening method to convert the board, whilst being able to use 5 Volt inputs is to run the board from 3.3 Volt and replacing the buffers with 74LVS245. In this case you don't have tow rewire / cut the VCC to each chip you replace, making damages less likely to happen. You only have to cut the trace to the direction pin and rewire that to GND.

Read here https://twitter.com/claude1079/status/1231194849350647808?lang=en


west flash --elf-file is not flashing using .elf file, but using zephyr.hex to flash

I used the following build and flash command and expecting that when we use a -elf-file flag for west flash, it should not require the zephyr.hex file, if i remove the zephyr hex file from the build folder, then the flash command is failing. with error "build\zephyr\zephyr.hex does not exist"
west build -b nucleo_f429zi -p
west flash --elf-file .\build\zephyr\zephyr.elf -r openocd --skip-rebuild --cmd-pre-init  "hla_serial 066CFF363336485043072710"
-- west flash: using runner openocd
-- runners.openocd: Flashing file: C:/XXXX/build/zephyr/zephyr.hex
Open On-Chip Debugger 0.11.0 (2021-11-18) [https://github.com/sysprogs/openocd]
Licensed under GNU GPL v2
libusb1 09e75e98b4d9ea7909e8837b7a3f00dda4589dc3
For bug reports, read
http://openocd.org/doc/doxygen/bugs.html
Info : The selected transport took over low-level target control. The results might differ compared to plain JTAG/SWD
Info : clock speed 2000 kHz
Info : STLINK V2J32M22 (API v2) VID:PID 0483:374B
Info : Target voltage: 3.232461
Info : stm32f4x.cpu: Cortex-M4 r0p1 processor detected
Info : stm32f4x.cpu: target has 6 breakpoints, 4 watchpoints
Info : starting gdb server for stm32f4x.cpu on 3333
Info : Listening on port 3333 for gdb connections
TargetName         Type       Endian TapName            State

0* stm32f4x.cpu       hla_target little stm32f4x.cpu       running
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
target halted due to debug-request, current mode: Thread
xPSR: 0x01000000 pc: 0x0800b0a0 msp: 0x2000a9c0
Info : device id = 0x20016419
Info : flash size = 2048 kbytes
Info : Dual Bank 2048 kiB STM32F42x/43x/469/479 found
auto erase enabled
wrote 262144 bytes from file C:/XXXXX/build/zephyr/zephyr.hex in 7.781227s (32.900 KiB/s)
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
target halted due to debug-request, current mode: Thread
xPSR: 0x01000000 pc: 0x0800b0a0 msp: 0x2000a9c0
verified 168160 bytes in 1.359219s (120.818 KiB/s)
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
Info : Unable to match requested speed 2000 kHz, using 1800 kHz
shutdown command invoked

Thanks for taking the time to file an issue. Please see https://github.com/zephyrproject-rtos/zephyr/issues/52262 for an explanation of why this is not working the way you think it should. To try to resolve your issue, check if your runner supports --file and use that instead if it does.


Can we make libxc available for Apple Silicon via conda-forge?

Comment:
Could you make libxc available also for Apple Silicon MacOS? With this I think it will be easier to install GPAW via pip or conda for Apple Silicon MacOS (currently it seems not easy). I naively guess that we just need to follow https://conda-forge.org/blog/posts/2020-10-29-macos-arm64/ particularly
build_platform:
  osx_arm64: osx_64

but I am not sure what else one should do. If someone knows what to do by heart, I would like to kindly ask. Otherwise I can give a try in future, but not sure when...

giving it a try with https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/4035

Thank you @loriab for this addressing the issue. I haven't succeeded to install libxc via conda yet. I guess this may be because I cannot so far use the latest version of conda (23.1.0). I hope this will be soon easily available after next update of anaconda. Then I would like to test this again.

I think #42 is ready-to-go, but one won't be able to actually install packages from conda until a few minutes after that PR is merged.


Replacing keras by tensorflow.python.keras causes AttributeError: 'ThreadsafeIterator' object has no attribute 'shape'

System information

OS Platform and Distribution (Windows 10 64bit):
TensorFlow installed from (pip):
TensorFlow version (1.13.1):
Python version: 3.6
CUDA/cuDNN version: 10
GPU model and memory: RTX 2060 6GB
Keras 2.2.4

Using PyCharm.
Describe the problem.
When I try my model with keras and tensorflow.python.keras I found difference.
The model on keras works but when I use tensorflow.python.keras throwing errors:
File "C:\python36_ver\lib\site-packages\tensorflow\python\keras\engine\training.py", line 1427, in fit_generator     initial_epoch=initial_epoch)
File "C:\python36_ver\lib\site-packages\tensorflow\python\keras\engine\training_generator.py", line 119, in model_iteration     shuffle=shuffle)
File "C:\python36_ver\lib\site-packages\tensorflow\python\keras\engine\training_generator.py", line 399, in convert_to_generator_like     num_samples = int(nest.flatten(data)[0].shape[0])
AttributeError: 'ThreadsafeIterator' object has no attribute 'shape'

@bwolfus  In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

I have the same issue. The generator passed to model.fit_generator is created with code
@threadsafe_generator
def generator_x(image_ids, data_dir, batch_size, do_shuffle=True, name=None,
                image_subdir='images', image_suffix='.png', mask_subdir='masks', mask_suffix='.png'):

    while True:
        # Shuffle the list of images from 'train_ids'
        if do_shuffle:
            ids = shuffle(image_ids)
        else:
            ids = image_ids

        for start in range(0, len(ids), batch_size):
            x_batch = []
            y_batch = []
            end = min(start + batch_size, len(ids))
            ids_batch = ids[start:end]
            for image_id in ids_batch:
                image_path = os.path.join(data_dir, image_subdir, '%s%s' % (image_id, image_suffix))
                img = imread(image_path)
                # Convert color space, if needed
                img = from_bgr(img, color_space)

                mask_path = os.path.join(data_dir, mask_subdir, '%s%s' % (image_id, mask_suffix))
                mask = imread(mask_path, cv2.IMREAD_GRAYSCALE)
                mask = np.expand_dims(mask, axis=2)

                x_batch.append(img)
                y_batch.append(mask)

            x_batch = np.array(x_batch, np.float32) / 255
            y_batch = np.array(y_batch, np.float32) / 255
            yield x_batch, y_batch

That generator is passed in tf.keras code to convert_to_generator_like that is not able to use it (the error as above).

Automatically closing due to lack of recent activity from the author. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!
@jpsacha Please post a new issue by filling all the information asked by the template. Thanks!

This problem still presist


Python files are excluded during frogbot PR Scan

Describe the bug
I have python files and while running frogbot, those files are excluded.
21:36:04 [Debug] The path '/tmp/jfrog.cli.temp.-1708407363-3912552052/smaq/integration-tests/test_kill_relaunch_procs_smaq.py' is excluded
Current behavior
I have python files and while running frogbot, those files are excluded.
21:36:04 [Debug] The path '/tmp/jfrog.cli.temp.-1708407363-3912552052/smaq/integration-tests/test_kill_relaunch_procs_smaq.py' is excluded
Reproduction steps
No response
Expected behavior
No response
JFrog Frogbot version
Frogbot version: 2.19.9
Package manager info
No package manager
Git provider
Azure DevOps
JFrog Frogbot configuration yaml file
steps:
included comment

task: CmdLine@2
condition: eq(variables['Build.Reason'], 'PullRequest')
displayName: 'Download and Run Frogbot Scan'
env:
JF_WATCHES: $(xray_watch_policyname)
JF_GIT_PULL_REQUEST_ID: $(System.PullRequest.PullRequestId)
JF_GIT_BASE_BRANCH: $(System.PullRequest.TargetBranch)
JF_GIT_PROJECT: $(System.TeamProject)
JF_GIT_API_ENDPOINT: $(System.CollectionUri)
JF_GIT_TOKEN: $(access-token-pat)
JF_VULN_CONTEXTUAL_ANALYSIS: TRUE
JF_URL: $(artifactoryurl)
JF_ACCESS_TOKEN: $(jfrog-identity-token)
JFROG_CLI_LOG_LEVEL: "DEBUG"
JF_GIT_OWNER: "KLA-GPG"
JF_AVOID_PREVIOUS_PR_COMMENTS_DELETION: TRUE
JF_GIT_REPO: "virgo"
JF_RELEASES_REPO: ""
JF_GIT_PROVIDER: "azureRepos"
JF_FAIL: FALSE
JF_INCLUDE_ALL_VULNERABILITIES: FALSE
JF_WORKING_DIR: ${{ parameters.componentName }}/${{ parameters.subcomponentname }}
JF_AVOID_EXTRA_MESSAGES: true  # to avoid extra messages in the comment from JFROG
JF_PR_COMMENT_TITLE: ${{ parameters.componentName }}/${{ parameters.subcomponentname }}Report
inputs:
script: |
export HOME="/usr/share/maven"
export M2="$HOME/bin"
export MAVEN_OPTS="-Xms256m -Xmx512m"
export PATH="$M2:$PATH"
mvn --version
getFrogbotScriptPath=$(if [ -z "$JF_RELEASES_REPO" ]; then echo "https://releases.jfrog.io"; else echo "${JF_URL}/artifactory/${JF_RELEASES_REPO}"; fi)
curl -fLg "$getFrogbotScriptPath/artifactory/frogbot/v2/[RELEASE]/getFrogbot.sh" | sh
./frogbot spr
#./frogbot scan-and-fix-repos $(Build.SourceBranch)
continueOnError: true

Operating system type and version
ubuntu 18.04
JFrog Xray version
No response

Hello @vinodhini-devops and thank you for using Frogbot!
The issue you reported is in fact not a bug, but rather an intended behavior.
There is a default exclude pattern for Frogbot if you don't set your own excluded pattern using JF_PATH_EXCLUSIONS in frogbot-config.yml. The default pattern is: .git;node_modules;target;venv;test
since the file you describe has  the word 'test' in it and we have it as a regexp in our default pattern- this file is intentionally excluded.
If you want to avoid this exclusion you need to set your own exclude pattern (can be empty).
This suppose to fix you issue and Ill be happy to hear if it actually did. If not please feel free to leave another comment here

Hello @vinodhini-devops
Did the above answer helped resolving your issue? if so- can I close this ticket?

Hello again @vinodhini-devops,
We didn't get a response from you in a while, I assume my explanation explained the behaviour you are experiencing.
If you still facing any issues on this matte, feel free to re-open this issue.
And if you have any further questions, please open another GitHub issue and we will be glad to assist :)


Export to XLSX produces corrupt file

b### Description
since version DBeaver 23.1.3, the XLSX export produces a corrupt file.
Here is the error message: Excel cannot open the file '20230803-164007.xlsx' because the file format or file extension is not valid. Verify that the file has not been corrupted and that the file extension matches the format of the file.
DBeaver Version
Community Edition 23.1.3
Operating System
Windows x64
Database and driver
Database: Apache Hive 1.2.1<IP_ADDRESS>.0-292
Driver: hive-jdbc-2.0.0-standalone
Steps to reproduce

I connect to the database
I execute my sql script
I push the Excel button which again executes my sql script and produces an XLSX file
This XLSX file is corrupt since version DBeaver 23.1.3

Additional context
No response

Please provide a sample table ddl and table data to reproduce the issue.

When testing with the sample table data, the error disappeared.  So, I suppose this is due to an error on the server side.


Update hetzner CCM to v1.17.1

Which issue(s) this PR fixes:
Xref #2782
What type of PR is this?
/kind chore
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Update hetzner CCM to v1.17.1

Documentation:

NONE

/retest


AAP-18228: Handle user's trial expiry

Hello,
See https://issues.redhat.com/browse/AAP-18228
This PR is related to https://github.com/ansible/ansible-wisdom-service/pull/761
Thanks in advance!
Roger

Hi @romartin, @ganeshrn,
Unit tests and final Trial expiry message is added in the new PR: https://github.com/ansible/vscode-ansible/pull/1037 this PR can be closed.
Thank you!

PR superseded by https://github.com/ansible/vscode-ansible/pull/1037


I started to get a UI API called from a background thread: Queue name: com.facebook.react.ShadowQueue,

Hi, I need a bit of support here, I'm not sure if I did something, but I think I didn't besides updating the package to the latest versions, and all of the sudden I started getting this warning on app start:

I already combed through my app code, I'm sure I'm not using shadows anywhere, and the error is there any time I use a  component, any idea of how to get rid of this?

Hi @ospfranco: it is a legitimate bug in the repo that we make this UI Thread checker assert if a RCTDynamicColor is used.   But it should only be encountered if a color prop is assigned a dynamic value, e.g. color: { dynamic: { light: 'black', dark: 'white' } }.   Do you explicitly use a dynamic color in your app?   Otherwise none of the built-in components should use a dynamic color.   The RNTester does, but that shouldn't impact your code.

Yes, I do, and extensively it is necessary to have consistent color schemes to pass apple's app review guidelines

Ok.   We need to fix it.    Upstream, for iOS, we are able to implement dynamic colors using UIColor initWithDynamicProvider.  The macOS NSColor originally did not have an equivalent so we made this RCTDynamicColor.   In macOS 10.15 and later there is now a NSColor colorWithName:dynamicProvider: that looks promising -- so we could use it if running on 10.15 or later and fallback to RCTDynamicColor otherwise.

I would actually like to keep this bug and open. I was not aware that there was a way to make Dynamic Colors thread safe. With RN-macOS 0.66, we are bumping the minimum to 10.15 so this might actually be a good solution.

This should be fixed with the latest RN-macOS 0.66.13 +


Add retry support to every request

This PR added a retry function applicable to all the requests sent
to openwhisk services. Since openwhisk supports HA for controllers,
which can pick up requests and handles them in a parallel fashion,
we need to avoid the race condition.

I’m really confused what the goals are. The 409s you’re seeing are specific to your test cases. Why not add retries in the tests for wskdeploy?
Nicks suggestion for which codes to retry makes sense. Beyond that it’s dubious what you’re trying to get out of a retry.

It is not specific to my test cases. It will happen to all the situations, if multiple requests are sent to openwhisk in a very short time.
As long as the user using client side code will bump into this issue, if requests are sent fast enough.

That’s right: the client which is the user of this library should decide on the retry policy. Not this go library itself. What does it mean to retry a conflict which inherently means an invariant might have been violated. The compensation and resolution will be specific to a scenario. Like the test cases.

Let’s focus on a specific example where you first observed the 409.
You disable a rule.
Then you delete the rule.
Say the delete succeeds. The disable will fail with a 404. Does it make sense to retry in this case? You’ll have to adjust your test case anyway.
If you retry the delete because the disable won the race, you’ll get the test outcome you want. However since this is a race already you have to adjust the test case either way.
So can we work through a couple of example so that we can clarify what you’re trying to accomplish?

I agree with @rabbah - let's get specific on these HA/race-condition issues and then decide on the best practice way to handle these.  developers using the REST API will need to know how to deal with these issues as well.


Revert "feat: RND-119: Add token usage in inference output (#183)"

This reverts commit f1cbffb728e8bdf0c7cb418b6a25890f4a6f1ede.

Codecov Report
Attention: Patch coverage is 67.74194% with 10 lines in your changes missing coverage. Please review.

Project coverage is 61.17%. Comparing base (f1cbffb) to head (ddd4f66).
Report is 1 commits behind head on master.

Files with missing lines
Patch %
Lines

adala/runtimes/_litellm.py
66.66%
10 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
- Coverage   61.57%   61.17%   -0.41%
==========================================
  Files          44       44
  Lines        2064     2032      -32
==========================================
- Hits         1271     1243      -28
+ Misses        793      789       -4

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

Commit https://github.com/HumanSignal/infra/commit/3e4f8a7b5d011cdff4850ca80da837172b7ee901 is created.

Commit https://github.com/HumanSignal/infra/commit/4990b5fb89e5db96c4a93da1f65026339c2059e9 is created.


Feature: Select All Locations

First of all, you are my hero for this Week! I was looking literally hours for a script/program/website to convert Semantic location Data (.json) to csv files.
Now I have a feature request.
If it would be possible top select all Locations (to export them) this tool would again save me hours of Work.
(My mother made a Moto Trip around South America- now i want to visualize her route outside of Google Maps)
best regards!

@MueJosh Glad you find this app helpful!  I have added Select All, Deselect All, and Invert Selection buttons. Hope this makes your life easier!
Let me know how it goes.
Best,
Ryan


Long-term anticipation

Hi, thanks for repo!
Excuse me, could you please tell how to perform long-term anticipation as showed in the paper? Have I missed this part in code, or it's not implemented here yet?

Hi @RodinIvan thanks for your interest in this work! I don't have a config for it since we only used it for some visualizations, however you should be able to get that output by setting the following and running the evaluation with a breakpoint around here
+model.future_predictor.avg_last_n=-1
data_eval.num_frames=3  # To give it only 3 frames to start predicting from
+model.future_predictor.output_len=10  # To predict 10 steps into the future instead of 1
+dataset_eval.load_long_term_future_labels=${model.future_predictor.output_len}

It will need a bit more code to generate the figures shown in the paper but I haven't gotten around to add that yet


Update search-limits-quotas-capacity.md

Updating Create and Delete Index limits: per second instead of per minute

#hold-off per Angie, this needs more testing.

@HeidiSteen
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

Closing this one as the information showing in this page is accurate.


how to get include and lib without Compilation?

for some reason, the network in my country is very bad!!
so i want to use the lib and dll directly.
and finally i found no where to download it?
compile it is so hard for me, is there a solution?

you can get the latest appveyor build artifacts from
win32
https://ci.appveyor.com/project/zdenop/tesseract/build/master.993/job/c8rtk21uqliscckn/artifacts
win64
https://ci.appveyor.com/project/zdenop/tesseract/build/master.993/job/c6uplulmcrk7bew8/artifacts

@Shreeshrii Thanks for the links! How can I use this build in my QT C++ project?

@Shreeshrii
thanks for you help.
i try download it, but i can not find any .lib file there.
i perfer files below

libtesseract304d.lib
giflib416-static-mtdll-debug.lib
giflib416-static-mtdll-debug.lib
giflib416-static-mtdll.lib
giflib416-static-mtdll.lib
libjpeg8c-static-mtdll-debug.lib
libjpeg8c-static-mtdll-debug.lib
libjpeg8c-static-mtdll.lib
libjpeg8c-static-mtdll.lib
liblept171-static-mtdll-debug.lib
liblept171-static-mtdll-debug.lib
liblept171-static-mtdll.lib
liblept171-static-mtdll.lib
liblept171.lib
liblept171.lib
liblept171d.lib
liblept171d.lib
libpng143-static-mtdll-debug.lib
libpng143-static-mtdll-debug.lib
libpng143-static-mtdll.lib
libpng143-static-mtdll.lib
libtiff394-static-mtdll-debug.lib
libtiff394-static-mtdll-debug.lib
libtiff394-static-mtdll.lib
libtiff394-static-mtdll.lib
zlib128-static-mtdll-debug.lib
zlib128-static-mtdll-debug.lib
zlib128-static-mtdll.lib
zlib128-static-mtdll.lib

@weituotian: we provide source code only - e.g. we do not produce and distribute windows libraries (with header files), that you can use in your project.

@zdenop and you know where to get what we want?

@weituotian, get the code and compile it yourself. You won't save much network bandwidth if you could get libraries.

@weituotian. you can try to ask on tesseract user forum.


Upgrade to broccoli-persistent-filter

@shinnn @stefanpenner

Landed in https://github.com/shinnn/broccoli-clean-css/commit/b13e42d65c373aaca96b5db58f7a5100aa6fe4ff.
Thanks, @thoov!

@thoov @stefanpenner
I'm planning to release the next version as v2.0.0 which uses ES2015 syntax as possible and drop Node 0.x. Can I just go as planned, or should I still support Node v0.x?

or should I still support Node v0.x?

Ember-cli continues to support node 0.10.x (my employers default infrastructure is still tied to 0.10.x, so I will need to continue its support) I would prefer this library also remain 0.10.x+, otherwise we wont be able to upgrade.

otherwise we wont be able to upgrade.

Hmm, it seems that Ember-cli currently doesn't depend on broccoli-clean-css. https://github.com/ember-cli/ember-cli/commit/489f68b6a3bdca1bb4aadefb71c5ba75a6b22bbc#diff-b9cfc7f2cdf78a7f4b91a753d10865a2L78

it does via its dependency on the preprocessor registry https://github.com/ember-cli/ember-cli-preprocessor-registry/blob/master/package.json#L20 which includes csso as the default fallback: https://github.com/ember-cli/ember-cli-preprocessor-registry/blob/2fc5043a871cd7491f3cd374c10b0f08c0de7509/preprocessors.js#L116-L117

OK, I released the latest version as v1.1.0 which still supports Node 0.10.x.

@shinnn thank you kindly.


Configurable default measurement units

remove partly wrong default measurement units from database ('weight' and 'volume' unit isn't 'mm')
allow configuration of default measurement units in config file shipping.php with 'default' => true key/value pair
refactoring attribute getter methods to attribute casting accessor with correct default values

Because this pull request is a fix and an enhancment, it could be better to merge it into main branch?

Do you see any changes this would cause that would need explaining in the upgrade guide?

@netzknecht could you retarget to 0.8 branch please?

This PR seems to have lots of updates not associated to the functionality specified. Closing, but feel free to reopen if you can make a PR that is focused on a single issue.


Update toolchain to 2024-09-23

Changes required due to:

https://github.com/rust-lang/rust/pull/130593 Sync from rustfmt
https://github.com/rust-lang/rust/pull/124895 Disallow hidden references to mutable static

With the exception of changes to rust-toolchain.toml, rustfmt.toml, and library/kani/src/futures.rs all changes were automatically created by running scripts/kani-fmt.sh.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Not upgrading to 2024-09-24 as that version ICEs when trying to build http-body.

Not upgrading to 2024-09-24 as that version ICEs when trying to build http-body.

Will be fixed by https://github.com/rust-lang/rust/pull/130775 (which reverts earlier changes).


⚠ Fortnite has degraded performance

In 2ba4cdc, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1551 ms

Resolved: Fortnite performance has improved in 211c5c3.


sys.exit import missing in supert_metric.py

https://github.com/Yale-LILY/SummEval/blob/1eeb182cf35b2ff9ecaa9833e46e34e7e9056470/evaluation/summ_eval/supert_metric.py#L22
Hello,
When I try to run supert_metric() I get the error:

NameError: name 'exit' is not defined

Which is perhaps due to the fact that from sys import exit is missing in the file

Hi,
What Python version are you using (and any other environment details would be helpful)? I don't believe it needs to be imported.

Hi Alex,
I'm trying to run it in google colab notebook, and the python version it shows there is: Python 3.7.10

It looks like this is only a problem when running in a colab/ipython notebook. It still prints the export statement to be run, stops the program (albeit with an error), and won't throw an error when the env variable is exported - I'll modify this when I clean up the code next.

Hi Alex,
When I did specify the python path where summ-eval is installed by running:
%env PYTHONPATH="/env/python:/usr/local/lib/python3.7/dist-packages/summ_eval"
Then I get the error stack trace below. I think it is also related to google colab perhaps. Can you guide me regarding this? Thanks!
`INFO:root:Load pretrained SentenceTransformer: bert-large-nli-stsb-mean-tokens
INFO:root:Did not find a / or \ in the name. Assume to download model from server
INFO:root:Downloading sentence transformer model from https://public.ukp.informatik.tu-darmstadt.de/reimers/sentence-transformers/v0.2/bert-large-nli-stsb-mean-tokens.zip and saving it at /root/.cache/torch/sentence_transformers/public.ukp.informatik.tu-darmstadt.de_reimers_sentence-transformers_v0.2_bert-large-nli-stsb-mean-tokens.zip
100%|██████████| 1.24G/1.24G [00:47<00:00, 26.4MB/s]
INFO:root:Load SentenceTransformer from folder: /root/.cache/torch/sentence_transformers/public.ukp.informatik.tu-darmstadt.de_reimers_sentence-transformers_v0.2_bert-large-nli-stsb-mean-tokens.zip
ModuleNotFoundError                       Traceback (most recent call last)
 in ()
1 import logging
2 logging.getLogger().setLevel(logging.INFO)
----> 3 rnd_supert = get_supert(mlsum_rnd_sum, mlsum_src)
14 frames
 in get_supert(hypothesis, sources, avg)
111
112 def get_supert(hypothesis, sources, avg=True):
--> 113     metric = supert_metric.SupertMetric()
114     metric.bert_model = SentenceTransformer('xlm-r-100langs-bert-base-nli-stsb-mean-tokens')
115     if avg:
/usr/local/lib/python3.7/dist-packages/gin/config.py in gin_wrapper(*args, **kwargs)
1067       scope_info = " in scope '{}'".format(scope_str) if scope_str else ''
1068       err_str = err_str.format(name, fn_or_cls, scope_info)
-> 1069       utils.augment_exception_message_and_reraise(e, err_str)
1070
1071   return gin_wrapper
/usr/local/lib/python3.7/dist-packages/gin/utils.py in augment_exception_message_and_reraise(exception, message)
39   proxy = ExceptionProxy()
40   ExceptionProxy.qualname = type(exception).qualname
---> 41   raise proxy.with_traceback(exception.traceback) from None
42
43
/usr/local/lib/python3.7/dist-packages/gin/config.py in gin_wrapper(*args, **kwargs)
1044
1045     try:
-> 1046       return fn(*new_args, **new_kwargs)
1047     except Exception as e:  # pylint: disable=broad-except
1048       err_str = ''
/usr/local/lib/python3.7/dist-packages/summ_eval/supert_metric.py in init(self, ref_metric, sim_metric)
25 class SupertMetric(Metric):
26     def init(self, ref_metric='top15', sim_metric='f1'):
---> 27         self.bert_model = SentenceTransformer('bert-large-nli-stsb-mean-tokens')
28         self.ref_metric = ref_metric
29         self.sim_metric = sim_metric
/usr/local/lib/python3.7/dist-packages/summ_eval/sentence_transformers/SentenceTransformer.py in init(self, model_name_or_path, modules, device)
73                 modules = OrderedDict()
74                 for module_config in contained_modules:
---> 75                     module_class = import_from_string(module_config['type'])
76                     module = module_class.load(os.path.join(model_path, module_config['path']))
77                     modules[module_config['name']] = module
/usr/local/lib/python3.7/dist-packages/summ_eval/sentence_transformers/util.py in import_from_string(dotted_path)
67         raise ImportError(msg)
68
---> 69     module = importlib.import_module(module_path)
70
71     try:
/usr/lib/python3.7/importlib/init.py in import_module(name, package)
125                 break
126             level += 1
--> 127     return _bootstrap._gcd_import(name[level:], package, level)
128
129
/usr/lib/python3.7/importlib/_bootstrap.py in _gcd_import(name, package, level)
/usr/lib/python3.7/importlib/_bootstrap.py in find_and_load(name, import)
/usr/lib/python3.7/importlib/_bootstrap.py in find_and_load_unlocked(name, import)
/usr/lib/python3.7/importlib/_bootstrap.py in _call_with_frames_removed(f, *args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py in _gcd_import(name, package, level)
/usr/lib/python3.7/importlib/_bootstrap.py in find_and_load(name, import)
/usr/lib/python3.7/importlib/_bootstrap.py in find_and_load_unlocked(name, import)
ModuleNotFoundError: No module named 'sentence_transformers'
In call to configurable 'SupertMetric' (<class 'summ_eval.supert_metric.SupertMetric'>)

NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.
To view examples of installing some common dependencies, click the
"Open Examples" button below.
---------------------------------------------------------------------------`

Can you please confirm that PYTHONPATH has been set correctly (make sure "! echo $PYTHONPATH" returns "/env/python:/usr/local/lib/python3.7/dist-packages/summ_eval") and that the folder "/usr/local/lib/python3.7/dist-packages/summ_eval/sentence_transformers" exists?

Yup the path is correct, and the directory path exists

That's odd - I'll try to figure this out, but for now, as a workaround, you can just add a cell with '% cd /usr/local/lib/python3.7/dist-packages/summ_eval/'

Hi Alex,
Thanks a  lot. It worked using  the workaround!
Best,
Zohaib


fix db:seed data

the save_metadata update broke this.  Now it is fixed.

👍

Coverage remained the same at 96.213% when pulling 85b9844b232797e57215a68e394ea0394d38e201 on DEC-1085 into 621d9637050e42b0f946bdee8486882d097c9d7b on master.

👍


require.context with variable for directory not string

Doing something like require.context(var) doesn't work. I'm building a CLI tool that use webpack to build web sites and want to be able to dynamically create contexts when the CLI tool is run but can't as require.context seems to only work with string directories.

Is there any update on this?


RouteDefinition validation not pass

with my config like:
spring:
  application:
    name: gateway
  cloud:
    gateway:
      routes:
        - id:  flow
          uri: lb://WORKFLOW/
          predicates:
            - Path=/flow/**

application cannot startup
***************************
APPLICATION FAILED TO START
***************************

Description:

Failed to bind properties under 'spring.cloud.gateway.routes[0]' to org.springframework.cloud.gateway.route.RouteDefinition:

    Property: spring.cloud.gateway.routes[0].uri
    Value: lb://WORKFLOW/
    Origin: class path resource [bootstrap.yml]:14:16
    Reason: HV000083: The specified annotation javax.validation.constraints.NotEmpty defines no attribute 'groups'.

Action:

Update your application's configuration

debug info:

But this configuration has no problems in other projects.I don't know where the problem is.

the reson is RouteDefinition id filed has annotation NotEmpty
and pom.xml add
<dependency>
    <groupId>net.sf.oval</groupId>
    <artifactId>oval</artifactId>
    <version>1.90</version>
</dependency>

and spring cloud gateway using
<dependency>
    <groupId>javax.validation</groupId>
    <artifactId>validation-api</artifactId>
    <version>2.0.1.Final</version>
</dependency>

they have same package javax.validation.constraints
and the validator use oval's,and don't have groups filed

If you exclude
<dependency>
    <groupId>javax.validation</groupId>
    <artifactId>validation-api</artifactId>
    <version>2.0.1.Final</version>
</dependency>

does it work?
Can you not use validation-api?

@ryanjbaxter
exclude validation-api have same problem.
in my project the validation-api is import by spring-boot-starter-webflux only.
my solution is changing the order in pom.xml like:
       <dependency>
            <groupId>org.springframework.boot</groupId>
            <artifactId>spring-boot-starter-webflux</artifactId>
        </dependency>
        <dependency>
            <groupId>net.sf.oval</groupId>
            <artifactId>oval</artifactId>
            <version>1.90</version>
        </dependency>

No, I'm afraid ordering is the only solution.


feat(hello_bot): add code example with current location

An example to extract lat and long of the current location and returning the address via FB message through accessing the Google Maps Public API.

Note: in this commit I've indented the carousel example and added the part with if "message" in messaging_event:to indicate where the example can be added to to run it immediately.


Change directory to a string

Directory path must be a string but it's set to boolean

Oops, typo from my side! Thanks for the PR 🙏

:tada: This PR is included in version 2.0.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


CREATE add on CSRF TOKEN UBER DRIVER

Hi! I'm not sure what you're asking, very sorry. Can you clarify?

Closing from no response. I can always re-open if you clarify 👍


[Feature Request]: Add icon for bluesky

It would be very nice to have an icon for bluesky, similar to how we have it for mastodon :)

@satnaing i would like to pick this issue, can you please assign it to me.

I've added to my website
website: https://yogeshpaliyal.com
https://github.com/yogeshpaliyal/yogeshpaliyal.com/commit/587aa0a4ec257cb075b42c60ea6e55d1e02150cd


[KYUUBI #3990] Support for translating SELECT TABLE_CAT FROM system.jdbc.catalogs to Getcatalogs operation in Trino front protocol

Why are the changes needed?

close https://github.com/apache/incubator-kyuubi/issues/3990, Kyuubi introduce Trino front protocol in https://github.com/apache/incubator-kyuubi/issues/3901, this issue as a subtask to support for translating SELECT TABLE_CAT FROM system.jdbc.catalogs to Getcatalogs operation.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

How about we integrate the Trino front-end protocol with the parser module this way? @ulysses-you


Update to be compatible with ESPHome 2023.3.0 release

Update to be compatible with ESPHome 2023.3.0 release  https://github.com/esphome/esphome/pull/4275

Thanks for the update!


IM Reporting layer incorrectly applies hold-off for chunked messages

Problem
The minimum reporting interval is used to govern the minimum time separation between successive Report transactions. Within a given Report transaction, it's possible to send multiple ReportData messages in the case of chunked data.
The set of chunked messages that are part of the same logical ReportData action should not however be subject to minimum reporting intervals. However, our current implementation incorrectly applies that restriction.
To repro the problem:

Launch the REPL server
Setup a subscription with full wildcard to it from another chip-tool/repl instance with min-interval = 5s.
Create a new endpoint on the REPL server consisting of a a number of clusters, including the test cluster. This ensures it will exceed an MTU later.
Wait at least 5s from the time the sub was created.
Finalize the endpoint.

You should see chunked reports going out every 5s, when in fact, they should be sent immediately.

This will affect subscriptions to hubs mostly, especially when new endpoints come online.

We should add a unit-test to validate the eventual fix FYI.

V1 review: acceptable for master changes for v1 as this is fixing spec.

Fixed by https://github.com/project-chip/connectedhomeip/pull/22139


Can't install extensions to WSL2 while connected to VPN

Issue Type: Bug
While connected to a VPN, I'm unable to install extensions to WSL: UBUNTU-20.04
After exiting VSCode and closing my VPN connection, I restarted VSCode, and was able to install extensions to WSL Ubuntu.

Can you please provide following logs while installing the extension

F1 > Set Log Level > Trace
Help -> Toggle Developer Tools -> Console
View -> Command Pallette... -> Toggle Shared Process -> Console
View -> Open View… -> Log (Shared)
View -> Open View… -> Log (Window)

Here you go.
Thanks,
Oren.
sharedLog.txt
shared-process-1619821574128.log
rendererLog.txt
dev-tools-console-1619821460950.log

Seems the logs does not include any info. May be these are new ones.
Can you please try to provide logs while you are reproducing the issue?
Can you also  please include Remote Logs - View -> Open View... -> Log (Remote)


build.sh updates for ansible module changes

Susequent PR to support https://github.com/ppc64le-cloud/k8s-ansible/pull/17

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: piyushgupta1551
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @mkumatag
/assign @piyushgupta1551

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag, piyushgupta1551
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


which backbone is better?

There are two backbone nets used in  lanenet,VGG and BiseNetV2, which is better?   vgg is larger than BiseNetV2,so vgg is better in case not care about weight size?

@hitbuyi BiseNetV2 is better. VGG is too old. Maybe you can try ocrnet or other sota segmentation architecture:)


api.setColumnState(state) not render cell group

When call api.setColumnState grid not render cell Group

Is this happening on the demo page for you? What you mean by cell group?

Hi ceolter,
this is demo
http://plnkr.co/edit/VYOmzXjAcz9ARqzjWJR6?p=preview

thanks for raising. i see what you mean. i'll need to look into it and get back to you.

it's because your are pivoting by an imaginary column. if you stick this onto the end of your columns, it will work:
{headerName: "field1", field: "field1", hide: true}
not sure if this is a bug, or a feature, i'll have to think about it. it's not simple to 'fix' as the setColumnState() sets the state, including pivoting. maybe i include a flag 'suppressPivot' so it doesn't overwrite pivot info. or can just say it doesn't support the pivot by non-columns.

pivoting and grouping has undergone a lot of changes since this issue. if this is still an issue, please reopen.