Window line-wrapping can cause colored output to break copied text

Windows Terminal version
1.17.11461.0
Windows build number
10.0.22621.0
Other Software
bat 0.19.0
Steps to reproduce
I have found this very specific scenario for reproducing the issue reliably:

Install bat.

Create EXAMPLE.md with the following contents (including the markdown code fence, because we want syntax highlighting to color the output):
```shell
echo '1 22 333 4444 55555<PHONE_NUMBER>777 88888888 999999999<PHONE_NUMBER> 1 22 333 4444 55555<PHONE_NUMBER>777 88888888 999999999<PHONE_NUMBER>'
```

Resize the terminal to be narrower than that long echo line.

Run the following:
/usr/bin/batcat --color=always --style=header --decorations=always EXAMPLE.md

Copy the output.

Paste the output in a wider window.

It's worth noting that it works fine (i.e. "as expected") if the window is wide enough to not wrap when the command is run. If I resize the window later, the problem is not there.
Something that really surprises me is that piping the output via cat fixes the issue while looking exactly the same visually:

This bug is very similar to, or actually the same as, #5113.
Expected Behavior
For clarity, I think we need to distinguish between the line breaks introduced by Windows Terminal for the purposes of displaying a long line in a wrapped format and the line breaks that are part of the original string. I will call them wrapping line breaks and string line breaks respectively.
When copying a string from the terminal, I expect to copy the string line breaks but not the wrapping line breaks, so that when I paste it I get the "original" string (not the one with line wrappings).
Actual Behavior
When I copy text under these circumstances, it keeps the wrapping line breaks as well as he string line breaks.

Alright, I've figured it out. And as always, it's a very very very specific edge case. It's when I use bat (aka. batcat) to output colored text, with certain flags set.

Install bat.

Create EXAMPLE.md with the following contents (including the markdown code fence, because we want syntax highlighting to color the output):
```shell
echo '1 22 333 4444 55555<PHONE_NUMBER>777 88888888 999999999<PHONE_NUMBER> 1 22 333 4444 55555<PHONE_NUMBER>777 88888888 999999999<PHONE_NUMBER>'
```

Resize the terminal to be narrower than that long echo line.

Run the following:
/usr/bin/batcat --color=always --style=header --decorations=always EXAMPLE.md

Copy the output.

Paste the output in a wider window.

It's worth noting that it works fine (i.e. "as expected") if the window is wide enough to not wrap when the command is run. If I resize the window later, the problem is not there.
Something that really surprises me is that piping the output via cat fixes the issue while looking exactly the same visually:

