Add AV1 color space parsing in MP4 atom parser

This adds av1C parsing based on the AV1 bitstream specification: https://aomediacodec.github.io/av1-spec/av1-spec.pdf
This is needed to have correct color space when using hardware AV1 decoder.

I'm going to send this for internal review now. You may see some more commits being added as I make changes in response to review feedback. Please refrain from pushing any more substantive changes as it will complicate the internal review - thanks!

@haixia-meta
Actually would you be able to add a unit test to the Mp4ExtractorTest file with a small av1 sample with color info for testing purposes?

@haixia-meta
Actually would you be able to add a unit test to the Mp4ExtractorTest file with a small av1 sample with color info for testing purposes?

I'd love to but do you have any example or documentation on how to generate the prerecorded dump files associated with the av1 sample mp4? Thanks

@haixia-meta
Actually would you be able to add a unit test to the Mp4ExtractorTest file with a small av1 sample with color info for testing purposes?

I'd love to but do you have any example or documentation on how to generate the prerecorded dump files associated with the av1 sample mp4? Thanks

If you were to add your test and run the Mp4ExtractorTest, it should fail with an error in DumpFileAsserts. This is due to the default DUMP_FILE_ACTION being COMPARE_WITH_EXISTING. The failure would tell you to change the action to WRITE_TO_LOCAL to generate the dump file. Make sure you change the DUMP_FILE_ACTION back to COMPARE_WITH_EXISTING afterwards though!

@haixia-meta
Actually would you be able to add a unit test to the Mp4ExtractorTest file with a small av1 sample with color info for testing purposes?

I'd love to but do you have any example or documentation on how to generate the prerecorded dump files associated with the av1 sample mp4? Thanks

If you were to add your test and run the Mp4ExtractorTest, it should fail with an error in DumpFileAsserts. This is due to the default DUMP_FILE_ACTION being COMPARE_WITH_EXISTING. The failure would tell you to change the action to WRITE_TO_LOCAL to generate the dump file. Make sure you change the DUMP_FILE_ACTION back to COMPARE_WITH_EXISTING afterwards though!

thank you for explaining! forgive my n00b question but how do I run the Mp4ExtractorTest? thanks again

If you were to add your test and run the Mp4ExtractorTest, it should fail with an error in DumpFileAsserts. This is due to the default DUMP_FILE_ACTION being COMPARE_WITH_EXISTING. The failure would tell you to change the action to WRITE_TO_LOCAL to generate the dump file. Make sure you change the DUMP_FILE_ACTION back to COMPARE_WITH_EXISTING afterwards though!

thank you for explaining! forgive my n00b question but how do I run the Mp4ExtractorTest? thanks again

A simple way to do it is open the project in Android Studio, go to the Mp4ExtractorTest file and click the play symbol next to the class declaration to run the test file. Or you can right-click on the file in the list in the Project-Pane and select "Run 'Mp4ExtractorTest'".

If you were to add your test and run the Mp4ExtractorTest, it should fail with an error in DumpFileAsserts. This is due to the default DUMP_FILE_ACTION being COMPARE_WITH_EXISTING. The failure would tell you to change the action to WRITE_TO_LOCAL to generate the dump file. Make sure you change the DUMP_FILE_ACTION back to COMPARE_WITH_EXISTING afterwards though!

thank you for explaining! forgive my n00b question but how do I run the Mp4ExtractorTest? thanks again

A simple way to do it is open the project in Android Studio, go to the Mp4ExtractorTest file and click the play symbol next to the class declaration to run the test file. Or you can right-click on the file in the list in the Project-Pane and select "Run 'Mp4ExtractorTest'".

Apologies for the delay but I'm having trouble with the current main branch, as loading Android Studio encounters the error:

Caused by: org.gradle.api.ProjectConfigurationException: A problem occurred configuring project ':lib-decoder-midi'.

I wonder if this is a known issue and if I can work around this by checking to a previous stable checkpoint? Alternatively I can share an AV1 sample mp4 but would need help to generate the dump files. Thanks

@microkatz - as requested, added new test case for Mp4ExtractorTest with sample AV1 test file with color info and the corresponding dump files.

Hey @haixia-meta,
I ran your unit test and I don't think it is testing the code that you added. When I ran the unit test it appears that the code exits in line 2246 with obu_type != OBU_SEQUENCE_HEADER with obu_type being 0. The colorInfo in your generated unit test files comes from a colr type Atom. Did you notice this as well? Do you have any other samples you can provide?

hey @microkatz - apologies; this was broken by a merge with the recently merged https://github.com/androidx/media/pull/491 which modified the parser byte position
I've updated the parser to skip ahead 1 byte instead of 4 which fixed the issue
Also updated the test mp4 file which manually removed the colr atom to make sure we're parsing color from av1c.

Hey @haixia-meta,
I ran your unit test and I don't think it is testing the code that you added. When I ran the unit test it appears that the code exits in line 2246 with obu_type != OBU_SEQUENCE_HEADER with obu_type being 0. The colorInfo in your generated unit test files comes from a colr type Atom. Did you notice this as well? Do you have any other samples you can provide?

Makes sense and great! Glad we caught that.
I noticed your new sample file has about ten times the number of samples that your old one had. Any chance you can shorten the mp4 sample you just added?

Makes sense and great! Glad we caught that.
I noticed your new sample file has more than ten times the number of samples that your old one had. Any chance you can shorten the mp4 sample you just added?

done - I reverted to the previous shorter mp4 file with the "colr" atom manually renamed to "free"

@haixia-meta
Thank you for your patience. The code is still in review. If I may ask, where in the av1c spec does the requirement that the obu_size must not be greater than 127? Its in your Parser logic just after reading for obuHasSizeField.

@haixia-meta
Thank you for your patience. The code is still in review. If I may ask, where in the av1c spec does the requirement that the obu_size must not be greater than 127? Its in your Parser logic just after reading for obuHasSizeField. Just trying to match the very-specific parsing logic to the AV1 Bitstream & Decoding Process Specification.

@microkatz thanks for asking! The obu_size in the spec is a leb128() type. See 4.10.5.: unsigned number represented by a variable number of little-endian bytes - the byte sequence is terminated when the MSB is 0 (i.e. byte value ≤ 127). If we can assume that sequence header will never exceed 127 bytes long, then the parsing of leb128() can be simplified as a f(8).


Struct cache should compare requested and cached type and force_reload if they differ.

Line: 225
https://github.com/cxnder/ktool/blob/ea7e6f9f5beecd235e1f73f35c9efb92eb6528d8//src/ktool/dyld.py#L222-L232

So, fixing this directly honestly performs more operations than I'd prefer on 99.9% of struct loads where we dont need to check this. Dropping this.


Log Buffer

[ ] Checkout commit
[ ] revert commit (https://github.com/TimUntersberger/neogit/issues/22#issuecomment-898465278)
[ ] improve visual representation of log (https://github.com/TimUntersberger/neogit/issues/22#issuecomment-913260485)

I would like for the log buffer to re-apper after i've finished looking at a specific commit (I think magit works this way too).
So I would type L, find a commit, type Enter, type q when I'm done, and I'd be shown the log buffer again.

@asymmetric how about adding a seperate keybinding for that? Something like Shift+Enter or Control+Enter?
I have been thinking about introducing something similiar to hooks in magit for neogit, which would make something like this easy to introduce in user config.

Would be cool to have a map to revert a commit from the list! Flow would be something like:
L -> l -> move cursor to commit -> r or R etc -> results in git revert <selected-commit>

Hello!, great plugin.
At the moment during log exploration (L->b or any other option), commits of one a feature branch and commits that occurred on the main branch appear to be at the same level.
In neogit:

Using git log --oneline --decorate --graph --all:

Is there a way of showing them separated currently and I'm just missing an option?
Also, it would be a great addition if the log buffer showed branch locations!

@asymmetric  how about adding a seperate keybinding for that? Something like Shift+Enter or Control+Enter?

@TimUntersberger I'd say that incurs more of a cognitive load than thinking of it as "navigating through a stack". In the stack metaphor, it just makes sense that q pops, so you'd have to type it a couple times to get back to where you want to be.
I would rather have a special keybinding to pop everything from the stack - that would actually be quite useful :)

I would like to filter logs affecting only one file, something like git log src/foo.c.


clusters not deploying on in-cluster krak8s API

As mentioned in Slack last night, the API currently does not work.  Just opening an issue to track progress.

I’ve followed the readme examples for the API and I don’t seem to be able to actually create a new cluster.  Here are my exact requests/responses (run from inside the Launchdock pod within the cluster).  The created cluster never leaves the create_requested state and no new nodes have been created.

# create project
curl -X POST -H "Content-Type: application/json" -d '{"name":"acme"}' http://<IP_ADDRESS>:8080/v1/projects

{
  "created_at": "2017-08-29T00:03:31.043193048Z",
  "id": "f614ab6e",
  "name": "acme",
  "namespaces": null,
  "type": "project"
}

# create namespace
curl -X POST -H "Content-Type: application/json" -d '{"name":"acme-prod"}' http://<IP_ADDRESS>:8080/v1/projects/f614ab6e/namespaces

{
  "applications": null,
  "created_at": "2017-08-29T00:06:07.85514329Z",
  "id": "298f91ef",
  "name": "acme-prod",
  "resources": null,
  "type": "namespace"
}

# create cluster
curl -X POST -H "Content-Type: application/json" -d '{"namespace_id":"298f91ef", "nodePoolSize": 5}' http://<IP_ADDRESS>:8080/v1/projects/f614ab6e/cluster

{
  "created_at": "2017-08-29T01:07:28.912574145Z",
  "id": "b0ef89bd",
  "namespace_id": "298f91ef",
  "nodePoolSize": 5,
  "state": "create_requested",
  "type": "Resource",
  "updated_at": "0001-01-01T00:00:00Z"
}

# note that cluster is sitting in "create_requested" state 30 mins later
curl http://<IP_ADDRESS>:8080/v1/projects/f614ab6e/cluster/b0ef89bd

{
  "created_at": "2017-08-29T01:07:28.912574145Z",
  "id": "b0ef89bd",
  "namespace_id": "298f91ef",
  "nodePoolSize": 5,
  "state": "create_requested",
  "type": "Resource",
  "updated_at": "0001-01-01T00:00:00Z"
}

@jshimko - there are couple of things that we are working to resolve that affect the functioning of the API service.  1) the api requires that the config.yaml file for the cluster be annotated with a pair of marker strings that indicated to the service where the automation entries belong in the file.  2) the container image for the api service requires that several environment variables be passed in to the container so that it can access the cluster state resources (local) and the AWS resources required (cloud based).

With respect to issue 1) the procedure for annotating the config.yaml is documented here: https://github.com/samsung-cnct/krak8s#kraken-configuration-file-integration  but we will work through this to validate the configuration file is setup correctly.

With respect to 2) we have created a new AWS user for the reaction commerce account krakenbot that we will use as the credentialed user for the AWS interaction.  @joejulian is currently working on getting the rest of the environment in to the appropriate helm chart configuration for re-deployment to your cluster.

PR Merged: https://github.com/reactioncommerce/infrastructure/pull/5

