can't run mutation tests on epic/mutation-switching

replacing stuff in github actions i realised that you can't run mutation testing anymore... it seems there are some problems with configurations:
@stryker-mutator/core: 14:33:11 (2884) INFO ConfigReader Using stryker.conf.js
@stryker-mutator/core: 14:33:11 (2884) INFO ConfigReader Stryker can help you setup a `stryker.conf` file for your project.
@stryker-mutator/core: 14:33:11 (2884) INFO ConfigReader Please execute `stryker init` in your project's root directory.
@stryker-mutator/core: 14:33:11 (2884) ERROR Stryker Invalid config file. Inner error: TypeError: Cannot set property 'module' of undefined
@stryker-mutator/core: TypeError: Cannot set property 'module' of undefined
@stryker-mutator/core:     at Object.<anonymous> (/home/runner/work/stryker/stryker/packages/core/stryker.conf.js:4:27)
@stryker-mutator/core:     at Module._compile (internal/modules/cjs/loader.js:1137:30)
@stryker-mutator/core:     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
@stryker-mutator/core:     at Module.load (internal/modules/cjs/loader.js:985:32)
@stryker-mutator/core:     at Function.Module._load (internal/modules/cjs/loader.js:878:14)
@stryker-mutator/core:     at Module.require (internal/modules/cjs/loader.js:1025:19)
@stryker-mutator/core:     at require (internal/modules/cjs/helpers.js:72:18)
@stryker-mutator/core:     at ConfigReader.loadConfigModule (/home/runner/work/stryker/stryker/packages/core/src/config/ConfigReader.js:60:32)
@stryker-mutator/core:     at ConfigReader.readConfig (/home/runner/work/stryker/stryker/packages/core/src/config/ConfigReader.js:26:35)
@stryker-mutator/core:     at readConfig (/home/runner/work/stryker/stryker/packages/core/src/config/readConfig.js:7:25)

This part could be fixed by changing config file name or adding a parameter to CI. We handle  stryker.conf... but ours is stryker.parent.conf so it doesn't see it.
Then there is a problem with dashboard option in core.
it could be solved by
settings.dashboard ? settings.dashboard.module = moduleName : settings.dashboard = { module: moduleName };

then problem with build...
then with tests - i can't actually solve this one...
error messages:
@stryker-mutator/core: ><EMAIL_ADDRESS>stryker C:\Users\groszek\Desktop\GitHub\stryker\packages\core
@stryker-mutator/core: > node bin/stryker run
@stryker-mutator/core: 17:44:25 (4396) INFO ConfigReader Using stryker.conf.js
@stryker-mutator/core: 17:44:25 (4396) INFO BroadcastReporter Detected that current console does not support the "progress" reporter, downgrading to "progress-append-only" reporter
@stryker-mutator/core: 17:44:25 (4396) WARN InputFileResolver Globbing expression "typings/**/*.ts" did not result in any files.
@stryker-mutator/core: 17:44:25 (4396) INFO InputFileResolver Found 103 of 188 file(s) to be mutated.
@stryker-mutator/core: 17:44:27 (4396) INFO Instrumenter Instrumented 103 source file(s) with 2817 mutant(s)
@stryker-mutator/core: 17:44:27 (4396) INFO ConcurrencyTokenProvider Creating 2 checker process(es) and 2 test runner process(es).
@stryker-mutator/core: 17:44:41 (4396) INFO Sandbox Running build command "tsc -b && lerna run build" in the sandbox at "C:\Users\groszek\Desktop\GitHub\stryker\packages\core\.stryker-tmp\sandbox8835757".
@stryker-mutator/core: 17:44:41 (4396) INFO DryRunExecutor Starting initial test run. This may take a while.
@stryker-mutator/core: 17:44:42 (4396) INFO DryRunExecutor Initial test run succeeded. Ran 0 tests in 17 seconds (net 0 ms, overhead 5 ms).
@stryker-mutator/core: 17:44:42 (4396) ERROR Stryker No tests were executed. Stryker will exit prematurely. Please check your configuration.

Yeah, we didn't update the config for Running Stryker on Stryker. I've changed the title (because it's not an issue with Stryker itself).

@stryker-mutator/core: 17:44:41 (4396) INFO Sandbox Running build command "tsc -b && lerna run build" in the sandbox at "C:\Users\groszek\Desktop\GitHub\stryker\packages\core.stryker-tmp\sandbox8835757".

That command actually failed, but because you're running on windows and https://github.com/nicojs/node-link-parent-bin/issues/20 you don't notice it.
I think "tsc -b" should be the build command of a sub package.

@nicojs if you were able to fix it I would be grateful, these reports would be very handy in refactoring and optimising code :)

I'm running into this issue:
nicojs@nicoj03:~/stryker/packages/core$ npm run stryker

><EMAIL_ADDRESS>stryker /home/nicojs/stryker/packages/core
> node bin/stryker run

17:54:44 (15197) INFO ConfigReader Using stryker.conf.js
17:54:45 (15197) WARN InputFileResolver Globbing expression "typings/**/*.ts" did not result in any files.
17:54:45 (15197) INFO InputFileResolver Found 104 of 189 file(s) to be mutated.
17:54:49 (15197) INFO Instrumenter Instrumented 104 source file(s) with 2830 mutant(s)
17:54:50 (15197) INFO ConcurrencyTokenProvider Creating 2 checker process(es) and 2 test runner process(es).
17:54:52 (15197) INFO Sandbox Running build command "tsc -b" in the sandbox at "/home/nicojs/stryker/packages/core/.stryker-tmp/sandbox8668860".
17:55:16 (15197) ERROR Stryker an error occurred Error: Command failed with exit code 1: tsc -b
src/config/OptionsValidator.ts(108,13): error TS2578: Unused '@ts-expect-error' directive.
src/config/OptionsValidator.ts(120,312): error TS2578: Unused '@ts-expect-error' directive.
    at makeError (/home/nicojs/stryker/packages/core/node_modules/execa/lib/error.js:59:11)
    at handlePromise (/home/nicojs/stryker/packages/core/node_modules/execa/index.js:114:26)

We're getting the "unused @ts-expect-error" because the strip-comments dependency isn't stripping the comments for some reason. See  problem 3 discussed in #2438
We need to find another way to deal with typescript compile errors.


Hyperion APT repo is broken/docs have the wrong URL

[x] I confirm that this is an issue rather than a question.

Bug report
There appears to be an issue with the installation instructions on the main hyperion website, I run into errors when I add the repository and install as described.
Steps to reproduce
Go through the apt repo setup as normal, proceed to install step:
sudo apt-get update && sudo apt-get install hyperion
Hit:1 http://deb.debian.org/debian bookworm InRelease
Hit:2 http://deb.debian.org/debian-security bookworm-security InRelease
Hit:3 http://deb.debian.org/debian bookworm-updates InRelease
Hit:4 http://archive.raspberrypi.com/debian bookworm InRelease
Hit:5 https://apt.hyperion-project.org bookworm InRelease [1,838 B]
Fetched 1,838 B in 1s (1,287 B/s)
Reading package lists... Done
N: Skipping acquire of configured file 'main/binary-arm64/Packages' as repository 'https://apt.hyperion-project.org bookworm InRelease' doesn't support architecture 'arm64'
N: Skipping acquire of configured file 'main/binary-armhf/Packages' as repository 'https://apt.hyperion-project.org bookworm InRelease' doesn't support architecture 'armhf'
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package hyperion

EDIT: The fix: I confirmed that the issue is the given URL on every set of hyperion docs is incorrect. If you dive into the source code for Hyperbian, you see the actual APT url is https://apt.releases.hyperion-project.org, not https://apt.hyperion-project.org. This is the cause of the repo apparently not supporting any valid architecture.
What is expected?
I expected hyperion to install normally.
What is actually happening?
Hyperion fails to install as the repo claims it doesn't support arm64 or armhf.
System

> uname -a
Linux raspberrypi 6.1.0-rpi7-rpi-v8 #1 SMP PREEMPT Debian 1:6.1.63-1+rpt1 (2023-11-24) aarch64 GNU/Linux

> lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 12 (bookworm)
Release:	12
Codename:	bookworm

The repository in the documentation is the official current one. Unfortunately, at the time of the latest release there had been no Bookworm arm build environment available which is why you get the error.
We are in the process of reworking the apt repositoryand  this is why you already see the other url.
With the new repo Bookworm and Trixie will be a available for arm64 and armhf.

The repository and installation instructions are now up to date.


Limitations with Azure Firewall

[Enter feedback here]
Requesting the docs be updated to provide clearer limitations with Azure Firewall, stating something like below.

Azure Firewall Premium SKU is not supported
Azure Firewall with VMs deployed in peered VNETs is not supported
Azure Firewall Standard and VMs must be deployed in the same VNET

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0e7f551c-e02a-1a23-6d40-5b0b838f8929
Version Independent ID: 084e658e-1d11-d976-46ef-af93460f3ad8
Content: Understanding just-in-time virtual machine access in Azure Security Center
Content Source: articles/security-center/just-in-time-explained.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@seanstark
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @seanstark, thanks for the feedback and for taking the time to help us improve the docs.
This documentation page is going to get refreshed in the coming days, and we'll incorporate the information discussed here.
I'm going to close this issue, but feel free to comment again here or open a new one if you have additional requests.
#please-close


Splinter headed (i.e. with browser) does not retrieve results.

I've found that when you use the SPLINTER browserHandler, the headed version just does not work.
In fact, I discovered quite a few things, and after some debugging, got the following:
The browser is NOT signed in. Normally this is good, but as a result, you get the "Come here often?" box, which ruins everything. When you get the HTML, you only get the search bar and doodle, and this box...no results.
This is the 'x' button on the box, to quit it.
×
I'm not sure how to get around it...clicking the link seems to make no difference.

Fixed with v1.6.3


Update Excel.Hyperlinks.Delete.md

Add remarks similar to Range.ClearHyperlinks to note that formatting will be removed when invoking this method.

Docs Build status updates of commit 4815066:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Hyperlinks.Delete.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@IvenBach Thanks for your contribution. I made some editing tweaks; changes will be live tonight. Here's the final:
Calling the Delete method on the specified Hyperlinks object is equivalent to using both the Clear Hyperlinks and Clear Formats commands from the Clear drop-down list in the Editing section of the Home tab. Not only hyperlinks will be removed; cell formatting will be removed also. If you only want to remove the hyperlink, see the Range.ClearHyperlinks method.


