Clarify LZX vs. LZXD

As I'm looking through this library, it isn't too apparent whether this library actually implements the LZX-DELTA algorithm.
For example, the header block is parsed according to the LZX specification as opposed to the LZX-DELTA specification, and this library is able to successfully decode an LZX bitstream.
In addition, a similar C library provides the ability to set reference data for the algorithm to process, whereas this library does not.
It might be better to rename this library to lzx and limit the functionality to that offered by the LZX algorithm (such as limiting the window sizes).
Or, alternatively, the mspack library could be used as reference for extending this library to support both LZX and LZX-DELTA.

It might be better to rename this library to lzx and limit the functionality to that offered by the LZX algorithm (such as limiting the window sizes).

Oh, no, my goal was absolutely LZX-DELTA in order to decompress Terraria's XNB files. But I have near zero experience with (de)compression algorithms, so there's a good chance I got some things wrong. However, given I (tried to) follow Microsoft's lzxd encoding specification, I was hoping this would be LZX-DELTA.
I won't be renaming the library (it's also on crates.io as lzxd already). But given "LZX" is part of the name "LZX-DELTA", it would be a good idea for this library to properly support both.

Or, alternatively, the mspack library could be used as reference for extending this library to support both LZX and LZX-DELTA.

That's using GNU GPL license, so I don't know if we can borrow anything from there.

Ah - well this library can definitely support both!
In that mspack repository, you can search for the string is_delta to immediately see the differences between LZX-DELTA and LZX.
While we definitely can't reuse the code from that library because it's GPL and it's C, it does serve as a good code reference.

Hmm, I think a good test file (properly compressed .lzxd, and not .lzx, file) plus the latest version of [MS-PATCH]: LZX DELTA Compression and Decompression should be enough to verify whether we have a working implementation or not. Alas, I've had a hard time finding compressors (and not decompressors) for the LZX-DELTA format (and this library is yet another example).

Ditto - same here! I only just learned that Microsoft's makecab seems to be the best option at getting samples of LZX.


player controller lock up

I threw a strong punch, then a fast punch, then got hit by an attack, and locked up with the fast punch state, showing the strong punch animation, and with no keyboard control.
Seems like fast punch didn't release, but I don't know why it'd still show the strong punch animation.

I haven't seen this since I stopped using AnimationPlayer.stop(), since the player relies on some signals from _on_animation_player_animation_finished(anim_name)

This is still an issue.
Sometimes the player shows the strong punch animation, with the fast punch state, but controls are locked.
It might have to do with transition from strong punch to fast punch during escape frames.

I think I fixed this.


Correct AMD define for plugins

Related to https://github.com/aFarkas/lazysizes/issues/470
Using requirejs if a plugin is required, the function callback is called before plugin initialization.
With define it works correctly because requirejs wait for plugin  initalization.

Thank you for your contribution.
@emkayy
Are you fine with this change?

Yes, that's actually right!


Feature add support for markdown and mentioned people in /messages POSTs

Added code to support markdown in messages.   This is particularly needed in order to test bots in group spaces when the bot will only respond to commands where the bot is mentioned.
This commit also includes a set of postman based tests that sends markdown messages to the Spark platform and to the emulator and compares the return values.

Forgot an update in tokens.json.  Will re-open momentarily.


[DO NOT MERGE] Migrate Place

Migrate the Place format, such that it is rendered through the Publishing API.
Do not merge yet, as the merging/release of this needs to be coordinated with Frontend.

This should now add a document_type method to the Presenter, see https://github.com/alphagov/publisher/pull/581

This could be squashed

WFM :sheep: :it:

We've previously added a test to check that #migrated? returns true for each migrated format here https://github.com/alphagov/publisher/blob/master/test/unit/edition_test.rb#L75-L110) so might be good to add for consistency?

Same here https://github.com/alphagov/publisher/blob/cefe2eecde3f6b788638323825cd9d2821804904/test/unit/services/edition_presenter_factory_test.rb

We've previously added a test to check that #migrated? returns true for each migrated format here so might be good to add for consistency?

Yep, I've done this now :)

Looks good, merge when ready! 👍


Handling SocketException in NamedPipesServer

Solves #235 by catching the SocketException and handling it the same way, that ObjectDisposedException is currently handled.

Thanks again for this PR, and it actually does point to a more serious underlying issue with disposed tracers. See #244 for my attempt at fixing this.

Ok, great to see you've fixed the root cause of this! With your fixes in, I assume we can abandon this PR.


Response returned with this_block is always null

I have started the project, I use nodeos from WAX testnet. Everything is fine, but the following response with this_block is always null. Can you help me? (my start_block = 0 )

you probably started node from a snapshot. If not then try setting start_block to 2 but in current versions this should be fixed

I have started the project, I use nodeos from WAX testnet. Everything is fine, but the following response with this_block is always null. Can you help me? (my start_block = 0 )

I,m interested
Can u contact me?


Show Rules error

If you only provide the package name and no layer information, this will result in a KeyError. Please provide   name for now.

#59 Open, working on it.

Resolved by providing to user available layers. No user input required.


chai

I have an error on saberhq  and I don't understand what the problem

@ZouaouiK can you provide us more info on what you're trying to do?

Please migrate to yarn 3: https://yarnpkg.com/getting-started/migration#step-by-step


[thank-you page] re-add the special first QGIS-users group picture

Background discussion: https://lists.osgeo.org/pipermail/qgis-community-team/2022-November/006082.html

related to: https://github.com/qgis/QGIS-Website/issues/1077

Request:

re-add the special group picture from the first-ever QGIS-users event, to the thank-you page

rock on 2015 @jmckenna .....!
@anitagraser what do you think? Bring it back and make @jmckenna happy or kill it and make the suits happy?

+1 for this nice group picture on the thank you page
I don't think suits will see this 😉

Thank-you @Xpirix and all, what a nice way to start the day, a nice memory.