Can you try this again on Terminal Preview, v1.19/? (manual download link: https://github.com/microsoft/terminal/releases/tag/v1.19.2831.0) We just recently re-wrote how resize/reflow works, and I have a feeling that it may have just fixed this ☺

I don't think v1.19 will make any difference. If you run batcat inside a script session and look at the log, you can see it's actually splitting the content into multiple lines with a line break between them, so it's expected that a resize won't treat that as a wrapped line.
I also noticed there's an issue in their bug tracker which sounds like it might be related (see https://github.com/sharkdp/bat/issues/1740).

Can you try this again on Terminal Preview, v1.19/? (manual download link: https://github.com/microsoft/terminal/releases/tag/v1.19.2831.0) We just recently re-wrote how resize/reflow works, and I have a feeling that it may have just fixed this ☺

Thanks for the suggestion. I just tried with Preview version 1.19.2682.0 and it had the exact same behavior, unfortunately.

I don't think v1.19 will make any difference. If you run batcat inside a script session and look at the log, you can see it's actually splitting the content into multiple lines with a line break between them, so it's expected that a resize won't treat that as a wrapped line.
I also noticed there's an issue in their bug tracker which sounds like it might be related (see sharkdp/bat#1740).

Interesting! That bug does indeed sound very similar to my problems. It is beyond be how there can be a line wrap that affects copying but not resizing.

It is beyond be how there can be a line wrap that affects copying but not resizing.

The fact that the resizing "works" for you is actually a bug. When you have a line that's the exact width of the screen, it is mistakenly considered to be wrapping when the screen is resized (see issue #3088). That was supposed to have been fixed recently, but it still doesn't seem to be working correctly in Windows Terminal.

Can you try repro'ing this with the debug tap to get a trace of all the input and output? Once the bug starts occurring, send us a screenshot/copy-paste of the output and we might be able to figure out what the Terminal thinks it's getting here.

I'd love to, but:

It looks like there was a typo in Mike's link. It should be:
https://github.com/microsoft/terminal/wiki/Troubleshooting-Tips#enabling-the-debug-tap

(thanks - forgot that I had fixed that typo but not the links in my quick replies 🤦)

Here you go! :)
␛[K␍␊
␛[K␛[5;3H␛[?25h␛[13;28;13;0;32;1_␛[200~/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣/tmp/tmp.sbiNSfFwSY/EXAMPLE.md␛[201~␛[7m/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣/tmp/tmp.sbiNSfFwSY/EXAMPLE.md␛[27m␛[K␛[13;28;13;1;32;1_␛[?25l␛[?1l␛>␛[?2004l␛[5;3H/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣/tmp/tmp.sbiNSfFwSY/EXAMPLE.md␍␊
␛[?25h␛[13;28;13;0;32;1_File:␣␛[1m/tmp/tmp.sbiNSfFwSY/EXAMPLE.md␛[38;5;231m␛[22m␍␊
```␛[38;5;141mshell␛[38;5;231m␍␊
echo␣'1␣22␣333␣4444␣55555␣666666␣7777777␣88888888␣999999999␣0000000000␣1␣22␣␍␊
333␣4444␣55555␣666666␣7777777␣88888888␣999999999␣0000000000'␍␊
```␛[m␍␊
␛[K␍␊

Oh whoops I clearly never hit enter on my reply last week. The debug tap obviously showed a manual line break, that's what we're trying to figure out 🤦
Can you try with script instead/? https://github.com/microsoft/terminal/wiki/Troubleshooting-Tips#script
That'll actually capture what was emitted straight from the application, rather than what conpty re-emitted.
(thanks for your patience)

(thanks for your patience)

Thanks for the help!
␛[K␍␊
␛[K␍␊
␛[K␛[8;5H␛[?25h␛[O␛[200~/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣EXAMPLE.md␛[201~␛[I␛[7m/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣EXAMPLE.md␛[27m␛[K␛[13;28;13;1;32;1_␛[13;28;13;0;32;1_␛[?25l␛[?1l␛>␛[?2004l␛[8;5H/usr/bin/batcat␣--color=always␣--style=header␣--decorations=always␣EXAMPLE.md␍␊
␛[?25h␛[?25l␛]9;9;\\wsl.localhost\Ubuntu\tmp\tmp.pxjFyYQaGS␛\␛]7;file://conoa/tmp/tmp.pxjFyYQaGS␛\␛[?1h␛=␛[?2004hFile:␣␛[1mEXAMPLE.md␛[38;5;231m␛[22m␍␊
```␛[38;5;141mshell␛[38;5;231m␍␊
echo␣'1␣22␣333␣4444␣55555␣666666␣7777777␣88888888␣999999999␣00000␍␊
00000␣1␣22␣333␣4444␣55555␣666666␣7777777␣88888888␣999999999␣00000␍␊
00000'␍␊
```␛[m␍␊
<EMAIL_ADDRESS>╰─␛[1m$␛[22m␛[K␍␊
␛[K␍␊
␛[K␍␊

Looks like the bat utility inserts hard line breaks. This issue on the bat repository might be helpful: https://github.com/sharkdp/bat/issues/897
Thanks for filing!

It is beyond be how there can be a line wrap that affects copying but not resizing.

The fact that the resizing "works" for you is actually a bug. When you have a line that's the exact width of the screen, it is mistakenly considered to be wrapping when the screen is resized (see issue #3088). That was supposed to have been fixed recently, but it still doesn't seem to be working correctly in Windows Terminal.

@carlos-zamora please read the above.

At the very least, there are a couple issues here.

bat is emitting hard line breaks.
When you resize, sometimes Terminal unwraps a line of text even though it shouldn't. This is reproduceable with...

PowerShell: "a"*$host.ui.rawui.windowsize.width

for 1

When I copy text under these circumstances, it keeps the wrapping line breaks as well as he string line breaks.

This is because bat is emitting hard line breaks, visible below:

for 2
Now, we should not treat hard wrapped lines as wrapped text under any circumstances. However, we used to have an issue where text would unwrap on resize even though we knew it had a hard line break[^1]
I can reproduce this in 1.17 (where you originally filed the bug):

... and in 1.18 ...

but not in 1.19 (preview) or 1.20 (canary):

If I try your repro from script (thanks for sharing it!) on 1.19, it no longer unwraps when I resize it :)
(window width 76 as specified in repro)

(window resized to larger)

That is what @j4james was saying was a bug. Resize should have never unwrapped the lines, which would have made it more obvious that bat was inserting hard line breaks. 😁
[^1]: The resizing code was... uh. It was not great. It is much better now!

@DHowett FYI, I can still reproduce the bug with the bash test case from issue #3088 in version 1.19.2682.0 of Windows Terminal. However, your PowerShell test case does not reproduce the bug for me.
Looking at the debug tap, it seems that conpty is not actually writing out the expected CR LF at the end of the line in the bash test, but it is doing so in your PowerShell test. So I think the wrapping bug has indeed been fixed, but there's potentially now a conpty EOL bug that is causing the same problem.

@j4james I cannot reproduce it in 1.19.3172.0.
Pretty sure I did reproduce it in Terminal Preview before, but maybe not. ¯\_(ツ)_/¯


Fix Index not found Notices

I use the zend annotation module, it scans the whole Module directory with this the ClassScanner class from this module. In the config directory it writes some notices to the output - I can't avoid this without raise the php log level, so I think it's an issue of this module to avoid that output.
Notice: Uninitialized string offset: 1 in /.../vendor/zendframework/zendframework/library/Zend/Code/Scanner/ClassScanner.php on line 974

Notice: Uninitialized string offset: 1 in /.../vendor/zendframework/zendframework/library/Zend/Code/Scanner/ClassScanner.php on line 974

Notice: Uninitialized string offset: 1 in /.../vendor/zendframework/zendframework/library/Zend/Code/Scanner/ClassScanner.php on line 974

Notice: Undefined offset: 267 in /.../vendor/zendframework/zendframework/library/Zend/Code/Scanner/ClassScanner.php on line 942

Notice: Undefined offset: 681 in /.../vendor/zendframework/zendframework/library/Zend/Code/Scanner/ClassScanner.php on line 942

Added one simple reproducible example in #70


Please explain how to setup this program on Ubuntu-based distributions

This program is not in the package manager for my distribution, that distribution being Mint. So I used git clone to get the code. At that point I discovered that there are no instructions for how to setup and run the code (other than by using a package manager - npm - that I do not have) and no install file and nothing, so far as I can see, that can be compiled. Please explain.

Please see this.

Aha. Thanks. I have never heard of npm. Perhaps your README file should make it clearer that (1) one needs npm, (2) npm will work on any distribution.

Upon executing npm install -g crontab-ui (as root - yes?)  I see that you have 4.2.6.  Try sudo ln -s /usr/bin/nodejs /usr/bin/node

Thanks. However. .
Re 1: the website you link has binaries for what it calls 'Linux on
Power Systems' and 'Linux on System'. To which my response is, frankly:
er, what? If I have to download and compile the source myself, then this
is more trouble than it is worth. (I investigated your program as an
alternative to gnome-scheduler, for that latter had involved me in all
sorts of complications that ultimately did not yield a useable program.)
Re 2: this does not work on its own. I still get an error message when
trying to run your program. Still, I take it that I need to sort out
problem 1 first.
On 19/03/2017 03:52, Suresh Alse wrote:

Hi,

You need to install latest node. I see that you have 4.2.6.  Try |sudo ln -s
/usr/bin/nodejs /usr/bin/node|

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alseambusher/crontab-ui/issues/46#issuecomment-287592438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJiR_gVH5ZmjAObGMyNEav1vEs1P0Vw3ks5rnKaRgaJpZM4MhnfN.

You have to download binaries. There will be binaries of node and npm in the bin folder. You don't have to compile anything.

Ah, I didn't see (can I blame the page layout?) the standard Linux binaries.
Having download them, and unpacked them, I ran the downloaded version of
`npm' with these argments: install -g crontab-ui. I still could not get
it working. I am giving up.

Removing nodejs installed through apt-get
As mentioned earlier add the bin folder of the downloaded node to the PATH.


Save last settings (quality, game)

It would be nice, if the application saves the last selected game and quality settings.

Added in https://github.com/muffe/livestreamer-tool/commit/385253f366b47d2503fbf831e906fa84b4230973


Performance issue after latest pull

The admin area seems to lag after last pull, anyone experienced that?

Hey @BogdanChayka,
can you specify where do you found these performance issues? There was a bootstrapping refactoring which loads the plugins asynchronous after the login. This can cause a small delay between the login and the final interface. All other aspects in the administration should be the same or better.
Thanks,
Jannis

@jleifeld maybe I have to execute commands after pull? Is there anything to migrate? Or do I have to recompile the admin?

Do you use the watcher for development or the production build? The best way is to reinitialize the administration with ./psh.phar administration:init

Closing this due to inactivity


Sticky Lists not working on Chrome

Hi, I am happily testing Framework7 right now :) very impressed.
I found anyhow a bug, running kitchen-sink-material:

On chrome all sticky headers are not sticking. Impacted:
List View
Contact View

Example Chrome vs. Safari:

System:

Mac OSX El Capita, 10.11 (latest Update)
Chrome: Version 46.0.2490.71 64-bit (latest Update)

it doesn't work on chrome because it uses position:sticky   which was temporarily  removed from chrome.
You can read more about it Here.

... temporarily removed... for over a year now... (-.-)
Support of position:sticky looks really terrible right now

Any idea for a workaround to bridge the behaviour to chrome (latest)? 18% Global support is really a bummer.

You can find few polyfills in the end of this article http://www.sitepoint.com/css-position-sticky-introduction-polyfills/

I think we can close it as Chrome-specific issue, not the F7


Clarify licence of FFTW

[skip ci]

I want to say I'm really intrigued by your package(s). E.g.
https://github.com/pabloferz/DLPack.jl (and https://github.com/rejuvyesh/PyCallChainRules.jl)

supports interfacing through both PyCall and PythonCall.

In general, if you can't beat them (Python), join them! And that goes for AI/ML in Python (but also in R).
I don't know much about making Python C extension, but 1) I believe this may be simpler, 2) nor about making extensions for R, but at least 3) I don't think extensions for one language is compatible with the other, because of extension API, do you know if there's a project to make extensions for both? The wrapper C code could still be the same (I however think C++ might be (more used) with R, and C more with Python?
I really would like for Julia to replace C/C++ for both those languages, mainly for selfish reasons, not that I need to call from either, but to attract their developers to the Julia ecosystem. You could just let other do similar work for R and/or should it be part of this package (if anyone wants to help)?
The package seems to work as advertised, while I might add one line to the docs:

This is how you use it:
from example import jl_mat_mul
import jnumpy as np

The example failed without the latter import, and I also tried rater "import numpy as np", and it doesn't seem to matter, I guess why the name of your package, since it imports numpy. That's probably sensible, since most users are going to be calilng Julia for array (2D+) support.
I see in the roadmap "JNumba", can you clarify, is it just to support, numba, and why needed?
As expected this approach here has the same (small) TTFX latency, but I noticed your other work, that likely will help: https://discourse.julialang.org/t/a-prototype-of-pkgimage-binary-cache-system-for-reducing-latency/85610

Thanks for adding the license clarification, we should really notice this earlier.

It seems JNumPy is sort of redundant with PythonCall and pyjulia, except to allow more transparent use and installation?

JNumPy uses TyPython under the hood. TyPython is highly inspired by PythonCall, but it focuses on Python users instead of Julia users. The functionality should be covered by PyCall v10 or PythonCall v10, but not now.
Currently, the major difference (of our interest) between TyPython and PythonCall is:

TyPython does not load external CPython symbols at Julia's __init__ time. We leave the tasks/rights to downstream developers.
TyPython does not (intend to) support any implicit data conversion.
TyPython is specially designed with relocataility in mind to support relocatable system images.

You could just let other do similar work for R and/or should it be part of this package (if anyone wants to help)?

I'm not familiar with R's low level APIs, but it is considerably feasible to use the same interfaces for R and Python. I'm willing to help one to create a Julia repo for R extensions using JNumPy's approach.

The package seems to work as advertised.

Sorry.. I'd clarify that we have not yet advertised it. I'm not the author of that HN post, and we're not ready to do any advertising.

That's probably sensible, since most users are going to be calilng Julia for array (2D+) support.

We're considering replacing some slow functions in numpy with julia implementations. Ideally, in the future, using JNumPy instead of numpy gives free lunch without any compatibility issue.

I see in the roadmap "JNumba", can you clarify, is it just to support, numba, and why needed?

I'm not ready to introduce JNumba as it highly depends on the progress of https://github.com/JuliaCN/Py2Jl.jl .
The goal of JNumPy is to provide the same user APIs as those from Numba, with extra support such as generic data structures and aggressive const propagation from the Julia ecosystem. Then users only need to replace import numba with import jnumba to migrate the code and get better code abstraction via multiple dispatches, as well as optimization-related stuffs.

Will merge this on the next weekday😄


Ergin java implementation

Completed task using Java and Spring Boot framework. Unit tests and integration tests are included. For more information please refer to Readme file.

Thank you for taking the time to apply and do the assignment.
We are reviewing all candidates and we will get back to you soon with feedback.


Shortcut keys

issue https://github.com/Raathigesh/majestic/issues/124
Added shortcut keys for run all tests, run selected test, open search and toggle watch

This looks good to me. Hooks are so good for stuff like this. Thanks for sending the PR 💯
I just left a single comment. Let me know what you think.

Looks good. I'll try to make a release this weekend.

Yayyyy!!! Great work guys. Been waiting for this!


Add new build configurations for desktop CLR

Add new "Debug.Desktop" and "Release.Desktop" build configurations that produce binaries targeted at the desktop CLR
Add abstractions for platform detection and setting the environment of launched processes to support running on Mono

Please review:
@jacdavis @gregg-miskelly @chuckries @wiktork @richardstanton @caslan

This will get merged into the release branch first, and the remaining CR feedback will be addressed separately before merging back to master.


Suggestion: Change PHP file icons

It would be nice to have nice PHP icons ))

Yes it would. But I haven't found any good idea to make better PHP icons...
Do you have any suggestion ?

Some icons from the web:

More like this ?
I think I'll keep this color as it's the one from the official logo

Maybe try to set text color to white?

The earlier one was using transparent text, so it would appear white with a light theme and black with a dark theme

Like your last one. For me white text looks more appealing)
You also have .html, .css, .sass icons with white text on it. Just checked with light color theme. Looks good)

The reason these have white text is because the icons are made using the official logos (which use white text). The official PHP logo uses black text. However if you are using a light theme, then the transparent text version looks like this :

OK, I got it about tranparency ) I just like the white text color on both light and dark theme )
Anyway, decision is up to you )


updates for chia-blockchain-1.8.1

Update setup.py to include chia-blockchain 1.8.1
Use changed output from conditions_dict_for_solution

Looks like there are a couple other places the fee_str pattern could be applied.


Web: PhysicalModelLayer doesn't check elevation

Currently physical_model_test.dart disables some tests because we do not check elevation on the Web the same way mobile Flutter does (or at all). We need to match the mobile behavior.

These tests are still skipped on the master
https://github.com/flutter/flutter/blob/018467cdb13bb6bce5d2d0d98ca754d874706092/packages/flutter/test/widgets/physical_model_test.dart#L264-L268


Avoid passing null

Issue created from code comment with imdone.io

TODO: @Sandy Avoid passing null id:139
src/vs/workbench/services/configuration/node/configuration.ts:458
@imdone - Efficiently manage your project's technical debt. imdone.io

Issue closed by removing a comment.

TODO: @Sandy Avoid passing null id:139 gh:154
src/vs/workbench/services/configuration/node/configuration.ts:458
@imdone - Efficiently manage your project's technical debt. imdone.io


Many compiler warnings for Swift3

Hi Stuffrabbit,
I like your CoAP implementation of Swift.
But when using XCode 7.3 it seems to already have support for Swift3, i.e next version after Swift2.2. Thus when using SCClient, SCServer or SCMessage about 20 compiler warnings appear, for example:

SCClient.swift:208:129: Use of string literal for Objective-C selectors is deprecated; use '#selector' instead
SCClient.swift:205:34: '++' is deprecated: it will be removed in Swift 3
SCMessage.swift:574:67: 'init(start:end:)' is deprecated: it will be removed in Swift 3.  Use the '..<' operator.
'sendAsynchronousRequest(_:queue:completionHandler:)' was deprecated in iOS 9.0: Use [NSURLSession dataTaskWithRequest:completionHandler:] (see NSURLSession.h

Will you make an update of SwiftCoAP to fix those compiler warnings?
See here for more details about Swift 3:
https://github.com/apple/swift-evolution
https://swift.org/blog/swift-3-api-design/

Hey, yes I am aware of this. I will provide an update, as soon as I'll find the time :)

Ok, I've tested to fix the warnings by myself and the compiler had suggestions for most of them, the other ones I solved manually. So I will close this question.


Added the autocomplete attribute to the signup-form and turned it off.

Fixes: #250
Signed-off-by: Jürgen Löhel<EMAIL_ADDRESS>

LGTM


Migration: Chef-server migration to A2HA

User Story
Chef-server Migration to A2HA Cluster
Setup the Infra

Install the Stand-alone chef-server.
Setup the A2HA (3 Node Cluster).

Note :Need to check dependency on the version on ES and PG.
Add Data to Chef-Server

Add the data to the chef-server node like Org, nodes, cookbooks, databag etc.
2.  Add at-least 3 Org, with more than 2 node in each org.
Each org will have number of cookbooks, data-bags etc...

List of the item store in the PG, ES and locally.

Testing

To do the backup (chef-server-ctl / knife) commands.Purpose of both command and difference.
Take the backup either in the zip/tar files.
Restore the backup to chef-server
Post restore, how the node will get to know the new a2ha-cluster URL.
Populate the new server url to all the node.
Make sure that it would received the data from the node.

note: Is there any way to move data directly from oe machine to another (not to create a zip.tar).
Definition of Done

List out the stand alone chef-server version to A2HA cluster(what we supports).
List out the data will migrate via command.
List out the data need to copy/migrate manually (which are the part of DB).
Steps by step guide to do the steps.
Capture the error in the FAQ section.(keep the error in text and image both the way separately).
Post restore the

Demo Script / Repro Steps

Script to do the task
ReadMe section

Closing this issue, as chef server migration to HA effort is getting tracked in below list of issues:
https://app.zenhub.com/workspaces/the-delta-quadrant-5fc7c4c3922dc10021323fa6/issues/chef/automate/5518
https://app.zenhub.com/workspaces/the-delta-quadrant-5fc7c4c3922dc10021323fa6/issues/chef/automate/5522


just: automate setting up RHEL containers

It should be easy for people who work with RHEL to use our stuff. Looks like what we need is documented here:
https://debarshiray.wordpress.com/2023/08/25/fedora-meets-rhel-upgrading-ubi-to-rhel/
It'd be neat to set it up exactly how they want it with toolbx too, let's not make editorial decisions on their preferred tools.

Sounds like this is simply adding a just directive (or set of them) which would do this: toolbox create --distro rhel --release 9.2 maybe with parameter... and ideally using distrobox....
But... maybe this would also be a reason to stop removing toolbox, too... RHEL users may really want it as a fallback for their use case.

My Head Hurts
So, the landscape is confusing in regards to Red Hat Toolbox Containers. I'm going to ask some questions tomorrow to hopefully get a better understanding of what customers and individuals should be using.
UBI
Images are available to use completely for free and can be accessed from registry.access.redhat.com without a subscription. The FAQ clearly indicates that anyone can use these images. https://developers.redhat.com/articles/ubi-faq#
quay.io RHEL toolbox images
Timothy has been managing these and they seem to be mostly up to date. They are slightly behind the official toolboxes which I will get to next. These exist here: https://quay.io/repository/toolbx-images/rhel-toolbox?tab=info
Official Toolboxes
These require a subscription since they are hosted at registry.redhat.io This would require you to be a customer (or maybe a developer account can access these?) These seem to be the most up to date with current RHEL releases and are specifically designed to work inside of Toolbox.
RHEL 8
https://catalog.redhat.com/software/containers/rhel8/toolbox/611bd18eeb34b73652492a75?architecture=amd64&image=6543c1d79fc22886ccc4df72&container-tabs=overview
RHEL 9:
https://catalog.redhat.com/software/containers/rhel9/toolbox/61532be3bc74e0be7ca66e40?architecture=amd64&image=6571634acb7b2ef3e5ae8937&container-tabs=overview
Other Toolbox Images
From the offical toolbox repo there are several out of date containers there for RHEL: https://github.com/containers/toolbox/tree/main/images/rhel
Conclusion
My gut tells me we should use the official toolbox images from registry.redhat.io, but this will require the user to log into the registry in order to pull the images. This also offers more options since every minor release is available in case they don't want just :latest.
However, if we think just having one for rhel-ubi8 and rhel-ubi9 would be good, I can add those as separate ones and those shouldn't require a login.

I want to clarify that I assumed the use of distrobox in my earlier comment, but then re-read (and I think discussed with @castrojo) and I believe we'd expected toolbox to be used for these RHEL *boxes.


Since Symfony 2.4.0, the "timer" and "timeline" no longer work in the Profiler.

Since the upgrade to Symfony 2.4.0, the toolbar no longer lists the execution time, and the profiler timelines do not work either. As far as I can tell, this is because of the changes in Symfony\Component\HttpKernel\DataCollector\TimeDataCollector, but it might be something else.
Minimal testcase:
composer.json:
{
    "require": {
        "silex/silex": "~1.1",
        "silex/web-profiler": "1.0.*"
    }
}

index.php:
<?php

use Silex\Provider;

require_once __DIR__.'/../vendor/autoload.php';

$app = new Silex\Application();

$app['debug'] = true;

$app->register(new Provider\ServiceControllerServiceProvider());
$app->register(new Provider\TwigServiceProvider());
$app->register(new Provider\UrlGeneratorServiceProvider());

$app->register(new Provider\WebProfilerServiceProvider(), array(
    'profiler.cache_dir' => __DIR__.'/../cache/profiler',
    'profiler.mount_prefix' => '/_profiler', // this is the default
));

$app->get('/', function () {
    return "Hi!";
});

$app->run();

Result:

This is happening to me yet... using Silex 1.* and silex/web-profiler 1.0.*. Any suggestion?


Support Wide Displays

Is your feature request related to a problem? Please describe.
Currently Refract only works with Looking Glass Portrait and only displays the center 1/3 of the game.
Describe the solution you'd like
Support for wide screen LKG displays and full-frame games.
Describe alternatives you've considered
N/A
Additional context
I can think of 3 potential ways of doing this:

Switch to rendering horizontal. The problem with this is that you have to get the game to render in half the vertical resolution but at the 2:1 compress ratio. This has to be done in Reglass (Reshade) before you can try to stretch it back out in Refrat to the full height and proper aspect ratio. You would need to add another shader pass in Reglass and expose an option for it in the shader settings. In Refract, you'd have to change the parameters on the UDD (desktop capture) object to capture the vertical half rather than the horizontal half. And you'd have to add an extra pass in the Refract displacement shader to stretch the result.

Interlace. Rather than splitting the screen half and half, you would do every-other line. This might require less effort because you wouldn't have to squish. You'd only have to stretch. But the process would be similar to the above.

The right way to do it would be to write a Reshade native plugin. These are new and are in Reshade 5. They're written in C++. The plugin would theoretically have access to the full color frame and the full depth buffer frame without screen splitting at all. The C++ plugin would then have to send this data to Refract over ICP (inter-process communication). Refract would need to be modified to receive frames over ICP rather than screen scraping. This is no small undertaking, but it's really the right way to do it if possible. It would get around the desktop duplication limitations some users are facing, and it would allow Refract to work on a much wider range of displays.

I haven't started this work because I don't have a wide screen display to test it on. If any users are thinking about implementing this, I highly recommend trying to implement it this way.

I would potentially be up for tackling this, but I'd need a wide format display here myself to do it. This is because I would need to write a C++ plugin for Reshade and I would need to do a bunch of constant testing to get it right. If the folks at Looking Glass would like to donate a display for the purposes of supporting other orientations, I'd be happy to look into it. But it would probably be later this year before I could get to it due to other projects I'm working on right now. FYI, one benefit to this work would probably also be lower latency due to no longer needing to do screen scraping.


fix heap overflow with in-predicate on nullable columns (#14319)

Losing segmentid info will mess up the _segment_id_to_value_in_dict_flags map in InListPredicate,
causing two distinct segments to collide and crash the BE at last.
Root causes of the problem:

NullableColumn use nested column but the rowsetid&segmentid info cannot pass on to the nested one.
So overrided set function should be provided instead of use the dummy virtual one in the base class.
Set function only called in short_cir_eval and pre_eval while delete_range and delete_bloom_filter are left.

Signed-off-by: freemandealer<EMAIL_ADDRESS>Co-authored-by: Pxl<EMAIL_ADDRESS>Proposed changes
Issue Number: close #14319
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

need merge this to 1.2


Remove selenium-driver, update rubyzip

Our code depends on rubyzip but we only pull it in transitively from selenium-driver, which we're not actually using.

selfie


Do not send back node data when specified

remove node and prevNode when noDataOnSuccess is set

This addresses issue #3363
Not crazy over the flag name but I am open for other names.


SAML: fixed userid cookie Path

Description
Fixes: #6365
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[ ] Critical
[X] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
Please, see the details in #6365

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6366 (SL-JID-1530)


Add accessibility statement for signon

Trello

This looks good to me @CristinaRO! Could you add a screenshot of the new page to the PR description? This helps team members who don't have a development environment set up to more easily check/understand the changes (when they result in end-user visable changes).

Could we add an item asking for screenshots of any visual changes to the PR template please?

This looks good to me @CristinaRO! Could you add a screenshot of the new page to the PR description? This helps team members who don't have a development environment set up to more easily check/understand the changes (when they result in end-user visable changes).

Could we add an item asking for screenshots of any visual changes to the PR template please?

It's not always needed, so I don't think it's necessary to add it to the template. It's more of a team convention - you can check some of the recent PRs to this repo to get a feel for how we currently do things.


Feature Request: Extract Method when nesting depth exceeds limit

Hey,
could you add a feature which offers to extract part of a method if the nesting depth of the current method exceeds a certain limit? I have a on going struggle with some of my developers who fail to keep their methods short and concise and this way, they would at least get a warning as well as a proposal on how to fix their method.
Thank you for your great extension! I have downloaded it last week and really liked it so far.
Arne

Would be interesting to write an analyzer for that.


fix: improve error message if subnetID is not provided for Storage Module

Description
Update the subnetID to be conditionally required based on enableVNET. This provides a more clear error message to users about what has gone wrong if they enableVNET without provided a subnetID.
Previous Error Message
status: Failed
error:
  code: DeploymentFailed
  target: >-
    /subscriptions/edf507a2-6235-46c5-b560-fd463ba2e771/resourceGroups/dcibTestDeploy2/providers/Microsoft.Resources/deployments/main.test-230407-2204
  message: >-
    At least one resource deployment operation failed. Please list deployment
    operations for details. Please see https://aka.ms/arm-deployment-operations
    for usage details.
  details:
    - code: ResourceDeploymentFailure
      target: >-
        /subscriptions/edf507a2-6235-46c5-b560-fd463ba2e771/resourceGroups/dcibTestDeploy2/providers/Microsoft.Resources/deployments/test1
      message: >-
        The 'AzureAsyncOperationWaiting' resource operation completed with
        terminal provisioning state 'Failed'.
      details:
        - code: DeploymentFailed
          target: >-
            /subscriptions/edf507a2-6235-46c5-b560-fd463ba2e771/resourceGroups/dcibTestDeploy2/providers/Microsoft.Resources/deployments/test1
          message: >-
            At least one resource deployment operation failed. Please list
            deployment operations for details. Please see
            https://aka.ms/arm-deployment-operations for usage details.
          details:
            - code: StorageAccountOperationInProgress
              message: >-
                An operation is currently performing on this storage account
                that requires exclusive access.

New Error Message
status: Failed
error:
  code: DeploymentFailed
  target: >-
    /subscriptions/edf507a2-6235-46c5-b560-fd463ba2e771/resourceGroups/dcibTestDeploy2/providers/Microsoft.Resources/deployments/main.test-230407-2204
  message: >-
    At least one resource deployment operation failed. Please list deployment
    operations for details. Please see https://aka.ms/arm-deployment-operations
    for usage details.
  details:
    - code: InvalidTemplate
      message: >-
        Deployment template validation failed: 'Template parameter 'subnetID'
        was provided an invalid value. Expected a value of type 'String, Uri',
        but received a value of type 'Null'. Please see
        https://aka.ms/arm-create-parameter-file for usage details.'.
      additionalInfo:
        - type: TemplateViolation
          info:
            lineNumber: 1
            linePosition: 709
            path: properties.template.parameters.subnetID.type

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Enable GitHub Worksflows in your fork to enable auto-generation of assets with our GitHub Action.
To trigger GitHub Actions after auto-generation, add a GitHub PAT as a secret in your forked repository called PAT.
Adding a new module

[ ] A proposal has been submitted and approved.
[ ] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.

Updating an existing module

[ ] This is a bug fix:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.
[ ] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[ ] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

[ ] I have updated the examples in README with the latest module version number.

/azp run


[preview]docs(dpg): move sample/data schema out of comments into xml …

…files
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.

closing since it's covered in #2774


Fix protbuf version

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: failed

Thus the PR was not passing and not merged.

@conda-forge-admin, please rerender

Hi! This is the friendly conda-forge automerge bot!
Commits were made to this PR after the automerge label was added. For security reasons, I have disabled automerge by removing the automerge label. Please add the automerge label again (or ask a maintainer to do so) if you'd like to enable automerge again!

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11965523827. Examine the logs at this URL for more detail.


Allow Krikri::Indexer#run to receive an (unused) activity_uri

Despite Krikri::Indexer using Krikri::EntityConsumer to identify the activity for which to index records, Krikri::Indexer#run still needs an activity_uri parameter to meet the expectations of the interface
for Krikri::Job#self.run.

This addresses the issue I described in Slack late yesterday, where indexing jobs were failing after being enquequed. The jobs would fail with an ArgumentError when the #run method would be invoked (wrong number of arguments (1 for 0)).

Also, as far as I can tell, unless this is merged we won't be able to index records from enqueued jobs at all.

It sounds like we have a problem in Job.  Maybe it ought to send empty arguments if activity_uri is nil.  I may submit a second commit to this branch doing that--or just adding some shared examples for SoftwareAgents, but will merge tomorrow unless I have something I can do in short order.
:+1:

Yeah, after looking at this further, what I was overlooking was the relevance of activity_uri parameter is different depending on the implementation of SoftwareAgent. Krikri::Harvester, Krikri::Mapper, and Krikri::Enricher use activity_uri to add PROV-O assertions, unless it is nil. Admittedly, it's not necessary for Krikri::Indexer since it doesn't modify resources in Marmotta.

Does that look better?

Yep! I started working on something similar, but your solution is a bit more thorough than mine. Do you want to rebase to get rid of my original commit and your reversion out?

Will do.

:squash:`d


for Version 1.1.1 game does not start on FF3PR

The file export completes successfully, but the game does not start.
BepInEx_UnityIL2CPP_x64_3a54f7e_6.0.0-be.571+ v2024.05.02
LogOutput.log

I tried again and it was exactly the same.
LogOutput_01.log
LogOutput_02.log
LogOutput_03.log

@melmount , looks like import of something leads to the crash. Could you turn off import and try again? (Or just remove exported files from SteamingAssets\Assets folder). If it'll help, you need to try find the file that cause the crash. For example remove half of exported files and try to run the game.

Thank you for your reply. I apologize for the delay in responding and expressing my gratitude.
Regarding your points, I would like to report what I have confirmed.
From the logs and your observations, it appears that the files causing the crash are likely image files. Therefore, I deleted the files in the folder StreamingAssets\Assets\GameAssets\Serial\Res.
After that, I started the game, and it launched successfully. I also confirmed that the memoria function (You can increase game speed (Default Key: F1)) is working.
Since I am not using any mod application features, it is sufficient for me that the modified CSV files are reflected in the game.
(I confirmed that non-sellable items appear in the shop and can be purchased.)
I have only confirmed this with FF3 so far, but I plan to try the same with other titles.
Thank you very much.

Got it. Looks like: https://github.com/Albeoris/Memoria.FFPR/issues/37 and https://github.com/Albeoris/Memoria.FFPR/issues/23 :(


Documentation is woeful

Why is the Readme.md so sparse? There are even comments making excuses for this - "by design", so it's a least partially acknowledged. If it is by design, it's a horrible design.
Contrast your readme to this from the nodejs mqtt package: https://github.com/mqttjs/MQTT.js
Some basic samples of subscribing and publishing are surely an entry level requirement for the readme? The code in samples is woeful, showing a toy console project that doesn't even properly decode the payload but rather simply serializes the entire MqttApplicationMessageReceivedEventArgs which does including the payload byte[] but zero samples of how to convert this.
Even including this makes things more obvious:
  var utf8 = Encoding.UTF8.GetString(e.ApplicationMessage.Payload);
  var json = JsonConvert.DeserializeObject<MqttNotification>(utf8);

I work with open source repo's all day and every day and have never felt the need to raise an issue like this. It's properly inadequate.

I was thinking the same thing. Spent some time trying to find a website or something to look for proper docs, only to discover they are nonexistent. I am open to helping document where possible, but it needs to be something the authors are open to.

This project is open source... So you could just be the change you want to see

We already made a huge progress by adding comments to the methods etc. including protocol details.
I agree that adding some quick start information to the README is beneficial for new users. I will add this to the README soon.
Since this is an open source project you are welcome to create a PR with proposals for how to make this project a better one.


Add way to specify executorservice to be used when returning commercetools SDK result

Is your feature request related to a problem? Please describe.
Modern async Java applications need control on which Threadpool which tasks are running.
Currently, by default, all responses returning from commercetools SDK will continue running in a thread managed by a threadpool from the SDK.
Describe the solution you'd like
Add a way (potentially via Middleware?) to automatically return from any call to commercetools SDK running on a thread from a configurable threadpool AKA Executor
Describe alternatives you've considered
Alternatively, we could manually do:
ctApi.<omitted>.execute().thenApplyAsync(e -> e, myExecutorService);
on each call to commercetools. However, this would clutter the code and can be easily forgotten. Same for any explicit decorator that would be needed.

It's already possible to define the executor for the underlying HTTP client. E.g. for OkHttp:
ApiRootBuilder.of(new CtOkHttp4Client(new ForkJoinPool(), 64, 64))
   .defaultClient()

As some middlewares use also executors e.g. RetryMiddleware or QueueMiddleware you will have to configure there the executor too (available in the builder since 8.10.0).
ApiRootBuilder.of().withRetryMiddleware(new ForkJoinPool(), 3)

For the OAuthMiddleware it's a bit more complicated as the token provider has to be instantiated before
ExecutorService executorService = new ForkJoinPool();

ClientCredentials credentials = ClientCredentials.of()
        .withClientId(CommercetoolsTestUtils.getClientId())
        .withClientSecret(CommercetoolsTestUtils.getClientSecret())
        .build();

TokenSupplier supplier = new InMemoryTokenSupplier(
        executorService,
        new ClientCredentialsTokenSupplier(
                executorService,
                credentials.getClientId(),
                credentials.getClientSecret(),
                credentials.getScopes(),
                tokenEndpoint,
                HttpClientSupplier.of().get()
        )
);

ApiRootBuilder.of().defaultClient(apiEndpoint).withOAuthMiddleware(
        OAuthMiddleware.of(
                executorService,
                new OAuthHandler(supplier)
        )
);

We will still investigate how to simplify this or use a centrally defined executor.

Besides this you could always add a middleware specifying the executor:
ApiRootBuilder.of().addMiddleware((request, next) -> next.apply(request).thenApplyAsync(Function.identity(), executorService));

FYI: In the more recent versions the Client & ApiRootBuilders have methods which allow to specify the ExecutorService to be used for each of the middlewares and handlers.
Still checking if a central defined executor may be useful


Query on Morph Net Structure Exporter

I have a Keras Sequential Model and I train the model using,
model.fit(x_train.values, y_train.values, batch_size=1024, epochs=10, verbose=2, validation_data=(x_valid,y_valid))
I have configured morphnet to this model. In this case, is there a way to save the model structure using structure exporter?

I think that the StructureExporter should still work regardless on how you trained it.

Thank you for your reply! I will look into it.


Reset index function

Is there a function on the data frame similar to the pandas reset_index method for adding the values in the index to the dataframe. I often find myself wanting to re-incorporate the index back into my dataframe after using group_by.
I don't see anything and wanted to check before trying to add it.

Should be a good addition. How will the new vector that the index will create be named? Also, the new index that the dataframe will get will be numbered from zero right?


Bound Streaming Visualizations

I looked at the source code briefly and did not see any sort of array limit for streaming visualizations. This limit is necessary to bound visualization data from appending perpetually.
One case where this is observable is on the http://lightning-viz.org/visualizations/streaming/#streaming page. If left to continue for some time in chrome, the tab will crash.
I apologize if I'm missing a config parameter or something else, but this seems like it would be a perhaps necessary feature for streaming visualizations.
Looking forward to the talk tomorrow!

true ... i think quick workaround is to put instead of post the new data (considering including some previous values) ..


Critical Information Exposure

.NET Critical Information Exposure
Examination of the binaries of the different .NET Core based Xamarin platforms for the same application reveals that information which might be usable by cyber-criminals for using/attacking web-services is embedded. For improved security and stability of applications and systems, such information should be encrypted in the binaries and decrypted as needed during run time. Yes there may be a performance impact, but minimal as the number of elements needing this added level of security typically is small. Perhaps it should be a build option. Examples:
From an UWP exe:
TAction4https://server.domain.com/AcctTReplyAction  T SupportFaultsQ  TAction3https://server.domain.com/PIDTReplyActionQ
From an iOS exe:
TRequestNamespace0https://server.domain.com/TResponseNamespace0https://server.domain.com/TU€†System.Web.Services.Description.SoapBindingUse, System.Web.Services, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3aUse TU€ˆSystem.Web.Services.Protocols.SoapParameterStyle, System.Web.Services, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3aParameterStyle ò 3https://server.domain.com/PID
From an Android APK:
TRequestNamespace0https://identityservices.identitycheckpoint.com/TResponseNamespace0https://server.domain.com/TU€†System.Web.Services.Description.SoapBindingUse, System.Web.Services, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3aUse TU€ˆSystem.Web.Services.Protocols.SoapParameterStyle, System.Web.Services, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3aParameterStyle ò 3https://server.domain.com/PID
Assuming that the hackers obtain a copy of the application and has access to the executables, with that access, simply using Notepad. and in the .exe file, if they search for "http", they can obtain the domain address of a service, e.g. "https://server.domain.com", which represents an endpoint.   In their analysis of the .dll, they can discover the parameters and their types, e.g.:

Further study reveals the objects, properties and the associated constraints that might be in use:
< E n t i t y T y p e N a m e = " A c c t " > < K e y > < P r o p e r t y R e f N a m e = " I D " / > < / K e y > < P r o p e r t y N a m e = " I D " T y p e = " E d m .    Now they know enough to potentially, with some effort and skills, build their own code to access the webservice/api/interface. The example is based on identity use. With Dark Web information, access and subsequent actions could enable criminal activity.

Thanks for the report, but this appears to be security by obscurity. There are even easier ways to get this information by putting a proxy like fiddler between a client and the servers it calls, this doesn't even need an attacker to acquire the executables. It seems to me that obfuscating text in a binary is a very low hurdle to overcome.
For next time though please report security bugs to<EMAIL_ADDRESS>rather than via github. You may end up eligible for a bug bounty that way.

Microsoft Support suggest I start a discussion on the issue here for discussion.
Yes, in the unencrypted world, the proxy approach works nicely, but we have been moving away from that for years. From your perception, you only need to lock the front door as the crook isn't smart enough to check if the back door is unlocked. Maybe in your neighborhood that is normal.
We don't need full cyber-security for games, but for business, government and the military applications we do. I didn't say obfuscating. I said encryption which isn't a muddying of the waters. Every piece of information, if exposed, might become the key to opening the door of the safe to those not authorized. That is what cyber-security is suppose to PREVENT. In .NET based applications, the exposure of this type of information is a major implementation flaw.

I'm going to close this issue -- I don't think it makes sense for us to implement, for reasons given by @blowdart. Thanks.

@alaskanrogue If you're talking about HTTPS, then that secures communication from the client machine to the server machine. But it doesn't prevent somebody with access to the client machine to get to that information.
If you're giving a binary to someone, you should assume that they can understand what exactly that binary can do. No amount of "encryption" (but it is more accurate to call it obfuscation) will prevent that.


Help wanted with DynamoDbItemEncryptorException: Configuration mismatch partition or sort key does not exist in item.

Hi,
I am observing an error software.amazon.cryptography.dbencryptionsdk.dynamodb.itemencryptor.model.DynamoDbItemEncryptorException: Configuration mismatch partition or sort key does not exist in item.  during reading from a global secondary index. The query operation returns successfully, but during reading the returned response the above mentioned error occurs.
Here is everything that i verified:

Table config includes plaintextOverride(PlaintextOverride.FORBID_PLAINTEXT_WRITE_ALLOW_PLAINTEXT_READ)
Table description looks accurate

{
    "Table": {
        "AttributeDefinitions": [
            {
                "AttributeName": "pk",
                "AttributeType": "S"
            },
            {
                "AttributeName": "sk",
                "AttributeType": "S"
            },
            {
                "AttributeName": "gsi1_sk",
                "AttributeType": "S"
            },
            {
                "AttributeName": "gsi2_sk",
                "AttributeType": "S"
            }
        ],
        "TableName": "pii-details",
        "KeySchema": [
            {
                "AttributeName": "pk",
                "KeyType": "HASH"
            },
            {
                "AttributeName": "sk",
                "KeyType": "RANGE"
            }
        ],
        "TableStatus": "ACTIVE",
        "CreationDateTime": "2023-09-07T14:48:44.776000-04:00",
        "ProvisionedThroughput": {
            "LastIncreaseDateTime": "1969-12-31T19:00:00-05:00",
            "LastDecreaseDateTime": "1969-12-31T19:00:00-05:00",
            "NumberOfDecreasesToday": 0,
            "ReadCapacityUnits": 0,
            "WriteCapacityUnits": 0
        },
        "TableSizeBytes": 4524,
        "ItemCount": 4,
        "TableArn": "arn:aws:dynamodb:us-west-2:000000000000:table/pii-details",
        "TableId": "958b7ee0-df96-419d-a378-83fc8e70fca1",
        "BillingModeSummary": {
            "BillingMode": "PAY_PER_REQUEST",
            "LastUpdateToPayPerRequestDateTime": "2023-09-07T14:48:44.776000-04:00"
        },
        "GlobalSecondaryIndexes": [
            {
                "IndexName": "gsi1",
                "KeySchema": [
                    {
                        "AttributeName": "pk",
                        "KeyType": "HASH"
                    },
                    {
                        "AttributeName": "gsi1_sk",
                        "KeyType": "RANGE"
                    }
                ],
                "Projection": {
                    "ProjectionType": "INCLUDE",
                    "NonKeyAttributes": [
                        "identityId",
                        "createdAt",
                        "nickName"
                    ]
                },
                "IndexStatus": "ACTIVE",
                "ProvisionedThroughput": {
                    "ReadCapacityUnits": 0,
                    "WriteCapacityUnits": 0
                },
                "IndexSizeBytes": 4524,
                "ItemCount": 4,
                "IndexArn": "arn:aws:dynamodb:ddblocal:000000000000:table/pii-details/index/gsi1"
            },
            {
                "IndexName": "gsi2",
                "KeySchema": [
                    {
                        "AttributeName": "pk",
                        "KeyType": "HASH"
                    },
                    {
                        "AttributeName": "gsi2_sk",
                        "KeyType": "RANGE"
                    }
                ],
                "Projection": {
                    "ProjectionType": "INCLUDE",
                    "NonKeyAttributes": [
                        "identityId",
                        "createdAt",
                        "nickName"
                    ]
                },
                "IndexStatus": "ACTIVE",
                "ProvisionedThroughput": {
                    "ReadCapacityUnits": 0,
                    "WriteCapacityUnits": 0
                },
                "IndexSizeBytes": 4524,
                "ItemCount": 4,
                "IndexArn": "arn:aws:dynamodb:ddblocal:000000000000:table/pii-details/index/gsi2"
            }
        ],
        "Replicas": []
    }
}

Items are correctly populated in the index, snippet below

"Items": [
        {
            "sk": {
                "S": "LATEST#1ff1b931-4a24-46ec-97ec-5ccd54f8ef2a"
            },
            "createdAt": {
                "S": "2023-09-07T18:51:13.634084084Z"
            },
            "pk": {
                "S": "<tenantId>"
            },
            "gsi1_sk": {
                "S": "LATEST#BENEFICIARY"
            },
            "identityId": {
                "S": "1ff1b931-4a24-46ec-97ec-5ccd54f8ef2a"
            },
            "nickName": {
                "S": "nickName"
            }
        },

Using 2.20.86 version of AWS SDK (more context https://github.com/aws/aws-sdk-java-v2/issues/3923#issuecomment-1597924182)
All the setters for the attributes are public
Also read this class where the error is thrown, but i am not sure what is the correct approach here.

Can you help me with some direction? Also it would be worth adding some examples for Query and Scan operations for this new encryption client. TIA :)

Hi @motica1,
This error would occur when an attribute configured as your partition key or sort key in your DynamoDbTableEncryptionConfig object is not present in the item returned by the query.
Could you provide the code you use to create your DynamoDbTableEncryptionConfig object? An example is here.
These should be set to the partition key and sort key for the table, not for the GSI.
With respect to Query and Scan examples:
Queries and scans with the AWS SDK for Java using the Database Encryption SDK for DynamoDB are performed the same way as you would without the DB-ESDK.
For examples using these actions, see this link.
Our examples primarily focus on how to configure the DB-ESDK in your code to work with the AWS SDK for Java's DynamoDB clients.
We do not explicitly maintain examples demonstrating Queries and Scans because, once you configure your DynamoDB client to use the DB-ESDK, you can perform these actions in the same way as described in AWS SDK for Java documentation.
That said: we have a handful of examples where we create QueryRequests and use those to query our table.
We create these in the same way as the examples for the AWS SDK for Java.
These queries exist in the context of examples for our searchable encryption feature since this feature slightly modifies Query behavior.
You can reference the construction and usage of these queries for your own implementation without setting up searchable encryption.
See here for a basic example, or here for a directory with more of these examples.
You are correct that we do not have examples demonstrating Scan operations.
You can reference the AWS SDK for Java's examples demonstrating Scan operations.
Please provide us with your DynamoDbTableEncryptionConfig config and let us know if we can help further.
Best,
Lucas

Thank you so much @lucasmcdonald3 for your detailed response. I understand that the config should define partition key and sort key accurately and this is where i think there might be an issue.

I am using DynamoDbEnhancedTableEncryptionConfig , referencing the EnhancedPutGetExample. This is how i define my encryption config and it doesn't have an option to define partition and sort key explicitly in the config.

final Map<String, DynamoDbEnhancedTableEncryptionConfig> tableConfigs = new HashMap<>();
tableConfigs.put(
        DDB_TABLE_NAME,
        DynamoDbEnhancedTableEncryptionConfig.builder()
            .logicalTableName(DDB_TABLE_NAME)
            .keyring(kmsKeyRing)
            .allowedUnsignedAttributePrefix(UNSIGN_ATTR_PREFIX)
            .schemaOnEncrypt(TableSchema.fromBean(DetailsDynamoDbBean.class))
            .build());

My schema is such that I share partition key from the base table with GSIs. So essentially i always use the same partition key, with different sort keys (as you would have probably seen from my describeTable output above). My hunch is that my DynamoDbBean class is either missing something or i can't use @DynamoDbPartitionKey and @DynamoDbSecondaryPartitionKey(indexNames = {"gsi1", "gsi2"}) on the same attribute. Here is how i define my schema.

@DynamoDbBean
@Data
public class DetailsDynamoDbBean {

  private String pk; // tenantId
  private String sk; // version#identityId
  private String gsi1_sk; // version#identityType
  private String gsi2_sk; // version#nickName
  private String identityId;
  private String createdAt;
  private String nickName;

  @DynamoDbPartitionKey
  @DynamoDbAttribute(value = "pk")
  @DynamoDbSecondaryPartitionKey(indexNames = {"gsi1", "gsi2"})
  public String getPk() {
    return pk;
  }

  @DynamoDbSortKey
  @DynamoDbAttribute(value = "sk")
  public String getSk() {
    return sk;
  }

  @DynamoDbAttribute(value = "gsi1_sk")
  @DynamoDbSecondarySortKey(indexNames = {"gsi1"})
  @DynamoDbEncryptionSignOnly
  public String getGsi1_sk() {
    return gsi1_sk;
  }

  @DynamoDbAttribute(value = "gsi2_sk")
  @DynamoDbSecondarySortKey(indexNames = {"gsi2"})
  @DynamoDbEncryptionSignOnly
  public String getGsi2_sk() {
    return gsi2_sk;
  }

  @DynamoDbAttribute(value = "identityId")
  @DynamoDbEncryptionSignOnly
  public String getIdentityId() {
    return identityId;
  }

  @DynamoDbAttribute(value = "createdAt")
  @DynamoDbEncryptionSignOnly
  public String getCreatedAt() {
    return createdAt;
  }

  @DynamoDbAttribute(value = "nickName")
  @DynamoDbEncryptionSignOnly
  public String getNickName() {
    return nickName;
  }

  public void setPk(String pk) {
    this.pk = pk;
  }

  public void setSk(String sk) {
    this.sk = sk;
  }

  public void setGsi1_sk(String gsi1_sk) {
    this.gsi1_sk = gsi1_sk;
  }

  public void setGsi2_sk(String gsi2_sk) {
    this.gsi2_sk = gsi2_sk;
  }

  public void setIdentityId(String identityId) {
    this.identityId = identityId;
  }

  public void setCreatedAt(String createdAt) {
    this.createdAt = createdAt;
  }

  public void setNickName(String nickName) {
    this.nickName = nickName;
  }
}

Hi @motica1,
Thank you for the information!
We released a new version of this library (3.1.0) today that improves error messaging in this situation.
This should help us debug your issue.
Could you update to the latest version (3.1.0) and let us know what error message you see?
Thank you,
Lucas

Thank you for the details error handling, i see more descriptive error now.
DynamoDbItemEncryptorException: On Decrypt : Partition key 'pk' does not exist in item. Sort key 'sk' does not exist in item. Item contains these attributes : createdAt identityId nickName.

I think i have to include the partition key and sort key in my attributesToProject. Let me try that out.

well, now i am seeing this error DynamoDbItemEncryptorException: Encrypted item missing expected header and footer attributes.  It wasn't there when i ran the test yesterday.


Styling within chrome-extension:// URL?

I would like to apply styling to a URL that has the pattern chrome-extension://......
My main reason being that the "Recent Tabs with your browsing history" extension (https://chrome.google.com/webstore/detail/recent-tabs-with-your-bro/khjgklaeknnibmeeanmbfjcnjablcpil) only displays as 135px wide in Vivaldi, when it should be 550px wide - see screenshot below. There are other scenarios where being able to style the extension popups or dialogs like this would be useful.

it might be a chrome security policy  that prevents this


How can I get the rqdata value?

How can I get the rqdata value? I couldn't find anything about it.

Rqdata is optional argument. But if you trying to bypass website like discord, you should get rqdata from response. In discord, response looks like {"captcha_sitekey": ..., "captcha_service": "hcaptcha", "captcha_rqdata": , "captcha_rqtoken": ...}. If there is no rqdata in response, then you dont need it.

I will use it on Discord, how can I get it?

You should make request without captcha_key, in response will be captcha_rqdata and captcha_rqtoken. After you solve captcha, make request with captcha_key and captcha_rqtoken parameters

Does this value change each time? (One Time/Daily/Daily etc.)
Or do I have to do this every time?

Rqdata changes every time because it's base64 of random bytes


Camelcase to hyphen?

I'm using hyphenated property names (mostly because Yoga uses hyphenated property names and I didn't want to deal with any mapping), which can be annoying when writing a javascript object literal because you have to quote the key names.
I'm not totally sure I want to add this feature, but it might be a nice to have: some camelCase to hyphenated library so that you can write js object literals without quotes and the renderer will map to hyphenated before pushing to the native code.

Adding this to beta, will close momentarily with #190. Thanks @olidacombe !


Disable test reporting for flaky tests on Helix

While I look into https://github.com/aspnet/AspNetCore-Internal/issues/2482. We might want to disable test reporting of flaky Helix tests so the PR checks are more meaningful.
WIP don't review yet.

aspnet/AspNetCore-Internal#2482 will likely take some feature work to resolve so I think we probably want to merge this and make the helix test check marks more meaningful. @natemcmaster @dougbu . Note that even this test run isn't green but that's because there are flaky tests that haven't been marked as such.

Note that even this test run isn't green but that's because there are flaky tests that haven't been marked as such.

Let's apply [Flaky].

Yup @ @alexperovich  is looking into devops features and workarounds.

Seems like I mis-spoke the helix test failures are not flakiness, they are consistently failing. Will file followup issues.


fix: ensure atom before check compiled

We have something like
defmodule MyApp.Post do
  use MyApp.Resource, comment: "blah blah"
end

and the "blah blah" is passed as an extension to spark formatter.
so before calling Code.ensure_compiled/1,
is_atom/1 or any other type validation should be performed, I think.

All sections are configured by the end-user in the same place, in the entity configuration. i.e
configure :spark, :formatter,
  "Ash.Resource": [
    section_order: [....]
  ]

So you or any users of your extension would need to put the sections in that section_order list.


[TRACKING ISSUE] MC-1075 - fix ReliableTopic totalPublishes metric

The tracking issue for the Java side PR.
See https://github.com/hazelcast/hazelcast/pull/19642 for details.

Previously ReliableTopic stats (published/received messages) were reported only when ReliableTopicProxy was used. No stats were reported If ClientReliableTopicProxy was used. Since ReliableTopic is always backed by a RingBuffer, we can report ReliableTopic stats from the RB operations if the RB name has a prefix "_hz_rb_"
Fixes https://github.com/hazelcast/hazelcast/issues/19555
Breaking changes (list specific methods/types/messages):

none

Label added and therefore reported accidentally. Closing the issue.


Mark C# auto-generated code and enable nullable annotations

Mark C# file contents as auto-generated code.
ref. https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/configuration-options#exclude-generated-code

.NET code analyzer warnings aren't useful on generated code files, such as designer-generated files, which users can't edit to fix any violations. In most cases, code analyzers skip generated code files and don't report violations on these files.

ref. https://github.com/dotnet/runtime/issues/64541#issuecomment-1026018990

From my perspective, the <auto-generated/> is valuable in source to highlight to someone reading the source that trying to change it will be futile as it will be regenerated,  the .g.cs suffix is valuable for someone looking at files on disk to know what's been created by a tool rather than by hand, and [GeneratedCode] is useful for tools that  inspect metadata.

ref. https://github.com/dotnet/runtime/issues/64541#issuecomment-1026123595

The one we are trying to standardize on is the compiler heuristic around generated code.
https://sourceroslyn.io/#Microsoft.CodeAnalysis/InternalUtilities/GeneratedCodeUtilities.cs,1c5dbec99946c19e,references
This takes into account several popular existing hueristics, including extensions. That is genenerally not a great approach. The one we'd prefer is that people have the <auto-generated> header.

Also explicitly enable nullable annotations, since auto-generated C# code is nullable-oblivious by default.
ref. https://learn.microsoft.com/en-us/dotnet/csharp/nullable-references#nullable-contexts

Important
The global nullable context does not apply for generated code files. Under either strategy, the nullable context is disabled for any source file marked as generated. This means any APIs in generated files are not annotated.

Together, this should help verify that nullable annotations are accurately generated, and avoid spurious warnings in consuming apps, e.g.
src/generated/Generated_CS.cs(77,68): warning CS8632: The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
src/generated/Generated_CS.cs(153,22): warning CS8632: The annotation for nullable reference types should only be used in code within a '#nullable' annotations context.
etc.

@nil4 👍 for the auto-generated stuff. Not so sure about enabling nullable. The auto-generated C# code is not (!) annotated with nullability specifiers. Only the target language bindings are annotated with nullability. Maybe there are some exceptions to this for some static code (like converter helper functions, etc.) but all the generated C# code is not nullability aware.

@nil4 I think the static code is where the confusion comes from. I originally wrote this in a nullable enabled context, then put it in beyond.net as is which now causes warnings because everything else is not annotated.
So I think the correct approach is to remove the handful of annotations and actually mark the file with nullable disabled.

Yup, definitely seems that #nullable is best left off disabled here.


OpenAPI spec for webhook call

We expect third parties to implement the webhook contract in order to provide a convention. OpenAPI is the canonical way to describe a REST API. We should publish an OpenAPI spec for this endpoint.

@katmutua Is this done?

The spec is defined here https://github.com/vmware-tanzu/cartographer-conventions/blob/main/api/openapi-spec/conventions-server.yaml


Download/Installation Tutorial

There should be a simple 1 minute video (linked in the README or Cirdles page) that visually shows the user how to download, install, and run sample data. There should be different videos for Windows/Mac as with Mac users will get a popup asking them to alter their System Preferences to run the .jar file.

This is an excellent idea, although keeping the video up to date might be difficult as quickly as the software is changing.
If anyone would like to be assigned this issue, please comment below.

I can make a short video for mac installation if you'd like. My idea for this included putting all the information a new user might need into the README so that all the info is in one place. Easier for them to follow and get set up quickly.

You got it. This might also belong in the soon to be website. @Jmack2, any input?

I think that is a brilliant idea, as well. It would be even more convenient for users!


Is it possible to display whitespace in output?

I'm working on creating some sample code that is heavily commented (as a teaching tool) and rustfmt was unhappy with trailing whitespace when I was using // on a line by itself to break up long blocks of text. the output showed
         // Payload: The response from the function, or an error object.
-        //
+        //
         // For our example, we are just going to interact with the payload
         // portion of the response
         Ok(resp) => {

Which is rather confusing until you copy/paste the response and examine that one contains a space and the other doesn't. Would it be possible for rustfmt to identify this whitespace in some way? maybe like with the little dots that some IDEs use?

Thanks for reaching out! I'd be open to anything that provides more visual cues in these cases, but to be fully transparent doing so is a very low priority. Part of the challenge is that highlighting spacing can produce some very noisy/difficult to process diffs when you consider the entire realm of formatting changes beyond the simple trailing whitespace trim case shared in the description
There's a lot of other considerations that would fall under this bucket as well, such as conversions between tab and space characters, newline styles, etc.

Understandable. I didn't see an Issue for it so I figured I'd open one to track it. I think If I had read the docs before using rustfmt I probably wouldn't have been caught up by this.


Portable mode

Hello,
In the latest version, 1.50.23, there's a new feature called portable mode. By enabling it, I was expecting Wox to let me select a folder to keep all the files and settings, but that wasn't the case and I haven't noticed a difference between normal and portable modes.
The way I understand portability involves having all the files and settings of a programme in a folder that you've selected and that is located in a path known by you, so you can copy everything and move it to a different computer.
The old approach where you could download the ZIP file with the portable version was perfectly fine to me. I wonder why this change was made.
Thank you for your time and sorry if I'm asking about something obvious.

So, my understanding is that it moves everything from ROAMING to LOCAL folder under your username. And presumably you can now move this folder wherever you want... Gonna test this soon.
BTW, I managed to break it by enabling and disabling portable mode a couple of times - and so all my settings were nulled to default. Original setting files were still available in the Roaming folder.

Hello,
In the latest version, 1.50.23, there's a new feature called portable mode. By enabling it, I was expecting Wox to let me select a folder to keep all the files and settings, but that wasn't the case and I haven't noticed a difference between normal and portable modes.
The way I understand portability involves having all the files and settings of a programme in a folder that you've selected and that is located in a path known by you, so you can copy everything and move it to a different computer.
The old approach where you could download the ZIP file with the portable version was perfectly fine to me. I wonder why this change was made.
Thank you for your time and sorry if I'm asking about something obvious.

Hi @vivadavid so when enabled, it removes start menu, desktop and startup shortcuts, and removes the uninstaller entry  for Wox. As a final act it will restart and move your roaming user data folder (%userprofile%\AppData\Roaming\Wox) into the Wox's program folder.
When likewise when disabled, it will reinstate the above.
The Wox program folder is located at %userprofile%\AppData\Local, this is where the default location of where Wox is installed. So when you have it enabled, you can then move that folder around (obviously exit Wox first), and run it from any other location. It is like your previous zipped download.
This is a great question, obviously I work on Wox so I would naturally know where Wox is installed, so do you think after enabling portable mode a prompt to tell the user where the Wox folder is would be nicer or directly open up the Wox program location?

BTW, I managed to break it by enabling and disabling portable mode a couple of times - and so all my settings were nulled to default. Original setting files were still available in the Roaming folder.

haha ok do tell, how you break it? Did you wait for it to fully finish including restarting then trigger disable again or did you do the repetitive tick untick without waiting?

So, my understanding is that it moves everything from ROAMING to LOCAL folder under your username. And presumably you can now move this folder wherever you want... Gonna test this soon.

Yep, can confirm that putting the wox folder into a cloud works & syncs the settings on both computers.

haha ok do tell, how you break it @Satyrars? Did you wait for it to fully finish including restarting then trigger disable again or did you do the repetitive tick untick without waiting?

Well, I've read somewhere that repetitive clicking makes it copy faster. Or Wox doesn't support this feature yet? ˚ ̆ ˚
No-no, I've properly waited till it launches back - but nvm for now, it's way more likely it's something on my end.

Wox is installed, so do you think after enabling portable mode a prompt to tell the user where the Wox folder is would be nicer or directly open up the Wox program location?

I think, the best way would be to prompt user where he wants to put the portable folder (at least, I've expected this behaviour). But a prompt with something like "Hey, your folder is here now. You can move it anywhere you want (but close wox repetitively first)" will do.

@jjw24 , thank you for the thorough answer.
I think it'd be nice if users could get a prompt with a link to access the Wox folder. Directly opening up the Wox folder would be good too.
Like MyGFHasBigTitsAndYouAreStillAVirginHaHa (thank you for your explanation), I was expecting Wox to ask me where I wanted to save Wox, but the workaround above should be helpful enough.

@vivadavid @JustACuteGirlLookingForAManToFixHerBugs PRed. Thank you for your suggestion.


Rename Dashboard Component

The Dashboard component is just a NavBar that is rendered when the user is signed in. We should rename this to SignedInNavbar, and import it at the top of our application like so
Since this NavBar should only be shown when the user is authenticated, you can do something like
render() {
  return (
    authenticated ? <SignedInNavbar /> : <NavBar />
    // rest of stuff here
  )
}

Closed by #181 and #168


Public Key Hash Extraction From Address

Takes an address (e.g. 16UwLL9Risc3QfPqBUvKofHmBQ7wMtjvM) and decodes the public key hash (HASH160) contained within it using an outside Base58Check library.

using an outside Base58Check library.

You haven't done it yourself? D:

If it doesn't break on regtest, :+1:

Feel free to replace the library with a hand-made implementation (i.e. copying the code from somewhere). I'll gladly deny the PR. :+1:
;) Merging.


Add loops in lua language

while loop
for loop
repeat-until loop

Deployed all 3 codes:
https://onelinerhub.com/lua/for_loop


bug: rpc reads on local node

While conducting load-tests on a local Madara node using this tool, we observed the following behavior:

Sending write transactions work well to some extent (see #818)
The tool is supposed to check transactions in parallel to make sure they have correctly been processed by the sequencer and thus have the ACCEPTED_ON_L2 status.

However the witnessed behavior is that the more connections to the node we have, each connection being in a different thread, the slower RPC calls get. This goes up to the point where 100 connections can lead to RPC calls taking a few minutes which is unacceptable and a huge DOS factor.
This clearly shows that some lock is taken by the RPC service and release only after the query has been processed thus slowing down all pending calls.
In the end, it's impossible to efficiently check thousands of transactions in parallel on a local madara node.
This might also be not a big deal if we take into the account that in production environments, we will have RPC only nodes and specialized actors that make sure the load is distributed correctly.
Expected Behavior
Katana on the other hand is able to correctly process read calls in parallel.

Related to https://github.com/keep-starknet-strange/gomu-gomu-no-gatling/issues/18

Can I take this one?


Support Kubernetes 1.22 for ingress

The ingress API version got changed in Kubernetes 1.22.
Thus, all the charts don't work on Kubernetes >= 1.22 when ingress is enabled.

Resolved in the commits:
https://github.com/utkuozdemir/helm-charts/commit/c4591ee7161e2a001d8f65d629bc531621bd4805
https://github.com/utkuozdemir/helm-charts/commit/8339261c3a9092874d3709c7eff0b736e8d26afe
https://github.com/utkuozdemir/helm-charts/commit/c5e88088e6cabc8b0886222abf2a663c0abc606e


rejecting hostname with non-English characters

Describe the bug
the add-on does not accept a hostname with non-English characters.
To Reproduce
put a random hostname with non-English characters in external home assistant host name field and start.
(e.g. "あああああ.jp"; you can register those domains with non-English characters if you may not know...)
Expected behavior
"FATAL: 'あああああ.jp' is not a valid hostname." error does not show up and successfully creates a tunnel.
Screenshots

Add-On / Home Assistant Information (please complete the following information):

Add-On Version  5.1.22
Supervisor Version 2024.10.2
Core Version 2024.10.3
Operating System 13.2
Devices Raspberry Pi 4 Model B

Thanks for raising this. Can you please open an issue in the development repository, since this is only the wrapper repository for the HA installation.


loss=nan occurs during training

When I train with images of the same resolution, loss=0 occurs. It doesn't matter if I reduce my learning rate to one-tenth of what it was.

@dongyyyy  did you solve this ?

I encountered the same problem. I notice that this happens when input batch contains NaN values.

Hi, I would suggest checking your data -- does it contain NaN in the input image or ground truth?

@dongyyyy did you solve this ?

Yes, thank you

@dongyyyy Hi, could you tell me how you solved this problem?


@material-ui/system conflicting with StylesProvider

Expected Behavior 🤔
The expected behavior is for the <Box> (unstable_Box) component to work correctly along with the <StylesProvider>.
Current Behavior 😯
The current behavior is, when there is the <StylesProvider> with the generateClassName and two nested <Box> components, the child inherits the parent's stylization.
Steps to Reproduce 🕹
https://codesandbox.io/s/jv5k40q2jw
Your Environment 🌎

Tech
Version

Material-UI
v3.8.1

Material-UI Styles
v3.0.0-alpha.6

Material-UI System
v3.0.0-alpha.0

React
v16.7.0-alpha.0

Browser
Google Chrome v71.0.3578.98-1

This is because you are using dangerouslyUseGlobalCSS

This is because you are using dangerouslyUseGlobalCSS, if you remove it it works: https://codesandbox.io/s/rm7k9xo2xp

Thank you, it's working.

@weslenng There is an underlying problem here that will fix this problem without disabling dangerouslyUseGlobalCSS though. Currently static rules are being processed incorrectly


Add command to Millumin

Be able to change Board
Command:
/millumin/action/selectBoard (board Number)

added into the core please test

Just tested it,  works great. Thank you


#11514 New cop to catch unescaped ']' in RegExp

This pull request adds a new cop that will catch and provide an auto-fix for unescaped closing square bracket meta characters in a regular expression.

I wonder if the cop is not way too specific in its current form and shouldn't be expand to account for other similar scenarios.

Happy to try and expand this. Did you have anything in mind?

@owst may have some suggestions. I recall he used to work on some Regexp related checks.

@owst may have some suggestions. I recall he used to work on some Regexp related checks.

Yeah, this cop feels closely related to Style/RedundantRegexpEscape.
Having poked about a bit, I think Ruby will warn for an unescaped ] unless it appears as the first character in the pattern (tested with 3.3.0):
$ ruby -e 'p /]/'
/]/
$ ruby -e 'p /]X/'
/]X/
$ ruby -e 'p /X]/'
-e:1: warning: regular expression has ']' without escape: /X]/
/X]/

I agree with @bbatsov - this change feels overly specific at least as-is.
However, perhaps the behaviour in this PR could be achieved by adding a style to the existing cop (perhaps after renaming it?) that allows/requires ] to be escaped even when it is redundant to do so (at the start of the pattern). Similarly, as per https://github.com/rubocop/rubocop/pull/11152 escaping a - is redudant at the start or end of a character class, but I think a similar argument could be made to always require - to be escaped within a character class. There are possibly other cases, too.
As a name to improve on, how about a style name of require_position_dependent to require escapes that are only redundant in certain positions?

I'm closing this PR due to no recent activity. Feel free to re-open it if you ever come back to it.


pythonPackages.karton-core: 4.2.0 -> 4.3.0

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

16 packages built:

python38Packages.karton-asciimagic
python38Packages.karton-autoit-ripper
python38Packages.karton-classifier
python38Packages.karton-config-extractor
python38Packages.karton-core
python38Packages.karton-dashboard
python38Packages.karton-mwdb-reporter
python38Packages.karton-yaramatcher
python39Packages.karton-asciimagic
python39Packages.karton-autoit-ripper
python39Packages.karton-classifier
python39Packages.karton-config-extractor
python39Packages.karton-core
python39Packages.karton-dashboard
python39Packages.karton-mwdb-reporter
python39Packages.karton-yaramatcher


Generalized idm to LaneFollowingDriver type

This allows us to use the PrincetonDriver interchangeably as an idm.

Codecov Report

Merging #10 (42bca01) into master (7ed151a) will decrease coverage by 0.75%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   78.36%   77.60%   -0.76%
==========================================
  Files           7        7
  Lines         513      518       +5
==========================================
  Hits          402      402
- Misses        111      116       +5

Impacted Files
Coverage Δ

src/driving_models.jl
79.14% <0.00%> (-1.93%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good to me!


Implement full server-side codegen

This is the follow-up to issue #26

Now that #26 is closed, we should be clear what is left to implement here (if anything).

Here we should make sure we can generate code for receiving all Thrift types, not only those that are used in the tutorial - for instance, sets are not used there.

I'm looking at the render_rs_type function and it looks like it handles all types.

Good news :) Then I'll close it after all types are implemented (#13)


SplitButton style

split button toggle seems not right

you can solved with import css file on your entry file
import 'kata-kit/dist/style.css';


x/tools/gopls: support converting array to slice when completing

When working with arrays you often need to convert them to slices. It would be nice if gopls offered completion candidates that did this for you:
package main

import "os"

func main() {
	var a [32]byte
	os.Stdout.Write(<>) // offer `a[:]` as a candidate
}

Happy to take this one if no one else is already working on it!
/cc @stamblerre @muirdm

I'm not working on it. Happy to give pointers if needed!

@muirdm I am still interested in tackling this, but was out of office last week so I haven't gotten to it yet. Would likely have time this weekend, but certainly understand if the team needs to go ahead and move forward :)

No worries, take your time.


Support for parsing more than one filter suffix.

Added support to expand abbreviations with several suffixes (see https://github.com/Microsoft/vscode/issues/35984 )
Example input:
ul.search-form._wide>li.-querystring+li.-btn_large|bem|c

(Or, equivalently, ending with |c|bem)
Example output:

|

|

@ramya-rao-a would you take a look?


README: Add information about xclip

I signed it.

Sorry I'm just getting to look at this now.  I've been super busy with my new job.
Speaking of which, it means I no longer have access to merge pull requests here!  Somebody else who still works at Google will have to do that.  :)

Hmm, forgot about that. I can merge it if it looks good to you.

Done.

LGTM.
Thanks for pointing this out and helping us fix it!


Feature: sign up and login to the dashboard

Create login and signup screens.
use sso signup and login using either:
supabase management
auth.js with github/google login

Can i take this issue?

@OshriAsulin assigned to you

done using supabse auth!


Mixed search results

Expand search to issues and users. Options:
Mixed results. Show top 4-5 results for repo/user/issue. Simpler impl, easy UX, but limited results. Maybe have a “show more” cell?
Seg control (scope API on search bar). Worse UX (have to tab) but shows a lot more results.
I lean towards mixed. Thoughts?

Mixed feels like the way to go for sure imho.

Using V3 APIs (https://developer.github.com/v3/search/#search-commits) there's actually quite a few other options we can do also

Search Commits
Search Code
Search Issues
Search Repositories

with Topic
written in Language

Search Users
Search Organisations

Proooobably want to keep it simple to start. Can expand once there's demand.

Definitely wasn't suggesting we do it all now 😅 Just saying what we could do

Suggestion: On searchBarTextDidEndEditing, show two options what to search.
What do you think? @rnystrom
Btw, this issue has a high priority since November. Are we planning to implement it soon?


[WIP] Added content for failed queries graph

Closes #2223
Questions for @vilterp @piyush-singh:

Does the 10-second average apply to this graph?
It shows the failed SELECT/INSERT/UPDATE/DELETE statements, right?

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4f807c5e6adb1330d0c9c5dbe8c44cb5cb2eb992/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/545f7ae45a3c626189c3909ef9de0266c55fbbe3/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3e9b0fdfab7ccc4fc18987b937aa9457ddb31462/

Yes, it's a rate over a quantity sampled every 10 seconds, like many of our other charts.
I think it includes all statements, not just SELECT/INSERT/UPDATE/DELETE. E.g. if you issue a CREATE INDEX on a table that doesn't exist, that is a planning error and will show up here.

@vilterp Need help answering @piyush-singh 's questions

v2.1/admin-ui-sql-dashboard.md, line 44 at r4 (raw file):
Previously, vilterp (Pete Vilter) wrote…
Yes, 99% sure a retry error will be counted here. Here's where the counter is incremented: https://github.com/cockroachdb/cockroach/pull/26731/files#diff-c73c08e8a876b7c79e4e43889d30fcddR95
@andrei, the SQL executor master, can confirm…

The PR says planning and execution errors are captured: https://github.com/cockroachdb/cockroach/pull/29380

Does this make sense: "In the node view, the graph shows the 10-second average of the number of SQL statements issued to the node that returned a planning or runtime error."

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/22b908bf5819e78be3622b921c6ce57eaaa038ee/


Add instances for Int8-64, Word8-64 and Float

Fixes #12

I've updated the code. Also added some comments distinguish the similar looking Int code blocks, and for the other types to get things a bit consistent.

Great, thanks!


BUG: Not working in new editor

related to #26, but it also doesn't work in Source Mode, as far as I can tell.
findUnderWord probably needs to be updated to CM6, and the sidebar buttons also does not seem to work, too.

@chrisgrieser I believe I've fixed it for CM6. Please give it a try on the latest release and let me know

can confirm, works now 👍


RichTextBoxTarget LayoutWithHeaderAndFooter support

Type (choose one): Feature request
NLog version: 4.4.6 (NLog), 4.2.3 (NLog.Windows.Forms)
Platform: .Net 4.5
Current NLog config (xml or C#, if relevant)
LoggingConfiguration newConf = new LoggingConfiguration();

RichTextBoxTarget richTextBoxTarget = new RichTextBoxTarget();
richTextBoxTarget.TargetForm = this;
richTextBoxTarget.TargetRichTextBox = LogBox;
richTextBoxTarget.UseDefaultRowColoringRules = true;
richTextBoxTarget.SupportLinks = false;
newConf.AddTarget("richtextbox", richTextBoxTarget);

LayoutWithHeaderAndFooter fullLayout = new LayoutWithHeaderAndFooter();
fullLayout.Header = $"#Software: {Program.ProductName}\r\n#Version: {Program.VersionName}\r\n#StartTime: ${{date}}";
fullLayout.Layout =  @"[${date} ${logger}] ${level:uppercase=true}: ${message}";
richTextBoxTarget.Layout = fullLayout;

LoggingRule debugAllRule = new LoggingRule("*", LogLevel.Trace, richTextBoxTarget);
newConf.LoggingRules.Add(debugAllRule);

LogManager.Configuration = newConf;

I had a brief look at the RichTextBoxTarget and was unsure why LayoutWithHeaderAndFooter was not used there. I can imagine it behaving similar to a console and printing these. Is there a deeper reason behind this design decision that I have not thought of?

I think it's difficult (in performance) to insert / replace the text in a RichTextBox?
@mikhail-barg any idea?

Yes, but in this case, I don't believe we'd be doing either. We will just be prepending some content in the beginning and appending at the end (Unless I'm confused as to what LayoutWithHeaderAndFooter does).

Sorry for the delay,
the LayoutWithHeaderAndFooter layout write a header, multiple logevents, and when done the footer. e.g. a file. When the file is created the header, and when the file is closed, the footer.
If you need to append info before/after a logevent, you could just append them in the layout (and maybe using `${newline}'

I assume your question has been answered, if not, let us know!


editor-standalone

WIP PR for merging editor-standalone functionality back into editor-ui if required

no longer needed with the go live of https://github.com/RaspberryPiFoundation/editor-standalone