Fix: make device config card easier to use

Problem
Editing fields in the Device Config card is confusing due to multiple checkmarks for changes.
Solution
Make it less confusing
Testing

Go to a LTE device page.
Try to edit a field and see how confusing it is.
Switch to this branch. (Note: I had to edit the package.json file to point to this branch on line 197 ("react-json-view"))
Compare the difference

Documentation
No documentation change needed, small internal change to visibility.
Jira
https://projecttools.nordicsemi.no/jira/browse/IRIS-4217

This isn't the nrf cloud project, please rewrite your PR request with that in mind.


refactor: enums

📝Changed routes:

/

(view full size)

Commit 51b3c42113b586cdc7565035cc5665c5b8090acd (https://secret-git-bu-sec-26-update-enums.bukinoshita.now.sh).


Doesn't work with pytest-asyncio

pytest-asyncio overrides pytest's collection, preventing pytest-cleanslate from protecting against state pollution.

It works with pytest-asyncio v0.21.1, but not with versions since.

Resolved in v1.0.2.


XML with Proxy is wrong

The "system.net" should be at the sam elevel as "<system.identityModel.services>" and not inside of "<system.identityModel.services>".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 626caf29-7e88-0b56-2013-5c5b7c2a5917
Version Independent ID: 2376192f-793d-0166-46c7-870cd0c41abb
Content: Hybrid Modern Authentication overview and prerequisites for using it with on-premises Skype for Business and Exchange servers
Content Source: Enterprise/hybrid-modern-auth-overview.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@FrankCarius Thank you for your feedback. As per system.net documentation it looks like the syntax is placed at the correct place. Please see https://docs.microsoft.com/en-us/dotnet/framework/configure-apps/file-schema/network/system-net-element-network-settings for more details.
Thanks
Imran

Not really. The XML in the doc has the hirarchy
<system.identityModel.services>
<system.net>

And <system.identityModel.services>  is below   A normal reader would now try to add the setting at:

But if you check https://docs.microsoft.com/en-us/dotnet/framework/configure-apps/file-schema/network/system-net-element-network-settings then is should be

My customer had it at

and it was not working. Removing the "  " solved the problem.
Or better.  Replace  with
i hope i explained it better now.

@FrankCarius Can you please cofirm that removing the <system.net> works as expected and solved the issue.

Sorry for the bad formatting. Github remove some of the <>-brackets . I try to write it again and hope is it readable now..
`The XML in the doc has the hierarchy
<system.identityModel.services>
<system.net>

And <system.identityModel.services>  is below the
My customer added it like that:

But if you check https://docs.microsoft.com/en-us/dotnet/framework/configure-apps/file-schema/network/system-net-element-network-settings then is should be direct below configuratoin

My customer had a configuration with

<system.identityModel.services>
<system.net>

and it was not working.
Removing the "<system.identityModel.services>" solved the problem.
Or better.  Replace <system.identityModel.services> with
i hope i explained it better now.
`

Sorry for the bad formatting. Github remove some of the <>-brackets . I try to write it again and hope is it readable now..
`The XML in the doc has the hierarchy
<system.identityModel.services>
<system.net>
<defaultProxy>
And <system.identityModel.services>  is below the <configuration>
My customer added it like that:
<configuration>
<system.identityModel.services>
<system.net>
<defaultProxy>
But if you check https://docs.microsoft.com/en-us/dotnet/framework/configure-apps/file-schema/network/system-net-element-network-settings then is should be direct below configuratoin
<configuration>
<system.net>
<defaultProxy>
My customer had a configuration with
<configuration>
<system.identityModel.services>
<system.net>
<defaultProxy>
and it was not working.
Removing the "<system.identityModel.services>" solved the problem.
Or better.  Replace <system.identityModel.services> with <configuration>
i hope i explained it better now.
`

@FrankCarius - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/commit/f665d3e7916b846aafe4ccd1ded789e28876bdc3.
Thank you for your contribution to make the docs better! Much appreciated!


We might have to dump Opus.. here's why!

This is long and open to discussion. I was thinking about modernization of the audio engine. The main main reasons why Godot's audio engine was written as it is were:
Traditionally, devices had little amount of memory until a few years ago. Additionally, decoding compressed audio by traditional libraries was CPU intensive and required allocating/deallocating memory (which is forbidden and must never be done in audio threads).
This combination of facts resulted in audio implementations where streaming audio (music) and samples (sound effects) had to be divided. Streaming happened from disk (OGG files in Godot are streamed from disk), while samples are played from memory, generally uncompressed.
The method for streaming music works like this:
-A thread decodes the audio from disk, then fills a ringbuffer
-The audio thread reads from the ringbuffer and writes audio
This method has the drawback that streaming or any kind of change to the audio results in high latency, and seeking, resampling, etc. is hard.
Nowadays computers and mobile devices have plenty of memory, so loading the ogg fles into memory and streaming them from there (just like you would do for regular sound effects) could probably be possible without much of a problem... however one problem remains:
Both libogg and libvorbisfile perform memory allocations during decode, which makes the library unsuitable for streaming from the audio thread.
So it crossed my head today, what if there was a way to stram vorbis, mp3 or any other format from memory directly? without allocating memory? nowadays you can have a full thread for this.. did no one think of this? We could unify the SAMPLE/STREAM classes in Godot if this was possible.
This is when I remembered stb_vorbis.c , which is an alternative implementation of Vorbis. I  discarded it back in the day because it lacked seeking support, so I went to check today and I found this:
// Originally sponsored by RAD Game Tools. Seeking sponsored
// by Phillip Bennefall, Marc Andersen, Aaron Baker, Elias Software,
// Aras Pranckevicius, and Sean Barrett.
Aha! Someone did this reasoning before, Aras is one of the leads of Unity, so I definitely was on the right track :)
So, for the new audio engine, my proposal is to ditch all audio formats that are RT Unsafe (Opus and Vorbis bia libvorbisfile) and simply use std_vorbis and wav (via ima-adpcm)  instead.  This would allow us to use AudioStream for pretty much everything and enormously simplify the audio engine.
The only reason we could still support Opus is due to the very good voice compression ratio (great or in-game voice), but we would not support it as an AudioStream (unless we can figure out a way to stream it in a realtime-safe mode somehow).
Any thoughts?

The only real problem I see is that abandoning the traditional sample playback code will make streaming have more latency in the HTML5 backend :| right now we are using a webaudio implementation of samples and it works pretty well..  having a single stream API will undoubtly force it to go via the regular HTML5 streaming, which is processed in the main thread (having more lag).
Eventually AFAIK after web assembly is out, browsers will add thread support for it soon, so this will be a thing of the past, but we are talking a year or more ahead..

@reduz If webassembly gets thread support eventually I'd go for the audio change. I'd rather design for the future than to maintain an old system if you're breaking compatibility anyway. The GLES3 change already leaves out older devices and the web export will at least work. What magnitudes of latency are you we talking about in the web version?
But then again I'm no audio expert..

probably around 100-200ms as opposed to the 15-50 you usually expect from a
game
On Tue, Jan 10, 2017 at 4:12 PM, Karroffel<EMAIL_ADDRESS>wrote:

@reduz https://github.com/reduz If webassembly gets thread support
eventually I'd go for the audio change. I'd rather design for the future
than to maintain an old system if you're breaking compatibility anyway.
The GLES3 change already leaves out older devices and the web export will
at least work. What magnitudes of latency are you we talking about in the
web version?
But then again I'm no audio expert..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7496#issuecomment-271668429,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z27mDfthVT-dSqHXJcKzYGVlpF1Gtks5rQ9gCgaJpZM4LfxKJ
.