PR Merged: https://github.com/reactioncommerce/chart-krak8s-api/pull/1/

👍  Let me know when things are working and I'll give it another test.

We are working on reinstalling the chart with the required values now, will provide an update as soon as we have something testable.

Commit: https://git.launchdock.io/reactioncommerce/krak8s/commit/d2d2f43e0a60ff13ca0f4bd6ba4ccc37b175be96

changes release version tag to 0.3.0, changes the release state to beta and tag's this as latest.

Several changes to:

allow git cli commands to behave correctly w/o a global config https://github.com/samsung-cnct/git-archivist/commit/d07a35d3bb0927681b51deb0b2d1b51d70a229b1
separate git-archivist functions in to two logical components

first: an init container https://github.com/samsung-cnct/git-archivist/commit/6947183f7535ffdf1e6293ece59bd3194377c8c3
second: a long running synchronizer.  https://github.com/reactioncommerce/chart-krak8s-api/pull/2/commits/2ccf7c6818a8faa205111bcd75e68e9680ac56c2 and https://github.com/reactioncommerce/chart-krak8s-api/pull/2


Add Rust language

Similar to #102, it might be nice to add another more "practical" language to the list of languages (and, selfishly, I want to practice my Rust skills more!)
Golfing
I haven't done a ton of golfing in Rust, but here are some initial thoughts on it.
Pros

Frequent use of iterator patterns in standard library
Terse syntax for various patterns (ranges, slices, closures, C-like operators, no parenthesis required in conditionals)
Unicode support by default (UTF-8 source and char/str literals)
Pattern matching for flow control
Every block has a return value
Macros – although this may have limited use
Builtin tuple types
Whitespace agnostic

Cons

Requires let for immutable variables and let mut for mutable variables
Requires unsafe to abuse memory, reinterpret casting, etc.
At minimum requires chars fn main(){}
Language design generally prefers explicit to implicit, especially when it comes to taking references
argv requires std::env::args() at minimum

Implementation
Pros

Existing Docker images
cargo run should be pretty simple to hookup

Cons

As far as I can tell, will probably require copying stdin to src/main.rs and a boilerplate Cargo.toml to work with cargo easily
New rustc version releases every six weeks, which makes it kind of hard to maintain latest stable version

If you'd like, I'd be willing to start working on a pull request to add support. Is #79 a good example of the changes that would be required? Let me know what you think of this proposal.

@ian-h-chamberlain The more the merrier! Even if its not very golfable, its generally more fun to compete with the same language on a hole and Rust is definitely popular enough to get a few solutions. I'm hoping #112 will make alternative languages even more popular. In terms of the actual implementation, having never touched Rust before, I'll be more than happy to leave the PR to you, #79 is a fairly recent example if you want to copy that, and running on dev should mostly work though I should really add more docs. The wrapper idea from #79 is fine since I'm guessing that a simple cargo run - -- arg1 arg2 < main.rs doesn't work?


Create smoke tests

Smoke tests used to verify that our app has been deployed and is standing up.  More in depth requests can come later.

Whoops.  Closing.


Feature request: Setting custom localization string keys

VS Code localization currently uses a hash of [English string] + [comment] as a key for localization strings in .xlf files. It would be beneficial to allow having a custom, stable key for each localization string. For example, if we make a small change to an English string that has already been translated, it would be good to keep the old translation for as long as it takes to re-translate the new version, instead of showing untranslated English strings during that period of time. Similarly, it could be useful to have the old translation at hand to use as a reference for the new translation.
In terms of implementation, the custom key could be provided as an optional parameter to the vscode.l10n.t() function and parsed by @vscode/l10n-dev together with the English strings and comments.

This is a duplicate of https://github.com/microsoft/vscode/issues/193738

@TylerLeonhardt I apologize about the duplicate. I filtered by Issues created by me in this repo and didn't found the one I had posted so I thought I had accidentally discarded it before creating the issue. I didn't realize that I had created it in another repo. My bad. Thanks for linking to it from here.


docs: add readme

updates main readme for readability
adds extensive connect readme

@getify Feedback addressed 🏁
I also added a banner graphic to the main README ✨


Automated migration tools for QUnit 2.0 syntax

Hi folks,
I've been working (occasionally) on a QUnit migrator tool, platinumazure/qunit2-migrator. Today I learned of another tool being developed, apsdehal/qunit-migrate.
Before I get too much further, I'd like to confirm with the QUnit team: Are there any plans to create or support a migration tool? If so, what do I need to do to either win support for what I have done or to gracefully fold in my tool's capabilities to whatever the "official" migration tool may be? And if there are no plans to support a tool, what is the best way for me to request extra eyes on it and get feedback without spamming people?

As a quick note: We should add links on both the plugins and the upgrade guide pages to available migration tools.
I'll get back to the actual question later.

It looks like your tool is based on AST transforms, while @apsdehal's tool is using regex. He mentioned (https://github.com/apsdehal/qunit-migrate/issues/4#issuecomment-145562918) that he'd like to switch to using JS AST, so that seems like a good opportunity to collaborate. It would be great to have one really solid tool instead of two that mostly solve the same problem.
@apsdehal since you mentioned moving to AST, would you consider contributing to @platinumazure's tool?

@jzaefferer I can contribute to @platinumazure's tool, but the tool I developed came into existence mostly because of the custom needs of jquery-projects and the custom syntax. Thus I further modified it to a more general support. I can't immediately halt the development on it until @platinumazure's tool take care of different syntax, good wikis and a few other stuff. We can work together in that direction.

@apsdehal The syntax tree transformation I selected, recast, was chosen specifically to try to preserve existing syntax and spacing and things, although it does have a few bugs when new statements are generated. Could you please either provide me some test cases using jQuery or other custom syntax, or try running the tool against said syntax, and open issues (or otherwise let me know) for any problems that show up? Thanks!

@platinumazure You can select data number 8, 9 and 10 from https://github.com/apsdehal/qunit-migrate/tree/master/tests/data. These all have been taken from jquery projects.

Understood. I'll give it a try in the next couple of days.
On Oct 6, 2015 11:37 PM, "Amanpreet Singh"<EMAIL_ADDRESS>wrote:

@platinumazure https://github.com/platinumazure You can select data
number 8, 9 and 10 from
https://github.com/apsdehal/qunit-migrate/tree/master/tests/data. These
all have been taken from jquery projects.
—
Reply to this email directly or view it on GitHub
https://github.com/jquery/qunit/issues/868#issuecomment-146075169.

It is great to see how this is going. Ping me if you need any help.