Add : Signup and signin page [GSSOC'23].

I would like to add a signup and signin page as these pages are must in any website.
This will increase the user interaction.
so ,please assign this issue to me under GSSOC'23.
Thanking You.

I want to work on this issue as a contributor.
Pleasure Assign so that I can show my skills under GSSoC'23.

hello can i work on this issue as a GSSOC 23 contributor. can you assign this to me so can i work on it

I am a full-stack developer, so adding this feature is not a big task for me. so please assign me this and I am also a GSSOC 2023 contributor

Hello, I already worked on a food recipe app, so I'm good to create these types of pages easily. please assign me this issue.
I'm GSSOC'23 contributor.


[OtrkeyFinderBridge] Add bridge for otrkeyfinder.com

A bridge for otrkeyfinder.com.
The site already has an RSS feed, but it will only return the entries on the first page. Since one day has already 25 pages worth of entries, this feed is useless if not retrieved constantly 24/7. It also offers no filter function for running time.
This bridge fixes that: It can retrieve more than one page and also has a running time filter.
Be aware of the ToS of this site: It prohibits "automatic readout of data". This issue is discussed in #1631.

Hi, @mibe! Tried out this bridge.
Could you give me example of query for demonstration? My query does not give anything:
https://feed.eugenemolotov.ru/pr1712/?action=display&bridge=OtrkeyFinder&searchterm=Terminator&station=ARD&type=&minTime=90&maxTime=120&pages=5&format=Html

@em92 You get zero results because "Terminator" never ran on ARD. Try using "RTL" as the station name or leave it emtpy.
Try "tagesschau" as a search term on station "ARD" without any running time filters. You should get tons of results, since this is one of the main television news program.
I personally use this bridge to get the latest movies which were aired in high definition. I'm using an empty search term & station name, "HD" as media type and a minimum running time of 60 minutes.

gj @mibe !


docs/Formula-Cookbook: several content updates

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[N/A] Have you written new tests for your changes? Here's an example.

[N/A] Have you successfully run brew tests with your changes locally?

Several updates for docs/Formula-Cookbook:

move all rubydoc.info link URLs to footer to keep the text easier to manage. Also convert each to HTTPS.
link to specific revisions of formula examples so line highlighting doesn't drift over time.
monospace formula link text.
add newer download strategies, and list authenticated strategies in separate table.
mention that Requirements can specify a minimum version using a String, and that both dependencies and Requirements accept lists.

@EricFromCanada Have you validated all of those URLs? Are they all still active and correct?

move all rubydoc.info link URLs to footer to keep the text easier to manage.

I'm sorry to say I don't agree with this change and think it makes the raw markdown harder to follow.
Otherwise I'm 👍 👏 on everything else.

Very well then. [wields rebase command]


Question: is there a way to disable select on a particular row or rows?

We need select checkboxes for delete operations, but have a "default" item that cannot be deleted and wish to NOT show a checkbox for that row. Seems something that could be handy for other uses as well (user can only edit some rows depending on some value). Is there any current way to accomplish that?

@L5eoneill, sorry for lately reply and react-bootstrap-table doesn't support this feature currently. I thinks it's a particular feature or requirement. BTW, I think this requirement is not very hard to implement, maybe you can try it. Feel free to ask me anything if have question.
BTW, this inspire me a idea for supporting a unselect row. however, i thinks I can support it in near future, if it done, and you can hide the disable checkbox through css(I can give this disbled checkbox a classname). Maybe its a perfect workaround.

Sweet! Excellent work, Allen. It works great (other than the "select all" checkbox will never indicate "all" are selected.)
By the way, to hide the disabled one, you'll need css saying input[type="checkbox"][disabled] { display: none;}

@L5eoneill, oh I forgot the select all, I'll patch it on next version. BTW, I think I just make the checkbox disabled, I dont want to hide it. Anyway, it's a customizable case for user. I'm planing to make the ckeckbox columns more customizable in v3.0.0. ckeck #497
Thanks.

@L5eoneill, the select all bug has been fixed on v2.3.7.


Add support for categorical/dictionary types

Arrow has a very useful dictionary/categorical type (https://arrow.apache.org/docs/python/generated/pyarrow.dictionary.html). This data type has significant speed, memory and disk benefits over pa.string() when there are only a few unique text strings in a column.
Unfortunately, huggingface datasets currently does not support this type. So huggingface datasets cannot natively read many parquet files that use this datatype .This PR adds support for Huggingface Datasets to read categorical/dictionary data.
Note: This PR functions by simply converting those dictionary/categorical types to strings. This means that huggingface datasets cannot take advantage of the compute benefits of categoricals, but it significantly simplifies logic. At this time, I do not think it makes sense to optimize categorical support within huggingface datasets and that we should only try to optimize later, if necessary.
Closes #5706

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@lhoestq  Thanks a ton for helping this get merged!


[ZEPPELIN-5184] Update related to entity name of credentials

What is this PR for?
The entity name of the credential has changed.
Please refer to these. https://github.com/apache/zeppelin/pull/3806, https://github.com/apache/zeppelin/pull/4008.
The documentation and web UI need to be updated.
Change the entity name from [Interpreter Group].[Interpreter Name] to [Interpreter Name].
What type of PR is it?
[Improvement | Documentation]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-5184

How should this be tested?
Access the web page.
Screenshots (if appropriate)
Web UI

http://[zeppelin-server]:[zeppelin-port]/#/credential
Change the entity name from [Interpreter Group].[Interpreter Name] to [Interpreter Name].
default UI

before

after

next UI

before

after

Documentation

http://[zeppelin-doc-server]:[zeppelin-doc-port]/setup/security/datasource_authorization.html#how-to-save-the-credential-information

Change the entity name from [Interpreter Group].[Interpreter Name] to [Interpreter Name].

before

after

http://[zeppelin-doc-server]:[zeppelin-doc-port]/usage/interpreter/overview.html#credential-injection

Fix broken image

before

after

Questions:

Does the licenses files need update? N
Is there breaking changes for older versions? N
Does this needs documentation? Y

LGTM

LGTM

Will merge if no more comment

Will merge if no more comment

@zjffdu Thank your for your help!

@zjffdu Thank your for your help!

@zjffdu Thank your for your help!

No problem, welcome more contributions to zeppelin

@zjffdu Thank your for your help!

No problem, welcome more contributions to zeppelin


Fix slowness when finding a large number of items

addOrUpdateList was duplicating work, and the keyedById object
was being cloned, handed to Vue to be made reactive before being thrown away,
over and over again through the loop. Instead, now do as much work as possible
by batching addItem mutations together.

Thank you for this!

Released in<EMAIL_ADDRESS>


Feature Request: Notify about old notes

It's great I can see old note warning on the top. But it'd be cool to get notified about old notes once per specified time using built-in Obsidian notifications.

Thank you for your FR, @decatetsu :)

get notified about old notes once per specified time using built-in Obsidian notifications.

I think it is a bit noisy. However, it might be helpful to get all notes expired in the vault.

OK, then I close the issue


Attempted to call an undefined method named "loadUserByUsernameAndPayload" of class "Symfony\Component\Security\Core\User\ChainUserProvider"

JWTAuthenticator throws an error when ChainUserProvider is used. When you iterate through $this->userProvider->getProviders() you call loadUserByUsernameAndPayload on $this->userProvider rather then on specific $provider in chain.

I also have the same problem and #972 solves the problem :+1:

Not sure about the solution. It has not solved it for me.
Firstly, I get method_exists(PayloadAwareUserProviderInterface::class, 'loadUserByIdentifierAndPayload') as false
$this->userProvider is the ChainUserProvider as is being iterated over, so will not have the method loadUserByIdentifierAndPayload or loadUserByUsernameAndPayload - the chain is being looped over, I don't know why we would then use the chain $this->userProvider and not the provider within $provider
The solution that works for me is changing to
if ($provider instanceof PayloadAwareUserProviderInterface) {
                        if (method_exists($provider, 'loadUserByIdentifierAndPayload')) {
                            return $provider->loadUserByIdentifierAndPayload($identity, $payload);
                        } else {
                            return $provider->loadUserByUsernameAndPayload($identity, $payload);
                        }
                    }

Not sure about the solution. It has not solved it for me.
Firstly, I get method_exists(PayloadAwareUserProviderInterface::class, 'loadUserByIdentifierAndPayload') as false
$this->userProvider is the ChainUserProvider as is being iterated over, so will not have the method loadUserByIdentifierAndPayload or loadUserByUsernameAndPayload - the chain is being looped over, I don't know why we would then use the chain $this->userProvider and not the provider within $provider
The solution that works for me is changing to
if ($provider instanceof PayloadAwareUserProviderInterface) {
                        if (method_exists($provider, 'loadUserByIdentifierAndPayload')) {
                            return $provider->loadUserByIdentifierAndPayload($identity, $payload);
                        } else {
                            return $provider->loadUserByUsernameAndPayload($identity, $payload);
                        }
                    }


DO NOT update to 05.65.10

my 5 year old oled just received an update after many many years of neglect. LG is awake just to patch this exploit.

Can you write if you checked it?
In the sense, did you do an update and the exploit stopped working?

the exact same update was done on my c1 and it stopped working. not risking
and trying it on my 2016 oled.
On Tue, Oct 11, 2022, 1:12 PM Tomasz C. @.***> wrote:

Can you write if you checked it?
In the sense, did you do an update and the exploit stopped working?
—
Reply to this email directly, view it on GitHub
https://github.com/webosbrew/webos-homebrew-channel/issues/121#issuecomment-1275015315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABI3UWZ22TMAC6KEMDOCESTWCWNWHANCNFSM6AAAAAAQ76U6MU
.
You are receiving this because you authored the thread.Message ID:
@.***>

tomasz was asking because what you have written is not clear. What is not working?
Is it your TV that is not working?
Or it works, but the homebrew channel app disappeared?
Or that and also you are not able to install it back again?
Please answer all questions above.
From what you have written, it is not clear whether you tried reinstalling the app.

FYI: Same here: today LG awake to upgrade my 5y old UJ6309 from 06.00.20 to ...25 after years of ignoring this model.
Even option "automatically upgrade" was unchecked, there was a warning displayed from Homebrew to reboot since a crash happened. Just pressed "cancel" here, but don't know if this will re-appear. Don't know yet if I should upgrade (how) or not.

See also #112

your version is <IP_ADDRESS> and not the version in question. never update your lg tv after they ghosted you for years. this new update wont fix anything or add any feature. the whole update is to only remove the ability to install homebrew

I can now install 5.70.30 in dev mode? does anyone have it?

Unrelated to Homebrew Channel. We have always advised people not to perform any software updates.


Add WAYF in eduID

This issue is imported from pivotal

- Support decentralised verification process

Make WAYF friendly for externals
(Marlies Rikken - Sep 10, 2024)

Is this for account linking?
eduID user will already get an WAYF screen in eduID: are you a student/employee? Instead of directly being dropt to the SURFconext WAYF.


[HUDI-3895] Fixing files partitioning sequence for BaseFileOnlyRelation

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Fixing files partitioning sequence for BaseFileOnlyRelation to make sure we efficiently bucket small files. This brings Hudi tables on par w/ raw Parquet tables.
Brief change log

Make sure we reverse sort the files before bucketing

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
This pull request is already covered by existing tests, such as (please describe tests).
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@nsivabalan i don't think there's a way for us to assert this easily. We can try to assert the number of partitions after optimizing Spark plan. Will create a ticket to follow-up on it.

sure, sg.


fix host in request header

Following instruction at https://aws.amazon.com/blogs/compute/creating-a-searchable-enterprise-document-repository/, I  deployed the sam template  with the endpoint url of my elastic search instance.
The last lambda function in the pipeline, AddToESFunction, fails since the host in the request header to the elastic  search  service is set to the full URL , instead of just the domain  name part
The fix is a one line change

Thank you for this contribution!


Mid0061 TighteningErrorStatus - wrong bit field encoding?

If I set the Rehit flag in the TighteningErrorStatus entry of Mid0061, I get an encoded value of 4194304.
Rehit is documented to be bit 18. Because they seem to count from 1, it should be 2^17 = 131072. But it isn't. Why?
And the bits in StrategyOptions count from 0 in the documentation, but you start with 1. Hmmm...
As long as encoding and decoding both is done by your OpenProtocolInterpreter, this may work. But what happens, if another code counts and converts in a different way. The bits all would be messed up and nothing would work.

SetByte method from BitConverter is setting bits "left to right" orientation, not "right to left", which causes the incorrect byte array.

So this is a bug on your side?
I cannot verify this, because both applications, I use for testing, do not support bit fields.
And I cannot find any details in the OpenProtocol documentation.
Do you have any detailed documentation on the different bit fields?
A potential client is waiting for a proof of concept demonstration...

Yes, it is, just commenting for documenting purpose. Also tagged as a bug.
I'm going to fix it ASAP, probably until the weekend!

Hello Henrique,
I’m afraid, there is a major problem with encoding and decoding of bit fields as they are used in Mid0061 and others.
You seem to encode theses bit fields binary, which results in non ASCII characters including 0x00.
Only the Rehit bit set results in (hex values): 32 30 00 00 40 00 00 00 00 00 00 00
I think, this is wrong. Bit fields should be encoded as integer values to ASCII characters.
Neither the binary nor the ASCII method is documented in the Open Protocol manual.
So I rely on some hints, I found:

MID 0033, Revision 3, Job list:

Job step type = Two ASCII characters, range 00-99     Batch step = 1     Reserved = 2-6
Including the reserved, there can be up to 6 bits, which creates a maximum value of 63 – 2 ASCII characters.
The examples shows, that Bit 1 (the lowest bit) is encoded as “01” (lowest bit on the right).

MID 0061 and 0065, Revision 6, Tightening error status 2:

Including the reserved, there are up to 32 bits documented. 32 bits give a maximum decimal value of<PHONE_NUMBER>.
This requires 10 ASCII characters, exactly the size, which is specified for this field (517-526).
Just a coincidence? I don’t’ think so. A binary encoding only requires 4 bytes of the available 10. Makes no sense for me.

Chapter 2.2 says: “Most of the information sent over the communication links is in ASCII format. Some messages can also contain binary data and this is described at each MID.”

But there is no description of non ASCII in the Mids 0061, …
From my understanding, the Rehit bit in Mid 0061 (Bit 18, counting from 1) must be encoded to 2^17 = 131072.
The encoded entry therefore should be “200000131072” and in hex: 32 30 30 30 30 30 31 33 31 30 37 32
I`m not a 100% sure, but let’s say 95%, no 98%. 😉
Also there seems to be a different result by Pack() (decimal character digits, but wrong number) and PackBytes() (binary)
Please carefully think about it. It’s not enough, if you package can send and receive correctly.
The communication must work with other programs from other developers and companies as well.
Bye,
Inge
Von: Henrique Dal Bello @.>
Gesendet: Mittwoch, 19. Mai 2021 16:09
An: Rickedb/OpenProtocolInterpreter @.>
Cc: Ruppio, Inge @.>; Author @.>
Betreff: Re: [Rickedb/OpenProtocolInterpreter] Mid0061 TighteningErrorStatus - wrong bit field encoding? (#56)
Yes, it is, just commenting for documenting purpose. Also tagged as a bug.
I'm going to fix it ASAP, probably until the weekend!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRickedb%2FOpenProtocolInterpreter%2Fissues%2F56%23issuecomment-844141665&data=04|01|inge.ruppio%40stahlwille.de|af5c3b8329c1444fbcef08d91acfa358|ed198749807c4ca38d7d9c86b93b9853|0|0|637570301373772166|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=dqRty7ZIY3E88cDutQHbgObdPK3ay6MkzPoYL7wbkpo%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPPFHTDRRTWPUOEA7S3VM2DTOPA7JANCNFSM4472Y22Q&data=04|01|inge.ruppio%40stahlwille.de|af5c3b8329c1444fbcef08d91acfa358|ed198749807c4ca38d7d9c86b93b9853|0|0|637570301373782122|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QrTK9Z%2B1WxVH5oogfFrV%2F74zQ1tdDWLl%2BQ9GzUbUZuk%3D&reserved=0.

For Mid0061, I'm sure it's been converted to integer before we convert to an ASCII value, that ensures we are not encoding wrongly.
What happens is that, it parses correctly, however when packing, it's creating the byte array reversed, for example, in StrategyOptions we have a Unit tests that it's integer value is 67 which stands for Torque, Angle and Rundown. (Bytes 0, 1 and 6)
When packing the integer value was 194:
67 => 0100 0011
194 => 1100 0010
This is a defect in BitConverter class, that threats it reversed. I'm already working for the solution.
This affects StrategyOptions, TighteningStatus and TighteningStatus2.

Pack() now seems to create a correct entry for TighteningErrorStatus (Rehit now is "0000131072"). 👍
But PackBytes() still creates a binary entry, so the Integrator cannot read Mid0061 correctly. 👎

In the Controller, I now use
byte[] msg = Encoding.ASCII.GetBytes(mid.Pack());
instead of
byte[] msg = mid.PackBytes();
and it works. The Controller can send Rehit and the Integrator does receive it correctly.
But I have no idea, why you use totally different code for Pack() and PackBytes() - both should always create identical results (besides string vs byte array). Just delete the "public override byte[] PackBytes() {}" and it should work fine!? Same for Parse(). Just an idea, only you know...

I'm already working on it.,The reason of it is what you said 13 days ago, besides the use of a bit approach, it is always sent as an ASCII.
What happens is that when using PackBytes() it just creates the byte array in it's "pure form".
What it needs to do is to create the byte array, convert to an Int32 or Int64 value and then encode the generated number (padded left with 0) to ASCII again.
Mid0061 is already fixed, however I'm working on the same issue for Mid0065 now


Git flow Assignment Git flow Initialization

Git flow Assignment

Git flow initialized
Make sure the root branch is develop
Push from featurebranch to develop

Hi @Aoamos
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights
✔ Good commit history
✔Linters are passing
✔Very professional README
✔Correct Git flow
✔Descriptive PR
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


Desktop App: Implement translations

Moving to icebox per Daniel and Ilia. Translations for electron to be implemented later


Remove CentOS 6 from CI

Overview
According to https://wiki.centos.org/FAQ/General#What_is_the_support_.27.27end_of_life.27.27_for_each_CentOS_release.3F  CentOS 6 is now discontinued. This PR removes it from the matrix.
Testing recommendations

Related Issues or Pull Requests

Checklist

[ ] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

+1 from me but would like @wohali or @janl to comment also before we merge.

I'll wait for @wohali or @janl 's +1 then before merging.

I'm generally +1, but would also defer to @wohali for a final call :)

@bessbd nice work discovering the bug, it was a rather tricky one!

Motivation: as I was working on #3394 to get a green main, I was looking at the broken https://github.com/apache/couchdb/blob/main/src/mango/test/17-multi-type-value-test.py#L28 and I've noticed it's green everywhere but centos6.
I fired up the centos6 image in docker and realized couchdb main always returns one row less there than in my native (osx) environment.
After a long tracing and debugging session (with plenty of help from @nickva - thanks for that!), we've discovered that
couch_ejson_compare:less([{[{<<"forename">>,<<"Eddie">>}]}], [{[{<<"ÿÿÿÿ">>,<<>>}]}]). always returns 1 on centos6 and -1 elswehere.
When I was trying to bump the libicu version, I've noticed the yum commands are failing so I looked up the supported centos versions and that's when I realized it's now discontinued - hence this PR.
@wohali : please let me know if I missed any important details here.


Raspberry Pi npm install failure

This seems similar to Issue 195, but is on a different OS.
I am running Raspbian GNU/Linux 8 (jessie), which is a flavor of Debian for Raspberry Pi.
When running "npm install mdns" I recieve the below build error:
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
See the attached log for more details:
2017-08-14T18_17_05_277Z-debug.txt

Noticed that the log and the debug.log do show the "same" contents. Here is the full error from execution:
pi@raspberrypi:~/nodejs/mdns $ npm install mdns

<EMAIL_ADDRESS>install /home/pi/nodejs/mdns/node_modules/mdns
node-gyp rebuild

make: Entering directory '/home/pi/nodejs/mdns/node_modules/mdns/build'
CXX(target) Release/obj.target/dns_sd_bindings/src/dns_sd.o
In file included from ../src/dns_sd.cpp:1:0:
../src/mdns.hpp:32:20: fatal error: dns_sd.h: No such file or directory
#include <dns_sd.h>
^
compilation terminated.
dns_sd_bindings.target.mk:150: recipe for target 'Release/obj.target/dns_sd_bindings/src/dns_sd.o' failed
make: *** [Release/obj.target/dns_sd_bindings/src/dns_sd.o] Error 1
make: Leaving directory '/home/pi/nodejs/mdns/node_modules/mdns/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Linux 4.9.35-v7+
gyp ERR! command "/usr/bin/nodejs" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/pi/nodejs/mdns/node_modules/mdns
gyp ERR! node -v v8.2.1
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/home/pi/nodejs/mdns/package.json'
npm WARN mdns No description
npm WARN mdns No repository field.
npm WARN mdns No README data
npm WARN mdns No license field.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!
Based on Stack Overflow's [error: dns_sd.h: No such file or directory] I needed to install the below packages.
(https://stackoverflow.com/questions/19585117/error-dns-sd-h-no-such-file-or-directory)
sudo apt-get install libavahi-compat-libdnssd-dev

No cigar.  Mine failed with same error, later ubuntu version. Had already installed libavahi-compat-libdnssd-dev

Guys.. I am facing the same issue
I am trying to push a node js application into Heroku. I am facing the below error "
n file included from ../src/dns_sd.cpp:1:0:
../src/mdns.hpp:32:20: fatal error: dns_sd.h: No such file or directory
#include <dns_sd.h>
^
compilation terminated.
make: *** [Release/obj.target/dns_sd_bindings/src/dns_sd.o] Error 1
make: Leaving directory /tmp/app/node_modules/mdns/build' gyp ERR! build error gyp ERR! stack Error: make` failed with exit code: 2
I have made changes already like adding the dependencies libavahi-compat-libdnssd-dev but still facing the error. Can you please suggest what would be the possible reason?

Where do I find that dns_sd.h on Windows?


feat(runtime-config): export applyEnv and getEnv from nitropack

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This adds applyEnv and getEnv as exports, as well as allowing them to operate on a custom env object instead of reading from process.env.
This would be a useful feature for frameworks to provide more accurate runtimeConfig at the build time. For example, we already inline these utils in https://github.com/nuxt/test-utils/pull/655 and would need to do the same for https://github.com/nuxt/nuxt/issues/24224 and https://github.com/nuxt/nuxt/issues/26960.
Of course, that would be fine, but it would nicer to be able to simulate the runtime behaviour of Nitro with these utils, as suggested in https://github.com/nuxt/nuxt/issues/24224#issuecomment-1805495557.
I've included them in the core nitropack export but I would almost prefer nitropack/utils, perhaps, to allow space for other similar utilities?
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks for PR ❤
I think this needs to be more thorough thinking before exposing from nitro runtime utils. Diverging env definition one more level and allowing to complete override process.env (while for use cases like testing makes sense) can open the door to many more mistakes and implicit issues.
Therefore putting PR on draft state until have a chance to think more.

I've included them in the core nitropack export but I would almost prefer nitropack/utils, perhaps, to allow space for other similar utilities?

A shared (build time) util via nitropack/kit seems good idea.

As another alternative approach we can think of different methods of runtime config expansion (via a util that is not env) so we don't add over complexity to "environment" definition but also allow a runtime method of customizing runtime config (this would help to somehow replicate "app config" extendability feature that is going to be deprecated in Nitro v3). I love to hear your thoughts about this method.

It would be nice not to hard-code the use of process.env, even if we don't change Nitro runtime behaviour - it would help to avoid duplicating these utils (which would run the risk of diverging more quickly from Nitro)...
Are you thinking that we just rename these utils so they aren't environment-focused (ie. naming for clarity), so they can be used generically for expansion, either as build-time kit utils or within the runtime (first in env, and then later with new feature)? That sounds very doable - say the word and I'll refactor.
If you think this isn't so straightforward, I think we can go ahead and inline the existing nitropack behaviour in Nuxt for v3.12 like we did for nuxt/test-utils, and then take the time to get it right here, and switch downstream to using a forward-consistent implementation when it ships in nitropack/kit...
What do you think?

Thanks for the clear explanation. Yes, what was in my mind was purely about build-time utilities. The change from process.env was not meant to affect runtime behaviour, as these utilities were not being exposed within the runtime/ utils. (But very happy with your proposed runtime override solution!)
(The kind of usage I'm imagining was normalising nuxt.options.runtimeConfig - but I guess your suggestion of useRuntimeConfig and updateRuntimeConfig might be safer. Is that your take?)

Splitting into subtopics:

Runtime updateRuntimeXConfig (~> https://github.com/unjs/nitro/issues/2427)
Build time utils (~> https://github.com/unjs/nitro/issues/2428)

I strongly belive in making a new source/definition of environment variables (as proposed with opts.env) in this PR can be unsafe but also understand there are cases like testing that need it and love to discuss how to approach it better.

Thank you! One more use-case to mention for allowing non-process environment is to 'preview' the resolved config if environment variables were to be set. I'm very much not proposing using arbitrary overrides for the actual resolved Nitro runtime config, just trying to make the underlying logic reusable in different contexts so it doesn't have to be duplicated.

to 'preview' the resolved config

I guess at least all common runtimes allow to modify environment variables by directly modifying their env object (Deno.env.set and process.env object) if testing the exact env mapping for runtime-config is a must-have in a scenario. (*) and it is safer because:
The benefit of keeping runtime-config behavior strictly based on one standard (**) source (that is what environment variable in the runtime context means) is, that environment variables and how runtime config implicitly works with them won't diverge in a project that uses both environment variables and runtime config, behavior is consistently same. It makes tests more reliable too and also build and runtime utils consistent.

(*) I understand that today, for testing modules with different sets of runtime configs, we strictly need this and also updating global env looks more hacky but let's consider that 1) updateRuntimeConfig didn't exist and 2) updating an env in test/dev isolates gives the safety reason above.
(**) Today, we have already multiple definitions of environment variables: Actual system environment variables (that itself yet has no unified standard to access), .env augmentations, bundler bundled environment variables and strange runtimes like cloudflare that combine them with other values and make accessible in second stages. I belive both us and test runners have to help to at least keep on these known things rather than introducing more ways to change env variable access behavior and small places like this where we have alternatives that helps to avoid adding new sources.

@danielroe Shall we close PR and later continue discussing about this perhaps after updateRuntimeConfig util landed? 🙏🏼

Closing for triage. Followup:

runtime: https://github.com/unjs/nitro/issues/2427
kit util: https://github.com/unjs/nitro/issues/2428


SPDX element/relationships should be defined for the container image being described

What would you like to be added:
At the moment, when scanning container image, SPDX documents contain the list of packages that the target contains.. However, there are no elements/relationships that link back to the container that is being described..
Why is this needed:
Without specifying the container image element/relationships to the packages, the ingestion of SPDX documents resulting in detached packages without being able to query for the containers that use the packages. It will be useful to be able to link them up so that queries can be made for finding entities that use certain packages.
Additional context:

Example graph without links to entity:

The SPDX element to be defined can reference the OCI PURL (https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#oci), and should include sha256 checksum of the container image when possible.

👋 Nice find @lumjjb!
As far as the core syft data model do you think this would be a new field for the packages?
https://github.com/anchore/syft/blob/91eece47ffc5de009dccb916458b9ba734864319/schema/json/schema-4.0.0.json#L864-L985
Right now we have a purl field that describes the package, but we could add a field that also describes the distro it's related to and then transpose that into the spdx relationship^1.
We could also codify this as a relationship, but I'm not sure what the parent would be in this case that all packages could be linked back to since we identify the distro as it's own separate field outside of the package list ^2

Note: I think adding an ID field to the source object might be the best bet here.
We could also potentially update relationships to be id:container:["id", "id", ...] so we don't have to generate every single new relationship.
This would give us a good basis for then incorporating 2.3 spdx when it's ready

@kzantow I think we can also use this issue to discuss changes to the core model that need to be made before we're ready to adopt this for spdx 2.3

Hi - pinging back on this issue! Wondering if this is something that is being worked on.. We are still special casing this in GUAC and likely will be doing a filter for SBOMs that don't have this soon going forward - as the heuristics are unreliable (sometimes files become containers and vice versa).


Certificate XXX was not found

I have just deployed the Function and tried the AddCertificate. But i always get af "Certificate XX was not found" error. I have looked trough your code but can´t seem to find why and where this is a problem.
There is no certificates ind either the App Service or in the Storage account. Where should the Certificate be stored?

@jacobmohl
Is Web App and App Service Plan in another Resource Groups?

Yes. The Function, storsge ect. from your deploy link is in its own Resource Group and the app service plan and web app is in another.

Currently, I'm working to solve the problem.

Fixed. Please read again a README.md


How to get the alpha video

python -m demo.video_matting.custom.run --video
I can run demo.video_matting.custom.run.py.But I want to get the alpha video , can you tell me how to get it?

Hi, thanks for your attention.
I hope you have address the problem.

@mayingwuhu Hi! How did you get the alpha matte for a full video? I appreciate the answer.


Fix sanity.external criu functional test failure

Updated packages of external functional-test for ubuntu 22 docker image
Related Issue: https://github.ibm.com/runtimes/infrastructure/issues/7688#issuecomment-54036796

Test Link: grinder 31429


[Doc] Modify description about max_error_number and max_filter_ratio in RL

Why I'm doing:

Description about max_error_number and max_filter_ratio in RL are not correct.

What I'm doing:

Error data rows will be loaded when there are too many error rows before the load job pauses.
Default value of max_filter_ratio is 1 when it's not set.

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

ignore backport check: 3.3.3


Enhance GPT system.role to be concise and precise with ROS distribution

Two things need to be considered here.

in general, system.role can be more described to be helpful assistant. i think we can add more descriptions for query and execute sub-commands.
when user can ros2 ai command, that means the workspace is already sourced. that said we can read the ROS distribution depends on user workspace, and that can be also added to system.role to get the more precise answer for the distribution that user is using.

New images are released https://hub.docker.com/repository/docker/tomoyafujita/ros2ai/tags?page=1&ordering=last_updated, closing


river water infinite source

Put 2 water sources on 3 blocks: source - nothing - source
"nothing" becomes a water source
do the same with river water
-> you have an infinite source of tears because it doesn't work

We could make river water a renewable liquid like normal water, although it would probably not be a good idea. River water was specifically coded not to be renewable since it is used to generate rivers, and sometimes, rivers fall down a hill or a small pile of nodes and their configuration would make river water sources multiply themselves and ruin the generated illusion of a river.

Yes, it's a normal feature and we won't add the infinite river water source for biome reasons

ok, then I will make a river with standard water.


VSCode ignoring tsconfig.json

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

VS Code Version: 1.63.2
OS Version: Windows 10 20H2

Steps to Reproduce:
VSCode (apparently) completely ignores the tsconfig.json file, for some reason. As a result, the "paths" defined in the TypeScript settings file at the root of the project are not resolved, and errors are generated (specifically, TS:2307).
Initially I thought it might be related to #21473, but I realized the problem was occurring because the VSCode is installed on one disk (C:/), and the project is on another (H:/). To reach this conclusion, I made tests by copying the project to the C:/ disk, with exactly the same structure, and the problem no longer exists.
The obvious solution that many can point me to is to simply create the projects on the C:/ disk. However, for reasons of free disk space, I have no way to do that (that's exactly why I installed the new disk).
I tried multiple solutions (even modifying the NPM directory), and none worked.
Any light?

Please share a small project that demonstrates the issue and share steps to reproduce this issue using it


protocol/packp: add report status message.

Current coverage is 76.64% (diff: 90.90%)

Merging #162 into master will decrease coverage by 0.60%

@@             master       #162   diff @@
==========================================
  Files            83         85     +2
  Lines          5330       5394    +64
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4117       4134    +17
- Misses          756        808    +52
+ Partials        457        452     -5

Powered by Codecov. Last update 8f9cdf3...842b47d


Update sbt-scoverage to 2.1.1

About this PR
📦 Updates org.scoverage:sbt-scoverage from 2.0.12 to 2.1.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scoverage", artifactId = "sbt-scoverage" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, commit-count:1

coverage: 44.828%. remained the same
when pulling 46266338d49b4780abb2bbdb07403f3ee8bb39cb on scala-steward:update/sbt-scoverage-2.1.1
into 20266a3209b02490ae17fae8a036e173a6130e96 on evolution-gaming:master.

Superseded by #130.


azure-portworx-credentials.sh fails silently

script fails silently, see attached screenshot running script and running manually

Script fails silently here if the user does not have permission to create a role:
https://github.com/cloud-native-toolkit/automation-solutions/blob/main/boms/software/maximo/files/azure-portworx-credentials.sh#L111
User is required to have permission to create role & service principal.

Tracking issue created on azure-portworx module: https://github.com/cloud-native-toolkit/terraform-azure-portworx/issues/23

@triceam may also want to add in the readme azure user permissions necessary to run the script


Python Transpilation Engine

Allows for skol code to be transpiled into valid Python 3 code. This would merely be a way to neatly represent the AST as Python code, but also allows Python interop.
The Python engine in its current state is not the best, and is missing most of the API that would make Python interop fully possible.

Note that the Python engine is updated alongside the actual language, and a feature of the language exists due to the need for better Python interop. Python will become a real-deal transpilation target.


Hamburger icon

Hi, i've just started to implement your library but i've come across an issue that you may not be aware of.
The hamburger icon doesn't change from the back to the hamburger icon if you swipe the nav drawer away.
And i also have a feature request
You can't flagnoextractui (I know its little but i'd prefer to have it)
So please take these into consideration,
Thanks,
Jack

I'm sorry i found out how to fix all issues i'm sorry 👍

Hi @JackRoberts1998,
I am glad you found a solution, the sample app demonstrates how to deal with the nav hamburger.
Regards,

I do apologise, your library is amazing !!!!!!!!!!!!!!!


RSS failed

One recipe makes RSS and ATOM dead.

Yup. Looks like that character will have to be escaped with: &.

Why not make script for thats?


Adds more logging to test_unique_host_constraint

Changes proposed in this PR

adding a log with the number of total, running, and waiting jobs

Why are we making these changes?
To make failures easier to track down.

Build is red


Check dependencies license compatibility using fossa

What's needed?
We need to make sure the dependencies we use have a compatible license. We should try fossa.

Superseded by https://github.com/frequenz-floss/frequenz-repo-config-python/issues/26


improve readability

Pull Request Description:
Improve readability
Checklist before requesting a review:

[ ] I set the appropriate labels on the pull request.
[ ] I have added/updated the release note draft (if necessary).
[ ] I have updated the documentation to reflect my changes (if necessary).
[ ] All function and files are well documented.
[ ] All function and classes have type hints.
[ ] There is a licenses in all file.
[ ] The function and variable names are informative.
[ ] I have checked for code duplications.
[ ] I have added new unittest (if necessary).

@irenaby
Please specify the main changes made in this PR in the PR description.


Error loading the plugin after updating Strapi to version 4.17.1

Error loading the plugin after updating Strapi to version 4.17.1
Description:
After updating Strapi to version 4.17.1, I encountered the following error when trying to use the strapi-plugin-publisher:
Environment

Plugin Version: 1.5.7
Strapi Version: 4.17.1
Node.js Version: 20.10.0
Operating System: macOS
yarn: 1.22.19

Steps to Reproduce:

Install strapi-plugin-publisher version 1.5.7.
Update Strapi to version 4.17.1.
Start Strapi and observe the error.

Error Message
Error: Error loading the plugin publisher because publisher is not installed. Please either install the plugin or remove it's   ││   configuration.                                                                                ││   at Object.loadPlugins                                                                                ││   at Object.loadPlugins

I am unable to reproduce this issue with the steps provided. This issue is most likely due to your specific setup as opposed to his plugin. Closing this issue for now, happy to re-open if this turns out to not be the case.


AdaGrad rule yields NaN on first iteration

I'm running simple tests on small networks for variety of learning rules, and am finding that AdaGrad currently doesn't seem to work on a couple machines I tried:
Traceback (most recent call last):
  File "pylearn2/training_algorithms/sgd.py", line 470, in train
    raise Exception("NaN in " + param.name)
Exception: NaN in Output_Linear_W

This is a linear layer, but the problem seems to happen regardless of layer type.
Am I missing anything?

Try using NanGuard to detect where it gets produced? Be aware of #1465.

OK, here's what I got:
Traceback (most recent call last):
  ...
  File "pylearn2/pylearn2/training_algorithms/sgd.py", line 455, in train
    self.sgd_update(*batch)
  File "/Library/Python/2.7/site-packages/theano/compile/function_module.py", line 595, in __call__
    outputs = self.fn()
  File "/Library/Python/2.7/site-packages/theano/gof/link.py", line 797, in f
    raise_with_op(node, *thunks)
  File "/Library/Python/2.7/site-packages/theano/gof/link.py", line 795, in f
    wrapper(i, node, *thunks)
  File "/Library/Python/2.7/site-packages/theano/gof/link.py", line 810, in wrapper
    f(*args)
  File "pylearn2/pylearn2/devtools/nan_guard.py", line 105, in nan_check
    do_check_on(x, node, fn, False)
  File "pylearn2/pylearn2/devtools/nan_guard.py", line 84, in do_check_on
    assert False
AssertionError:
Apply node that caused the error: Elemwise{true_div,no_inplace}(DimShuffle{x,x}.0, Elemwise{sqrt,no_inplace}.0)
Inputs types: [TensorType(float32, (True, True)), TensorType(float32, matrix)]
Inputs shapes: [(1, 1), (16, 4)]
Inputs strides: [(4, 4), (16, 4)]
Inputs values: [array([[ -9.99999997e-07]], dtype=float32), 'not shown']

Backtrace when the node is created:
  File "pylearn2/pylearn2/training_algorithms/learning_rule.py", line 367, in get_updates
    * grads[param])

HINT: Use the Theano flag 'exception_verbosity=high' for a debugprint and storage map footprint of this apply node.

So it looks like it's specifically the delta_x_t of AdaGrad that's causing the issue.  Maybe the T.sqrt() returns zero, or less likely, new_sum_squared_grad is negative?

I would set a breakpoint in NanGuard and investigate from there. See if
it's either the input or the output, etc.
If your not inclined to do that yourself I would appreciate a simple
example that I can test myself (yaml, dataset) because I'm a bit short on
time.
In the long run some form of tool to run the graph step by step would be
useful, at least for me.
I intend to implement it as soon as possible. Not today, tho.

So, new_sum_squared_grad is zero and T.sqrt() returns zero also and the division on line 366 yields NaN.  Adding an epsilon in either of those places (1E-9) hacks around the problem.
The case I'm testing is doing training on a test dataset that's created as follows:
a_in, a_out = numpy.zeros((8,16)), numpy.zeros((8,4))

If I change this to the following, it works fine (no NaN and no hacks required):
a_in, a_out = numpy.ones((8,16)), numpy.zeros((8,4))

There's something about entirely zero inputs to the layer that causes the NaN.  I understand this is very unlikely to happen in a real dataset, but it could cause rare instabilities?

I'm baffled by this line
https://github.com/lisa-lab/pylearn2/blob/master/pylearn2/training_algorithms/learning_rule.py#L335
.
Anyway, for new_sum_squared_grad  to be 0 both sum_square_grad and
T.sqr(grads[param]) need to be 0.
sum_square_grad seems to be set to 0 by that funky line
grads[param]) - the gradient for the parameter - may very well be 0
Disclaimer - i'm just looking at learning_rule.py. I should be reading the
paper http://www.magicbroom.info/Papers/DuchiHaSi10.pdf.

Right, having a minimum value for the denominator would be a good idea for these corner cases. Something like T.maximum(eps, T.sqr(...)), where eps could be an additional parameter of AdaGrad.
@TNick This line defines how the update rule should build an expression for the parameter updates, starting from the gradients, learning rates, and individual learning rate multipliers for each parameter.
It also returns updates for the persistent shared variables that track moving averages for the gradients.
That function is executed only once, to build the symbolic graph computing that expression, so sum_square_grad is initialized to 0, but will be updated each time the training function gets executed.

Thanks Pascal.
The param.get_value() * 0. inside sharedX didn't made sense to me. I
understand now that the purpose is to create a shared variable of the same
shape as param.get_value() but initialized to 0.
You want me to make a pull request?
def get_updates(self, learning_rate, grads, lr_scalers=None, eps=1e-7):
    """
    Compute the AdaGrad updates
    Parameters
    ----------
    learning_rate : float
        Learning rate coefficient.
    grads : dict
        A dictionary mapping from the model's parameters to their
        gradients.
    lr_scalers : dict
        A dictionary mapping from the model's parameters to a learning
        rate multiplier.
    eps : float, optional
        TODO
    """
    updates = OrderedDict()
    for param in grads.keys():

        # sum_square_grad := \sum g^2
        sum_square_grad = sharedX(param.get_value() * 0.)

        if param.name is not None:
            sum_square_grad.name = 'sum_square_grad_' + param.name

        # Accumulate gradient
        new_sum_squared_grad = (
            sum_square_grad + T.sqr(grads[param])
        )

        # Compute update
        epsilon = lr_scalers.get(param, 1.) * learning_rate
        delta_x_t = (- epsilon / T.maximum(eps,

T.sqrt(new_sum_squared_grad))
* grads[param])
        # Apply update
        updates[sum_square_grad] = new_sum_squared_grad
        updates[param] = param + delta_x_t

    return updates

2015-04-24 0:03 GMT+03:00 Pascal Lamblin<EMAIL_ADDRESS>
Right, having a minimum value for the denominator would be a good idea for
these corner cases. Something like T.maximum(eps, T.sqr(...)), where eps
could be an additional parameter of AdaGrad.
@TNick https://github.com/TNick This line defines how the update rule
should build an expression for the parameter updates, starting from the
gradients, learning rates, and individual learning rate multipliers for
each parameter.
It also returns updates for the persistent shared variables that track
moving averages for the gradients.
That function is executed only once, to build the symbolic graph computing
that expression, so sum_square_grad is initialized to 0, but will be
updated each time the training function gets executed.
—
Reply to this email directly or view it on GitHub
https://github.com/lisa-lab/pylearn2/issues/1496#issuecomment-95718192.

@TNick Thanks, a pull request along those lines would be great. Actually, rather than defining eps as an additional parameter, I think it would be better to use max_scaling as 1. / eps, as defined in RMSProp. That way, the interfaces of different rules are more consistent with each other.
The PR should also have a unit test that checks that no NaN gets generated in the case reported by @alexjc

Done in #1521.


Remove references to v prefix in Helm charts

Issue #, if available:
https://github.com/aws-controllers-k8s/community/issues/1715
Description of changes:
This pull request removes all references to Helm charts and images prefixed with v. Some parts of the release documentation still reference mono-repo style image tags. For those parts, I have partially updated them where applicable, but honestly they need to be entirely rewritten to match the current structure.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/hold
Deploying all v prefix changes at once

/unhold
All container images and Helm charts have been republished. Documentation changes are ready to go live

/lgtm


Console spam

We are getting spam in our server log with the following. We were able to trace it back to this mod and were wondering if there is a fix or way to turn it off.
[de.npe.gameanalytics.EventHandler$RESTHelper:sendData:209]: Failed to send analytics event data. Result of attempt: ok | Authentication hash used: a9b26ba7c8b91f892782818e8b7a99f6 | Data sent: [{"install_publisher":"1.8.0_73-b02","install_site":"server","install_campaign":"3.13.0-24-generic","install_adgroup":"amd64","install_ad":"Linux","install_keyword":"server","user_id":"server8074143c-0261-48a1-91f8-c48a9f1af099","session_id":"1456228982764","build":"591"}]
https://pastebin.com/TvxTEBEd
EDIT: We have also turned the config option for sending analytic to false and still spams

Try disabling Mc snooper. GA framework should disable itself if MC snooper is disabled.
There seems to be two bugs:

in the framework: fixed in https://github.com/NPException/GameAnalyticsAPI/commit/822ecb9f7a45adb2251dfdecdf2edb490359759f
https://github.com/pixlepix/Aura-Cascade/blob/master/src/main/java/pixlepix/auracascade/main/Config.java#L59 analytics variable is never set when reading config file

Why is there an analytics tool anyway? It seems very shady to include one in a mod.

@Galaxy2Alex Thats what I am thinking
@jikuja I have disabled MC snooper and resolved the issue.

Please reopen. Bugs are still in the code.

@Galaxy2Alex, just to lower your concerns: My GA-API that is used by aura cascade enables mods (or any game written in Java) to send anonymous usage data to GameAnalytics. No information about your MC account is ever sent to them. The only data that is collected anonymously by default are:

Language that is set in MC
Type of CPU
Name of Operating System
Version of Operating System
Java version used
Whether this is a Client or a Server

Mods can queue the sending of additional events, in case of Aura Cascade for example (if I interpret the code correctly):

A quest was finished
An Aura of some color was used

As far as I can tell, Aura Cascade is not sending anything that could be directly linked to users MC account, and is therefore collecting only anonymous data.


Add support for feature flags in Prometheus

Related to prometheus-operator/prometheus-operator#3878

Personally, I'd vote for allowing the enable-feature param just for the sake of cleaner code 🙂. Since we already use those parameters for other things today, I do prefer:
local kp =
  (import 'kube-prometheus/main.libsonnet') +
  {
  values+:: {
    prometheus+: {
      externalLabels: {
        cluster: 'cluster-1',
        enableFeature: ['feature-x', 'feature-y'],
      },
    },
  },
};

{ ['prometheus-' + name]: kp.prometheus[name] for name in std.objectFields(kp.prometheus) }

Over:
local kp =
  (import 'kube-prometheus/main.libsonnet') +
  {
  values+:: {
    prometheus+: {
      externalLabels: {
        cluster: 'cluster-1',
      },
    },
  },
  prometheus+: {
    prometheus+: {
      spec+: {
        enableFeatures: ['feature-x', 'feature-y']
      },
    },
  },
};

{ ['prometheus-' + name]: kp.prometheus[name] for name in std.objectFields(kp.prometheus) }

Since there is interest in this, let's revive this feature.

I am happy to continue with the work here. I have reopened the PR and rebased it on main. LMK if anything else in needed :)

Thanks! :100:


WIP: maybe measure then set

probably not due this, just update the value instead, but its possible to go this route

too much code for now


KAFKA-16944: Rewrite Range Assignor

The server side range assignor was made to be sticky i.e. partitions from the existing assignment are retained as much as possible. During a rebalance, the expected behavior is to achieve co-partitioning for members that are subscribed to the same set of topics with equal number of partitions.
However, there are cases where this cannot be achieved efficiently with the current algorithm. There is no easy way to implement stickiness and co-partitioning and hence we have resorted to recomputing the target assignment every time.
In case of static membership, instanceIds are leveraged to ensure some form of stickiness.

TargetAssignmentBuilderBenchmark.build            100                         10           100  avgt    5    0.432 ± 0.003  ms/op
TargetAssignmentBuilderBenchmark.build            100                         10          1000  avgt    5    4.139 ± 0.013  ms/op
TargetAssignmentBuilderBenchmark.build           1000                         10           100  avgt    5    4.332 ± 0.049  ms/op
TargetAssignmentBuilderBenchmark.build           1000                         10          1000  avgt    5   43.449 ± 0.058  ms/op
TargetAssignmentBuilderBenchmark.build          10000                         10           100  avgt    5   47.766 ± 0.389  ms/op
TargetAssignmentBuilderBenchmark.build          10000                         10          1000  avgt    5  487.833 ± 3.459  ms/op

Here are the results of the benchmarks based on the last commit:

Benchmark                                       (assignmentType)  (assignorType)  (isRackAware)  (memberCount)  (partitionsToMemberRatio)  (subscriptionType)  (topicCount)  Mode  Cnt    Score    Error  Units
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false            100                         10         HOMOGENEOUS           100  avgt    5    0.052 ±  0.001  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false            100                         10         HOMOGENEOUS          1000  avgt    5    0.454 ±  0.003  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false           1000                         10         HOMOGENEOUS           100  avgt    5    0.476 ±  0.046  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false           1000                         10         HOMOGENEOUS          1000  avgt    5    3.102 ±  0.055  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false          10000                         10         HOMOGENEOUS           100  avgt    5    5.640 ±  0.223  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false          10000                         10         HOMOGENEOUS          1000  avgt    5   37.947 ±  1.000  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false            100                         10       HETEROGENEOUS           100  avgt    5    0.172 ±  0.001  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false            100                         10       HETEROGENEOUS          1000  avgt    5    1.882 ±  0.006  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false           1000                         10       HETEROGENEOUS           100  avgt    5    1.730 ±  0.036  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false           1000                         10       HETEROGENEOUS          1000  avgt    5   17.654 ±  1.160  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false          10000                         10       HETEROGENEOUS           100  avgt    5   18.595 ±  0.316  ms/op
ServerSideAssignorBenchmark.doAssignment             INCREMENTAL           RANGE          false          10000                         10       HETEROGENEOUS          1000  avgt    5  172.398 ±  2.251  ms/op
JMH benchmarks done

Benchmark                                     (memberCount)  (partitionsToMemberRatio)  (topicCount)  Mode  Cnt   Score   Error  Units
TargetAssignmentBuilderBenchmark.build                  100                         10           100  avgt    5   0.071 ± 0.004  ms/op
TargetAssignmentBuilderBenchmark.build                  100                         10          1000  avgt    5   0.428 ± 0.026  ms/op
TargetAssignmentBuilderBenchmark.build                 1000                         10           100  avgt    5   0.659 ± 0.028  ms/op
TargetAssignmentBuilderBenchmark.build                 1000                         10          1000  avgt    5   3.346 ± 0.102  ms/op
TargetAssignmentBuilderBenchmark.build                10000                         10           100  avgt    5   8.947 ± 0.386  ms/op
TargetAssignmentBuilderBenchmark.build                10000                         10          1000  avgt    5  40.240 ± 3.113  ms/op
JMH benchmarks done

@dajac @rreddy-22 It looks like this commit broke the build since there were new tests added here a few hours before this was merged that were not updated to use the new MemberSubscriptionAndAssignmentImpl constructor.
Can we either revert this commit or publish a fix PR ASAP?

Ah, looks like a fix has been published: https://github.com/apache/kafka/pull/16526


v4 is not working for select environments

Placeholder issue to have details later
Some discussion: https://github.com/ember-template-imports/ember-template-imports/pull/206/

This is caused by https://github.com/emberjs/babel-plugin-ember-template-compilation/pull/30

This is resolved.
For folks who run in to any issues with v4, make sure that your lockfile states that babel-plugin-ember-template-compilation is at least 2.2.1.
if it's not, delete the entry in the lockfile and re-run your package manager -- that should update it.

To aid future searchers like me: the advice in this ticket fixed an issue for me when using ember-template-imports v4 in a v1 addon, imports referenced within template tags were removed / elided / missing.


Create Component for Top Repo Section

This section needs to be developed

please assign this issue

Hi @shiv35 have you had a chance to dive into the issue yet? Need any help?

Hey @thtauhid I would like to work on this issue. Please assign me.


python310Packages.jupyterlab: 3.6.3 -> 4.0.3

Description of changes
Changelog: https://github.com/jupyterlab/jupyterlab/blob/v4.0.3/CHANGELOG.md

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

closes https://github.com/NixOS/nixpkgs/pull/223814

jupyterlab-lsp is not yet compatible with 4.0.0

If that's the only failure I think we should merge anyway.
No other package depends on jupyterlab-lsp. Should it maybe use buildPythonApplication and be removed from python3Packages?

This was the first failure, so I stopped here. I'm running it again.

tensorflow 2.11.1 is also failing. 2.13.0 is out, so maybe this can just be bumped.
I'll let nixpkgs-review run and post the final result.

Result of nixpkgs-review pr 244209 run on x86_64-linux 1

40 packages marked as broken and skipped:

python310Packages.batchspawner
python310Packages.batchspawner.dist
python310Packages.dalle-mini
python310Packages.dalle-mini.dist
python310Packages.dockerspawner
python310Packages.dockerspawner.dist
python310Packages.elegy
python310Packages.elegy.dist
python310Packages.jupyterhub
python310Packages.jupyterhub-ldapauthenticator
python310Packages.jupyterhub-ldapauthenticator.dist
python310Packages.jupyterhub-systemdspawner
python310Packages.jupyterhub-systemdspawner.dist
python310Packages.jupyterhub-tmpauthenticator
python310Packages.jupyterhub-tmpauthenticator.dist
python310Packages.jupyterhub.dist
python310Packages.oauthenticator
python310Packages.oauthenticator.dist
python310Packages.treex
python310Packages.treex.dist
python311Packages.batchspawner
python311Packages.batchspawner.dist
python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.dockerspawner
python311Packages.dockerspawner.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.jupyterhub
python311Packages.jupyterhub-ldapauthenticator
python311Packages.jupyterhub-ldapauthenticator.dist
python311Packages.jupyterhub-systemdspawner
python311Packages.jupyterhub-systemdspawner.dist
python311Packages.jupyterhub-tmpauthenticator
python311Packages.jupyterhub-tmpauthenticator.dist
python311Packages.jupyterhub.dist
python311Packages.oauthenticator
python311Packages.oauthenticator.dist
python311Packages.treex
python311Packages.treex.dist

6 packages failed to build:

python310Packages.jupyterlab-lsp
python310Packages.jupyterlab-lsp.dist
python311Packages.jupyterlab-lsp
python311Packages.jupyterlab-lsp.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist

165 packages built:

cantor (libsForQt5.cantor ,plasma5Packages.cantor)
ihaskell
jupyter
jupyter.dist
labplot
manim
manim.dist
python310Packages.ansible-kernel
python310Packages.ansible-kernel.dist
python310Packages.arviz
python310Packages.arviz.dist
python310Packages.augmax
python310Packages.augmax.dist
python310Packages.bambi
python310Packages.bambi.dist
python310Packages.einops
python310Packages.einops.dist
python310Packages.encodec
python310Packages.encodec.dist
python310Packages.experiment-utilities
python310Packages.experiment-utilities.dist
python310Packages.jupyter
python310Packages.jupyter-collaboration
python310Packages.jupyter-collaboration.dist
python310Packages.jupyter-contrib-core
python310Packages.jupyter-contrib-core.dist
python310Packages.jupyter-contrib-nbextensions
python310Packages.jupyter-contrib-nbextensions.dist
python310Packages.jupyter-lsp
python310Packages.jupyter-lsp.dist
python310Packages.jupyter-nbextensions-configurator
python310Packages.jupyter-nbextensions-configurator.dist
python310Packages.jupyter-server
python310Packages.jupyter-server-fileid
python310Packages.jupyter-server-fileid.dist
python310Packages.jupyter-server-mathjax
python310Packages.jupyter-server-mathjax.dist
python310Packages.jupyter-server.dist
python310Packages.jupyter-ydoc
python310Packages.jupyter-ydoc.dist
python310Packages.jupyter.dist
python310Packages.jupyterlab
python310Packages.jupyterlab-git
python310Packages.jupyterlab-git.dist
python310Packages.jupyterlab.dist
python310Packages.jupyterlab_launcher
python310Packages.jupyterlab_launcher.dist
python310Packages.jupyterlab_server
python310Packages.jupyterlab_server.dist
python310Packages.jupytext
python310Packages.jupytext.dist
python310Packages.k-diffusion
python310Packages.k-diffusion.dist
python310Packages.layoutparser
python310Packages.layoutparser.dist
python310Packages.mkdocs-jupyter
python310Packages.mkdocs-jupyter.dist
python310Packages.nbclassic
python310Packages.nbclassic.dist
python310Packages.nbdev
python310Packages.nbdev.dist
python310Packages.nbdime
python310Packages.nbdime.dist
python310Packages.nikola
python310Packages.nikola.dist
python310Packages.nix-kernel
python310Packages.nix-kernel.dist
python310Packages.notebook
python310Packages.notebook-shim
python310Packages.notebook-shim.dist
python310Packages.notebook.dist
python310Packages.notedown
python310Packages.notedown.dist
python310Packages.pdfplumber
python310Packages.pdfplumber.dist
python310Packages.phik
python310Packages.phik.dist
python310Packages.pymc
python310Packages.pymc.dist
python310Packages.pymoo
python310Packages.pymoo.dist
python310Packages.qgrid
python310Packages.qgrid.dist
python310Packages.unstructured-inference
python310Packages.unstructured-inference.dist
python310Packages.vector
python310Packages.vector.dist
python310Packages.vega
python310Packages.vega.dist
python310Packages.xarray-einstats
python310Packages.xarray-einstats.dist
python311Packages.ansible-kernel
python311Packages.ansible-kernel.dist
python311Packages.augmax
python311Packages.augmax.dist
python311Packages.einops
python311Packages.einops.dist
python311Packages.encodec
python311Packages.encodec.dist
python311Packages.experiment-utilities
python311Packages.experiment-utilities.dist
python311Packages.jupyter
python311Packages.jupyter-collaboration
python311Packages.jupyter-collaboration.dist
python311Packages.jupyter-contrib-core
python311Packages.jupyter-contrib-core.dist
python311Packages.jupyter-contrib-nbextensions
python311Packages.jupyter-contrib-nbextensions.dist
python311Packages.jupyter-lsp
python311Packages.jupyter-lsp.dist
python311Packages.jupyter-nbextensions-configurator
python311Packages.jupyter-nbextensions-configurator.dist
python311Packages.jupyter-server
python311Packages.jupyter-server-fileid
python311Packages.jupyter-server-fileid.dist
python311Packages.jupyter-server-mathjax
python311Packages.jupyter-server-mathjax.dist
python311Packages.jupyter-server.dist
python311Packages.jupyter-ydoc
python311Packages.jupyter-ydoc.dist
python311Packages.jupyter.dist
python311Packages.jupyterlab
python311Packages.jupyterlab-git
python311Packages.jupyterlab-git.dist
python311Packages.jupyterlab.dist
python311Packages.jupyterlab_launcher
python311Packages.jupyterlab_launcher.dist
python311Packages.jupyterlab_server
python311Packages.jupyterlab_server.dist
python311Packages.jupytext
python311Packages.jupytext.dist
python311Packages.layoutparser
python311Packages.layoutparser.dist
python311Packages.mkdocs-jupyter
python311Packages.mkdocs-jupyter.dist
python311Packages.nbclassic
python311Packages.nbclassic.dist
python311Packages.nbdev
python311Packages.nbdev.dist
python311Packages.nbdime
python311Packages.nbdime.dist
python311Packages.nikola
python311Packages.nikola.dist
python311Packages.nix-kernel
python311Packages.nix-kernel.dist
python311Packages.notebook
python311Packages.notebook-shim
python311Packages.notebook-shim.dist
python311Packages.notebook.dist
python311Packages.notedown
python311Packages.notedown.dist
python311Packages.pdfplumber
python311Packages.pdfplumber.dist
python311Packages.qgrid
python311Packages.qgrid.dist
python311Packages.unstructured-inference
python311Packages.unstructured-inference.dist
python311Packages.vega
python311Packages.vega.dist
quarto
sage
sageWithDoc
tts
tts.dist
wolfram-notebook

Ok, I might not be interpreting nixpkgs-review output correctly, but anyway here is what I got about tensorflow:
ERROR: /build/source/tensorflow/python/eager/BUILD:18:11: Compiling tensorflow/python/eager/pywrap_tfe_src.cc failed: (Exit 1): gcc failed: error executing command /nix/store/bxic6j2whyg3z4h2x>
In file included from tensorflow/python/eager/pywrap_tfe_src.cc:55:
./tensorflow/python/util/stack_trace.h: In static member function 'static tensorflow::StackTrace tensorflow::StackTrace::Capture(int)':
./tensorflow/python/util/stack_trace.h:64:55: error: 'PyThreadState' {aka 'struct _ts'} has no member named 'frame'; did you mean 'cframe'?
   64 |     const PyFrameObject* frame = PyThreadState_GET()->frame;
      |                                                       ^~~~~
      |                                                       cframe
./tensorflow/python/util/stack_trace.h:66:56: error: invalid use of incomplete type 'const PyFrameObject' {aka 'const struct _frame'}
   66 |     for (; i < limit && frame != nullptr; frame = frame->f_back, ++i) {
      |                                                        ^~
In file included from bazel-out/k8-opt/bin/external/local_config_python/python_include/Python.h:42,
                 from ./tensorflow/python/lib/core/numpy.h:35,
                 from ./tensorflow/python/eager/pywrap_tensor.h:21,
                 from tensorflow/python/eager/pywrap_tfe_src.cc:51:
bazel-out/k8-opt/bin/external/local_config_python/python_include/pytypedefs.h:22:16: note: forward declaration of 'PyFrameObject' {aka 'struct _frame'}
   22 | typedef struct _frame PyFrameObject;
      |                ^~~~~~
./tensorflow/python/util/stack_trace.h:67:37: error: invalid use of incomplete type 'const PyFrameObject' {aka 'const struct _frame'}
   67 |       PyCodeObject* code_obj = frame->f_code;
      |                                     ^~
bazel-out/k8-opt/bin/external/local_config_python/python_include/pytypedefs.h:22:16: note: forward declaration of 'PyFrameObject' {aka 'struct _frame'}
   22 | typedef struct _frame PyFrameObject;
      |                ^~~~~~
At global scope:
cc1plus: note: unrecognized command-line option '-Wno-unknown-warning' may have been intended to silence earlier diagnostics

Ok, I might not be interpreting nixpkgs-review output correctly, but anyway here is what I got about tensorflow

Probably tensorflow isn't rebuilt by this PR. But k-diffusion is and fails because tensorflow does.

jupyter-server fails to build on aarch64-linux and aarch64-darwin but that's probably an issue with upstream

Result of nixpkgs-review pr 244209 run on x86_64-linux 1

42 packages marked as broken and skipped:

python310Packages.batchspawner
python310Packages.batchspawner.dist
python310Packages.dalle-mini
python310Packages.dalle-mini.dist
python310Packages.dockerspawner
python310Packages.dockerspawner.dist
python310Packages.elegy
python310Packages.elegy.dist
python310Packages.jupyterhub
python310Packages.jupyterhub-ldapauthenticator
python310Packages.jupyterhub-ldapauthenticator.dist
python310Packages.jupyterhub-systemdspawner
python310Packages.jupyterhub-systemdspawner.dist
python310Packages.jupyterhub-tmpauthenticator
python310Packages.jupyterhub-tmpauthenticator.dist
python310Packages.jupyterhub.dist
python310Packages.oauthenticator
python310Packages.oauthenticator.dist
python310Packages.treex
python310Packages.treex.dist
python311Packages.batchspawner
python311Packages.batchspawner.dist
python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.dockerspawner
python311Packages.dockerspawner.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.jupyterhub
python311Packages.jupyterhub-ldapauthenticator
python311Packages.jupyterhub-ldapauthenticator.dist
python311Packages.jupyterhub-systemdspawner
python311Packages.jupyterhub-systemdspawner.dist
python311Packages.jupyterhub-tmpauthenticator
python311Packages.jupyterhub-tmpauthenticator.dist
python311Packages.jupyterhub.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.oauthenticator
python311Packages.oauthenticator.dist
python311Packages.treex
python311Packages.treex.dist

16 packages failed to build:

python310Packages.experiment-utilities
python310Packages.experiment-utilities.dist
python310Packages.jupyter-contrib-nbextensions
python310Packages.jupyter-contrib-nbextensions.dist
python310Packages.jupyter-nbextensions-configurator
python310Packages.jupyter-nbextensions-configurator.dist
python310Packages.jupyterlab-lsp
python310Packages.jupyterlab-lsp.dist
python311Packages.experiment-utilities
python311Packages.experiment-utilities.dist
python311Packages.jupyter-contrib-nbextensions
python311Packages.jupyter-contrib-nbextensions.dist
python311Packages.jupyter-nbextensions-configurator
python311Packages.jupyter-nbextensions-configurator.dist
python311Packages.jupyterlab-lsp
python311Packages.jupyterlab-lsp.dist

153 packages built:

cantor (libsForQt5.cantor ,plasma5Packages.cantor)
ihaskell
jupyter
jupyter.dist
labplot
manim
manim.dist
python310Packages.ansible-kernel
python310Packages.ansible-kernel.dist
python310Packages.arviz
python310Packages.arviz.dist
python310Packages.augmax
python310Packages.augmax.dist
python310Packages.bambi
python310Packages.bambi.dist
python310Packages.einops
python310Packages.einops.dist
python310Packages.encodec
python310Packages.encodec.dist
python310Packages.jupyter
python310Packages.jupyter-collaboration
python310Packages.jupyter-collaboration.dist
python310Packages.jupyter-contrib-core
python310Packages.jupyter-contrib-core.dist
python310Packages.jupyter-lsp
python310Packages.jupyter-lsp.dist
python310Packages.jupyter-server
python310Packages.jupyter-server-fileid
python310Packages.jupyter-server-fileid.dist
python310Packages.jupyter-server-mathjax
python310Packages.jupyter-server-mathjax.dist
python310Packages.jupyter-server.dist
python310Packages.jupyter-ydoc
python310Packages.jupyter-ydoc.dist
python310Packages.jupyter.dist
python310Packages.jupyterlab
python310Packages.jupyterlab-git
python310Packages.jupyterlab-git.dist
python310Packages.jupyterlab.dist
python310Packages.jupyterlab_launcher
python310Packages.jupyterlab_launcher.dist
python310Packages.jupyterlab_server
python310Packages.jupyterlab_server.dist
python310Packages.jupytext
python310Packages.jupytext.dist
python310Packages.k-diffusion
python310Packages.k-diffusion.dist
python310Packages.layoutparser
python310Packages.layoutparser.dist
python310Packages.mkdocs-jupyter
python310Packages.mkdocs-jupyter.dist
python310Packages.nbclassic
python310Packages.nbclassic.dist
python310Packages.nbdev
python310Packages.nbdev.dist
python310Packages.nbdime
python310Packages.nbdime.dist
python310Packages.nikola
python310Packages.nikola.dist
python310Packages.nix-kernel
python310Packages.nix-kernel.dist
python310Packages.notebook
python310Packages.notebook-shim
python310Packages.notebook-shim.dist
python310Packages.notebook.dist
python310Packages.notedown
python310Packages.notedown.dist
python310Packages.pdfplumber
python310Packages.pdfplumber.dist
python310Packages.phik
python310Packages.phik.dist
python310Packages.pymc
python310Packages.pymc.dist
python310Packages.pymoo
python310Packages.pymoo.dist
python310Packages.qgrid
python310Packages.qgrid.dist
python310Packages.unstructured-inference
python310Packages.unstructured-inference.dist
python310Packages.vector
python310Packages.vector.dist
python310Packages.vega
python310Packages.vega.dist
python310Packages.xarray-einstats
python310Packages.xarray-einstats.dist
python311Packages.ansible-kernel
python311Packages.ansible-kernel.dist
python311Packages.augmax
python311Packages.augmax.dist
python311Packages.einops
python311Packages.einops.dist
python311Packages.encodec
python311Packages.encodec.dist
python311Packages.jupyter
python311Packages.jupyter-collaboration
python311Packages.jupyter-collaboration.dist
python311Packages.jupyter-contrib-core
python311Packages.jupyter-contrib-core.dist
python311Packages.jupyter-lsp
python311Packages.jupyter-lsp.dist
python311Packages.jupyter-server
python311Packages.jupyter-server-fileid
python311Packages.jupyter-server-fileid.dist
python311Packages.jupyter-server-mathjax
python311Packages.jupyter-server-mathjax.dist
python311Packages.jupyter-server.dist
python311Packages.jupyter-ydoc
python311Packages.jupyter-ydoc.dist
python311Packages.jupyter.dist
python311Packages.jupyterlab
python311Packages.jupyterlab-git
python311Packages.jupyterlab-git.dist
python311Packages.jupyterlab.dist
python311Packages.jupyterlab_launcher
python311Packages.jupyterlab_launcher.dist
python311Packages.jupyterlab_server
python311Packages.jupyterlab_server.dist
python311Packages.jupytext
python311Packages.jupytext.dist
python311Packages.layoutparser
python311Packages.layoutparser.dist
python311Packages.mkdocs-jupyter
python311Packages.mkdocs-jupyter.dist
python311Packages.nbclassic
python311Packages.nbclassic.dist
python311Packages.nbdev
python311Packages.nbdev.dist
python311Packages.nbdime
python311Packages.nbdime.dist
python311Packages.nikola
python311Packages.nikola.dist
python311Packages.nix-kernel
python311Packages.nix-kernel.dist
python311Packages.notebook
python311Packages.notebook-shim
python311Packages.notebook-shim.dist
python311Packages.notebook.dist
python311Packages.notedown
python311Packages.notedown.dist
python311Packages.pdfplumber
python311Packages.pdfplumber.dist
python311Packages.qgrid
python311Packages.qgrid.dist
python311Packages.unstructured-inference
python311Packages.unstructured-inference.dist
python311Packages.vega
python311Packages.vega.dist
quarto
sage
sageWithDoc
tts
tts.dist
wolfram-notebook

I'm not sure why these packages had been successfully built.
python310Packages.experiment-utilities
python310Packages.jupyter-contrib-nbextensions
python310Packages.jupyter-nbextensions-configurator
Should we patch them to use nbclassic or jupyter-server instead of notebook>=7?

Anyway, I think it should be reported upstream.

Anyway, I think it should be reported upstream.

https://github.com/ipython-contrib/jupyter_contrib_nbextensions/issues/1647
https://github.com/Jupyter-contrib/jupyter_nbextensions_configurator/issues/165

Should we patch them to use nbclassic or jupyter-server instead of notebook>=7?

Done. But I did not test their functionality.

Thanks! LGTM for x86_64-linux.
It seems to fail on aarch64-linux, I will check it later.