basically take the minimum FPS you might find bottom barely acceptable for
a game (I don't know, 10f-15fps?), the audio will be processed between
frames, so you have to leave enough room for the audio to be sent to
webaudio.
On Tue, Jan 10, 2017 at 4:26 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

probably around 100-200ms as opposed to the 15-50 you usually expect from
a game
On Tue, Jan 10, 2017 at 4:12 PM, Karroffel<EMAIL_ADDRESS>wrote:

@reduz https://github.com/reduz If webassembly gets thread support
eventually I'd go for the audio change. I'd rather design for the future
than to maintain an old system if you're breaking compatibility anyway.
The GLES3 change already leaves out older devices and the web export will
at least work. What magnitudes of latency are you we talking about in the
web version?
But then again I'm no audio expert..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7496#issuecomment-271668429,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z27mDfthVT-dSqHXJcKzYGVlpF1Gtks5rQ9gCgaJpZM4LfxKJ
.

I know that a lot of people use streaming formats as sound effects, but how
common is it that these effects are longer than 1-2 seconds? In that case
we could just convert them to adpcm on export, and if they're too long
stream them, then you still need to to have 1 sound player node for both
types (so unify the SamplePlayer and StreamPlayer, etc into one) Long sound
files are usually used for voice or background music, in that case it's not
crucial to have low latency, so there's no need to sacrifice formats.
There might be some corner cases where a long sound file needs to be played
with no latency (maybe a very complex sound game), I'm not opposed to
supporting those by having the best implementation of the streaming code,
but I don't know how much it would duplicate code to support both the
threadable and non-threadable streaming..
On 10 January 2017 at 16:29, Juan Linietsky<EMAIL_ADDRESS>wrote:

basically take the minimum FPS you might find bottom barely acceptable for
a game (I don't know, 10f-15fps?), the audio will be processed between
frames, so you have to leave enough room for the audio to be sent to
webaudio.
On Tue, Jan 10, 2017 at 4:26 PM, Juan Linietsky<EMAIL_ADDRESS>wrote:

probably around 100-200ms as opposed to the 15-50 you usually expect from
a game
On Tue, Jan 10, 2017 at 4:12 PM, Karroffel<EMAIL_ADDRESS>wrote:

@reduz https://github.com/reduz If webassembly gets thread support
eventually I'd go for the audio change. I'd rather design for the future
than to maintain an old system if you're breaking compatibility
anyway.
The GLES3 change already leaves out older devices and the web export
will
at least work. What magnitudes of latency are you we talking about in
the
web version?
But then again I'm no audio expert..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/godotengine/godot/issues/7496#
issuecomment-271668429>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AF-Z27mDfthVT-
dSqHXJcKzYGVlpF1Gtks5rQ9gCgaJpZM4LfxKJ>
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7496#issuecomment-271673327,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPT1yh3dVEHr3eLZQ1LKeO0amMj9lks5rQ9wzgaJpZM4LfxKJ
.

May lip sync be an issue for choice dependent dialog voice files? May be It can be corrected easily by delaying audio before hand but it just crossed my mind when reading.

@reduz FYI me being one of sponsors for seeking implementation has nothing whatsoever to do with Unity (otherwise it would be "Unity" in the list). I paid my personal money, because I thought that would be a good thing to do. Unity does not use stb_vorbis.

@aras-p Thanks for the clarification! And since you are here, do you think it would be possible to ask you how does Unity decode OGG audio data and sends it to the mixing thread?

@reduz I have no idea. I know we use FMOD as the audio engine, but whether Vorbis decoding is done by FMOD itself, or some other piece of code, I don't know...

What you seem to be missing is that Opus itself does not allocate memory, it's the container parser opusfile that allocates. And if you're dealing with containers, you're already not real time.
So maybe, if allocation is so deathly bad for the mixing process, pre-walk the entire Ogg stream into pre-sized arrays of packet sizes and packet pointers, and hand those packets off to libopus yourself instead of using opusfile?
I mean, cripes, talking about transcoding to ADPCM at the build stage, just to spare CPU cycles? If, then, what is the point of even supporting high compression lossy formats on input? There is no point of even using high compression lossy formats at the authoring stage, they're designed for distribution. Authoring should be in lossless formats, compression optional.

Yeah I don't like that people are using ogg or mp3 for sound effects, but
that seems to be the problem here..
On 29 January 2017 at 19:42, Christopher Snowhill<EMAIL_ADDRESS>wrote:

What you seem to be missing is that Opus itself does not allocate memory,
it's the container parser opusfile that allocates. And if you're dealing
with containers, you're already not real time.
So maybe, if allocation is so deathly bad for the mixing process, pre-walk
the entire Ogg stream into pre-sized arrays of packet sizes and packet
pointers, and hand those packets off to libopus yourself instead of using
opusfile?
I mean, cripes, talking about transcoding to ADPCM at the build stage,
just to spare CPU cycles? If, then, what is the point of even supporting
high compression lossy formats on input? There is no point of even using
high compression lossy formats at the authoring stage, they're designed for
distribution. Authoring should be in lossless formats, compression optional.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7496#issuecomment-275952476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPahCNtZEOosXUgjiqbnX4d0X1X-fks5rXRXngaJpZM4LfxKJ
.

@kode54 I don't think pre-parsing the opus block from the container to memory would be a problem, but I'm not sure that Opus does not allocate memory during playback, I think that's probably not the case, but have no idea how to find information about that.
@punto- No, the problem is that having two separate paths for playing audio, one for SFX and one for music is just a ton more code to maintain. Merging them is having a lot less code.

So I can confirm that libopus never ever allocates memory in anywhere outside of the functions with _create() in the name. That means it'll never do memory allocation during encoding or decoding. Opus is designed to run in real-time even in tiny DSPs, so it's very careful about allocation. In fact, you can even compile it without libc!
That leaves Ogg. I believe that the current libogg may do some buffer resizing on decode, but those should be rare, and likely only happen at the beginning of a stream. If that's still a problem, Ogg is a pretty simple format and if you're just decoding a file from the beginning without seeking, you can implement the Ogg parsing in very few lines.

@jmvalin Thanks for the information! Unfortunately no one in our team is familiar with the ogg format (and for anyone untrained in these kind of file formats I admit it's a bit scary), so I'm afraid it may be impossible for us to do this. If liboups or any of the official samples/code would provide a way to do decoding directly in the audio thread, we would be glad to add Opus back.

@reduz Have you actually tried using libopus+libogg? Did you observe anything going wrong? I had a look at the libogg code and basically it will only use realloc() when the current working buffer it's using is too small, which means it's going to be very rare. Not only that, but on modern systems a small realloc() is no longer a big deal RT-wise (no worse than many other things that can happen). I assume we're not talking about sub-millisecond latency, right?
Also, from what I read, the audio is originally on file anyway, which has much worse latency issues. If the idea is to read to memory and then have low latency decoding-from-memory, then it wouldn't be hard to simply do the Ogg unpacking on the "read-from-file" part, and then store the raw Opus packets in memory.
In any case, there's probably a dozen different ways this can work without much difficulty, assuming you're actually interested in Opus support. If so, you can ask further questions in #opus on irc.freenode.net or on the Opus mailing list.

@jmvalin Can you provide samples/code would provide a way to decode directly in the audio thread? It would helpful to retaining opus support.
@reduz is not familiar with the code and not having such example will mean dropping support for opus.

@jmvalin We are interested in Opus support, but honestly just need a ready to go solution. We implemented Vorbis from stb_vorbis instead of libvorbis/libogg because they provided this packaged and ready to use. Most contributors here do their work on free time and are not that experience with audio programming (and much less container formats).
You mention things such as "packets" that to you probably obvious, but we honestly have little idea what you are refering to. All we need is to read audio frames from a function, mix them back and eventually seek. That's it.

The Ogg specification is available here and here. For opus, you can assume two additional things to hold:

There is a maximum size in bytes for packets. This makes writing an allocation free ogg parser possible in the first place. Not sure about the exact number though, its not put explicitly in the spec...

There is only one logical stream, the main opus stream, per physical stream (for what "logical"/"physical" stream means, consult the spec :P)

Just out of curiosity, what's the current (3.1.1) behavior of ogg files?  This topic is fairly old, and I was curious if it would be safe to use .ogg files for all sound effects, or if those are still streaming and should be avoided for everything except for music.  It would be nice if small .ogg files simply loaded into memory decompressed at load time and played back quickly in realtime.  Should we still be using .wav files for performance?

What's the status right now regarding this issue? And what are the plans for 4.0?

Note that Godot 4.0 uses libvorbis + libogg instead of stb_vorbis. This opens the way to Opus integration with less overhead on binary size (as libvorbis is already included).


Integration of hpc-lcov into hpc

Hi,
I'm one of the new maintainers of the hpc binary: https://gitlab.haskell.org/hpc/hpc-bin
I would like to incorporate the functionality to emit lcov data directly into the hpc binary distributed with ghc. But since hpc is a binary distributed with ghc, we cannot depend on any third-party libraries, only boot libraries. So I would have to copy-paste your code from the src/ directory. I would of course add an attribution to you as the author of that code.
This would probably be covered by the terms of the BSD-3 licence, but I wanted to check with you first if this is ok with you.

Hi, sorry for the delay! You asked this right when my company was being acquired and I had to figure out how to get the repo transferred to my personal account.
Yes, I'd be honored to have this work merged into hpc! I look forward to better integration support within hpc itself!

No worries :) Thanks, I will hopefully find some time soon to integrate it into the hpc tool.


cannot save jupyter notebooks larger than 1M

I run notebook from repo https://github.com/arogozhnikov/GW150914, and as I save changes, it complains that notebook is too large.

Is it jupyter which complains? If so, it's nothing we can do, the problem with jupyter or juperhub I suppose.

Jupyter if run without jupyter hub can easily handle such large notebooks

Odd that jupyterhub should have a different limit. Have you investigated where in jhub it enforces the limit/raises the error?
Sidenote: for nteract we also briefly discussed the size limit of notebooks and there the conclusion was that this isn't a hard limit more and "accidental" one.

issue was on nginx-proxy side, that was running in front of everware.
Default value of client_max_body_size is 1M.
Changed to 10M.


Implemented Accessibility Evaluation and Modified the Relevant Changes

Accessibility Evaluation Report

Page Titles
Accessibility guidelines - in accordance with guidelines
Content - accurately descriptive
Other browsers - visible on all browsers

Image text alternatives
Total 36 images
both for mobile and desktop
alternative text applicable in all cases

Text headings
two sections for mobile - h1 first and h2 second
four sections for desktop - h1 first and four h2 second
no h3

Color Contrast
health score tester: score 81% passed
placeholder contrast ratio: 6.11 : 1

Resize
by default: all anchor elements are accessible with keyboard use
use of Aria interaction method: all html elements become more accessible

Interaction
form: three textfields accessible with keyboard Tab
labels: include inside the text
required fields: not indicated
asterisks: not indicated
general instructions: not indicated
dates: not available / not applicable
detail error handling: validation messages require javascript
html validation: no issue

Moving Content
Moving h1 - applicable / no issue
Distraction - not applicable
Flashing and blinking - no issue

Multimedia
Not applicable / no issue

Basic structure
CSS turn-off - in order
Menu-button turned-off - structure remains intact
Portfolio source image - alternative text displayed descriptively

Accessibility Changes

Image text alternatives:

Alternative texts for images improved appropriately

Heading:

Heading structure improved with h3 tag.
Added h3 tag to social media and technology areas.
h3 tag replaced paragraph (p) tag.

Forms, labels, and errors

Added instructions for filling the form.
Added general and individual field instructions.
Add labels to form elements
Labels were set to  invisible
Required fields are indicated by asterisks.
Border color added for validation.

Moving, Flashing, or Blinking Content:

Only the Primary Heading is refactored to move
appropriately, without distraction.

Your review is good.  Changes will done. Thanks

Thank you. Further suggestions will surely be looked at and worked on.


Follow-up to groups UI work in #24766.

First commit is to rename the search-container class as discussed here.
Second commit is to fix the messages shown when adding users to a group. (part of #24443)
Third commit is to fix the placeholder text in search input for members list.
Fourth commit is to fix placeholder text in pill input for members list.
Fifth commit is to add simplebar to group creation form.
Sixth commit is to fix the cursor when user is not allowed to add members.

Fixes:  #24443

Screenshots and screen captures:
Messages shown when trying to add users to a group -

Placeholder text -

When user is not allowed to add members for a group -

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

We do not live-update the UI currently (at least we do not cover all the cases). But we would probably be adding new settings like "Who can manage user groups", etc. so we can fix it while adding those settings.
The tooltip here is not correct because it is also shown when user is member of the group but the "Who can create and manage user groups" setting is set to admins only. We can fix this now also or we can handle it when adding new settings.

@alya can you have a look at this one? These are mostly fixes from #24443 and a couple of UI updates.

Thanks! A would make the following wording tweaks (as specified in #24443):

No user to add. -> No users to add.
Already added users: -> Already members:

I also noticed a couple of small issues:

[ ] In the editing UI, if you click "Save changes" without making any changes, nothing happens and the button freezes in the hovered state. If it's not too hard, we'd ideally just disable it if no changes were made.

[ ] Once you exit the editing UI, the tooltip on the "edit" pencil button unexpectedly appears:

If you'd prefer to fix some of these on a separate PR, that's of course totally fine. I think the issues with long group names (4,7,8) in the comment above are not blockers for moving over to the new UI.

In general, I did not test to check whether the issues above are present in the stream management UI as well.

Opened #26788 for the empty group names fix.

Made the wording tweaks as well -

Also, I checked the long-group names issues and they can be reproduced for streams as well. Will fix those in another PR.

These changes look great, merged, thanks @sahil839!


Add '--ignore-dead-nodes' option support to nodetool removenode

Since commit
commit 829b4c14380020fa4058335c4c43cefec135b3de
Author: Asias He<EMAIL_ADDRESS>Date:   Mon Nov 16 16:13:44 2020 +0800

    repair: Make removenode safe by default

Scylla added the ability to exclude dead nodes during removenode operation.
If the user want the removenode operation to succeed even if some of the nodes
are not available, the user has to explicitly pass a list of nodes that can be
skipped for the operation.
The nodetool option can be:
$ nodetool removenode --ignore-dead-nodes <list_of_dead_nodes_to_ignore> <host_id>

Example restful api:
$ curl -X POST "http://<IP_ADDRESS>:10000/storage_service/remove_node/?host_id=7bd303e9-4c7b-4915-84f6-343d0dbd9a49&ignore_nodes=<IP_ADDRESS>,<IP_ADDRESS>"

it seems https://github.com/scylladb/scylla-jmx/pull/178/files adds support for option
but it doesn't really pass on the option to the REST API
[root@7cde2899a85b /]# nodetool removenode --ignore-dead-nodes <IP_ADDRESS>,<IP_ADDRESS> f9a3b363-f2dd-4ff0-a86b-27d4294e2983
nodetool: Scylla API server HTTP POST to URL '/storage_service/remove_node' failed: std::runtime_error (removenode[b31b7293-d7d9-4b41-b4cb-57c3d91fa72b]: Nodes={<IP_ADDRESS>} needed for removenode operation are down. It is highly recommended to fix the down nodes and try again. To proceed with best-effort mode which might cause data inconsistency, run nodetool removenode --ignore-dead-nodes <list_of_dead_nodes> <host_id>. E.g., nodetool removenode --ignore-dead-nodes <IP_ADDRESS>,<IP_ADDRESS> 817e9515-316f-4fe3-aaab-b00d6f12dddd)

while direct
curl -X POST "http://<IP_ADDRESS>:10000/storage_service/remove_node/?host_id=f9a3b363-f2dd-4ff0-a86b-27d4294e2983&ignore_nodes=<IP_ADDRESS>,<IP_ADDRESS>"

worked
(found by @fee-mendes  )

was that option tested with multiple ignore nodes IPs ? @bhalevy ?

Yes. It's being tested regularly.
See https://jenkins.scylladb.com/view/master/job/scylla-master/job/dtest-release/1092/testReport/update_cluster_layout_tests/TestUpdateClusterLayout/test_simple_removenode_3_nodetool_True_/

Duplicate of #225

I'm closing this issue.
Please verify the versions of tools/java and jmx you're using
and if there's a problem, let's open a new issue for it.


Make dependees of muted node not crash, but simply be ignored

Hi,
Suppose a workflow:
[ random node X ]
        ↓
      [ rgthree Image Comparer ]
        ↑
[ random node Y ]

This works just fine.  This is very annoying and gets in the way. This is contrary to most built-in Comfy nodes which simply display a red circle on the missing input, and silently prevents the rest of the dependencies of that muted input from running, which is exactly what I want when I mute a node: kill all the downstream branch, without crashing.
I'm not knowledgeable about Comfy node API, but I'm going to assume that if Comfy can do it correctly, it's probably possible to replicate this behavior as a custom node. I guess it's something about making the inputs not strictly required, or handling the failure gracefully?
Thanks.

c8603bc Fixes this, thanks!


Added Functionality to query and conn to enable first simple connections

Horrible codestyle, but that'll be fixed later ;)


Chore/run e2e tests from within the repo

Context
https://dsdmoj.atlassian.net/browse/CAS2-140?atlOrigin=eyJpIjoiOGYwMzUwYmJkMWZkNDY3NGEyNGUyZTM3ZGFkYmIzYjciLCJwIjoiaiJ9
It’s important to note that e2e tests are required to pass for deploying the frontend to preprod and prod.
We found that it can be difficult to coordinate the frontend changes (a pull request in 1 repo) alongside the changes to the e2e tests (a pull request in another repo).
Both must be worked on and tested locally from different repos. We also have double the admin of proposing and reverting two pull requests, rather than just one.
We’d like to move these tests from a separate repo to the same repo.
Changes

We install playwright and copy over the e2e tests from the separate repo.
We add a very simple health check test to help quickly verify the e2e tests are set up correctly. We could drop this if needed?

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

Once this is reviewed I need to go back to the e2e test repo to collect any new changes!


Fix basic example

Following code gave me an error "Module not found: Error: Can't resolve 'brace/theme/github' in 'C:\web\ng2-ace-editor-master\examples\basic\src\app'"
import "brace/theme/github";
import "brace/mode/sql";

Changing it to following fixed it.
import "../../node_modules/brace/theme/github"
import "../../node_modules/brace/mode/sql"

Hi,
Now it ok?
I think yes


the app need profiling

Ate lots of time to unique this vector.

Solution:

[ ] Just use the first row since they are all the same from the observation data?
[ ] don't know


Release Wasmtime 18.0.1

This is an automated pull request from CI to create a patch
release for Wasmtime 18.0.1, requested by @alexcrichton.
It's recommended that maintainers double-check that RELEASES.md
is up-to-date and that there are no known issues before merging this
PR. When this PR is merged a release tag will automatically be
created, crates will be published, and CI artifacts will be produced.

This is dependent on https://github.com/bytecodealliance/wasmtime/pull/7966 and is intended to publish a full release with a fix for this issue


Manual Code Review 3

Review HA Repos for any specific files related to your Misuse/Use cases

Closing this as your branch has already been merged to master.


Handle new devices dynamically?

Does this project handle new devices being connected after the script started running? Thinking of using it for my setup, but that's a pretty big requirement for me.

it doesnt, it just grabs all devices currently present as it stands. the program itself merely combines what devices you give it, and the script just matches all device pointers and passes it to that.
workarounds would exist where you rerun it from a dbus rule on seeing a device being removed/added. due to it all being a bit taped together and the program in the end really doing near nothing complex besides merging what you pass it I've abandoned making it more featured.
could give it another look though, question has come up multiple times. and it could still keep its old working without any special flags

that would be great! it's also pretty critical for me

Since Ubuntu 23.04 I had a problem with my r-go split keyboard and your little program saved my day!
I found a way that works for me to handle hotplugging by using a systemd path unit to wath the device directory for changes, and restart the mergeinputs service when necessary.

PR: https://github.com/Ckath/mergeinputs/pull/9


ninja can't build itself after ninja -t clean

Observe:
hummer:ninja thakis$ ./bootstrap.sh
Building ninja manually...
Building ninja using itself...
wrote build.ninja.
[17/17] LINK ninja
Done!
hummer:ninja thakis$ ./ninja -t clean
Cleaning... 34 files.
hummer:ninja thakis$ ./bootstrap.sh
Building ninja manually...
Undefined symbols:
"DepfileParser::Parse(std::basic_string<char, std::char_traits, std::allocator > const&, std::basic_string<char, std::char_traits, std::allocator >)", referenced from:
Edge::LoadDepFile(State, DiskInterface*, std::basic_string<char, std::char_traits, std::allocator >*)in ccwbQgLA.o
ld: symbol(s) not found
collect2: ld returned 1 exit status
This is probably because src/depfile_parser.cc is generated from depfile_parser.in.cc, and so the clean tool deletes it, even though it can't be regenerated and is checked in for that reason.

Oh, and
git checkout master -- src/depfile_parser.cc
makes the build go again.

I sorta intended bootstrap.sh for only using on the first build.  From then on, you can copy the generated ninja binary elsewhere and use that to develop ninja.  (And when you run /path/to/my/ninja instead of bootstrap.sh, it will correctly run re2c to generate the file.)

hummer:ninja thakis$ cp ninja build_ninja
hummer:ninja thakis$ ./ninja -t clean
Cleaning... 34 files.
hummer:ninja thakis$ ./build_ninja
[15/19] RE2C src/depfile_parser.cc
FAILED: re2c -b --no-generation-date -o src/depfile_parser.cc src/depfile_parser.in.cc
/bin/sh: re2c: command not found
ninja: build stopped: subcommand failed.
:-)
Makes sense, closing.


Will not attempt to authenticate using SASL

pulsar 2.8 version
run command:
./pulsar standalone
show bottom unknown error:
19:45:53.411 [main-SendThread(<IP_ADDRESS>:2181)] INFO  org.apache.zookeeper.ClientCnxn - Opening socket connection to server localhost/127.          0.0.1:2181.
19:45:53.411 [main-SendThread(<IP_ADDRESS>:2181)] INFO  org.apache.zookeeper.ClientCnxn - SASL config status: Will not attempt to authenticate using SASL (unknown error)

why ?
SASL config status: Will not attempt to authenticate using SASL (unknown error)

The issue had no activity for 30 days, mark with Stale label.


Synced local files to container files

Fixes Issue: -  Change in docker compose.yaml
Closes  #167
Changes proposed
Added a volume over the
Local files to container files i.e. .:/usr/src/app
And another for node modules so that they don't overlap i.e. /usr/src/app/node_modules

note: sorry for the delay in merging, I was clearing up disk space on the server (we ran out so deployments were no longer working 🤦♂ , not sure how we used 100GB in 1 week 😰 )


Migrate samples for ArcScVmm and Attestation

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


math.vectors.simd test breaks

Saw this on linux/x86.32 build machines today:
==== resource:basis/math/vectors/simd/simd-tests.factor

resource:basis/math/vectors/simd/simd-tests.factor: 253

Unit Test: { [ { } ] [ \ longlong-2 \ fixnum [ = ] check-vector-ops ] }

=== Expected:
{ }
=== Got:
{ ~array~ }
[Traceback]

It failed again:
==== resource:basis/math/vectors/simd/simd-tests.factor
resource:basis/math/vectors/simd/simd-tests.factor: 253
Unit Test: { [ { } ] [ \ short-8 \ fixnum [ = ] check-vector-ops ] }
=== Expected:
{ }
=== Got:
{ ~array~ }
[Traceback]

These reports aren't very useful without knowing what ~array~ is. Can we adjust the prettyprint params to go deeper here?

Just saw this in a Mac OS X 32-bit build report too. Someone should run this locally and see what the failure really is.

Doug ran it locally and found it was vlshift that was failing. For what values, we still need to figure out, since the test failures don't give that information.


Question: Jetbrains.Fantomas

I just found this on nuget and was wondering if it can be used outside of Rider?
A while ago I read that fantomas might end up in FSharp.Compiler.Service.
So now I'm curious where the team is heading with the usage of Fantomas. Will you eventually swap to the FCS version once it is stable?
On a side note is the formatting of F# code already available in Rider 2017.2?

@nojaf It is a version built against latest FCS, nothing is added, something not needed in our case is cut. This package should have been unlisted in NuGet, although technically it could be used outside Rider if a newer FCS is needed. A better approach would be to open a PR incorporating changes needed for the newer FCS in the Fantomas repo and ask its maintainers to publish a newer build to NuGet. I'm going to do it at some point if no one else does it before.
We'll definitely move to the Fantomas bundled in FCS once it's integrated (see a newer discussion here), but looking at the discussion I doubt it'll happen soon.
As of Rider 2017.2.1 selection formatting was disabled but should work for file level in cases where Fantomas is able to reformat it.

Thanks for the clarification. I tested the formatting on a file instead of selection, it indeed works.


Hard time loading data to amundsen using apache atlas

Hi all,
I've seen some examples to load data into amundsen using Neo4j (amundsendatabuilder/example/scripts/) but how can I load data using Apache Atlas? Is there any sample too? Should I use Atlas Rest API? Atlas client?
I'm finding it dificult to find examples. And quite confusing.
Thanks.

cc @verdan @mgorsk1
hey Verdan, Mariusz, I wonder whether it will be possible to have some guide or doc on how to boostrap Amundsen with Atlas. I don't have much experience with Atlas, so will leave it to you to decide :)

Since Amundsen in this context is displaying data present in Atlas, I'd suggest reading the Atlas guides for loading data, and/or hitting up their support community. If you get the data loaded but the display isn't as expected in Amundsen, we're happy to help debug that.


Pipeline enhance

refactor some classes from unicomplex to pipeline, most of them are from org.squbs.proxy, now moved to org.squbs.pipeline in squbs-pipeline project.

Simplify pipeline management, remove some classes like PipelineRegistry.

A few more comments. Couple of rules I need to ask for:

Please squash the commits into 1 commit.
Please put the issue URL into the commit text.

These rules and link to how to squash are here: https://github.com/paypal/squbs/blob/master/CONTRIBUTING.md
Thanks!

Please just take care of the nulls, squash, and ensure you have the issue# in the commit. I'm good to merge if that's in place. Thx!

Use https://github.com/paypal/squbs/pull/63 instead


supabaseUrl is required.

when run  npm run dev ,there is an error Error: supabaseUrl is required.
where can get supabaseUrl?

I am getting the same thing.  I actually tried on my local machine and then tried from a virtual machine with windows and both had the same error.  One suggestion was to see if I had anything running on port 3000 when launching the program.  I checked netstat and didnt see anything running.

I'm having the same issue. I was using port 3000 to run another app that required supabase (I'm not sure that would be the issue), however, I quit that app and I'm still having the same issue.

when run npm run dev ,there is an error Error: supabaseUrl is required. where can get supabaseUrl?

Supabase is part of the new update Supabase is required.
Create a Supabase project then copy and paste the supabase url & your anon key to your .env.local:
NEXT_PUBLIC_SUPABASE_URL="your-supabaseurl"
NEXT_PUBLIC_SUPABASE_ANON_KEY=your-anon-key

Have fixed this in #21


downgrade requirements of numpy

Home Assistant has numpy fixed on version 1.21.6.
So there is a conflict now.
Will the camera also work with 1.21.6?
@prairiesnpr

Just checked my production instance, it's running with numpy==1.22.4

thx.
Home Assistant uses this: https://github.com/home-assistant/core/blob/187d56b88ba5a5f5656937d6c6a687e2de96d600/requirements_all.txt#L1126
And with my production version HA 2022.6.6 I got this error:
Setup failed for custom integration husqvarna_automower: Requirements for husqvarna_automower not found: ['numpy>=1.22.4']

Interestingly, on my instance it just upgrades numpy silently.  Just tested and can confirm, you can downgrade to 1.21.6.


Documentation of Precision property

The documentation says Precision is the total number of digits. Apparently it ignores trailing zeros, for example the precision of 123450000 is 5, not 9.

Yeah, you can think of it as the number of digits when represented in scientific notation. I will clarify in the documentation, thanks for the report.

Fixed in v1.0 release.


Retry reannounce request if connection was dropped

Workaround for https://twistedmatrix.com/trac/ticket/7843

Coverage decreased (-0.18%) to 75.35% when pulling d5933f9478f080752366c5b9b237d6d6a67045d5 on guidow:restart_request_on_broken_connection into 9de89671fc70ef91bb5fcdbfc14be15ff395a53b on pyfarm:master.

Coverage decreased (-0.18%) to 75.35% when pulling d5933f9478f080752366c5b9b237d6d6a67045d5 on guidow:restart_request_on_broken_connection into 9de89671fc70ef91bb5fcdbfc14be15ff395a53b on pyfarm:master.

Coverage decreased (-0.23%) to 75.31% when pulling 097dfe0df10f5f2a2b0da35c1c4457c09de630b9 on guidow:restart_request_on_broken_connection into 9de89671fc70ef91bb5fcdbfc14be15ff395a53b on pyfarm:master.

Code looks good, but could we get a configuration variable for the number of retries for these errors?  We're already hard coded the value twice and will probably need it again elsewhere.

Coverage decreased (-0.36%) to 75.18% when pulling c73cbacda31cc8eb3875fac7c8fbe184ad705e93 on guidow:restart_request_on_broken_connection into 9de89671fc70ef91bb5fcdbfc14be15ff395a53b on pyfarm:master.

Looks good except for the one comment above, thanks for the update.  Once that's fixed feel free to merge this.


[DDW-846] Update react, react-router and other related dependencies

This PR updates react, react-router, mobx, mobx-react-router, and their related package versions
Jira Ticket
Testing Checklist

Slack QA thread
[ ] Regression testing on all screens of app and stories of storybook

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes
[ ] PR has default-sized Daedalus window screenshots or animated GIFs of important UI changes:

[ ] In English
[ ] In Japanese

[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

@tomothespian @DeeJayElly please review this PR 🙏

@tomothespian I left reply comments. I will handle those things once I return back to work.
CC @nikolaglumac

@yakovkaravelov take a look at my answers

@tomothespian Please review again.

@yakovkaravelov There is a scrollbar visible when you access the "Add Wallet" screen. It goes to an empty screen

@yakovkaravelov There is a scrollbar visible when you access the "Add Wallet" screen. It goes to an empty screen

@gnpf This was fixed. Please review again, thanks.

@yakovkaravelov The previous fix looks good.
I found something else:

When you access to View - Settings from the wallet view, you are redirected to the settings of that wallet. What is expected is to display the user settings
When you access to View - Settings from the user settings, you are redirected to an empty page

@yakovkaravelov The previous fix looks good.
I found something else:

When you access to View - Settings from the wallet view, you are redirected to the settings of that wallet. What is expected is to display the user settings
When you access to View - Settings from the user settings, you are redirected to an empty page

@gnpf This was fixed. Please review. Thanks.

@tomothespian
I really hate to handle these because this PR is for react package dependency updates.
It's really odd I should handle these already existing bugs in this ticket.
Anyway, will take a look at them.
Btw, would you please leave all the comments at one time as possible as you can?
It would have been nicer if you left all above fallback related comments when you did similar things last week all together at the same time.

@yakovkaravelov any news on this one?

@yakovkaravelov any news on this one?

Fixed all the comments from Tomislav except failing itn test
Fixing it.

@nikolaglumac @tomothespian Finally I fixed router action issue and several itn tests.
It should work fine now.
Please review. Thanks.

@yakovkaravelov I still see failing tests on the yarn nix:selfnode :(

@yakovkaravelov I also get failing tests on the ITN selfnode :(

@yakovkaravelov I still see failing tests on the yarn nix:selfnode :(

@yakovkaravelov revert these changes:
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-781f9069104c17460cdcb9868fabba82R65
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-b622efb61d13bd72d88f512936c25651R82
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-b622efb61d13bd72d88f512936c25651R91
And run tests outside watch mode. Edit wallet name test should work then

@yakovkaravelov I still see failing tests on the yarn nix:selfnode :(

@yakovkaravelov revert these changes:
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-781f9069104c17460cdcb9868fabba82R65
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-b622efb61d13bd72d88f512936c25651R82
https://github.com/input-output-hk/daedalus/pull/1966/files#diff-b622efb61d13bd72d88f512936c25651R91
And run tests outside watch mode. Edit wallet name test should work then

@tomothespian No need to revert above things. They were added to fix test errors. They are fine.

@nikolaglumac @tomothespian All test errors were fixed. Please review.
Thanks.


Update Assimp binaries

LGTM - but breaking changes

This branch seems to be having a panic attack due to the target change/squash of the original PR, so im going to let the assimp task run on the dxvk-native-packagse branch (i merged main into it) then merge into there and the package update should come with that PR


Sugar ORM save() does not return any value

save () must not have to be void ,
I want to perform some operation only when data is inserted successfully in the table but save () is of void type so it doesn't return anything .

Check https://github.com/satyan/sugar/blob/master/library/src/main/java/com/orm/SugarRecord.java
It returns a long


Unit Tests for Dynamic RR on Unbonding

x/stakeibc unit tests currently unbroken
still need to add additional unit tests

closing as the tests were no longer broken with the new approach


ClientUtils cleanups

Type of change

Refactoring

Description
This PR includes some cleanups in related to ClientUtils
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests

:x: Test Summary :x:
TEST_PROFILE: pr
TEST_CASE:
TOTAL: 266
PASS: 245
FAIL: 4
SKIP: 17
:heavy_exclamation_mark: Test Failures :heavy_exclamation_mark:

testTopicSubscribe{ArtemisManagement} in io.enmasse.systemtest.shared.brokered.clients.stomp.MsgPatternsTest
testDurableSubscriptionOnPooledTopic in io.enmasse.systemtest.shared.standard.TopicTest
testAccessLogs in io.enmasse.systemtest.isolated.CommonTest
testStandardPersistentMessages in io.enmasse.systemtest.isolated.PersistentMessagesTest


Error building site: render: failed to render pages: render of "page" failed

After finishing cp -r ./themes/eternity/example/eternity.bora.sh/* ., I runed command hugo serve, then I met the following errors:
Start building sites …
hugo v0.118.2-da7983ac4b94d97d776d7c2405040de97e95c03d windows/amd64 BuildDate=2023-08-31T11:23:51Z VendorInfo=gohugoio

ERROR render of "page" failed: "D:\github\xxx\yoursite\themes\eternity\layouts\_default\single.html:1:3": execute of template failed: template: _default/single.html:1:3: executing "_default/single.html" at <partial "header.html" .>: error calling partial: partial "header.html" timed out after 30s. This is most likely due to infinite recursion. If this is just a slow template, you can try to increase the 'timeout' config setting.
Built in 120038 ms
Error: error building site: render: failed to render pages: render of "page" failed: "D:\github\xxx\yoursite\themes\eternity\layouts\_default\single.html:1:3": execute of template failed: template: _default/single.html:1:3: executing "_default/single.html" at <partial "header.html" .>: error calling partial: partial "header.html" timed out after 30s. This is most likely due to infinite recursion. If this is just a slow template, you can try to increase the 'timeout' config setting.

I tried few version of hugo, but none of them works:

When using hugo v0.117.0, the problem exists;
neither can the latest hugo v0.118.2 ;
neither can hugo v0.104.0;
neither can hugo v0.102.2;
neither can hugo v0.96.0;
neither can hugo v0.88.1

I am using an environment of amd64 windows11
I am not a go learner but only learned some basic knowledge on how to write a theme, I can't fix it right now : (
Thanks for giving a help : )

Do you still have the issue? @VenusXK

This may happen if you have slow internet connection. Because the demo site downloads many pictures from the internet. Just delete some of posts and re-try please.

[!WARNING]
This project is currently not being actively maintained.
Please do not create new issues on GitHub.


Global Variable in Evaluate Expression Event doesn't show in Variable Uses

Describe the bug
When you check where a global variable is used in your project, by clicking on the variable and looking at the Variable Uses list, you won't see any variable use from the free text area part of the Evaluate Expression Event.
To Reproduce

Create a global variable foo.
Add a Evaluate Expression Event to an On Init part of a test scene.
Set $Local0 variable to expression $foo + 1.

Observed behavior
The Variable Uses list says "Variable hasn't been used in project".
Expected behavior
The test scene should be listed in the Variable Uses list.
Platform (please complete the following information):

OS: macOS 14.6.1 (23G93)
App version 4.0.2

Additional context
This bug also applies to any text in a Display Dialogue event and, potentially, any other event where the variable reference can be entered in a free text area. I just didn't want to file numerous bugs for what is probably a similar problem in each case.

It looks like the If Math Expression event part of this bug has been fixed in v4.1, so I am going to edit the Additional context section above and remove that part.

The bug I submitted has been fixed in version v4.1.1. I can file the Display Dialogue bug I mentioned in Additional context separately. Closing as fixed, thanks.

Is the Display Dialogue bug still happening for you? That should have been fixed with 4.1.1 also :-)

Yeah, here is my setup:

and my result:

I went ahead and filed the Dialog Box Event part of this issue as a separate issue because I still see it in v4.1.3:
https://github.com/chrismaltby/gb-studio/issues/1628


Refactor build tasks

Consider splitting each task into a separate script file.
This would be easier to manage and create new tasks.
Create a function to generate new build tasks

FooBar

blah

test2

test3

hello.txt


Create QuestionBox

UPDATED ACCEPTANCE CRITERIA
The Question box displays:

The Active question
The question counter

The question counter displays the active question # / total number of questions

The question example text
Follows the design to the pixel

Behaviour:

If the active question changes, the displayed question is updated to the active question

PREVIOUSLY:
Display question/answer

Re-opening for Waffle


General — Save all settings to an INI file

Save settings like "Emulator/Program to run", "Override Arguments", and "Mute sound" into a single .INI file that is formatted.
Following controls to be saved:

Emulator/Program to run file location
Override Arguments
Mute sound (Don't play a sound when the corruption is finished)
The checkbox states in the (Start Emulator Program) section
Stash Saves checkbox state
Display HEX instead of Denary checkbox state

This allows for less hassle setting everything up again.

Doesn't have to be an INI file. Just has to be SOME type file.


explainability tools for representation

We should extend the explainability tools developed for robust virtual staining to representation learning. I suggest the following:

Export a movie of features (input, PC of features) for a chosen track.
Effective receptive field of a chosen embedding dimension.

Effective receptive field of a chosen embedding dimension.

The embedding is the global average pooling of the deepest feature map, so by definition its ERF is the entire input image.

Related: #170, #162.


Add home page and details page and get data from api plus styling

Create Redux store
Configure Reduces and actions
Get data from API
Create home page and update the UI with data from api
Create details page

@Reem-lab Please i need more clarification on your recommended changes in order to proceed. You can also send reply me on slack if that ok. Waiting

@FrederickMih You made mention as follows:  In order to make sure each unit of our application is working as intended, I think it's important to add unit tests for data that comes from the backend(API) and your components are displaying them. You have also made mention as follow: **As mentioned above, you'll need to pass some mock data(home_team_score, season, period, etc..) relevant to what you want to test and make sure the mock data renders successfully 👍Your firrst statement is in contradiction to the second. {data that comes from the backend(API) and your components are displaying them) and then mock the data at the same time? I have made test for the search component with interacts with the redux store and a button which when clicked goes to where data is fetched from the API

@FrederickMih You made mention as follows: In order to make sure each unit of our application is working as intended, I think it's important to add unit tests for data that comes from the backend(API) and your components are displaying them. You have also made mention as follow: **As mentioned above, you'll need to pass some mock data(home_team_score, season, period, etc..) relevant to what you want to test and make sure the mock data renders successfully 👍Your firrst statement is in contradiction to the second. {data that comes from the backend(API) and your components are displaying them) and then mock the data at the same time? I have made test for the search component with interacts with the redux store and a button which when clicked goes to where data is fetched from the API

Hi @enocol, I don't understand what you meant by my statement being in contradiction to the other. I just gave more emphasis on what you may do.
For your entired application, you carry out only two integration tests(test for the back button and the search input). I was suggesting you add unit tests to the functionalities of your application. I mentioned that you may mock data on your components from the  API


Add SubscriptionOptions

Is your feature request related to a problem? Please describe.
It would be really cool if we added options for registration. Thus, the mutation, query and registration widgets would be standardized.
Is there any reason today for Subscription widget to not have options like Query and Mutation widgets?
Describe the solution you'd like
Replace the  params operationName and query with options.

This is the case in 4.0.0-alpha.0 check out #648

Thanks @micimize :D


Making the cursor more visible

This closes https://github.com/sdras/night-owl-vscode-theme/issues/79

I forgot to add a screenshot of the visual changes:
Before

After

Hm, though I really love that you spent time on this, (thank you so much for all the work you're doing here), I want to

keep the foreground of the selected item crisp, and
yes, I know it's a dull purple now, but I think when the value is in a higher key, the purple doesn't do well. I think this needs to become that blue color we use in other areas.

Thank you though!

@sdras you mean that the cursor should be blue?

yes, like the sidebar foreground :) a bright shade of blue :)

Ok, I can do that tonight if you haven't already! :)

If you don't mind taking it, that would be lovely! It's on my list but I have a ton of work for my job that's backed up right now. Much appreciated, Winner!


sacrthgap : pi zero v2 : licamera-apps build fails with rpicam_app.so.1.4.2 error

Hello,
I'm trying to add libcamera-apps to pi zero v2 64 bit  but i didn't succeed to find a solution for the problem mentioned below. It seems close from this one reported and solved  https://github.com/agherzan/meta-raspberrypi/pull/1273
here is my setup:
ubuntu 22.04
bblayers.conf
BLAYERS ?= "
/home/dingo/scarthgap/poky/meta
/home/dingo/scarthgap/poky/meta-poky
/home/dingo/scarthgap/poky/meta-yocto-bsp
/home/dingo/scarthgap/poky/meta-raspberrypi
/home/dingo/scarthgap/poky/meta-openembedded/meta-oe
/home/dingo/scarthgap/poky/meta-openembedded/meta-python
/home/dingo/scarthgap/poky/meta-openembedded/meta-networking
/home/dingo/scarthgap/poky/meta-openembedded/meta-multimedia
"
Local.conf
MACHINE = "raspberrypi0-2w-64"
IMAGE_ROOTFS_EXTRA_SPACE = "10000"
EXTRA_IMAGE_FEATURES += " package-management ssh-server-dropbear debug-tweaks"
PACKAGE_CLASSES = "package_ipk"
IMAGE_INSTALL:append = " libcamera-apps"
Here is the error message
ERROR: libcamera-apps-1.4.2+git-r0 do_package: QA Issue: libcamera-apps: Files/directories were installed but not shipped in any package:
/usr/lib/rpicam_app.so.1.4.2
Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
libcamera-apps: 1 installed and not shipped files. [installed-vs-shipped]
ERROR: libcamera-apps-1.4.2+git-r0 do_package: Fatal QA errors were found, failing task.
ERROR: Logfile of failure stored in: /home/dingo/scarthgap/poky/build/tmp/work/cortexa53-poky-linux/libcamera-apps/1.4.2+git/temp/log.do_package.5878
ERROR: Task (/home/dingo/scarthgap/poky/meta-raspberrypi/dynamic-layers/multimedia-layer/recipes-multimedia/libcamera-apps/libcamera-apps_git.bb:do_package) failed with exit code '1'
Attempts
befor creating a bbappend, I tried several options by updating  libcamera-apps_git.bb but it fails everytime. Eithet i didn't compil  or libcamera-apps didn't work
d```
o_install:append() {
rm -v ${D}/${bindir}/camera-bug-report
rm -v ${D}${libdir}/rpicam_app.so.1.4.2
}
not picked automatically, because it's missing common 'lib' prefix
FILES:${PN}-dev += "${libdir}/rpicam_app.so"
#FILES:${PN} += "${libdir}/rpicam_app.so.1.4.2"
#FILES_${PN} += "${libdir}/"
#FILES_${PN}-dev = "${libdir}/ ${includedir}"
#FILES:${PN}-dev += "${libdir}/rpicam_app.so*"
#FILES:${PN} += "${libdir}/rpicam_app.so*"
Would you have ay idea how to solve this?
Many thanks for your support
Michael

I would expect that FILES:${PN} += "${libdir}/rpicam_app.so.1.4.2" would allow it work, at least to create the package successfully. What do you mean by "libcamera-apps didn't work"? Do you get any errors?

Hello,
yes this new line enable compilation but when i run the following commznd, here is what i get
root@raspberrypi0-2w-64:~# libcamera-vid -o  test.h264
imx708@1a - Selected sensor format: 1536x864-SBGGR10_1X10 - Selected unicam form
at: 1536x864-pBAA
munmap_chunk(): invalid pointer
Aborted

do you have any idea of the solution?
kind regards.

Unfortunately my knowledge stops at this point without taking a deep dive...
But I can confirm that see exactly the same behavior also using Rpi3-64 with IMX219, even if I downgrade libcamera-apps and libcamera, and even with kernel 5.15, and after playing with the packageconfig too.

I couldn't help myself, and installed gdb, and made at least some partial progress. I found at least one error: this patch in libcamera is buggy, from meta-oe - this seems to be causing this crash.
You can try to remove this patch (SRC_URI:remove:pn-libcamera = "file://0001-rpi-Use-alloca-instead-of-variable-length-arrays.patch" in local.conf or similar), and try to build it. If it succeeds, then that's it (the patch only tries to add support for more compilers, otherwise doesn't change anything).
If it fails, then revert it, and try to apply the attached patch on libcamera from a bbappend. This patch tries to fix the bug from that other patch. However after this I faced another problem with displaying the video, that I think comes from my own image config, though I'm not sure. Any case, it might be different on your setup - if it isn't, then sorry, I think I'm really  running out of further immediate ideas.
9999-Fix-incorrectly-sized-malloc.patch.txt

thanks. i will try and let you know


Hot-fix: re-add missing recaptcha script to HTML template in the SEP24 registration flow

What
Hot-fix: re-add missing recaptcha script to HTML template in the SEP24 registration flow.
Why
It was removed in https://github.com/stellar/stellar-disbursement-platform-backend/pull/152.
Checklist
PR Structure

[x] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR title and description are clear enough for anyone to review it.
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[x] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr179.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr179.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr179.previews.kube001.services.stellar-ops.com


Crash on Crime Spree

Was playing Crime Spree and managed to trigger this shortly after someone joined.
[string "lib/units/enemies/cop/logics/coplogictravel.lua"]:137: attempt to index field 'team' (a nil value)

SCRIPT STACK

set_logic() lib/units/enemies/cop/copbrain.lua:387
_exit() lib/units/enemies/cop/logics/coplogicbase.lua:198
on_new_objective() lib/units/enemies/cop/logics/coplogicidle.lua:483
on_new_objective() lib/units/enemies/cop/logics/coplogicattack.lua:1486
set_objective() lib/units/enemies/cop/copbrain.lua:265
_execute_so() lib/managers/group_ai_states/groupaistatebase.lua:2482
_upd_SO() lib/managers/group_ai_states/groupaistatebesiege.lua:212
clbk() lib/managers/group_ai_states/groupaistatebesiege.lua:174
_execute_queued_task() lib/managers/enemymanager.lua:562
_update_queued_tasks() lib/managers/enemymanager.lua:577
update() lib/managers/enemymanager.lua:51
update() lib/setups/gamesetup.lua:723
update() lib/setups/networkgamesetup.lua:21
core/lib/setups/coresetup.lua:557

Thanks, looks like this is yet another "lack of a team" crash. I'll make sure to patch over this one too.

Fixed.


docs: updates MAINTAINER.md with corrections

This makes updates to the MAINTAINERS.md file to make corrections to current roles:

Add @degenaro as a community maintainer and org admin
Add @vikas-agarwal76 as a boostrap Oversight Committee member

The applicable GitHub Teams have been updated to reflect these changes.


tests: stablize test_download_sst, test_invalid_read_index_when_no_leader

Signed-off-by: 5kbpers<EMAIL_ADDRESS>
What have you changed?
Fix test_download_sst, test_invalid_read_index_when_no_leader
https://github.com/tikv/tikv/issues/7146 https://github.com/tikv/tikv/issues/7130
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?

Unit test

/merge


Cursor navigation gets dirty symbols in the terminal

Windows Terminal version
1.12.2931.0
Windows build number
10.0.19043.0
Other Software
GNU Midnight Commander 4.8.7
Built with GLib 2.42.2
mintty 3.5.1 (Git Bash)
Steps to reproduce

Use Git Bash

            {
                "acrylicOpacity": 1.0,
                "backgroundImageStretchMode": "uniformToFill",
                "bellStyle": "none",
                "colorScheme": "Vintage",
                "commandline": "C:\\Program Files\\Git\\bin\\bash.exe -i -l",
                "experimental.retroTerminalEffect": false,
                "font":
                {
                    "face": "JetBrains Mono",
                    "weight": "light"
                },
                "guid": "{22a05ebd-bd8c-4882-a99d-53f9a993c069}",
                "icon": "\ud83d\ude80",
                "name": "git bash",
                "padding": "10",
                "startingDirectory": "%USERPROFILE%",
                "tabTitle": "git bash",
                "useAcrylic": false
            }

Use a slower connection then usually
Connect to a remote host
Create a file check.txt with 50-100 empty lines
Open the file in mcedit
Сlick and hold the cursor (right or down)
You will see that symbols will appear in the file

This problem doesn't reproduce in Git Bash without WT
Expected Behavior

Actual Behavior

This is kinda like #4037, but that's mostly for the terminal<->conpty connection. This is more at the conpty<->API layer presumably, or the API<->client layer. The bit about

Use a slower connection then usual

is going to make this a little tricky to debug.
Does this repro if you run C:\Program Files\Git\bin\bash.exe -i -l in the vintage console (conhost.exe), and then try reproing the scenario? That might help narrow the issue down.

Does the problem get worse, possibly with Mojibake garbage (e.g., ∟►ÿá∟;►@ñ►@ [...])?

incomplete escape codes that were then "ghost typed" was tracked down recently as a bug in msys2-runtime (which git bash links with) as shown in git-for-windows/git#3579, which could explain this and that should be included in the next snapshot of git for windows.

Thanks for the context @carenas! I don't think I ever would have been able to track that down. I'm gonna leave this one open till there's a fresh msys2-runtime to download and try out with the fix. Hopefully the issue will just go away - if it doesn't, we'll take another look.

@carenas My apologies for being a bit dense, but do you know if there's a new release of the msys2 runtime with that fix that @karpulix could try our to see if this was in fact the root cause? I'm not really sure myself how things are ingested across the various different git repos 😅

do you know if there's a new release of the msys2 runtime with that fix

the current version (as of the time of the original post) of msys2 included it, so the issue (and fix) should be reproducible (if fixed) by :

install latest msys2 (https://github.com/msys2/msys2-installer/releases/download/2021-11-30/msys2-x86_64-20211130.exe)
update to latest (pacman -Syuuu)

Thanks! @karpulix can you see if that fixes this?

I'm not really sure myself how things are ingested across the various different git repos 😅

@karpulix: latest git RC[1] also includes the msys/cygwin fix, if testing with git is easier
[1] https://github.com/git-for-windows/git/releases/tag/v2.35.0-rc0.windows.1


Check if Next.js projects have sharp installed

Students sometimes forget to install sharp in their projects, leading to very slow images as a result of the Squoosh dependency.
Prevent this in projects with next in the dependencies object in package.json by checking if sharp is also in the dependencies.
Error state looks like this:
$ ../preflight/bin/preflight.js
🚀 UpLeveled Preflight v1.23.0
✔ All changes committed to Git
✔ node_modules/ folder ignored in Git
✔ No extraneous files committed to Git
✔ No secrets committed to Git
✔ Use single package manager
✔ Project folder name matches correct format
❯ No dependency problems
  ✖ Next.js project has sharp installed
    › Next.js projects should have sharp installed for better image optimization. Install it with:

        $ yarn add sharp

  ◼ No unused dependencies
  ◼ No dependencies without types

Released in<EMAIL_ADDRESS>


Can we reduce the # of components?

I think we can get away with just Inline, FlexRow, and FlexCol

Why FlexRow and FlowCol rather than just Flex?

I'm not sure reducing the number of components is useful, perhaps a focus on allowing code to be more expressive? Perhaps @petehunt could grab some metrics on common Component/prop combinations to aid in thinking up some more expressive names for the most common concepts?

Proposal:

Current
Future

Inline
✅
✅

Block
✅
✅

InlineBlock
✅
✅

Row
✅
✅

InlineRow

✅

Col
✅
✅

InlineCol

✅

Flex
✅

InlineFlex
✅

Table
✅

TableRow
✅

TableCell
✅

Most used components for me are Block, Row, then Col.
With Table and friends, I’m on the fence. I’m not sure there’s a benefit to using divs with display set to a table prop. Probably makes more sense to use actual table/tr/etc. by specifying component="tr" on one of the other components.

Additional thoughts:

Box might be a good generic name for a containing element.
inline-ness could be a boolean prop: <Box inline />, <Row inline />, etc. Only problem is InlineBlock. flexbox has mostly removed my need for display: inline-block though.
I like the idea of making primitive layout components a little bit abstract but not too mysterious.

Why remove Flex? and InlineFlex?

@hnordt flex-direction’s initial value is row. Flex and Row are essentially the same component. Row is a bit more explicit than Flex, which is a good thing IMO. InlineFlex becomes InlineRow.

Sounds good @meyer

Hmm. In the case where flex-direction needs to be set programmatically, overriding the prop on a Row component would be a bit strange.

I’m still thinking about this, but will close this issue for now.


Product of Array Except self (JAVA)

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
Added code for Product of Array Except self in java, and updated the README file.
Checklist

[yes] I've read the contribution guidelines.
[yes] I've checked the issue list before deciding what to submit.
[yes] I've edited the README.md and link to my code.

Related Issues or Pull Requests
PR for #5223

May I know if any changes have to be made for this PR? @ankitaggarwal23
also, would like to know why it has not been merged. thank you.

please assign this issue to me


[Snyk] Upgrade com.microsoft.sqlserver:mssql-jdbc from 12.6.1.jre8 to 12.6.2.jre8

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade com.microsoft.sqlserver:mssql-jdbc from 12.6.1.jre8 to 12.6.2.jre8.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.

The recommended version was released on a month ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

this PR was created to see look at Fossa vs snyk. Closing this PR


Request for Documentation Addition: Retrieving Peer ID from Local REVM Node

Describe the change
I am reaching out to request the addition of documentation regarding the process of retrieving the peer ID from a local REVM node. Currently, the available documentation does not provide clear guidance on this matter, leading to confusion for users attempting to interface with the node.
Despite utilizing the admin_nodeInfo RPC call, users, including myself, are encountering difficulties in obtaining the correct peer ID. The peer ID returned from this call does not seem to be compatible with the formatting required for certain applications, such as creating p2p node addresses.
I believe that comprehensive documentation detailing the steps required to retrieve the peer ID from a local REVM node, as well as potential troubleshooting tips for common issues, would greatly benefit the community.
Could the documentation team please consider adding a section or updating existing documentation to address this topic? Clear instructions and examples would assist users in effectively interfacing with their local REVM nodes.
Thank you for considering this request.
Additional context
Utilized admin_nodeInfo RPC call
Peer ID received (0xa9...deb38f45) not compatible with desired format

not compatible with desired format

what's the desired format?

Sorry @mattsse, updated the issue.
Here's what I'm trying to achieve: /ip4/<IPv4_NETWORK_ADDRESS>/tcp/<LISTENING_PORT>/p2p/<NODE_ID>.
16Uiu2HAkvxQyb62x55RrhxxYHm1ZJFMFFFhCZ67YVmTcS6kJwqQR was used as an example value in docs.


BUG: Navbar issue

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
problem :-
In the navbar,  some texts are written in Capitalize form and some are in lower case. Along with it some icons are bigger than text and others are smaller the text.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Thank you for giving me this opportunity. I will try to give my best.

i can fix this issue you can assign me this task


field.flat is not a function

Idk why but he bot is not posting the streams on general like i config the bot he is showing the stream activity on the playing box of the bot but he is not sending the embed, and he has the admin permissions

To solve that problem  has to be from version 12 to the latest node.js


YanFei-CUIot-MZ-6 should support the use of ECC keys in PKCS format

Location:
vendor/platform/YanFei-CUIot-MZ-6/src/port_crypto_default/boatplatform_internal.c

Already fixed.


update manifest.yml

Thank you for the PR, but this example will not work with the new versions of hyperledger fabric. Thus, I will close the PR.


Pukki update instruction for enable root

Proposed changes
Adding Pukki instructions how to enable root from the web interface, so that users can easier take extensions into use.
Checklist before requesting a review

[ ] I have followed the instructions in the Contributing and Styleguide documents.
[ ] My pull request passes the automated tests. If not, visit pull requests at Travis CI and have a look at the job log.
[x] I have included a link to the appropriate preview page .

Looks good to me. Only comment was about explaining better how to connect, I assume it is using a psql or similar client and use root as a user, the same way other non privileged users work, but I think that it would be better if it is explicit.

Notice that there were some documentation about connecting to the database, added link to the section.


Update page-index.html: Remove last section on main page: "Download the app"

OLD

NEW

The applink test fails. We can deactivate it.


Strange sound appears at the loop point of VGM

sn76489-joe.zip

The AY8910's frequency registers were over-optimized across the loop.
Fixed at 0.8.19.


Remove XahauGenesis Test Workaround

Issue Description

Some of the test cases are ran with features - featureXahauGenesis. All test cases should be run with all features enabled.

[ ] Disable "black hole genesis" in standalone

Steps to Reproduce

Expected Result

Actual Result

Environment

Supporting Files

.

.


Add favicon icon

I would like to try tell me which image  u want as a favicon..

Try to take this one


mod proxy status export

mod-proxy hooks in mod-status to provide additional metrics of interest.
Would be nice if those could be parsed and exported too.

I don't have the time to do it.
mod_proxy lines for mod_status in ?auto mode (= AP_STATUS_SHORT) are generated here, they start with ProxyBalancer

Merged #102


Ensure compatibility check readiness

Coming from https://github.com/opensearch-project/opensearch-devops/issues/114#issuecomment-1548868449
Overview
In order to detect the breaking changes between OpenSearch (core/min) and plugins, we recently onboarded a compatibility check gradle task with OpenSearch. See the pull request.
The task is dependent on publishToMavenLocal task (that publishes the core engine artifacts to maven local). It then clones the plugins' repository in temporary directory and runs ./gradlew assemble.
Action Items
In order to avoid false positives and false negatives, we need to make sure the tasks in below checklist are complete:

[ ] The build.gradle file in this repository uses mavenLocal() as the first entry in repositories() section. This will ensure that when core/min dependencies are published to maven local, gradle looks at maven local first and gets the required core/min dependencies rather than fetching from snapshots or maven central resulting in false checks.
[ ] ./gradlew assemble task passes for the plugin build. If the task itself is broken, checking compatitibility is irrelevant as it will always fail.

Please close this issue once the above checklist is complete.
Thanks!

@praveensameneni @lezzago can you please provide an update?

Will have this completed by Monday

Maven local is the first repository: https://github.com/opensearch-project/alerting/blob/main/build.gradle#L30
./gradlew assemble passes


WorkflowServiceInstanceService logs unredacted request

I noticed a private key being logged out while testing today. It turned out to be coming from App Broker in the case where service instance creation fails. It's not safe in the general case to log the request here, since it may contain credentials.
2019-06-26 12:47:17.632 ERROR 7 --- [ry-client-nio-2] o.s.c.a.s.WorkflowServiceInstanceService : Error creating service instance ServiceBrokerRequest{platformInstanceId='null', apiInfoLocation='api.cf.purple.springapps.io/v2/info', origin
atingIdentity=null}AsyncServiceBrokerRequest{asyncAccepted=true}AsyncParameterizedServiceInstanceRequest{parameters={git={uri=https://scs-gitlab.springapps.io/acceptance/backup-test.git, privateKey=-----BEGIN RSA PRIVATE KEY-----...

duplicate of #278. resolved in #279.


chore: improve external secret reconciliation sequence

Secret client is created only if we are going to refresh. This skip an unnecessary check/request in the case we are not going to do anything.
(hello guys, it's me again 👋 :P )

#done should be ready for milestone 0.6.0 ;)

damn, there's a conflict again :disappointed: can you please rebase again :pray:

@moolen #done, sry for the delay

/approve

/merge

Thank you again for your contributions!


Eclipse 2018-09 migration

We going to migrate to Eclipse 2918-09 platform.
This will fix quite a lot of platform bugs + add support of Java 11.

Done.
However I see problems on some Linux machines (mostly on recent Ubuntu)


Cluster Connection URL should be Cluster Connection String

Describe the bug
The wording on the Connect String is not the right standard. It should be Connection String according to the SDK specs.

Steps to reproduce
Go to the Create Connection screen in the plugin.
What did you expect to see?
Change the Cluster Connection URL to Connection String
What version did you use?
VSCode Version: 1.89
Extension Version: 2.0.2
Additional context
https://couchbase.slack.com/archives/C02Q5E70PU3/p1715867058404719

Released


feat: Backend Endpoint "extravios" added methods get post put and delete

Also added config.env and config.example (here you can find the mongodb connection string)

Great job @marcelooggero! Thanks! 🤗


[SPARK-42106][PYTHON] Hide parameters when printing remote URL in REPL

What changes were proposed in this pull request?
When the pyspark REPL is initialized with the --remote <url> option,
the URL provided is logged out to stdout as part of the REPL startup.
However, it may have auth tokens specified as query parameters.
This change is to print the network location, i.e., host and port, instead
of the full URL. Since Spark Connect servers will always be hosted at
the root and the URL scheme is always expected to be sc, only the
host and parts of the URL are relevant.
Why are the changes needed?
Security best practices require not logging auth tokens to standard
output or any destination.
Does this PR introduce any user-facing change?
Previously -
$ ./bin/pyspark --remote "sc://foo.com/?x=y"
Python 3.10.8 (main, Nov 24 2022, 08:08:27) [Clang 14.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /__ / .__/\_,_/_/ /_/\_\   version 3.4.0.dev0
      /_/

Using Python version 3.10.8 (main, Nov 24 2022 08:08:27)
Client connected to the Spark Connect server at sc://foo.com/?x=y

Now -
$ ./bin/pyspark --remote "sc://foo.com/?x=y"
Python 3.10.8 (main, Nov 24 2022, 08:08:27) [Clang 14.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /__ / .__/\_,_/_/ /_/\_\   version 3.4.0.dev0
      /_/

Using Python version 3.10.8 (main, Nov 24 2022 08:08:27)
Client connected to the Spark Connect server at foo.com

$ ./bin/pyspark --remote "sc://foo.com:8080/?x=y"
Python 3.10.8 (main, Nov 24 2022, 08:08:27) [Clang 14.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /__ / .__/\_,_/_/ /_/\_\   version 3.4.0.dev0
      /_/

Using Python version 3.10.8 (main, Nov 24 2022 08:08:27)
Client connected to the Spark Connect server at foo.com:8080

How was this patch tested?
Manually tested by running the pyspark binary. See above.

I'll leave this to @grundprinzip .

Merged to master.