How about we add links to both tools (while they're not merged, or whatever will happen) to the plugins page and the top of the migration guide? With those in place we can close this ticket, then update the links later as things progress.

@jzaefferer :+1:
@apsdehal Sorry for not getting back to you-- I've been incredibly busy at work. I'm hoping to try my tool out on your test cases this week.

@apsdehal @platinumazure was there any progress towards merging these tools?

No, and I've stopped working on my tool. @apsdehal has made good progress
on his tool. It can be the "official" one, if everyone agrees it is up to
snuff.
Sorry, I just don't have the bandwidth to maintain my migration tool on top
of my other projects right now :cry:
On Apr 20, 2016 4:40 PM, "Jörn Zaefferer"<EMAIL_ADDRESS>wrote:
@apsdehal https://github.com/apsdehal @platinumazure
https://github.com/platinumazure was there any progress towards merging
these tools?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/jquery/qunit/issues/868#issuecomment-212618322

Alright, thanks for letting us know.
I lost track of the previous discussion (and I'm not sure what might still be relevant). Seems like @apsdehal could maybe add your test cases to this project. Either way, we should link to it in the migration guide.

See https://github.com/jquery/qunitjs.com/issues/120 for adding the note to the migration guide.


No authentication problem

@Abyss777, can you please take a look at this issue:
https://www.traccar.org/forums/topic/traccar-mail-notifications
It seems like something fails if no authentication is provided.

I think it can be closed
https://www.traccar.org/forums/topic/traccar-mail-notifications/#post-7497

Thanks.


[Bug]

Addon Name and Version
PinnedMessageIcons | Just downloaded it from here.
Describe the bug
I get an error whenever I launch discord telling me "Could not be compiled. See console for details."
To Reproduce
Enable Addon Errors in Better Discords settings
Expected behavior
I expected it to work instantly or to not show that, I can disable addon error if need be but that's your call.

I'm not experiencing any issues or errors on my end. Could you send a screenshot of any errors in the console related to the plugin?

I'm not experiencing any issues or errors on my end. Could you send a screenshot of any errors in the console related to the plugin?

Sorry about not doing this before. I thought you'd have the error too so I didn't feel the need of doing all this, my bad.

 Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release. Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.   DevTools failed to load source map: Could not load content for https://discord.com/assets/524ace8dd3ef4b062178.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE DevTools failed to load source map: Could not load content for https://discord.com/assets/52d61a6f729359ce0517.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE 67862f280624dcfb158a.js:130 [ActivityIcons] version 1.2.5 has started. 67862f280624dcfb158a.js:130 [AutoIdleOnAFK] version 0.3.0 has started. 67862f280624dcfb158a.js:130 [BetterInvites] version 1.5.0 has started. 67862f280624dcfb158a.js:130 [BetterRoleColors] version 0.8.17 has started. 67862f280624dcfb158a.js:130 [ChannelTabs] version 2.6.1 has started. 67862f280624dcfb158a.js:130 [Routing/Utils] transitionTo - Transitioning to /channels/888071204576051241/888193309548367942 67862f280624dcfb158a.js:130 [ChatUserIDsRedux] version 1.0.24 has started. 67862f280624dcfb158a.js:130 [ClickToChat] version 0.0.1 has started. 67862f280624dcfb158a.js:130 [ClickToCopyUsername] version 0.0.2 has started. 67862f280624dcfb158a.js:130 [Copier] version 1.4.1 has started. 67862f280624dcfb158a.js:130 [DefaultReplyPing] version 1.0.6 has started. 67862f280624dcfb158a.js:130 [DoNotTrack] version 0.0.7 has started. 67862f280624dcfb158a.js:130 [DoubleClickVoiceChannels] version 2.0.4 has started. 67862f280624dcfb158a.js:130 [DownloadEmotes] version 1.1.2 has started. 67862f280624dcfb158a.js:130 [HideDisabledEmojis] version 0.0.7 has started. 67862f280624dcfb158a.js:130 [AppNotifications] version 1.0.6 has started. 67862f280624dcfb158a.js:130 [InMyVoice] version 1.0.2 has started. 67862f280624dcfb158a.js:130 [InvisibleTyping] version 1.2.0 has started. 67862f280624dcfb158a.js:130 [LinkChannels] version 1.2.0 has started. 67862f280624dcfb158a.js:130 [MentionChannels] version 1.0.3 has started. 67862f280624dcfb158a.js:130 [NoSpotifyPause] version 0.0.1 has started. 67862f280624dcfb158a.js:130 [OpenSpotify Links in Spotify] version 0.0.8 has started. 67862f280624dcfb158a.js:130 [PluginManager] Could not load PinnedMessageIcons.plugin.js: MetaError: META was not found.     at Object.extractMeta (<anonymous>:4:266698)     at Object..js (<anonymous>:4:267567)     at Function.o._load (<anonymous>:4:370428)     at Object.loadAddon (<anonymous>:4:267961)     at Object.loadAddon (<anonymous>:4:305909)     at Object.loadAllAddons (<anonymous>:4:270636)     at Object.initialize (<anonymous>:4:265083)     at Object.initialize (<anonymous>:4:305119)     at Object.startup (<anonymous>:4:367198) (anonymous) @ 67862f280624dcfb158a.js:130 t.<computed> @ 87e15c072fa1fdf42a13.js:4247 stacktrace @ VM471:4 loadAddon @ VM471:4 loadAddon @ VM471:4 loadAllAddons @ VM471:4 initialize @ VM471:4 initialize @ VM471:4 startup @ VM471:4 async function (async) startup @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:9 e.startsWith.e.startsWith.WebFrame.<computed> @ VM75 renderer_init.js:89 (anonymous) @ VM75 renderer_init.js:117 (anonymous) @ VM75 renderer_init.js:101 emit @ VM14 events.js:315 onMessage @ VM75 renderer_init.js:93 67862f280624dcfb158a.js:130 [QuickDeleteMessages] version 1.0.12 has started. 67862f280624dcfb158a.js:130 [QuickMessages] version 1.2.4 has started. 67862f280624dcfb158a.js:130 [QuickMuteChannels] version 1.0.3 has started. 67862f280624dcfb158a.js:130 [ShowSessions] version 1.3.0 has started. 67862f280624dcfb158a.js:130 [BetterDiscord] [PluginManager] Starting to watch plugin addons. 67862f280624dcfb158a.js:130 [BetterDiscord] [Startup] Loading Themes 67862f280624dcfb158a.js:130 [BetterDiscord] [ThemeManager] Starting to watch theme addons. 67862f280624dcfb158a.js:130 [BetterDiscord] [Startup] Removing Loading Icon 67862f280624dcfb158a.js:130 [BetterDiscord] [Startup] Collecting Startup Errors DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/embed-legacy.a36ad1c8.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/vendor~embed-legacy.5d01c516.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE Copier.plugin.js:686 Uncaught (in promise) TypeError: Cannot read property 'bind' of undefined     at Copier.onStart (Copier.plugin.js:686)     at Copier.start (0PluginLibrary.plugin.js:3945)     at Object.startPlugin (<anonymous>:4:308019)     at Object.startAddon (<anonymous>:4:307812)     at Object.loadAddon (<anonymous>:4:268511)     at Object.loadAddon (<anonymous>:4:305909)     at Object.loadAllAddons (<anonymous>:4:270636)     at Object.initialize (<anonymous>:4:265083)     at Object.initialize (<anonymous>:4:305119)     at Object.startup (<anonymous>:4:367198) onStart @ Copier.plugin.js:686 start @ 0PluginLibrary.plugin.js:3945 startPlugin @ VM471:4 startAddon @ VM471:4 loadAddon @ VM471:4 loadAddon @ VM471:4 loadAllAddons @ VM471:4 initialize @ VM471:4 initialize @ VM471:4 startup @ VM471:4 async function (async) startup @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:4 (anonymous) @ VM471:9 e.startsWith.e.startsWith.WebFrame.<computed> @ VM75 renderer_init.js:89 (anonymous) @ VM75 renderer_init.js:117 (anonymous) @ VM75 renderer_init.js:101 emit @ VM14 events.js:315 onMessage @ VM75 renderer_init.js:93 67862f280624dcfb158a.js:130 [Spotify] WS Connected 67862f280624dcfb158a.js:130 [Spotify] Devices updated for 3wyrmghvsufkk1nnilngmdfvg: [{…}] 67862f280624dcfb158a.js:130 [Spotify] Profile updated for 3wyrmghvsufkk1nnilngmdfvg: isPremium = true 67862f280624dcfb158a.js:130 [BDFDB] (v2.3.4) Finished loading Library 67862f280624dcfb158a.js:130 [BetterFriendList] (v1.4.5) has been started 67862f280624dcfb158a.js:130 [BetterSearchPage] (v1.2.0) has been started 67862f280624dcfb158a.js:130 [ClickableMentions] (v1.0.3) has been started 67862f280624dcfb158a.js:130 [CustomQuoter] (v1.2.9) has been started 67862f280624dcfb158a.js:130 [EditChannels] (v4.3.7) has been started 67862f280624dcfb158a.js:130 [EditRoles] (v1.0.8) has been started 67862f280624dcfb158a.js:130 [GameActivityToggle] (v1.1.2) has been started 67862f280624dcfb158a.js:130 [ImageUtilities] (v4.7.1) has been started 67862f280624dcfb158a.js:130 [MessageUtilities] (v1.9.4) has been started 67862f280624dcfb158a.js:130 [NotificationSounds] (v3.7.0) has been started 67862f280624dcfb158a.js:130 [PersonalPins] (v2.0.8) has been started 67862f280624dcfb158a.js:130 [PinDMs] (v1.9.1) has been started 67862f280624dcfb158a.js:130 [PluginRepo] (v2.2.9) has been started 67862f280624dcfb158a.js:130 [QuickMention] (v1.0.4) has been started 67862f280624dcfb158a.js:130 [ReadAllNotificationsButton] (v1.7.1) has been started 67862f280624dcfb158a.js:130 [ShowBadgesInChat] (v1.8.8) has been started 67862f280624dcfb158a.js:130 [ShowConnections] (v1.0.8) has been started 67862f280624dcfb158a.js:130 [StaffTag] (v1.5.0) has been started 67862f280624dcfb158a.js:130 [ThemeRepo] (v2.2.9) has been started 67862f280624dcfb158a.js:130 [ThemeSettings] (v1.3.4) has been started 67862f280624dcfb158a.js:130 [UserNotes] (v1.0.9) has been started 67862f280624dcfb158a.js:130 [RunningGameStore] games [] 67862f280624dcfb158a.js:130 [RTCLatencyTestManager] RTC region latency test completed:  (5) ["rotterdam", "stockholm", "bucharest", "madrid", "dubai"] DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/embed-legacy.a36ad1c8.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE vendor~embed-legacy.5d01c516.js:1 POST https://sentry.io/api/265628/envelope/?sentry_key=db1eaaeafb9044828c39cd41f9805164&sentry_version=7 429 (Too Many Requests) (anonymous) @ VM15 vendor~embed-legacy.5d01c516.js:1 e @ VM15 vendor~embed-legacy.5d01c516.js:1 (anonymous) @ VM15 vendor~embed-legacy.5d01c516.js:1 add @ VM15 vendor~embed-legacy.5d01c516.js:1 t._sendRequest @ VM15 vendor~embed-legacy.5d01c516.js:1 e.sendSession @ VM15 vendor~embed-legacy.5d01c516.js:1 e.sendSession @ VM15 vendor~embed-legacy.5d01c516.js:1 e._sendSession @ VM15 vendor~embed-legacy.5d01c516.js:1 e.captureSession @ VM15 vendor~embed-legacy.5d01c516.js:1 e._sendSessionUpdate @ VM15 vendor~embed-legacy.5d01c516.js:1 e.captureSession @ VM15 vendor~embed-legacy.5d01c516.js:1 Jt @ VM15 vendor~embed-legacy.5d01c516.js:1 (anonymous) @ VM15 vendor~embed-legacy.5d01c516.js:1 Yt @ VM15 vendor~embed-legacy.5d01c516.js:1 (anonymous) @ VM14 embed-legacy.a36ad1c8.js:1 3887 @ VM14 embed-legacy.a36ad1c8.js:1 r @ VM14 embed-legacy.a36ad1c8.js:1 (anonymous) @ VM14 embed-legacy.a36ad1c8.js:1 r.O @ VM14 embed-legacy.a36ad1c8.js:1 t @ VM14 embed-legacy.a36ad1c8.js:1 (anonymous) @ VM15 vendor~embed-legacy.5d01c516.js:1 DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/vendor~embed-legacy.5d01c516.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/vendor~embed-legacy.5d01c516.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE vendor~embed-legacy.5d01c516.js:1 POST https://sentry.io/api/265628/envelope/?sentry_key=db1eaaeafb9044828c39cd41f9805164&sentry_version=7 429 (Too Many Requests) (anonymous) @ vendor~embed-legacy.5d01c516.js:1 e @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 add @ vendor~embed-legacy.5d01c516.js:1 t._sendRequest @ vendor~embed-legacy.5d01c516.js:1 e.sendSession @ vendor~embed-legacy.5d01c516.js:1 e.sendSession @ vendor~embed-legacy.5d01c516.js:1 e._sendSession @ vendor~embed-legacy.5d01c516.js:1 e.captureSession @ vendor~embed-legacy.5d01c516.js:1 e._sendSessionUpdate @ vendor~embed-legacy.5d01c516.js:1 e.captureSession @ vendor~embed-legacy.5d01c516.js:1 Jt @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 Yt @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ embed-legacy.a36ad1c8.js:1 3887 @ embed-legacy.a36ad1c8.js:1 r @ embed-legacy.a36ad1c8.js:1 (anonymous) @ embed-legacy.a36ad1c8.js:1 r.O @ embed-legacy.a36ad1c8.js:1 t @ embed-legacy.a36ad1c8.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 sentry.io/:1 Failed to load resource: the server responded with a status of 429 (Too Many Requests) DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/embed-legacy.a36ad1c8.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE vendor~embed-legacy.5d01c516.js:1 POST https://sentry.io/api/265628/envelope/?sentry_key=db1eaaeafb9044828c39cd41f9805164&sentry_version=7 429 (Too Many Requests) (anonymous) @ vendor~embed-legacy.5d01c516.js:1 e @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 add @ vendor~embed-legacy.5d01c516.js:1 t._sendRequest @ vendor~embed-legacy.5d01c516.js:1 e.sendSession @ vendor~embed-legacy.5d01c516.js:1 e.sendSession @ vendor~embed-legacy.5d01c516.js:1 e._sendSession @ vendor~embed-legacy.5d01c516.js:1 e.captureSession @ vendor~embed-legacy.5d01c516.js:1 e._sendSessionUpdate @ vendor~embed-legacy.5d01c516.js:1 e.captureSession @ vendor~embed-legacy.5d01c516.js:1 Jt @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 Yt @ vendor~embed-legacy.5d01c516.js:1 (anonymous) @ embed-legacy.a36ad1c8.js:1 3887 @ embed-legacy.a36ad1c8.js:1 r @ embed-legacy.a36ad1c8.js:1 (anonymous) @ embed-legacy.a36ad1c8.js:1 r.O @ embed-legacy.a36ad1c8.js:1 t @ embed-legacy.a36ad1c8.js:1 (anonymous) @ vendor~embed-legacy.5d01c516.js:1 DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/embed-legacy.a36ad1c8.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE DevTools failed to load source map: Could not load content for https://open.scdn.co/cdn/build/embed-legacy/vendor~embed-legacy.5d01c516.js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE 67862f280624dcfb158a.js:130 [discord_protos.discord_users.v1.FrecencyUserSettings] Loading proto<PHONE_NUMBER>48367942:1 Uncaught (in promise) DOMException: The play() request was interrupted by a call to pause(). https://goo.gl/LdLk22  [ThemeRepo] (v2.2.9) Finished fetching Themes  [PluginRepo] (v2.2.9) Finished fetching Plugins

Screenshot:

Try downloading the plugin from GitHub again

Already have and still nothing

Also make sure the file downloaded properly and isn’t corrupt for whatever reason, and that BetterDiscord itself is up to date. If everything seems fine could you open the plugin file in a text editor and send the contents?

Strange, I used another browser, and it's now working... My luck must suck.


Ingawei/man 721 add roles to groups

Yay now our groups will be more structured and constrained!
ROLE MANAGEMENT:
Can now view group members and their respective roles, big groups will load in 50 members at a time. Group creators cannot leave the group and are automatically admins

Admins will have access to this dropdown to change the role of people within the group

Members can also remove themselves from a role

These update live yay!
 If you're not an admin or moderator, clicking the add button will open a create market modal within the group:

If you are an admin or moderator, you will get the option to add an existing market

Markets that are already in the group should not appear in this list, yay!
If you're an admin or moderator, you can remove markets from your groups with this dropdown:

I rewrote these functions to the backend for security (see add-contract-to-group.ts)
ADDING/REMOVING GROUPS FROM CONTRACT PAGE:
I changed the look of the market groups modal

If you're a group admin or moderator or the market creator, you'll have the option to remove the groups upon hover

You'll only see groups that you are an admin or moderator of. If you are the market creator, you will see all the groups you are a member of

Same for the create market page

OTHER SMOL CHANGES:

changed button name from 'follow/unfollow' to 'join/leave'

got rid of anyonecanjoin permissions and checks for now

KNOWN WEIRD BUGS:
I think there were one or two groups where I was just bombarded with permission errors when I got on them, not sure if it's a problem with my code or it was just some legacy test group. Will list the slugs here if I come across them again.

@jahooma sorry this is kind of a big boi :(
@mantikoros I would appreciate a sanity check on supabase things I wrote!


Update enabling-inline-creation-mode-or-empty-rows-mode-for-table-ent…

…ries-cfb04f0.md
"controlConfiguration" should be inside "settings"

Hi @miyasuta, thanks for your contribution! We'll check your PR and get back to you.

Hi @miyasuta,
You are absolutely correct. I checked with our product owner and this line was indeed missing.
I've updated the sample code.
The change will be visible with our next release 1.110.
Best regards,
JannaLisa


Parameters used for training

Good afternoon,
May I ask if what were the parameters used for the training in the final version of the model, whose results were shown in the article? Are those all the same as the defaults from the code + flags from the https://github.com/silviazuffi/smalst/blob/master/scripts/smalst_train_run.sh? Also, are the parameters from the first (commented) line of the script are used, or from the second line (in that case, what is nokap)?
Additionally, may I ask what is the difference between flags "is_optimization" and "is_refinement"? The have the same comment in the flag definition, but they seem to mean different things in the code.
I have rewritten the code a bit to use Python 3 with the corresponding new Nvidia drivers, but I am now not sure that I would get the equivalent results due to the big number of parameters. (In case I am able to reproduce it, I would be also happy to share somehow the version for Python 3, so that more people would be able to use it with newer libraries - if you want and agree, of course.)
Thank you a lot in advance for any answer.
Best regards,
Valeria

Hi Valeria,

May I ask if what were the parameters used for the training in the final version of the model, whose results were shown in the article? Are those all the same as the defaults from the code + flags from the https://github.com/silviazuffi/smalst/blob/master/scripts/smalst_train_run.sh https://github.com/silviazuffi/smalst/blob/master/scripts/smalst_train_run.sh? Also, are the parameters from the first (commented) line of the script are used, or from the second line (in that case, what is nokap)?

The parameters are those in the code, the commented line was a test (only 40 epochs), the one used for the paper is the one that is not commented.
Nokap is a directory where I placed the training data, you can use any name.

Additionally, may I ask what is the difference between flags "is_optimization" and "is_refinement"? The have the same comment in the flag definition, but they seem to mean different things in the code.

is_refinement is something I tried for refining the network weights, but it is not used, is_optimization is explained in the paper, and refers to the optiization step that we perform on each test images.

I have rewritten the code a bit to use Python 3 with the corresponding new Nvidia drivers, but I am now not sure that I would get the equivalent results due to the big number of parameters. (In case I am able to reproduce it, I would be also happy to share somehow the version for Python 3, so that more people would be able to use it with newer libraries - if you want and agree, of course.)

Yes, of course, I think it would be very useful. You can contact me in private and we can have a chat if there is something not very clear.
Thanks!
Silvia

Thank you a lot in advance for any answer.
Best regards,
Valeria
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/silviazuffi/smalst/issues/20, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABNYVFMOSQRTBHWAGLBCU7TTG4Q7DANCNFSM42KJNAHQ.


Will be the web site link ever supported?

Yeah this would be a nice extension. Maybe I am gonna add a solution for that. You could already try to write your own implementation. With the following code you can
class WebImageThumbnailGenerator : ROThumbnailGenerator {

    var supportedExtensions:Array<String> = ["png", "jpg", "jpeg"]

    func getThumbnail(url:NSURL) -> UIImage {
        // Here fetch the image and return it
    }
}

// Add it
ROThumbnail.sharedInstance.addThumbnailGenerator(webImageThumbnailGenerator)


Discrepancy between source advisory file and osv.dev

I am comparing the content of the source advisory file GHSA-2cxf-6567-7pp6.json from github advisory database and the one in google ovs.dev and there is one important difference.
Source file
https://github.com/github/advisory-database/blob/b4fb55f2669be0216baf4cfb9a01fe4533007957/advisories/github-reviewed/2021/03/GHSA-2cxf-6567-7pp6/GHSA-2cxf-6567-7pp6.json
OSV File
https://osv.dev/vulnerability/GHSA-2cxf-6567-7pp6
The original file does not have "Affected versions" data and it is added into ovs.dev file.
This "Affected versions" information is not correct.
Where is this data added?
How can it be fixed?

Hi, thanks for the question!
We add "affected versions" as a data improvement by expanding the provided version ranges from the source.
In this case, the source has:
            {
              "introduced": "0"
            },
            {
              "fixed": "1.0.0-beta.9"
            }

So we expand it into individual versions as part of "affected versions".
As you pointed out, there's some weirdness here in that "1.0.0-beta10" and "1.0.0-beta11" are included here.
This is because "1.0.0-beta10" and "1.0.0-beta11" both actually sort before "1.0.0-beta.9" according to Maven's version ordering rules
Specifically, these strange rules:

A transition between digits and characters is equivalent to a hyphen.

else ".qualifier" = "-qualifier" < "-number" < ".number"

In this case specifically, it looks like the advisory intends to exclude "1.0.0-beta10" and "1.0.0-beta11", so we can fix this at the source by editing the introduced to start from 1.0.0-beta.6 instead.

I've submitted https://github.com/github/advisory-database/pull/1656 to fix this.

https://osv.dev/vulnerability/GHSA-2cxf-6567-7pp6 is now fixed since the GHSA PR has been merged. Thanks agai nfor reporting this!

Thanks you!


XMLTV Encoding issues

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
Threadfin has wrong encoding on the XMLTV output.
Server (please complete the following information):

OS - Docker
Arch Intel 64 Bit
Threadfin Version and Build 1.1

To Reproduce
As an example, this XMLTV programme title from the source feed:
<title>Hadi ve městě</title>

Gets borked into this in the threadfin XMLTV feed:
<title lang="">Hadi ve mst</title>

Screenshots
Titles are missing letters.

Check to see if you have Enable Non ASCII checked in settings. If you don't have it checked, please do so and restart threadfin

Seems like enabling that option fixes the issue. I did not notice it. Why isn't it on by default?
In Jellyfin though, the letters are still missing, maybe that is a separate issue with Jellyfin..


“test is not defined” jest/expect on client side

I get "test is not defined" Uncaught ReferenceError on the browser.
I dont use ESLINT, which according to this linkis the solution to "test is not defined". "test" which is a global in jest/expect testing library, it is expected to be available after the including the expect.min.js file in the test.html
In that case what could be the other reasons and their solution.
here is my code:

As suggested in the https://github.com/mjackson/expect#installation, I am using this link
So if I understand it right 'test' is not available on expect. But according to Jest official docs https://jestjs.io/docs/en/expect#reference 'test' is a global. This could mean that test is available only on the higher versions of expect.
If so can you suggest the library link to the version where I can use 'expect' along with 'test'.

Yes, it’s in the read me which i linked. Please read it :-)


Bug in fullintegrate in discframe

Wrong indexing of velocity component of orbits in l 1927 and 2088 in fullintegrate.

Fixed with commit https://github.com/sophialilleengen/mwlmc/commit/be9cf02727cbd04c78ab52459b0acd2bf3bc8ac9.


feat: python-is-python3 and move build essentials to base

Decision
As core maintainers we've considered all options and we've decided to go with the following solution:

go back to python2 for all target platforms (in base image) for Ubuntu 18.04 for and release v1.0.0, introduce BC
when Ubuntu 20.04 or 22.04 becomes recommended by Unity, we move to python3 and release v2.0.0, introducing another BC (we'll need to consider which editor versions will support 20.04)

Changes in this PR
Allows using Firebase with all images, by enabling the python command, referring to python3.

Removes python install from webgl and il2cpp as ~python3~ python2 is already be installed on the ubuntu image.
~Add python-is-python3 to base image (this is available from ubuntu 20.04 on and not desired in ubuntu 18.04)~
~Move build-essential, clang  from the editor image to the base image~
Move python-setuptools from the editor image to the base image
Cleanup any code that is no longer needed

Fixes: https://github.com/game-ci/docker/issues/100
Rationale
Status quo

ubuntu 18:04 (that we  use in our base image https://github.com/game-ci/docker/blob/4e88219ef1eebfa07934e00275786a7a87c531f5/images/ubuntu/base/Dockerfile#L1) has python2 installed
webgl  and il2cpp currently use python symlink, assuming  python is python2 https://github.com/game-ci/docker/blob/4e88219ef1eebfa07934e00275786a7a87c531f5/images/ubuntu/editor/Dockerfile#L202
android and iOS build on the fact that python3 is installed along with the target platform module. They then symlink python to python3 https://github.com/game-ci/docker/blob/4e88219ef1eebfa07934e00275786a7a87c531f5/images/ubuntu/editor/Dockerfile#L222

Discussion points

python symlink always used to be python2 so you could argue that developers will  use /usr/bin/python3  directly,  without the alias.
In 2012 PEP recommended to put python2 scripts in the shebang in
order to stay compatible: "in preparation for an eventual change in the default version of Python, Python 2 only scripts should either be updated to be source compatible with Python 3 or else to use python2 in the shebang line."
The goals of  Ubuntu were to set python3 as the default and only version in 18.04. They do not seem to have made that goal. Perhaps in 20.04?

Our options

install python2 in base image. Change all current python links to use python2 instead of python3. (downside: breaking change people will need to change their workflow if they were using python assuming  python3)
install python3 in the base image. Change all current python links to use python3 instead of python2 (maybe except for  WebGL, which broke on first sight) in a forward approach, without breaking changes. (downside: not every image will have the same link yet)
other?

Sources used

https://askubuntu.com/questions/320996/how-to-make-python-program-command-execute-python-3 (note that some posts  and comments are almost 10 years old)
Ubuntu python doc from 6~8 years  ago: https://wiki.ubuntu.com/Python/3
The more updated version of that https://wiki.ubuntu.com/Python
PEP-0394 https://legacy.python.org/dev/peps/pep-0394/

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)
[ ] I looked up the package sizes and this shouldn't add much at all, but still need to verify this
[ ] We'll need to verify that python works as expected still, on linux-il2cpp and webgl
[ ] Ideally we'd also check if this fixes the problem with firebase installs

There is this error: Unable to locate package python-is-python3
I guess apt-get doesn't have python-is-python3?

This is also an interesting read:
https://askubuntu.com/questions/1296790/python-is-python3-package-in-ubuntu-20-04-what-is-it-and-what-does-it-actually
the package only does a symlink so we can do this by hand instead. I’m sharing this so we don’t forget about pip which I think should also default to pip3

I'm a bit confused after this though:

#6 [4/7] RUN ln -s /usr/bin/python3 /usr/bin/python  && ln -s /usr/bin/pip3 /usr/bin/pip
#6 0.220 ln: failed to create symbolic link '/usr/bin/python': File exists
#6 ERROR: executor failed running [/bin/sh -c ln -s /usr/bin/python3 /usr/bin/python  && ln -s /usr/bin/pip3 /usr/bin/pip]: exit code: 1

So should we just install python as python3 for all images? What do you think. Would it introduce breaking changes?
I don't use python often enough to know the conventions very well. Please advise.

I feel like the transition from python 2 to python 3 has been slow and painful, but I think pointing python to python 3 should work fine in most if not all scenarios now.

I feel like the transition from python 2 to python 3 has been slow and painful, but I think pointing python to python 3 should work fine in most if not all scenarios now.

Maybe we should make this the 1.0.0 release then. Because all actions are currently rolling 0.x.x images automatically and this could be a BC for many people potentially.

Looks like the WebGL builds still expect python 2, so those are failing. We could simply install python 2, and we may reconsider switching to python 3 if Unity updates the version of python that WebGL builds use.
Even so, I think moving to 1.0.0 after adding python to the base image is a good idea.

The problem is that we use both python2 (for webgl) and python3 (for android and iOS).
Those were all added in individual fixes.
I think if python3 is indeed already used in some images it might be worth not installing python2 to more target platforms.
Does anyone have the right documentation for these Firebase SDKs that are used in games?
Perhaps we should go for a per-platform upgrade, or install python3 but not link it to python?

I haven't been able to find confirmation, but it seems to me that Firebase (for iOS and Android) expects python, but that can be either Python 2 or Python 3.
@dharmeshmp @mastef Maybe one of you can confirm?
If so, I think we should be OK to merge this PR and close the corresponding issue.

I haven't been able to find confirmation, but it seems to me that Firebase (for iOS and Android) expects python, but that can be either Python 2 or Python 3.
@dharmeshmp @mastef Maybe one of you can confirm?
If so, I think we should be OK to merge this PR and close the corresponding issue.

That's also what I'm thinking. Except I don't think that we should go from python 3 to python 2. Which is what it would  mean for iOS and Android if we merge it in the current state.
What about if we just install both (python2 and python3) and link python to python3 for all platforms, then mark it as a breaking change and call it v1.0.0? Or even remove python2 in the process? If we're moving to 1.0 it would be perfect to introduce that change now.

link python to python3 for all platforms

WebGL builds expect that python is python 2, so this would break all WebGL builds.
Maybe we could link python to python3 for all platforms except WebGL?

Yea I'm in favour of that too.

Not a necessity. The python = python 3 hack can be removed in favor of just
having python 2 installed

Not a necessity. The python = python 3 hack can be removed in favor of just having python 2 installed

I'm not  sure if I get your meaning.

AFAIK apt install python will install python 2, no?

Sure. But I believe you may have missed the past few comments.

We had the symlink python to python3 hack in there, just because python 3 was already installed on the system. Firebase is fine with having python as python 2. There is no necessity to have a "python is python 3" hack overall.
If apt install python is run on all systems, Firebase etc should be fine.

The problem is that we use both python2 (for webgl) and python3 (for android and iOS).

@webbertakken Ok I found your comment. Firebase just needs any python to be available at python. So if python 2 is installed, everything seems to be ok.

The current code of the PR looks great!

The current code of the PR looks great!

It's not complete. It has breaking changes for people that were already relying on python 3. As discussed above, we'd rather go up than down - maybe with the exception of WebGL which somehow needs python 2 specifically (I think?).
Installing python 3 and symbolic linking that would seem like a better forward solution.

It's not complete. It has breaking changes for people that were already relying on python 3. As discussed above, In case we introduce breaking changes we'd rather go up than down - maybe with the exception of WebGL which somehow needs python 2 specifically (I think?).
Installing python 3 and symbolic linking that would seem like a better forward solution.

Python 3 is already installed. If somebody needs to use Python 3, they can call python3.
The hack we used was to not add another dependency ( python2 ) - but since that is added now for all images, the hack is not needed any longer.
Do you have any use-case where Python 3 symlinked to python is needed? Because it's not needed for Firebase.

@mastef

Python 3 is already installed. If somebody needs to use Python 3, they can call python3.

This is misinformation. Python 3 is not installed on all images. That's exactly what we've been discussing above your first comment. See also this comment https://github.com/game-ci/docker/issues/100#issuecomment-1064919198 on the originating issue, from when this threw us off our game the first time. Please read the posts before your first comment on this thread.
We have concluded that ubuntu does not install python3 by default. Web target needs python 2, some other platforms include a python 3 installation. And so we have agreed on the following solution: https://github.com/game-ci/docker/pull/160#issuecomment-1065990987. Please let us know if you have any relevant input or concerns about our proposed solution.

Some how our base image contains python3. That's the reason this misinformation.

Confusion about which version is installed

@dharmeshmp: Some how our base image contains python3. That's the reason for misinformation.

This is likely because you're using Android or iOS as a target platform, which seem to indeed include a Python 3 installation.
In the base image it is never present though.
To Python3 or not to Python3
Regarding our python2 vs python3 discussion: We've discussed the matter amongst the core contributors of GameCI (cc: @davidmfinol @GabLeRoux @frostebite) and I've updated the opening post with our rationale, conclusions and decision on how to move forward.
cc: @mastef


[rviz_rendering] check if OGRE_PLUGIN_DIR is still required

Since we're using static linking, rather than dynamically loading ogre plugins, we may no longer need to set it from the build system:

https://github.com/ros2/rviz/blob/ca8f42e43ba9eaa6e15434b47906d7580da1c14a/rviz_rendering/CMakeLists.txt#L86
https://github.com/ros2/rviz/blob/ros2/rviz_rendering/src/rviz_rendering/resource_config.cpp#L47

This is where we statically load and avoid needing to load dynamically:

https://github.com/ros2/rviz/blob/ca8f42e43ba9eaa6e15434b47906d7580da1c14a/rviz_rendering/src/rviz_rendering/render_system.cpp#L183-L184

We checked it by just deleting the lines and rebuilding everything. No errors and it runs on Linux at least, so it seems to be unnecessary now.

Do you want a Pull-Request deleting this?

👍 Yes, please.


Can I know how did you transfer the policy learned in the simulator to a real-world TurtleBot 2i/ 3???

I would be also interested

Hi all,
For sim2real, you'll need a people detector and tracker to obtain the (x, y) positions of humans. The choice of detector and tracker depends on the sensors (camera, lidar, etc).
In addition, we also used a more realistic robot dynamics (unicycle with noises), and tried to reduce the gap between the behaviors of simulated people and real people.
I will consider releasing sim2real code for my next paper on crowd navigation, since quite a few people are interested. But I apologize that I don't have a specific date on when this will happen.
Follow-up questions are welcome if you have any.

@sumedhreddy90 @Mirorrn The sim2real tutorial is released here: https://github.com/Shuijing725/CrowdNav_Sim2Real_Turtlebot


xorg-x11-server: patch CVE-2024-21885

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
fast track version of https://github.com/microsoft/azurelinux/pull/8233. Will close cherry picked PR once this is merged
Change Log

CVE-2024-21885

Does this affect the toolchain?

NO
Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2024-21885

Test Methodology

Pipeline build id: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=520962&view=results

Auto cherry-pick results:

main :white_check_mark: -> Skipped

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=521696&view=results


Scheduling for Awake in PowerToys

Description of the new feature / enhancement
It would be great to have a feature to set schedules to Awake.
Scenario when this would be used?
To keep the display awake during business hours, but have a schedule where after hours the display would be allowed to sleep
Supporting information
No response

One possible, manual, workaround to achieve this can be to run Awake via task scheduler. With a command line parameter for a number of hours.
https://docs.microsoft.com/en-us/windows/powertoys/awake#command-line-interface-cli
Touching #11233

Or, is it possible to create two power plans and trigger them via task scheduler? 🤔 Then you wouldn't need Awake at all. 🤓

Using the command line, I was able to hack together this PowerShell script that would run PowerToys Awake and end by 6pm that day.
I've created PowerToysAwake.ps1:
Start-Process `
    -WindowStyle "Hidden" `
    -FilePath 'C:\Program Files\PowerToys\modules\Awake\PowerToys.Awake.exe' `
    -ArgumentList "--display-on true","--time-limit","$((Get-Date $(Get-Date -Format 'yyyy-MM-ddT18:00') -UFormat %s) - (Get-Date -UFormat %s))"

I was then able to use Windows Task Scheduler to run this on start up:

The script command is "C:\Program Files\PowerShell\7\pwsh.exe"
The arguments are -Command "D:\myScriptFolder\PowerToysAwake.ps1"

I'll close this as wont-fix because we'd effectively be re-implementing the Windows Task Scheduler 😊 The proposed solutions above should be simple enough for those that want to run Awake on a schedule to leverage.

Is this Task Scheduler with PowerShell script still an effective option?  I used this solution for the past 6 months or so but it no longer works after the latest Awake update.

Or, is it possible to create two power plans and trigger them via task scheduler? 🤔 Then you wouldn't need Awake at all. 🤓

No, because power plans are controlled by group policy.


Could anyone share a copy of Jester dataset？

This dataset is unavailable now on TwentyBN website. Could anyone share it on drive please? Thanks a lot.

Maybe you've already obtained Jester dataset.
20BN Jester and sth-sth v2 are now available in Qualcomm website.
Check this link.
https://developer.qualcomm.com/software/ai-datasets/jester


Support recursion for autojit functions

I'm also interested in this. Here is a basic example which currently fails and shows the behavior I would need:
@jit(nopython=True, nogil=True)
def _updateNodeRecursive2(last):
    if last > 3:
        return

    for i in range(4):
        last += 1
        _updateNodeRecursive2(last)

@jit(nopython=True, nogil=True)
def _constructTree2():
    last = np.int32(-1)
    _updateNodeRecursive2(last)

def test():
    _constructTree2()

which gives
Failed at nopython (nopython frontend)
Internal error at <numba.typeinfer.CallConstraint object at 0x2b69013be390>:
--%<-----------------------------------------------------------------
Traceback (most recent call last):
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/typeinfer.py", line 114, in propagate
    constraint(typeinfer)
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/typeinfer.py", line 305, in __call__
    self.resolve(typeinfer, typevars, fnty)
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/typeinfer.py", line 332, in resolve
    sig = context.resolve_function_type(fnty, pos_args, kw_args)
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/typing/context.py", line 126, in resolve_function_type
    return func.get_call_type(self, args, kws)
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/types.py", line 451, in get_call_type
    sig = template(context).apply(args, kws)
  File "python2.7/site-packages/numba-0.24.0-py2.7-linux-x86_64.egg/numba/typing/templates.py", line 280, in apply
    assert cases
AssertionError

Saw #1928, updated to master and this is now working! Nice timing.

Closing this issue for now.  If anyone needs something beside direct recursion, we should open a new issue.


fix UnrecognizedPropertyException in tuningConfig

Description

fix UnrecognizedPropertyException in tuningConfig
UnrecognizedPropertyException occurs when there is a property that does not exist in tuningConfig
Related Issue : #3828

How Has This Been Tested?

Added unit test
Patch an existing datasource and check the nested spec is preserving

Need additional checks?
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have added tests to cover my changes.

Additional Context

deploy to 2


fix: Add type to remove button

I wish to use ReactTags inside a form
To prevent tag's removing when I press on Enter (to submit the form), I do need to set a type to all my buttons.
ReactTags did not set any type to this RemoveComponent's button. So I added it.

Thanks for  the contribution @jdups-cordeco, can you add a snapshot for remove button as well ?

Thanks @jdups-cordeco for the contribution 🎉

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improvements to temperature panel resizing

This PR includes some improvements to the resizing of panels on the right-hand side of the GUI, as well as an aesthetic improvement to the buttons which toggle plot visibility.
Upon resizing of the window, only the panel containing the temperature plot is now free to expand vertically. However, as this panel contains a QGridLayout, which contains widgets with different QSizePolicys, a custom resize function is required to maximise the space taken up by the figure.
Closes #177

I've just found an issue with the resizing, so this needs a bit more work before reviews.

This looks much nicer. Good work! I think we should probs also move the OPUS panel somewhere else (maybe into the gap in the lefthand layout?) and then the plot can be even larger and then it'll look great. That's a job for another PR though.
I've made one small suggestion, but other than that I'm happy with it. The only thing is that I'm a bit concerned that the custom resize function feels like a bit of a hack. It's entirely possible that that's the only way to do what we want, but I've requested a review from @dalonsoa anyway to see if he's got any better ideas.

Yeah I completely agree that it feels like a hack. I was happy that I could at least set the figure position and size based on those of other widgets, which are decided by Qt.
I think the issue is that I set a minimum size for the figure. If I don't do this, then that panel is tiny by default - Qt seems to set its height to be the vertical distance between the top of the "Hot BB" button and the bottom of the "Cold BB" button. This itself, I think, is due to the size policies of the layouts. But we've seen what happens if we let Qt choose the size policies - the OPUS panel gets all the space. For this same reason, if I don't set the size policy of either the "Hot BB" and "Cold BB" buttons or the figure, the buttons get all the space.
In short, it seems we do need to override some Qt behaviour. I'm sure there will be improvements to the way I've done this, but I think something of this sort is necessary.

Starting to renew this. It looks good, but the process is not that robust. This is what I get after a couple of resizes (in MacOS):

I'm investigating PySide for the correct combination of options to make this resizing fluid. It should not be necessary to have this hack.

Starting to renew this. It looks good, but the process is not that robust. This is what I get after a couple of resizes (in MacOS):

I'm investigating PySide for the correct combination of options to make this resizing fluid. It should not be necessary to have this hack.

Interesting, that looks like it's ignored (or perhaps overwritten) the minimum size setting.

@dc2917 What's happening with this PR? Did we decide not to merge this one and merge @dalonsoa's fix instead? If so, would you mind opening a new PR for that one into main please?

Sorry @alexdewar this completely slipped my mind. Diego's fix has been merged into this one, I'll remove the custom resize function itself and push now.

@dc2917 Does anything else need doing or is this good to merge?


fractal lions

333 Lions: Captivating, rare, and ready to roar into your collection 🦁

please merge our collection, we have started minting


Emergency fix for Preferred Phone for 1.84

Critical Changes
In EDA 1.82, we released Preferred Phone functionality disabled by default for all existing customers. However, the EDA 1.83 push upgrade inadvertently activated our new Preferred Phone TDTM handler for all customers. The fix we are releasing in the EDA 1.84 emergency release now controls the Preferred Phone TDTM handler behind a new custom setting.
If you are an existing EDA customer who enabled the new Preferred Phone functionality when it was released in 1.82, you will need to re-enable the TDTM trigger by going to EDA Settings | Accounts and Contacts and checking the "Enable Preferred Phone sync" checkbox.
Changes
We've improved the Preferred Phone logic released in 1.82 to be controlled by a new custom setting. For new customers, the new Preferred Phone logic will be enabled by default. Existing customers will need to enable Preferred Phone by going to EDA Settings | Accounts and Contacts and checking the "Enable Preferred Phone sync" checkbox.
Existing customers must also uncheck "Disable Preferred Phone enforcement" if they wish to enforce that their contacts have a Preferred Phone value when more than one Phone field is specified on the contact record.
Issues Closed
Fixes #941
New Metadata
New Custom Setting: Hierarchy_Settings__c.Enable_New_Preferred_Phone_Sync__c
New Custom Labels: stgEnablePreferredPhoneSync and stgHelpContactPreferredPhoneSync
Deleted Metadata
Testing Notes

@lifewithryan  Ready for your approval/re-review


HHH-15040 Upgrade to Hibernate Commons Annotations 6.0.0.CR1

https://hibernate.atlassian.net/browse/HHH-15040

As requested by @gavinking , I triggered a release of HCANN and labelled it 6.0.0.CR1 to make this improvement available to ORM core:

https://hibernate.atlassian.net/browse/HCANN-120

Thank you sir


Add pod stats

Add a way to display the number of contributions between fellows. Make a little leaderboard or similar to motivate people to add new issues!

@OmkarAcharekar and I would love to work on this one!

Can I work on this? Actually, I started working on this issue but forgot to comment here.

@Mayank17M Sure feel free to continue working on it, no problem!


Lbaasv2

Hi guys,
This pull request is related to #340
We (Tieto Tecong folks) forked the repo, and did some modifications.
At this moment running OpenShift at our dev env requires quite amount of tuning (done in our fork),
so this pull request is not tested  -> it contains only the lbaasv2 fucntionality.
But still it may be good enough as a starting point.
Let's see how to work on from here, in case you are interested in this feature.
We have also other smaller features and fixes, so maybe we could propose further pull requests.
In ideal case, we would want to have all out modifications upstream.
-Vedran (on behalf of the tecong team)

Should I make a new pull request a new feature branch, instead of the master?

@bartpved thx++! I implemented a different solution adding a PoolMember in every master in master.yaml. I saw you used %index%. Your solution seems cool as you  didn't modify master.yaml

cheers! :-)

@bartpved I just added update_dns support for LBaaS. Check #330. Feedback welcome!

@bartpved I'm adding a lbaas configuration for the wildcard domain too (balancing nodes). Would you do it in the same lbaas.yaml or in a different one?

Hi Gang,
Still working on getting this deployed on my Openstack deployment, but somethings I noticed when deploying:

ERROR: The Parameter (external_network_subnet) was not defined in template.
ERROR: The Parameter (os_domain_name) was not defined in template.

I was able to get the stack to create by adding those parameters to openstack.yaml.
Not sure if this is helpful or correct. Let me know if I can provide additional information.
Thanks!
Connor

Also, I'm getting stuck on the deploy for the loadbalancer resource; any advice?
Resource Create Failed: Resources.Pool Members: Resources.Loadbalancer.Property Error: Resources[0].Properties: Property Master Count Not Assigned

@cwpenhale adding missing params is OK. About the HEAT error, looks like the param master_count is not passed properly, you will need to pass it if it's missing.

To me it worked nicely. I just modified:

the LB to point to the fixed_network avoiding to passing twice thru the external_network
removed from openshift.yaml all lbaasv1 staff which was now not useful
moved all templates to newton


gofmt gate in ci

Due to human error, v1.0.1 was released without a gofmt on the code. this isn't terrible for now but we should really have a step that runs as part of CI checking of prs that will block any code that gofmt still needs to be run on.

closed by 8f5673d


"Availability" and "StatusInfo" have a value 0  but CIM schema only defined +ve values for Win32_Bus profile

I am using a Windows 10 system.
When I query the WMI on the system for Win32_Bus, the Availability and StatusInfo attributes are returning 0.
In CIM Schema there is no 0 value corresponding to these attribute accepted values. Can you please check if this is really a behaviour?
URL: https://docs.microsoft.com/en-us/windows/win32/cimwin32prov/win32-bus
OS version Windows 10 - 1909 (18363.1139)

@drewbatgit Can you take a look?

@drewbatgit Can you take a look?

Though the scope of our feedback channel here on GitHub covers specific documentation fixes, I can help redirect you to the right support channel to get an answer to your question. The community of experts and Microsoft Engineers over at Microsoft Q&A offer timely answers to a multitude of technical questions surrounding windows-uwp.
If you do create a thread over at Microsoft Q&A, it would be fantastic if you could share that link here so I can track the progress and potentially incorporate the technical details into our documentation. Though I'm marking this Issue as closed now, please treat this thread as an open thread for follow-ups. Thanks!

Though the scope of our feedback channel here on GitHub covers specific documentation fixes, I can help redirect you to the right support channel to get an answer to your question. The community of experts and Microsoft Engineers over at Microsoft Q&A offer timely answers to a multitude of technical questions surrounding windows-uwp.
If you do create a thread over at Microsoft Q&A, it would be fantastic if you could share that link here so I can track the progress and potentially incorporate the technical details into our documentation. Though I'm marking this Issue as closed now, please treat this thread as an open thread for follow-ups. Thanks!


feat: CKEditor 5

Please provide enough information so that others can review your pull request:

Explain the details for making this change. What existing problem does the pull request solve?

Screenshots/GIFs

@dti-deploy create instance

@dti-deploy create instance

@dti-deploy create instance

@dti-deploy create instance

@dti-deploy create instance

An error occurred while executing the following command:
git checkout bs-apr-sprint-2
You may manually fix the issue using: ssh <IP_ADDRESS> -l frappe
Traceback:


Check attestation bit lengths

resolves #1262
replaces #1264
Note that the bitlists with too few bits had to be particular in it's signed and then modified to keep the signing helper from complaining

I find two lines more readable and more directly translatable to languages without magic syntax


Issue with writing to register on Sofar HYD 10KTL-3PH Inverter Using pysolarmanv5

Hello,
I am working with a Sofar Solar HYD 10KTL-3PH inverter, interfacing using a Solarman logger version LSW3_15_270A_1.68. I have successfully managed to read data from the inverter using the pysolarmanv5 library, but I encounter errors when attempting to write to the inverter's registers.
Issue Description:
Reading from the inverter's registers works without any issues; however, when I attempt to write to the registers to set new Reflux values, the process fails with a Modbus frame error indicating that the frame does not contain a valid Modbus RTU frame.
Technical Details:
Library Used: pysolarmanv5
Functionality Affected: Writing to registers
Error Message: "V5 frame does not contain a valid Modbus RTU frame"
Communication: Modbus TCP/IP
Inverter Type: Sofar Solar HYD 10KTL-3PH
Logger Version: LSW3_15_270A_1.68
Script Snippet and Full Hexadecimal Data:
Here is the relevant part of my script attempting to write to a register, along with the hexadecimal data logged by the script:
Code:
from pysolarmanv5 import PySolarmanV5
def main():
modbus = PySolarmanV5(
"192.168.x.x", 234xxxxxx7, port=8899, mb_slave_id=1, verbose=True
)
print("Current setting:")
reflux = modbus.read_holding_register_formatted(register_addr=0x1023, quantity=1)
print(f"Register: {0x1023:5}\tValue(0-reflux on, 1-no reflux, 2-limit): {reflux}")
new_reflux_value = 2
print(f"Attempting to write new value: {new_reflux_value} to register {0x1023:5}")
write_response = modbus.write_holding_register(0x1023, new_reflux_value)
print("Write response:", write_response)

print("Verifying change...")
reflux = modbus.read_holding_register_formatted(register_addr=0x1023, quantity=1)
print(f"Updated Register: {0x1023:5}\tValue: {reflux}")

modbus.disconnect()

if name == "main":
main()
Verbose Log Output:
DEBUG:pysolarmanv5.pysolarmanv5:Socket setup completed... <socket.socket fd=3, family=2, type=1, proto=6, laddr=('<IP_ADDRESS>', 50580), raddr=('<IP_ADDRESS>', 8899)>
DEBUG:pysolarmanv5.pysolarmanv5:SENT: a5 17 00 10 45 46 00 c3 94 e1 8b 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 03 10 23 00 01 71 00 20 15
DEBUG:pysolarmanv5.pysolarmanv5:RECD: a5 15 00 10 15 46 2b c3 94 e1 8b 02 01 52 3e 8c 02 90 0b 00 00 70 3c a7 63 01 03 02 00 01 79 84 e4 15
Register:  4131	Value(0-reflux on, 1-no reflux, 2-limit):  1
Writing new value:  2
DEBUG:pysolarmanv5.pysolarmanv5:SENT: a5 17 00 10 45 47 00 c3 94 e1 8b 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 01 06 10 23 00 02 fd 01 b2 15
DEBUG:pysolarmanv5.pysolarmanv5:RECD: a5 10 00 10 15 47 2c c3 94 e1 8b 02 01 54 3e 8c 02 92 0b 00 00 70 3c a7 63 05 00 e6 15
Traceback (most recent call last):
File "pysolarmanv5_write_test.py", line 36, in
main()
File "pysolarmanv5_write_test.py", line 26, in main
print(modbus.write_holding_register(register, new_reflux_value))
pysolarmanv5.pysolarmanv5.V5FrameError: V5 frame does not contain a valid Modbus RTU frame
Request for Assistance:
Has anyone experienced similar issues with writing to Sofar Solar inverters using the pysolarmanv5 library, or does anyone have insights into resolving this frame error? Any suggestions or recommendations would be greatly appreciated.
Thank you in advance for your support and assistance!

Hi @pmenclik,
Based on what you've supplied there, pysolarmanv5 seems to be working correctly. I've highlighted the Modbus RTU frame between ** below.
This is the read request and response from your output above:
DEBUG:pysolarmanv5.pysolarmanv5:SENT: a5 17 00 10 45 46 00 c3 94 e1 8b 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 **01 03 10 23 00 01 71 00** 20 15
DEBUG:pysolarmanv5.pysolarmanv5:RECD: a5 15 00 10 15 46 2b c3 94 e1 8b 02 01 52 3e 8c 02 90 0b 00 00 70 3c a7 63 **01 03 02 00 01 79 84** e4 15

While this is the write request and response:
DEBUG:pysolarmanv5.pysolarmanv5:SENT: a5 17 00 10 45 47 00 c3 94 e1 8b 02 00 00 00 00 00 00 00 00 00 00 00 00 00 00 **01 06 10 23 00 02 fd 01** b2 15
DEBUG:pysolarmanv5.pysolarmanv5:RECD: a5 10 00 10 15 47 2c c3 94 e1 8b 02 01 54 3e 8c 02 92 0b 00 00 70 3c a7 63 **05 00** e6 15

05 00 is not a valid RTU frame.
I suggest you talk to @Matthiasfranck. They reported the same thing with a Sofar inverter in #18.
Regards,
Jon

Hi, thx very much for answer. In meantime I found working solution if anybody might be concerned.
When writing to registers of Sofar you have to use modbus.write_multiple_holding_registers(register, reflux), where register is 4131 (decimal) and reflux is list of two values [2,71] 2 - feedin mode, 71 = 7.1KW limit power. If I want to limit feedin, I just write [2,0] which sets feedin-limit power. That works for me.
Thx for the library, it solves my problem!

Thanks for getting back and for sharing the quirk of the Sofar Modbus implementation.
FYI @Matthiasfranck @d-eggert


Add libfdo_data shared object into edge-api

This is an interim solution, till libfdo_data will start shipping with RHEL

Codecov Report

Merging #361 (70579a0) into main (9640a1f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #361   +/-   ##
=======================================
  Coverage   26.68%   26.68%
=======================================
  Files          30       30
  Lines        2747     2747
=======================================
  Hits          733      733
  Misses       1925     1925
  Partials       89       89

Continue to review full report at Codecov.

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


chore show progress bar when downloading third-party dependencies

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

Further comments

show progress bar when downloading third-party dependencies with wget, for better understanding in estimated time to wait, speed, and total size of file.
 --show-progress option is used with existed -q option , for showing progress bar only without other unhelpful request or response detailes.

This PR doesn't change the behavior of downlaod-thirdparty.sh, and the error message in CI run is irrelevant.
(https://github.com/apache/doris/actions/runs/4171352643/jobs/7221214733#step:4:283)
Unpacking odbcinst1debian2:amd64 (2.3.11) ...
dpkg: error processing archive /tmp/apt-dpkg-install-SY6NPA/43-odbcinst1debian2_2.3.11_amd64.deb (--unpack):
 trying to overwrite '/usr/lib/x86_64-linux-gnu/libodbcinst.so.2.0.0', which is also in package libodbcinst2:amd64 2.3.9-5
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)

Cleaned up code with mering #https://github.com/apache/doris/pull/16754 from master branch and squashed commits.

Hi @bowenliang123 , please rebase this PR on the branch master when you have time due to some checks failed.

Hi @bowenliang123 , please rebase this PR on the branch master when you have time due to some checks failed.

Alright, will do it. But I'm confused that the failures won't be related to my changes. No side effects should be brought from this pr.

Hi @bowenliang123 , please rebase this PR on the branch master when you have time due to some checks failed.

Alright, will do it. But I'm confused that the failures won't be related to my changes. No side effects should be brought from this pr.

Some cases are not stable in P0 regression, we should re-run it to let it pass. We can not merge a PR which made some checks fail.

Hi @bowenliang123 , please rebase this PR on the branch master when you have time due to some checks failed.

Alright, will do it. But I'm confused that the failures won't be related to my changes. No side effects should be brought from this pr.

Some cases are not stable in P0 regression, we should re-run it to let it pass. We can not merge a PR which made some checks fail.

Agree with it. 👍

run buildall

run buildall

What's buildall, a script for development or something?

run p0

run buildall

run buildall

What's buildall, a script for development or something?

A reply to trigger the checks, please reference to
https://cwiki.apache.org/confluence/display/DORIS/Github+Checks+Guidance.

run buildall


cc other people in email notification

in email_configs we can set the receivers by to, is it possible to add an option to set cc
who want to be included to the notification but not the owner need to take care the warning.

The email notifier configuration contains a header field to set arbitrary email headers. Setting the CC should accomplish what you want.


Shell auto completion

Working on issue  #74: Shell auto-completion
I've added a shared cobra command  - completion -  to be used as sub-command both for uyuniadm and uyunictl root commands.
It leverages cobra's built in generation of shell completion scripts.
I had to do a minor refactor of the Logger's init method in order to avoid redirecting also the info welcome messages to the generated shell completion scripts.

Thank you for your contribution, this is really cool stuff.
However, I'm not sure if the completion subcommand should be part of the existing commands. What do you guys think?
We could have a module that could be called to generate the completion for all flavors and it could be called at build time. to provide the data for the completion package.
@cbosdo @mbussolotto what do you think?

@rjmateus yes, I think we summarized everything here https://github.com/uyuni-project/uyuni-tools/pull/94#discussion_r1386313269 (not really readable since it was a comment on the code)

Thank you for your contribution, this is really cool stuff. However, I'm not sure if the completion subcommand should be part of the existing commands. What do you guys think? We could have a module that could be called to generate the completion for all flavors and it could be called at build time. to provide the data for the completion package. @cbosdo @mbussolotto what do you think?

If that is possible, then yes. Otherwise we could just hide those subcommands.

Note that this PR is built successfully for all distros but AlmaLinux8 in https://build.opensuse.org/package/show/home:cbosdonnat:branches:systemsmanagement:Uyuni:Master:ContainerUtils/uyuni-tools


LiveKit Gstreamer + SyncFlow Client Error

Once I try to stream any data over Gstreamer using the livekit-gstreamer crate to the SyncFlow server, I get the following error:
[2024-11-05T04:22:36Z INFO  livekit_api::signal_client::signal_stream] connecting to wss://livekit.syncflow.live/rtc?sdk=rust&protocol=15&auto_subscribe=1&adaptive_stream=0&version=0.6.0&access_token=...
thread 'tokio-runtime-worker' panicked at /home/nicole/.cargo/git/checkouts/livekit-gstreamer-54f42934dd926e8b/45cb93b/src/media_device.rs:158:51:
called `Result::unwrap()` on an `Err` value: StateChangeError
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
[2024-11-05T04:22:46Z INFO  example_client] Connected to room: session-peSs-Wr7L - RM_mPJJCtJoaCfA
[2024-11-05T04:22:46Z INFO  example_client] Published track with SID for one minute: "video-track-KYwCXfcc5bbOFKvc"

This error occurs if I run the example_client.rs. This was not an issue before when I was using the LiveKit auth directly.
Most interestingly, this doesn't cause the program to crash or leave the room. Plus the video track can be seen but it's a green screen for some reason.

Furthermore, this seems to be an error with video only:
Audio capability: AudioCapability { channels: 2, framerates: (44100, 192000), codec: "audio/x-raw" }
Configuration saved
{"sessionId":"b813a7aa-db5d-4eb1-b8f6-2130597fa4aa","sessionName":"session-tMmF-TuUN"}
Message: StateMessage { topic: "RabbitMQ", status: Success, comment: Some(Object {"sessionId": String("b813a7aa-db5d-4eb1-b8f6-2130597fa4aa"), "sessionName": String("session-tMmF-TuUN")}) }
RabbitMQ message: StateMessage { topic: "RabbitMQ", status: Success, comment: Some(Object {"sessionId": String("b813a7aa-db5d-4eb1-b8f6-2130597fa4aa"), "sessionName": String("session-tMmF-TuUN")}) }
Session ID: "b813a7aa-db5d-4eb1-b8f6-2130597fa4aa"
Session Name: "session-tMmF-TuUN"
Joining session with name and id: session-tMmF-TuUN - b813a7aa-db5d-4eb1-b8f6-2130597fa4aa
Connected to room: session-tMmF-TuUN - RM_HDQ9bSQmZqQY
Received room event: Connected { participants_with_tracks: [] }
Received room event: ConnectionStateChanged(Connected)
Received room event: LocalTrackPublished { publication: LocalTrackPublication { sid: TrackSid("TR_AMXMkXWctFRiwP"), name: "Built-in Audio Analog Stereo", kind: Audio }, track: Audio(LocalAudioTrack { sid: TrackSid("TR_AMXMkXWctFRiwP"), name: "Built-in Audio Analog Stereo", source: Microphone }), participant: LocalParticipant { sid: ParticipantSid("PA_FTpg5xCq7q2k"), identity: ParticipantIdentity("SyncFlowProjectClient"), name: "SyncFlow Project Client" } }
New room: Some(Room { sid: Some(RoomSid("RM_HDQ9bSQmZqQY")), name: "session-tMmF-TuUN", connection_state: Connected })
Received room event: ConnectionQualityChanged { quality: Excellent, participant: Local(LocalParticipant { sid: ParticipantSid("PA_FTpg5xCq7q2k"), identity: ParticipantIdentity("SyncFlowProjectClient"), name: "SyncFlow Project Client" }) }
Received room event: ParticipantConnected(RemoteParticipant { sid: ParticipantSid("PA_2rDNr4uF8t2n"), identity: ParticipantIdentity("edavalosanaya"), name: "edavalosanaya" })
Received room event: ConnectionQualityChanged { quality: Excellent, participant: Local(LocalParticipant { sid: ParticipantSid("PA_FTpg5xCq7q2k"), identity: ParticipantIdentity("SyncFlowProjectClient"), name: "SyncFlow Project Client" }) }

Determined that this was an issue from device configuration.


[MPG][Diff] properties of SystemData is marked as readonly

SDK generated with typespec:

SDK generated from swagger

@ArthurMa1978 , do we still need to keep this issue open?

@mikekistler if the readonly flag is mark in a normal object, should it propagate to its properties? e.g.
 "arbitraryObject": {
      "type": "object",
      "readOnly": true,
      "properties": {
        "aproperty": {
          "type": "string",
          "description": "The identity that created the resource."
        }
    }
}

Is aproperty readonly?

I doubt it should. I tried it out at https://editor-next.swagger.io/ and generated C# client. With readOnly on Bar and not on prop1, it essentially got ignored. Both setter and getter are generated for prop1.
So while ARM team may have intended for the properties to be readonly but incorrectly set on the definition instead rendered it useless, I suppose we would need to remove readOnly from all of SystemData's properties to match swagger then.
openapi: 3.0.0
info:
  title: (title)
  version: 0.0.0
tags: []
paths: {}
components:
  schemas:
    Bar:
      type: object
      required:
        - prop1
      readOnly: true    <----- placed here instead of on prop1
      properties:
        prop1:
          type: string
    Foo:
      type: object
      required:
        - bar
      properties:
        bar:
          allOf:
            - $ref: '#/components/schemas/Bar'
          readOnly: true

Fixed in pending PR. https://github.com/Azure/typespec-azure/pull/945


Windows node not joining the eks cluster

Describe the Bug
I am using below config to create a windows node group using the latest version of the module, node gets created but fails to join the cluster.
module "worker_label_green" {
source = "cloudposse/label/null"
namespace  = var.namespace
name       = var.name
stage      = var.stage
delimiter  = var.delimiter
attributes = var.attributes
tags       = merge(var.tags, {
"kubernetes.io/cluster/${var.cluster_name}" = "owned"
})
}
module "eks_web_node_group_green" {
source  = "cloudposse/eks-node-group/aws"
version = "3.1.0"
enabled = var.green_enabled
context = module.worker_label_green.context
instance_types     = var.instance_types
subnet_ids         = local.worker_subnet_ids
min_size           = var.min_size
max_size           = var.max_size
desired_size       = var.desired_size
cluster_name       = data.terraform_remote_state.eks_cluster.outputs.eks_cluster_id
kubernetes_version = var.kubernetes_version == null || var.kubernetes_version == "" ? [data.terraform_remote_state.eks_cluster.outputs.eks_cluster_version] : [var.kubernetes_version]
kubernetes_labels  = var.labels
ami_type = var.ami_type
before_cluster_joining_userdata = [
data.template_file.pre_eks_worker_nt.rendered
]
after_cluster_joining_userdata = [
data.template_file.post_eks_worker_nt.rendered
]
kubernetes_taints = [{
key    = "OS"
value  = "Windows"
effect = "NO_SCHEDULE"
}]
update_config = [{ max_unavailable = var.desired_size }]
capacity_type = var.capacity_type
detailed_monitoring_enabled = true
node_role_arn                = [data.aws_iam_role.worker_role.arn]
node_role_cni_policy_enabled = false #We use the Service Account as per best practice
associated_security_group_ids = [
data.terraform_remote_state.network.outputs.rancher_sg,
data.terraform_remote_state.network.outputs.ops_ssh,
data.terraform_remote_state.eks_cluster.outputs.security_group_id
]
Enable the Kubernetes cluster auto-scaler to find the auto-scaling group
cluster_autoscaler_enabled = var.cluster_autoscaler_enabled
create_before_destroy = true
node_role_policy_arns = ["arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"]
block_device_mappings = [
{
"delete_on_termination" : true,
"device_name" : "/dev/xvda",
"encrypted" : true,
"volume_size" : 90,
"volume_type" : "gp3"
}
]
node_group_terraform_timeouts = [{
create = "40m"
update = "40m"
delete = "20m"
}]
#Valid types are "instance", "volume", "elastic-gpu", "spot-instances-request", "network-interface".
resources_to_tag = var.capacity_type == "SPOT" ? ["instance", "spot-instances-request", "volume", "network-interface"] : ["instance", "volume", "network-interface"]
}
Expected Behavior
Node should join the cluster
Steps to Reproduce
If you have existing cluster just try creating the windows node group into that
Screenshots
No response
Environment
No response
Additional Context
No response

It's failing because the userscript contains the bootstrapper in the middle; but the script that is stored in the launch template contains the bootstrapper again at the end.

The change in how the windows nodes are assigned has caused this.
If the ami-type is defined and AWS is supplying the AMI it will show in the console as ami release version

The v2 module was fetching the windows ami so it was being set as 'custom' and showing the ami ala

The first one has the advantage that updates to the AMI show in the console; but AWS automatically augments your Userdata by adding the bootstrapper to the end of your userscript in the launch template. This doesnt show in the state when you do your plan.
It's not a huge issue to work-around but it does make the current user script broken; I assume it does the same for linux too.
If you have a before_cluster_joining_userdata and after_cluster_joining_userdata set and it's not a CUSTOM ami_type AWS will inject the EKSBootstrapScript execution at the end of the userdata.


[Bug]: Version not available under update card

Current Behavior
When I try to use the mushroom update card it does not allow me to select the binary sensor for the binary_sensor.docker_hub_update_available
This is based of the inbuilt Version Integration : https://www.home-assistant.io/integrations/version/#:~:text=From the configuration menu select,to complete the set up.
Expected Behavior
Home assistant version to appear
Steps To Reproduce
Within home assistant (docker)
On latest mushroom via hacs (v2.1.3)
Have Version Integration setup on docker
Environment
- OS: Ubuntu 22.04 lts
- Node: docker
- Version: 2.1.3
- Hassio: no - via HACS

Anything else?
No response

The update card is for update entity. Your entity is a binary sensor. You should use entity card.

Oh, brilliant.. sorry I had missed that.. could have sworn I looked at the ones it was seeing and they were binary sensors as well.
Thanks.


Updating IPFS hash checkpoint

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[material-ui] Handling complex themes/palettes

Summary
I've been using MUI lately on a larger project. Compared to building a custom UI Lib from scratch it had tons of advantages. With one downside: the color palette of the project was so versatile that I couldn't handle most parts of it with MUI's theme approach.
Examples
An exaggerated example to explain the issue:
We had a primary color red that is used as a background color for button variants on our light theme.
On our dark theme the primary color is blue. But not all buttons should have a 1:1 the blue color. They needed to have others shades or sometimes even different colors.
We had this issue in so many places that I basically ended up creating themes where I defined the colors for every component like this:

Which I injected into the theme and then used in my component overrides like this:
  {
          props: { variant: 'secondary' },
          style: {
            color: theme.styles.button.secondary.default.color,
            backgroundColor:
              theme.styles.button.secondary.default.backgroundColor,
            '&:hover': {
              backgroundColor:
                theme.styles.button.secondary.hover.backgroundColor,
            },
            '&:active': {
              color: theme.styles.button.secondary.active.color,
              backgroundColor:
                theme.styles.button.secondary.active.backgroundColor,
            },
          },
        },

But this has some downsides. Eg. using the color palette properties on components wouldn't reflect my theme anymore. And many others.
Motivation
My motivation is to find out are there any plans/considerations that something like this might also be handled with MUI? Or did you encounter similar issues in the past? Maybe there is another better way how this can be solved.
Search keywords: theme complex palette

@mightym I did not understand your issue. Could you provide a minimal reproduction? It helps us troubleshoot. A live example would be perfect. This StackBlitz sandbox template may be a good starting point.

hey @ZeeshanTamboli thanks for your reply!
Well this is less a technical question, more a conceptual question how to handle complex themes in MUI.
I try to rephrase in a single line:
What would you do if 2 different components A & B rely on primary for the background color in your light theme but on the dark theme the background color for A is still primary but on B its a slight variation of primary?

What would you do if 2 different components A & B rely on primary for the background color in your light theme but on the dark theme the background color for A is still primary but on B its a slight variation of primary?

I would use theme.palette.mode  to check for dark mode and apply the slight variation of primary which you have defined in your custom theme.

@ZeeshanTamboli see and thats why you will end up with something extremely complex like in my code example. Because in your approach you will end up with a lot of conditions based on the current theme. What you need is an intermediate level like my theme styles. But for sure you could write conditions in you components.

What you need is an intermediate level like my theme styles

Okay, so what's the problem with it? What issue is it causing? I see that you are saying that:

But this has some downsides. Eg. using the color palette properties on components wouldn't reflect my theme anymore. And many others.

But I can't check with a reproduction. It would be better if you can provide a reproduction.