[gws][notice] test coverage improvements

Coverage increased (+1.5%) to 72.138% when pulling 02023bc9fca57e09a27d5e7af90d67842eb6498a on sunny4381:features/gws-notice-coverage-improvements into 52e1224e3fbf2ca9c034da6dcad08bd0b078c508 on shirasagi:master.

Coverage increased (+1.5%) to 72.138% when pulling 02023bc9fca57e09a27d5e7af90d67842eb6498a on sunny4381:features/gws-notice-coverage-improvements into 52e1224e3fbf2ca9c034da6dcad08bd0b078c508 on shirasagi:master.


Issues in "fix pending" get held up when an instance has never worked

There are dozens of issues on the main syzkaller dashboard right now that probably don't need to be there.
They are in the "fix pending" list, their patched count is 20/21, and the only instance the fix is missing in is "missing on: [ci-qemu2-arm32]".  But the dashboard says that the ci-qemu2-arm32 instance has never worked.
Syzkaller should probably be smart enough to realize that if an instance has never worked, don't count it against the issues in the "fix pending" list, and move them to "fixed".
Here's the oldest example of this: https://syzkaller.appspot.com/bug?id=fbf9eaea2e65bfcabb4e2750c3ab0892867edea1

This is a one-off issue that only happens during new instance addition (when somebody is too eager to add a broken instance :)) and it's resolved now. If you want to contribute this improvement a PR is welcome.


Error handling

The problem
When gonb runs into errors, it generates HTML error reports. These are great for a jupyter notebook, but don't render in nbmake
The solution
Add an option to print out raw error messages instead of html reports, set by a command line flag. This works inside nbmake.

Thanks @bagel897 , this looks really great! Having it work well nbmake facilitates testing -- I have to start using, it also for GoNB (lots of things I didn't test because were not easy to).
A few suggestions (all very small "nit-picks"), I hope you don't mind.

In goexec/errorcontext.go, the small functions max, min and inBetween are all used in the new errorLine.go, so they may just be moved there.
errorcontext.go became really small. What do you think about merging it with error.go ? Either way is good, just something to think about.
Because klog injects many other flags, when reading the output of a gonb --help (or go run . --help), it's hard for someone to understand that --raw_error is about the GoNB cell execution error. Maybe comment that on the comment of the flag ? Something like (suggestion) "When GoNB executes cells, force raw text errors instead of HTML errors, which facilitates command line testing of notebooks" ?

Let me know if you would like to change them before merging (or if you disagree), or I'll just merge your PR as is do the small changes later (likely on Monday morning)
cheers, and thanks!!

Sounds good!
errorcontext.go still has the HTML template. But I understand combining the two files, it'd make it simpler. Alternatively, I can move the HTML report part back into errorcontext.go
Ill do that as well

I'll do the changes monday morning, is that alright?

That's perfect, thanks again Ellen.
On Sat, Aug 12, 2023, 08:24 Ellen Agarwal @.***> wrote:

Sounds good!
errorcontext.go still has the HTML template. But I understand
combining the two files, it'd make it simpler. Alternatively, I can move
the HTML report part back into errorcontext.go
Ill do that as well

I'll do the changes monday morning, is that alright?
—
Reply to this email directly, view it on GitHub
https://github.com/janpfeifer/gonb/pull/48#issuecomment-1675737521, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABY5KE7BY7LA3VKE2LEWAGDXU4OSZANCNFSM6AAAAAA3NKM4H4
.
You are receiving this because you commented.Message ID:
@.***>

Thanks for the feedback!

Done
I renamed it to htmlerror and moved all the HTML code back into the file. This leaves it over 100 lines, which should be good. Alternatively, we could move all the error files into their own error module, would that work better?
I updated the help flag


Upgrade libwacom package to version 2.9.0

As part of upgrade, introduce new package python-libevdev and move python-evdev from extended to mainline as dependency.
Merge Checklist

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

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

Summary

What does the PR accomplish, why was it needed?
Change Log

Upgrade libwacom to version 2.9.0 for 3.0
Moved python-evdev from extended to main
Introduce new package python-libevdev

Does this affect the toolchain?
NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

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

Thanks for the review @mfrw


"Save Credential" button does not do anything on OSX

Yubico Authenticator version: 4.3.3
Operating system and version: OSX Sierra 10.12.5
YubiKey model and version: Yubikey Nano FW 4.3.5
Bug description summary:  Save Credential button does nothing.

Steps to reproduce
Try to run Yubico Authenticator and add new keys. Clicking on "Save Credential" does not do anything.
Expected result
I was expecting that a new otp key could be added.
Actual results
The button action is ignored.
Other info
This is a brand new yubikey nano that arrived 1h ago.  Its primary purpose is OTP generation but it doesn't seem to be working. the U2FA feature does seem to work properly because it is already registered with a couple of services.
Logs do not show any relevant information:
2018-03-05T14:13:56+0000 DEBUG [ykman.descriptor.Descriptor.open_device:86] transports: 0x4, self.mode.transports: 0x7
2018-03-05T14:13:56+0000 DEBUG [ykman.descriptor.open_driver:141] Opening driver for serial: None, pid: PID.YK4_OTP_U2F_CCID
2018-03-05T14:13:56+0000 DEBUG [ykman.descriptor.open_driver:143] Attempt 1 of 10
2018-03-05T14:13:56+0000 DEBUG [ykman.descriptor.open_driver:148] Found driver: <ykman.driver_ccid.CCIDDriver object at 0x117c2e588> serial: 5972919, pid: PID.YK4_OTP_U2F_CCID

UPDATE: I've tried to restart my macbook and it is working now. I think I've seen another bug that only a restart would solve so this might be a duplicate.

Thanks for the report. Closing since a restart seemed to resolve it, but feel free to reopen if it happens again.

Same problem here, except it has never worked for me. I've yet to use my Yubikey, as #248 is an issue too.


Text exporter by encounter

Exports encounter-based reports to file output/text2. File TextExporter2.java isn't needed now that I merged it with TextExporter.java. Works when I run it but I'm sure there's parts to improve.

Before I look closely, 2 things:

If TextExporter2.java isn't needed anymore please delete it.
There are some conflicts that will have to be resolved before this can be merged. There are 2 ways to do that, with a rebase or with a merge from master onto this branch.  I usually prefer rebasing but it can be a pain if you have a lot of commits. I can help walk you through that if you're not sure what to do


No error, just a silent crash when creating a pipeline with bad ShaderSetDescription

I missed the part in the "Getting started" tutorial that says "CreateFromSpirv is an extension method defined in the Veldrid.SPIRV namespace", so I assumed the example projects are out of date, and I tried improvising:
ShaderSetDescription shaderSet = new ShaderSetDescription(
	new[] { ...	},
	new[]
	{
		graphicsDevice.ResourceFactory.CreateShader(
			new ShaderDescription(ShaderStages.Vertex, Encoding.UTF8.GetBytes(vertexShader), "main")
		),
		graphicsDevice.ResourceFactory.CreateShader(
			new ShaderDescription(ShaderStages.Fragment, Encoding.UTF8.GetBytes(fragmentShader), "main")
		)
	}
);

That obviously didn't work. The part that bothers me is that I got no error message, no exception; the application just exited with error code 0, on:
pipeline = factory.CreateGraphicsPipeline(
	new GraphicsPipelineDescription
	(
		...
		shaderSet,
		...
	)
);

I'm on Fedora 30, using .NET Core and VSCode, if it matters.

Experiencing a very similar issue on Arch Linux, .NET Core and VSCode, except I'm full on segfaulting + exit code 0.
Exact code I'm running: https://github.com/OpenSAGE/OpenSAGE/blob/1fcf348565d73edc9dbaa242826ecbb26460aecf/src/OpenSage.Game/Graphics/Shaders/ShaderMaterialShaderResources.cs#L44


ENHANCEMENT Improve and fix sync task

ENHANCEMENT Improve and fix sync task

Just for reference, what enhancements were added?

This was written last year and I can't recall.
Commit messages from the project in question aren't helpful.
Would have to dig through our conversations regarding this.

No worries, if I can't remember it either then they probably weren't major :)


Flow rendering refactor

Fix #1916
Fix #2176

Lines

Thick Lines

Thick 3D Lines

Fading Lines

Slowly Fading Lines

3D Fading Lines

Fading Lines With Direction

3D Fading Lines With Direction

Arrow Glyphs

3D Arrow Glyphs

Arrow Glyphs With Stride

3D Arrow Glyphs With Stride

Circle Glyphs

Sphere Glyphs

GUI Review Comments:

Eliminate current Radius widget, which operates in world space,  and just have unit-less radius control with default of 1 approximating a 1 pixel line.

Reduce precision of radius selector to maybe two digits after decimal

Replace "stream" in Render Type selector with "curve" or "line"

Add a grouping widget to better indicate related capabilities. E.g. fade controls under Fade Flow tails

Add a less expensive sphere primitive that is less likely to blow up inexpensive graphics cards

Arbitrary limits on sliders such as "Every N Samples" should be changeable by user

Add controls for illumination equation when lighting is enabled (e.g. ambient, specular, diffuse). Be consistent with lighting controls used elsewhere.

Remove "light from camera" option or make it the default

Support use case where user wants a single glyph displayed for each seed advected

Explore operating in units "time steps" instead of number of samples.

2 and 6 are require refactoring of VWidgets so @sgpearse could you take a look at them if you have time.
2.) Currently the number of decimals is a global property of VLineEdit and VSliderEdit and there is no way to change for one specific case without refactoring VLineEdit and VSliderEdit. I think we could have a system where the number of decimal places is automatically made as short as possible unless the user manually specifies a higher precision value as this would eliminate such issues.
6.) This is a feature that needs to be added to VLineEdit.

Is anyone able to compile this branch on cisl-vapor? It complains a QT problem, and I'm pretty I've put the latest 3rd party libraries on the systems directory.
Thanks!

Works for me on my mac

On Jun 3, 2020, at 12:50 PM, Samuel Li<EMAIL_ADDRESS>wrote:
Is anyone able to compile this branch on cisl-vapor? It complains a QT problem, and I'm pretty I've put the latest 3rd party libraries on the systems directory.
Thanks!
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.

John Clyne
National Center for Atmospheric Research
303.497.1236 (w),<PHONE_NUMBER> (c)
<EMAIL_ADDRESS>

Also, please make sure there are no compilation warnings and errors when compiling in release mode on cisl-vapor.


add workflows

The build is broken because of the chicken/egg problem. I merge the other PRs and then build is green.

The build is broken because of the chicken/egg problem. I merge the other PRs and then build is green.


Configurable browser to open in?

It'd be great if in .pocket-config there was an optional line about what "browser" to open it with - that way I could have pocket-cli open in lynx or elinks (or even pipe it to pandoc and then less) while not having to change my default browser.

Hi @uriel1998,
That sounds like a good idea. However, i can't find the time to work on this these days. Care to give it a try and propose a PR?


[ZEPPELIN-2479] Fix incorrect tooltip view inside of version control menu

What is this PR for?
This is to fix incorrect appearance of tooltip inside version control menu
What type of PR is it?
Improvement
Todos

[x] - fix/remove tooltip

What is the Jira issue?
ZEPPELIN-2479
How should this be tested?
go to version control menu in notebook action bar of note
Screenshots (if appropriate)
before:

after:

Questions:

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

@1ambda yeah that's better idea, I just pushed the changes and updated image

@khalidhuseynov Could you restart the failed travis JOB?

CI is green. https://travis-ci.org/khalidhuseynov/zeppelin/builds/227894706
LGTM.

Merge to master and branch-0.7 if no further discussions.


CFNetwork SSLHandshake failed (-9847)

Hi teams ,
I have some problem about the configuration edx iOS app , this is the error and i have google the problem. It showed it's about the SSL problem .
And i have try to add the OAUTH_ENFORCE_SECURE = "" in my lms.env.json , and have try to add in #Feature and out of #Feature ,  but it still error .
"Environment"
I build on localhost without https

I also try the method to add some properties on Info.plist.

How do i fixed this problem ?
Big Thanks,
MOH

Hrrrm. I haven't seen that before. Does it go away if you disable app transport security entirely?

Hi aleffer ,  I found the problem ,  is because i put the https://localhost:8000/ , https just only can merge direct url.
And now i change to http://localhost:8000/ it show wrong again, with the following error.

I have check the other issue, but also can't fix my problem .
Sorry for disturb, and hopefully you can help me .
Big Thanks,
MOH

We've disabled app transport security, so any problem here is probably a server configuration issue.


Soft WDT reset ESP8266

It says that
BSSL:_connectSSL: start connection
BSSL:CERT: 30 82 05 6b 30 82 04 53 a0 03 02 01 02 02 11 00 a5 a1 b9 f1 78 77 4a 63 0a 00 00 00 00 ed bc 63 30 0d 06 09 2a 86 48 86 f7 0d 01 01 0b 05 00 30 46 31 0b 30 09 06 03 55 04 06 13 02 55 53 31 22 30 20 06 03 55 04 0a 13 19 47 6f 6f 67 6c 65 20 54 72 75 73 74 20 53 65 72 76 69 63 65 73 20 4c 4c 43 31 13 30 11 06 03 55 04 03 13 0a 47 54 53 20 43 41 20 31 44 34 30 1e 17 0d 32 31 30 37 30 31 32 30 31 36 35 32 5a 17 0d 32 31 30 39 32 39 32 30 31 36 35 31 5a 30 19 31 17 30 15 06 03 55 04 03 13 0e 66 69 72 65 62 61 73 65 69 6f 2e 63 6f 6d 30 82 01 22 30 0d 06 09 2a 86 48 86 f7 0d 01 01 01 05 00 03 82 01 0f 00 30 82 01 0a 02 82 01 01 00 ac a3 4a b6 4c 76 d5 55 c5 77 4d ef c1 5f 2a ad b1 cb 26 fc 80 82 ab 6a 2c 80 89 f4 04 12 1f 2b 83 56 6f 55 da 3a cf 51 a3 2b 02 45 19
BSSL:CERT: a7 5c a6 9d 02 f9 bc 33 5f 4d 66 35 e2 d2 eb 8e b7 9b f7 1f c8 6e ea 9f 4b a6 75 9b a3 3a b7 e1 27 76 ac 76 42 e1 48 93 96 67 b9 17 be 09 44 64 df 63 e5 cf a4 e7 9c 2c 3d 9c 2b b9 5d 42 ba 4f 08 46 89 65 58 04 4c 02 97 97 97 2c 87 d6 a7 12 4d 08 06 7a 96 dc ed 2f 8e 47 5c e9 60 6a 2a ed c5 48 bd f7 b1 57 03 5f d7 4d ed 00 42 08 d3 12 87 72 0f b0 c2 8b 5b 50 da ed d8 9c c8 e9 94 d3 f3 16 6e 27 07 fd da f4 fb 14 96 ab 23 5a cd 4e fe a9 04 28 ff fd 97 b7 3b aa 6d 1f 79 9c 20 75 26 0a e9 09 f1 99 e7 f4 61 32 00 67 c6 dd 02 db e2 91 9f d8 5e 2a 1f de ed d8 0c 65 bb b3 07 d0 c5 cf ab 75 ce 38 3f 83 1d 6c 43 de fe f8 90 ac 91 47 65 02 03 01 00 01 a3 82 02 7f 30 82 02 7b 30 0e 06 03 55 1d 0f 01 01 ff 04 04 03 02 05 a0 30 13 06 03 55 1d 25 04 0c 30 0a 06 08 2b 06 01
BSSL:CERT: 05 05 07 03 01 30 0c 06 03 55 1d 13 01 01 ff 04 02 30 00 30 1d 06 03 55 1d 0e 04 16 04 14 ed 1b 1a 23 ee 1a a2 5e 8a a9 68 9e 79 f5 be 83 06 b9 60 28 30 1f 06 03 55 1d 23 04 18 30 16 80 14 25 e2 18 0e b2 57 91 94 2a e5 d4 5d 86 90 83 de 53 b3 b8 92 30 6d 06 08 2b 06 01 05 05 07 01 01 04 61 30 5f 30 2a 06 08 2b 06 01 05 05 07 30 01 86 1e 68 74 74 70 3a 2f 2f 6f 63 73 70 2e 70 6b 69 2e 67 6f 6f 67 2f 67 74 73 31 64 34 69 6e 74 30 31 06 08 2b 06 01 05 05 07 30 02 86 25 68 74 74 70 3a 2f 2f 70 6b 6
--------------- CUT HERE FOR EXCEPTION DECODER ---------------
Soft WDT reset

stack>>>

and the stack decode result is down below
Decoding stack results
0x402354e4: vsnprintf at /home/earle/src/esp-quick-toolchain/repo/newlib/newlib/libc/stdio/vsnprintf.c line 42
0x402273c1: uart_write(uart_t*, char const*, size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\uart.cpp line 509
0x402231a8: HardwareSerial::write(unsigned char const*, unsigned int) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266/HardwareSerial.h line 165
0x402236ac: Print::printf_P(char const*, ...) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\Print.cpp line 97
0x40250ca0: br_x509_decoder_run at src/x509/x509_decoder.c line 447
0x4025103c: br_x509_decoder_run at src/x509/x509_decoder.c line 672
0x4022319c: HardwareSerial::write(unsigned char const*, unsigned int) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266/HardwareSerial.h line 164
0x40247294: br_sha1_update at src/hash/sha1.c line 131
0x4021bfcd: BearSSL::insecure_append(br_x509_class const**, unsigned char const*, size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src\WiFiClientSecureBearSSL.cpp line 674
0x4024bddc: br_ssl_hs_client_run at src/ssl/ssl_hs_client.c line 1871
0x40249dc8: jump_handshake at src/ssl/ssl_engine.c line 1081
0x4024a2ad: br_ssl_engine_recvrec_ack at src/ssl/ssl_engine.c line 1206
0x4021cf6a: BearSSL::WiFiClientSecure::_run_until(unsigned int, bool) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src\WiFiClientSecureBearSSL.cpp line 537
0x4024a3d2: br_ssl_engine_hs_reset at src/ssl/ssl_engine.c line 1320
0x4021d1c8: BearSSL::WiFiClientSecure::_wait_for_handshake() at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src\WiFiClientSecureBearSSL.cpp line 561
0x4021d42f: BearSSL::WiFiClientSecure::_connectSSL(char const*) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src\WiFiClientSecureBearSSL.cpp line 1119
0x40100b79: check_poison_neighbors(uint16_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\umm_malloc/umm_local.c line 71
0x40244aa8: ip4_output_if_opt_src at core/ipv4/ip4.c line 1764
0x40100b79: check_poison_neighbors(uint16_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\umm_malloc/umm_local.c line 71
0x40100e98: umm_malloc(size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\umm_malloc\umm_malloc.cpp line 552
0x40100a64: get_poisoned(void*, size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\umm_malloc/umm_poison.c line 117
0x40244af0: ip4_output_if_opt at core/ipv4/ip4.c line 1577
0x401004dc: malloc(size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\heap.cpp line 232
0x40100a64: get_poisoned(void*, size_t) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\umm_malloc/umm_poison.c line 117
0x40244b16: ip4_output_if at core/ipv4/ip4.c line 1550
0x402456bf: ip_chksum_pseudo at core/inet_chksum.c line 395
0x40240c06: tcp_output at core/tcp_out.c line 1361
0x4023fef1: tcp_create_segment at core/tcp_out.c line 190
0x40225c86: __delay(unsigned long) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\cores\esp8266\core_esp8266_wiring.cpp line 54
0x4021b948: WiFiClient::connect(IPAddress, unsigned short) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src/include/ClientContext.h line 144
0x4021d597: BearSSL::WiFiClientSecure::connect(char const*, unsigned short) at C:\Users\user\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.7.3\libraries\ESP8266WiFi\src\WiFiClientSecureBearSSL.cpp line 228
0x40214891: FB_HTTPClient::connect() at C:\Users\user\Documents\Arduino\libraries\Firebase-ESP8266-master\src\wcs\esp8266\FB_HTTPClient.cpp line 134
i didn't understand, i try to remove some line of code, and it works, but after i re-upload it, it start error again, can some one explain to me what happend please?
i use this library
#include <FirebaseESP8266.h>
#include <ESP8266WiFi.h>
#include <WiFiManager.h>         // https://github.com/tzapu/WiFiManager
#include <NTPClient.h>
#include <WiFiUdp.h>
the code stop after print "Ok 4.", and always start printing BSSL:CERT

for now, it's working when i change this line of code
fbdo1.setBSSLBufferSize(2048, 2048);
fbdo1.setResponseSize(2048);
from 1024 to 2048, but 2 days ago, i try to change the size, it didn't change anything

This is due to incoming stream payload is larger than the reserved BearSSL buffer, only header can be read and SSL client stuck in somewhere when library is waiting for reading the unknown length stream payload and wdt overflown occurred caused by WiFiClientSecure class itself.
In Google RTDB http stream implementation, the content length header is not assigned and the client will not know the actual payload size to handle which is not possible for client to reserve the fix size memory for it which work for all possible situation.
Reserve the 8k memory of Rx can be handle all large payloads but it costs free memory available when Firebase Data object is in used.
I will add warning and error info for stream in the next update to let user know the risk situation of device crash of too large stream payload and defer the device crashed due to this situation.  We can only defer the crash due to WiFiClientSecure stuck in unknown payload length process.
The recommend is increase the Rx buffer of BearSSL or reduce the data being stream.

Oohh okay, it's make sense, cause when i change the buffer size, it's working normal
is there any side effect to use big size of buffer?

The direct effect is free heap reduced by the reserve mem amount as the stream connected and the memory free after disconnected.
The side effect is nothing except for the low memory situation depends on your code memory usage.

Ohh okayy.. thankyouu so much for the information sir..

Sorry sir, i want to ask again, i have 3 device, the first device is uploaded with the success code, but when i try to upload to other device, it get the same error, even i already increase the buffer rx
the same thing happen when i try to upload the first device, some times it work, some times it isn’t, is there are any advice that i could follow? Like how to check the current buffer size, my stream payload size, so i can solve it?

As I post above, the next update will add the stream error info and defer the device crashes.
The info included the stream payload size of streaming node of your database.

Anyway you need to minimize the size of your data under stream node and should not grow by using push function.


Periodic HashGrid and cylindrical grids

Hello, I would like to request two interrelated features for running physical simulations with Warp.

Periodic HashGrid:

Currently, it seems impossible to implement periodic boundary conditions with HashGrid in one or more dimensions.
I propose adding support for periodic boundary conditions to HashGrid, which would be useful for detecting collisions between particles located close to opposite boundaries.

HashGrid in cylindrical coordinate system:

While a normal grid covering the bounds of a cylindrical domain can be used, implementing periodic boundary conditions in the circular direction poses challenges.
Introducing a HashGrid specifically designed for cylindrical coordinate systems would simplify handling periodic boundary conditions in such scenarios.

If any of these features are already available or manageable, I would greatly appreciate any guidance on how to utilize them. Otherwise, please let me know if further clarification or assistance is required.

Hi @kmarchais, agreed this is very useful for a number of applications - the wp.hashgrid right now is tiling periodically, i.e.: if a particle lies on a boundary cell (as specified by the grid dimensions) then the radius query will wrap and return particles from the other side.
Particles are assigned to hash cells by essentially cell_coord = x%grid_dim and neighbors are returned from cells that overlap the query radius.
You would just need to make sure that your distance function is also periodic, and that your domain is equal to grid_dim*radius.
I haven't thought about cylindrical coordinate systems, I think it should be possible, the relevant code to look at is here: https://github.com/NVIDIA/warp/blob/main/warp/native/hashgrid.h#L59.
If you have suggestions on how to extend this to cylindrical coordinates that would be welcome.
Cheers,
Miles

Hello, thank you very much for your answer.
I didn't have time to try it yet but that's what I wanted indeed.
For the cylindrical grid, we can imagine a grid like this:

The grid would be defined by a curvature radius and its dimensions.
The coordinates would be something like:
x = np.linspace(-0.5, 0.5, N_POINTS)
y = np.linspace(-0.5, 0.5, N_POINTS)
z = np.linspace(-0.5, 0.5, N_POINTS)

x, y, z = np.meshgrid(x, y, z)

# make periodic
unit_theta = dim[1] / curvature_radius
n_repeat_to_full_circle = int(2 * np.pi / unit_theta)
unit_theta = 2 * np.pi / n_repeat_to_full_circle

rho = x + curvature_radius
theta = y * unit_theta

X = rho * np.cos(theta)
Y = rho * np.sin(theta)
Z = z


Creating clickable area

Can you please add a clickable area that's drawn in polygon

This is beyond the scope of the project and not well defined.


Functionality: No duck button is presented on-screen

Currently, there is no duck button (unlike the jump, left, and right buttons) on-screen. If anybody knows the source where we got the other three button images from, that would be an ideal place to a duck button image from.

@Germlord it is fairly easy use the existing buttons to create a new one in most editors like paint or Photoshop.

Tried to do a duck button, but didnt know which font was it so it looks kind of different

@Booxworm That looks good!

Tweaked it a bit to make it look better.
#177
Clicking it makes the character disappear though, but will try to fix that soon.

Completed by @Booxworm


[CosmosDB] Microsoft.DocumentDB 2024-02-15-preview API version

ARM (Control Plane) API Specification Update Pull Request

Tip: overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

 Click here to see the diagram footnotes
Diagram footnotes
[1] ARM review queue (for merge queues, see [2])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the Diagram Step 2, "ARM API changes Review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Breaking changes previously reviewed and approved in #26206

    "enablePerRegionPerPartitionAutoscale": {

Recommend enum over boolean, e.g. autoscaleMode: { None, PerRegionPerPartition, ... }

Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:8598 in 6419a79. [](commit_id = 6419a791c8857ab43dcb1dfe616c051d0b794987, deletion_comment = False)

    "enablePerRegionPerPartitionAutoscale": {

Same

Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:8778 in 6419a79. [](commit_id = 6419a791c8857ab43dcb1dfe616c051d0b794987, deletion_comment = False)

    "enablePerRegionPerPartitionAutoscale": {

Same

Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:8982 in 6419a79. [](commit_id = 6419a791c8857ab43dcb1dfe616c051d0b794987, deletion_comment = False)

    "restoreWithTtlDisabled": {

Prefer enum, something like timeToLivePolicy: { DisableOnRestore, Default, ... }

Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:11691 in 6419a79. [](commit_id = 6419a791c8857ab43dcb1dfe616c051d0b794987, deletion_comment = False)

    "restoreWithTtlDisabled": {

Prefer enum, something like timeToLivePolicy: { DisableOnRestore, Default, ... }
Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:11691 in 6419a79. [](commit_id = 6419a79, deletion_comment = False)

This property is just an optional toggle property within already existing RestoreParameter object to leave the target restored account with TTL disabled upon the restore completion. There is no reason to make it into an enum.

    "enablePerRegionPerPartitionAutoscale": {

Recommend enum over boolean, e.g. autoscaleMode: { None, PerRegionPerPartition, ... }
Refers to: specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2024-02-15-preview/cosmos-db.json:8598 in 6419a79. [](commit_id = 6419a79, deletion_comment = False)

Autoscale do not have any other mode, enablePerRegionPerPartitionAutoscale it is just a property we want to enable for some accounts. Since all such properties are defined in Boolean, we want to make it consistent. Also it would be easier for user to define it.


The speed of compiler

I am using Closure Compiler to obfuscate my JS code for a long time. Since I don't want to install the whole Java environment just because of this app, I compile my code by sending it to https://closure-compiler.appspot.com/compile  through HTTP.
My code has about 1.2 MB (40 000 lines), I use Advanced compilation, it takes almost a minute to compile.
Is this an expected time in my case? Would it be faster, if I was running it on my own computer? Are there any ways to make it faster (e.g. I don't care about browsers, that are more than a year old). My goal is just to change the names (of variables, functions ...), I don't need changing the syntax (but I don't mind it).

That sounds about right. However we don't recommend using the web service for that size of code base. You can try the experimental JavaScript version.

Thanks. Are there any flags, that would disable the optimization and just perform the renaming? Would it be faster then?

You can set --use_types_for_optimization=false, but you'll also get less renaming.


Please make isReady publilc or export its value via @Output

In my use-case, the full-size image might a very large file, so I first display a simple img pointing to the thumbnail, and then ngx-image-zoom would be ready, I'd replace the img with it.
But right now I cannot know when that's the case :(

That should be easy enough to sort, I'll look into it!

There's now a 1.0.0-beta.1 on npm as well that includes this change. Give it a spin. Since it's a beta release you have to specify version explicitly.
Should be a proper 1.0.0 out in a couple of days or so I think.

Thanks @wittlock, I will wait for 1.0.0 and then try it!


Add a "Shortcut" dialog

Is your feature request related to a problem? Please describe.
Not related to a problem
Describe the solution you'd like
Microsoft Teams and Discord for example have these nice dialogs which display all shortcuts. Would be nice to have something similar for Files as well as some point, since the amount of shortcuts is getting bigger and bigger.
Teams:

Discord:

#1660?

I hate it. I searched for "shortcut" and either I missed this one or it didn't show up. Sorry!

No problem xD


Merkle tree API improvements

Description
Was playing around with Jellyfish for a hobby project and ran into a couple of ergonomics issues in the Merkle tree API.
commit 699dea9956ff2d872704e74eec3badf38db168f8
Author: Jeb Bearer<EMAIL_ADDRESS>Date:   Wed Dec 14 11:43:07 2022 -0800
Make serde usable for MerkleTree types

The Merkle trees are ususally instantiated with cryptographic types,
e.g. field elements, which do not implement serde traits. This commit
changes the bound on the derived serde impls for Merkle tree types to
require ark_serialize impls on the element types and uses #[serde(with)]
to serialize these elements using ark_serialize.

We also add an instantiation of the Rescue hash scheme for field elements
as the _index_ type, since the BigUint instantiation just coverts to field
elements anyways, but BigUint does not implement serde _or_ ark_serialize,
while field elements do.

commit bdbd5b1e8be44cf22152d4e752b11ae95ffef41d
Author: Jeb Bearer<EMAIL_ADDRESS>Date:   Wed Dec 14 10:18:37 2022 -0800
Expose index and leaf element of Merkle proof.

Since MerkleNode is a private type, users cannot pattern match to
extract a MerkleNode::Leaf, so this is the only way to get the leaf
elemenet of a Merkle proof. This is also, therefore, the only way
to verify that a Merkle proof proves inclusion of the desired element,
since `verify` doesn't check this. (Technically you can also use
`remember`, but that includes side-effects that may be undesirable.)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (main)
[ ] Linked to GitHub issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[ ] Re-reviewed Files changed in the GitHub PR explorer

Yeah that's a good idea. mod field_elem actually already deals with T: CanonicalSerialize, so field_elem was already kind of a misleading name. I'll change it to mod canonical (#[serde(with = "canonical")] is concise an accurate).
Also totally agree with the benefit of BigUint. Only drawback is serialization. That's why I kept the BigUint Rescue scheme and added a new one with F as the index just in case user's need serialization. I also think indexing by F might be slightly more intuitive when you're interacting with circuits, since in that case you're mostly dealing with field elements everywhere, and BigUint just gets converted to a field element anyway.


cdk deploy and context with multiple subnets per AZ

Describe the bug
data.all assumes 1 Subnet per AZ when doing a deployment. But a customer might have 2 Subnets per AZ (in this case separate frontend and backend). VPC Interface Endpoints creation fails with a validation when there are more than 1 Subnet per AZ.
The quick manual solution was to use backend subnets in cdk.context.json and deleted the frontend subnets.
How to Reproduce
*P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.*

Expected behavior
No response
Your project
No response
Screenshots
No response
OS
n/a
Python version
n/a
AWS data.all version
V2.0.0
Additional context
No response

Docs added in #821


OCR Models

Feature request
Hi, I'm one of the chinese developers that knows transformer.js by a Front End salon than invited transformer.js engineer to talk about this lib.
Now i'm going to work with client OCR,  and i'm plan to use transformer.js, but i'm not seen any models of ocr is supported, i want to know is this right time to use transformer.js of ocr functions?
Motivation
OCR Models that supported chinese
Your contribution
not yet

Hi there! 👋 We do support a collection of OCR models (the TrOCR model family). For example, https://huggingface.co/Xenova/trocr-small-handwritten can be used as follows:
import { pipeline } from '@xenova/transformers';

// Create image-to-text pipeline
const captioner = await pipeline('image-to-text', 'Xenova/trocr-small-handwritten');

// Perform optical character recognition
const image = 'https://huggingface.co/datasets/Xenova/transformers.js-docs/resolve/main/handwriting.jpg';
const output = await captioner(image);
// [{ generated_text: 'Mr. Brown commented icily.' }]

However, as you state in your question, you are specifically looking for a model which performs Chinese optical character recognition. Do you know of a model (perhaps on the Hugging Face Hub) which you would like to run in the browser? If so, we can convert the model to ONNX so that it can run with transformers.js.

oh! thanks for responsing me at first, so pleasure!
I've already tried image-to-text models in hugging face before sending this issue, saddly these models is might not be ok to cover my issues of work: (
And to be honest, I'm not very well in using AI models(hah, just a FE developer who love ai now~), so there is no excactly model that i know is perfectly fitting me.
but what i need in works is pretty sure:
0.a model supported Chinese : )
1.it can generate all texts of the source image, and output them in JSON
2.also can tell the position of each single word(such as the word 'cat' in image is forbbiden, i should know where the 'cat' word is, or just gave me all the positions of text is fine too)
Do you have any advices for me to find the  perfect model?
I think these kind of  models is very big issue for informations publishing in Chinese, and also very matching the advantage of transformer.js~it's just in my opinion : ) may be we can buildup an OCR demo for this

I'm working on pick models those days, and changed the last comment info, add this comment just in case you've seen the old comment

Hi there, Here is the latest information by these days OCR Model researching:
I figured out that PaddleOCR model is a little different by other models ( the suffix of the model's file is '.pdmodel', it might be means 'paddle-model' i guess? )
But i do fond a way to conver the model file to ONNX pattern, but ONLY THE MODEL'S FILE ITSELF, which means i  can't using the tranformer.js converting tools but the PaddleOCR converting tools instead, because of the suffix of the model file name is  .pdomdel, so i don't have the other necessary files such as 'tokenizer' or 'config' files to run it in transformer.js ),is there any way to generate these files by hand or by other process to passover it?
And another problem is the PaddleOCR Model's output pattern is just like:
const output = {
  points: [
    // the first text block
    [
      // the first block's position of four corners
      1,2,3,4
    ]
  ],
  text: [
    // the first text info
    'first block text',
    'second block text'
  ]
}

but it seems like transformer.js only allows:
const output = [
  { generated_text: 'text' }
]

I don't know what's gonna happened when i using this model in transformer.js
Thanks for Response again * 2, hah! And waiting for responsing * 2 : )

Hi there! I deleted all the commet before, because i noticed that OCR Model is not as easy as what i think a few days before.
After a lot of gitlab OCR Models code reading and models knowlege learning by these days, i noticed that a OCR Model which can detect all the text blocks area and recognize each block of texts excactly in this image needs two model work together at least ( detect model and recognize model, maybe also need angle detect model and so on), and there should be a lot of works to glue them together with js to generate the usedful outputs, it's obviously not a easy work, converting the PaddleOCR now will late for my business researching for sure...
But to be honest, this kind of OCR workflow with in good performance, might be a very big issue for internet information publishing in China's enviroment (
just like input an image data, output all the text block positions and text charctors it self ),a lot of information companys requires this workflow and almost all using server to predict, and this is also a big advantage of tansformer.js, if you consider to conver this process to transformer.js, i think i could join it together, because i will focus on Web AI for a long time in my work ( ps: I do convert the model and run it in onnxruntime, but i'm not very good at ai's workding now.... )
Thanks for your response again! And wishing for your next response~

Hi again 👋 Apologies for the late response! The transformers.js library aims to bring the python transformers library to the browser and other JavaScript environments (e.g., Node, deno), so if you know of a suitable OCR model on the Hugging Face Hub (link), we can do our best to try support it.

This might soon be possible: https://github.com/VikParuchuri/surya

@xenova happy to help with the integration of the surya model, lmk what needs to be done. Should i just follow the instructions  in the readme to add a new model?

@xenova happy to help with the integration of the surya model, lmk what needs to be done. Should i just follow the instructions in the readme to add a new model?

I'd also be interested in surya in the browser, is this underway?

This seems like an interesting model too, at just 0.7B:
https://huggingface.co/stepfun-ai/GOT-OCR2_0


Crypt 2.munki recipe failing

Hello everybody,
the Crypt 2 munki recipe is failing since the newest release with the following error message and log (triple v):
FlatPkgUnpacker
{'Input': {'destination_path': u'/Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack',
           'flat_pkg_path': u'/Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/downloads/Crypt2.pkg'}}
FlatPkgUnpacker: Unpacked /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/downloads/Crypt2.pkg to /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack
{'Output': {}}
PkgPayloadUnpacker
{'Input': {'destination_path': u'/Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/payload',
           'pkg_payload_path': u'/Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack/Payload'}}
extraction of /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack/Payload with ditto failed: ditto: /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack/Payload: No such file or directory

Failed.
Receipt written to /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/receipts/ch.unibas.its.git.mcs.munki-receipt-20170919-143142.plist

The following recipes failed:
    com.github.grahamgilbert.Crypt2.munki
        Error in com.github.grahamgilbert.Crypt2.munki: Processor: PkgPayloadUnpacker: Error: extraction of /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack/Payload with ditto failed: ditto: /Users/***/Library/AutoPkg/Cache/com.github.grahamgilbert.Crypt2.munki/unpack/Payload: No such file or directory

Nothing downloaded, packaged or imported.

The download works fine, but if you take a look into the unpack folder you will only find a pkg instead of a folder like Payload or so.
Thanks in advanced!

Can you try the above PR and see if it fixes your issue?

@grahamgilbert thanks for the fix! I updated the path for the Payload unpacker, now the recipes works again for me. Can you confirm that (see PR)?

Thanks for the PR!


Use VitePress instead of VuePress

VitePress offers faster build times compared to VuePress (because it runs Vite under the hood) and it depends on Vue 3. Only issue I can see with it is that it alienates some older browsers due to its ESM-focused architecture, although the @vitejs/plugin-legacy plugin fixes that.
Honorarily pinging @emiyl for input on switching from VuePress to VitePress.

I wasn't aware VuePress beta is using Vite. I thought VuePress was in maintenance mode.


train.py_fault

File "h5py_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "C:\Users\XDU_Yu.conda\envs\RVIT\lib\site-packages\h5py_hl\dataset.py", line 768, in getitem
return self._fast_reader.read(args)
File "h5py_selector.pyx", line 376, in h5py._selector.Reader.read
OSError: Can't read data (can't open directory)
May I ask what is the reason for this error when running train.py?

Not sure. Have you checked whether the dataset path is pointing to the correct directory?

I assume the issue is solved, otherwise let me know

I had the same problem. Did you solve it？


Add ability to skip comma flag even if no comma output implemented

Addresses half of issue #71 by adding ability to ignore a comma flag -- e.g. #'d -- and not interpret it literally. The result is as if the comma flag is not present, e.g. #d. All existing tests work, and added a new test section for the comma flag.
This pull request at least makes this implementation of printf() compatible with existing code which uses the comma flag... At least seg faults and/or unwanted output no longer occur. Next step will be to implement the comma output with some kind of mechanism to turn it on; environment variable?

Codecov Report

Merging #73 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #73   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         359    360    +1
=====================================
+ Hits          359    360    +1

Impacted Files
Coverage Δ

printf.c
100% <100%> (ø)
:arrow_up:

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 d3b9846...cbe0314. Read the comment docs.


How we can we use front and back camera like as mobile tab

Hello all, I'm using react camera as barcode scanner in my app. By default, It is taking Back Camera, I want to  be able to switch between Front/Rear camera view for scanner. I'm not sure how to do this, any idea ? Any help appreciated!

@Kabijoy The <Camera> component takes a type property that can be used to choose whether you're using the front or back camera: https://github.com/lwansbrough/react-native-camera#type
The values are: Camera.constants.Type.front or "front", Camera.constants.Type.back or "back" (default)
So:
<Camera
  type={Camera.constants.Type.front}/>

You could add a button that toggles between setting the type to front/back to switch between them.


Auth notification and popup

hope it helps

Hi!
First of all, thanks for PR!
Showing popup notifications (Notifications.showAuthenticate) is annoying because of ton of auth errors. It's enough to notify user by showing another icon in the extension toolbar. It's already done.
The popup is almost that what David wants to see. The difference is that you placed the reauth button in the popup window. He suggested a link to the same button in the extension settings (see david-sabata/web-scrobbler#900):

The popup may then contain the message about the need to reauthenticate and maybe a link to settings page. I wouldn't authenticate users directly from the popup as it is unloaded and destroyed once it loses focus.

It's not hard to add the link to options page. But it's not enough.  Options page has collapsed "Options" sections, and we need to expand it somehow and to scroll to reauth button on clicking by reauth link in the popup window. Otherwise, users won't see this button. That's what I wanted to do, but I don't know how. Have you any ideas?
I don't know why it's important to have the link to optons page, not the reauth button. I tested this PR, and reauth works well by clicking on the button in the popup window. And this way is faster than going to options by clicking on the link and press the button in options page. Maybe we'll have some issues, if we'll use this way of reauth.
Anyway, if we can't find a solution, I will merge your changes and will propose a PR to upstream. It's exactly better than we have now (and we have nothing now).

Hm, looks nice! I'll test it soon.

			function auth (authUrlGetter) {
				var onHaveAuthUrl = function(authUrl) {
				chrome.tabs.create({ url: authUrl });
				};

				authUrlGetter(onHaveAuthUrl);
			}

			auth(LastFM.getAuthUrl);

And if you know how to simplify this mess, please tell.

Try this:
function reAuthenticate(tab) {
    console.log('Page action clicked in tab ' + tab);

    LastFM.getAuthUrl(function(authUrl) {
        chrome.tabs.create({ url: authUrl });
    });
}

I tested your code; the Last.fm auth page is opened twice on the icon click. My one works well for me.
In Web scrobbler Last.fm page is opened in a separate window, not in a tab:
window.open(authUrl, 'scrobbler-auth');

There's no difference how to open that page, but I think it should be the same in every place.

Is there anything you would like to add to this PR? Or is it ready to be merged?

Well, it looks good to me. Maybe we'll run into some conflicts when you update your base branch, it's pretty outdated.


to_netcdf fails for engine=h5netcdf when using dask-backed arrays

When using dask-backed datasets/arrays it does not seem possible to use the 'h5netcdf' engine to write to disk:
import xarray as xr
ds = xr.Dataset({'a': ('x', [1, 2])}, {'x': [3, 4]}).chunk()
ds.to_netcdf("test.h5", engine='h5netcdf')

results in the error:
...

h5py/h5a.pyx in h5py.h5a.open()

KeyError: "Can't open attribute (can't locate attribute: 'dask')"

Not sure if this is a xarray or h5netcdf issue - or some inherent limitation in which case apologies!

Thanks for the report!
This was definitely working at one point, but it appears we did not have test coverage and then it was broken by https://github.com/dask/dask/pull/2181.
I'm going to fix this on the h5netcdf side, but will leave this issue open as a reminder to write an integration test to catch issues like this.

This should be fixed in the latest h5netcdf release (v0.4.2), which is now up on pypi (so you can install it with pip).


[question] role mapper and user in-line

I create a user inline provider with one user (test), and i want to associate role to this user.
but how to configure ?
i try uid=test, but it doesn't work.
how to configure also user mapper to have field in jwt ?
version= 1.3.0

Hello @vdelattre,
Role mapping is not yet implementing for the in-line identity provider. I update the question issue to feature issue.


MetaSync cannot parse the library xml of iTunes 12.3

I use iTunes on Windows
(My username is edited out in the following snippets, but it contains only ASCII letters and numbers)
The plugin is configured like this:
metasync:
    source: itunes
    itunes:
        library: C:\Users\<username>\Music\iTunes\iTunes Music Library.xml

The file is actually there (and when it's not there the error message is different). Then this happens:
> beet -vvv metasync
user configuration: C:\Users\<username>\AppData\Roaming\beets\config.yaml
data directory: C:\Users\<username>\AppData\Roaming\beets
plugin paths:
Sending event: pluginload
library database: C:\Users\<username>\AppData\Roaming\beets\library.db
library directory: G:\Music\Music\beets
Sending event: library_opened
metasync: loading iTunes library from C:\Users\<username>\Music\iTunes\iTunes Music Library.xml
metasync: Unknown metadata source 'itunes'
metasync: No valid metadata sources found
Sending event: cli_exit

10-15 seconds pass after the message metasync: loading iTunes library... and before the next one. And previous ones are printed almost immediately after each other. So I'm guessing it actually loads and tries to parse the file

Thanks for the report! I just pushed a fix.

Thanks! Now it reads the file

Sorry for writing on this issue but I'm on OSX running iTunes <IP_ADDRESS> and having exactly the same problem. The path and filename are correct.
metasync
  source: itunes
  itunes:
    library: /Volumes/HDD/Music/iTunes/iTunes Library

and when running beet -vvv metasync I get exactly the same message:
user configuration: /Users/<username>/.config/beets/config.yaml
data directory: /Users/<username>/.config/beets
plugin paths: /usr/local/lib/python2.7/site-packages/beetsplug
Sending event: pluginload
artresizer: method is (2, (6, 9, 2))
library database: /Users/user/Music/Beets/database.db
library directory: /Volumes/HDD/Music/Library
Sending event: library_opened
metasync: loading iTunes library from /Volumes/HDD/Music/iTunes/iTunes Library.xml
metasync: Unknown metadata source 'itunes'
metasync: No valid metadata sources found
Sending event: cli_exit

after the metasync: loading iTunes library from... message it takes about 30 seconds and then it shows the metasync errors. Tested with 1.3.16

Hi, @christiankopac—can you check that you have the latest source from git? 1.3.16 isn't released yet, so this fix is only available if you're on the latest master branch.

Pulled latest master branch. It is working now. Thanks @sampsyo !


Treasures of Montezuma Arena [PCSB00422]

App summary

App name: Treasures of Montezuma Arena
App serial: PCSB00422
App version: 1.00

Vita3K summary

Version: v0.1.8
Build number: 3251
Commit hash: https://github.com/vita3k/vita3k/commit/0fb1cd7c
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Sam
OS: Windows 10
CPU: AMD 5500u
GPU: AMD  Radeon Graphics
RAM: 6 GB

Issues

Screenshots

Log
vita3k.log
Recommended labels

https://github.com/Vita3K/compatibility/labels/Playable
B?
C?

there is a problem with this game
when it comes up to this screen (after the developers screen)

it freezes up here 99% of the time unless you keep pressing the X button
it is supposed to automatically load to the menu screen


To my local staging - Change - Don't sleep after the last needed ping

In some cases, application needs to run infinitely.
I edited code, so that if ping count is set to 0 (or below, e.g., -1), httping cycle runs infinitely. This can be extremely useful.
I thought of creating another flag, so I looked through industry practices. MacOS and Linux pings are continuous pings by default, and Windows uses -t flag. However, what does -t means and why is it not -c as in -continuous - is unclear. So, modifying -count flag behavior seemed like the best option.
You can read more about Continuous Ping here

LGTM


When is the new non-game app revenue split coming?

There was word "later this year"(https://blogs.windows.com/buildingapps/2018/05/07/a-new-microsoft-store-revenue-share-is-coming/) around build conference time and it's almost end of year yet still nothing?

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

ID: 36abe7f6-20c0-e1ba-d585-9630bcfdcc9f
Version Independent ID: fd28df55-a61e-4742-37a1-b6691fd5fdf6
Content: Getting paid - UWP app developer
Content Source: windows-apps-src/publish/getting-paid-apps.md
Product: uwp
GitHub Login: @JnHs
Microsoft Alias: wdg-dev-content

Great question. This is still on track to occur by the end of year, although I don’t have an exact date to give you.

This has not occurred now.


Not All Valid String Values are Listed For ContentType

The page shows an example value of "text/plain" for a password, but certificates are also used in the parent class KeyvaultSecret, and through debugging, I found that the content type can be "application/x-pem-file" or "application/x-pkcs12". These should be listed in the documentation.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c07a8a21-3e21-c2ca-86de-060d6bcda6c9
Version Independent ID: 1cd6cd94-2651-754c-d993-f0bca2358d2f
Content: SecretProperties.ContentType Property (Azure.Security.KeyVault.Secrets) - Azure for .NET Developers
Content Source: xml/Azure.Security.KeyVault.Secrets/SecretProperties.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

"text/plain" is typical for most secrets. If you choose to store a certificate yourself using secrets instead of the Azure.Security.KeyVault.Certificates SDK to manage certificates, you can choose whatever content type you like. As the documentation states, "text/plain" is only an example.


Question about rdpwrap.dll

I took a PC offline (no Internet connection) and found that when I installed rdpwrap 1.6.2, rdpwrap.dll still gets installed.  So my question is, where is it getting installed from?  Is it embedded in one of the installer files like RDPWInst.exe?  Or is it perhaps getting compiled on-the-fly?  Thank you

Is it embedded in one of the installer files like RDPWInst.exe?

This.


Deploy workflow runs can fail without the whole team being notified

Cause
When a pull request is merged, the CI workflow first runs tests to ensure that the changes have merged cleanly. If the tests fail then only the user who clicked the merge button will be notified, via email.
If the tests succeed then a deploy will be started, with a deploy notification in the #design-system-dev Slack channel showing the status of the deploy.
Consequences
It is possible for the notified user to miss the email about the failed workflow run (see https://github.com/alphagov/design-system-team-internal/issues/527). It is also possible that the deploy is critical, or that the failure needs to be actioned by a developer (the merge can be performed by any member of the team). We should treat a failed deploy as something the whole team should be aware of.
Impact of debt
Medium
Reason (impact of debt)
We have flaky tests that fail fairly often, so it is not uncommon for deploys to fail without the whole team being aware.
Effort to pay down
Medium
Reason (effort to pay down)
Adding a Slack notification should be possible, although we may also want to consider sending email notifications.
Overall rating
Medium
Reason (overall rating)
We should fix this soon, but it is perhaps not as high priority as making it impossible for the team to accidentally deploy old code https://github.com/alphagov/govuk-design-system/issues/1946

We've got Slack notifications for deployments of the design system website now, which tell us if they've been successful or failed. This is probably enough for now - we can re-open this if something crops up in future where those Slack messages haven't been enough.


enabling to use double underscore delimiters

Thank you for developing a very useful and simple library.
In my project, I needed to use a hierarchical configuration, so I added it.
Using (__ ) for SecretID, covert to :. (It will be interpreted as a delimiter by the configuration provider https://github.com/aspnet/Configuration/blob/master/src/Config.Abstractions/ConfigurationPath.cs#L17)
This implementation is based on the following documentation and source code.
https://docs.microsoft.com/aspnet/core/fundamentals/configuration/?view=aspnetcore-5.0#environment-variables
https://github.com/aspnet/Configuration/blob/master/src/Config.EnvironmentVariables/EnvironmentVariablesConfigurationProvider.cs#L65-L68
Thank you.

Codecov Report

Merging #1 (438203c) into develop (6789ba8) will increase coverage by 0.52%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop       #1      +/-   ##
===========================================
+ Coverage    87.87%   88.40%   +0.52%
===========================================
  Files            5        5
  Lines           66       69       +3
===========================================
+ Hits            58       61       +3
  Misses           8        8

Impacted Files
Coverage Δ

...ger.Provider/SecretManagerConfigurationProvider.cs
100.00% <100.00%> (ø)

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 6789ba8...438203c. Read the comment docs.

Great! Always glad to hear that someone is using this nuget :)
Could you also add test case for this?
Thank you!

OK, I'll try it. please wait a moment.

I appreciate your kindness and support!
Thank you very much. 🤗


mobile/android: Use API level 19 to prevent a crash on some devices

Original issue: https://github.com/iotaledger/trinity-wallet/issues/1135
In optimized builds for armeabi-v7a, the memcpy function is sometimes replaced with ARM's __aeabi_memcpy. Because of changes in later versions of the Android NDK, crashes can sometimes occur where this function is undefined. This is fixed by using API level 19 (see https://github.com/android-ndk/ndk/issues/126#issuecomment-334672371).
Test Plan:
No longer crashes on Motorola Moto G4 (Android 6.0.1) and LG K3 (Android 6.0.1), which were previously affected by this issue

Closing this because I'm solving via revert of #808 via #843


Local Secondary Index can't be created by override.ts before table creation.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.13.0
Amplify CLI Version
12.1.1
What operating system are you using?
Windows 10
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug

I'm trying to make 'ToDoLast' Table in schema.graphql.
I also wanted to add local secondary index. so, I coded in override.ts which is generated file by 'amplify override api'

First Try of override.ts code is failed. The error code of amplify-cli is below.

Reason: Resource handler returned message: "Number of attributes in KeySchema does not exactly match number of attributes defined in AttributeDefinitions" (RequestToken: 90c326b5-38b3-9061-0bdc-1d60710bf928, HandlerErrorCode: InvalidRequest)

Second Try of override.ts is failed.

Name: ToDoLastTable (AWS::DynamoDB::Table), Event Type: create, Reason: Resource handler returned message: "An attribute referenced in a KeySchema element is not defined in AttributeDefinitions" (RequestToken: a1d440cd-7854-573b-ef3b-40e9bd25d052, HandlerErrorCode: InvalidRequest), IsCustomResource: false

Third Try of override.ts is failed.

Name: ToDoLastTable (AWS::DynamoDB::Table), Event Type: create, Reason: Resource handler returned message: "An attribute referenced in a KeySchema element is not defined in AttributeDefinitions" (RequestToken: 465d6ce3-7e61-a5fe-e2d2-371dec8a2b82, HandlerErrorCode: InvalidRequest), IsCustomResource: false

Last Try of override.ts is failed.

Name: ToDoLastTable (AWS::DynamoDB::Table), Event Type: create, Reason: Resource handler returned message: "An attribute referenced in a KeySchema element is not defined in AttributeDefinitions" (RequestToken: e933b26d-0409-fdb4-2782-15eee8d29b0e, HandlerErrorCode: InvalidRequest), IsCustomResource: false

Expected behavior
The Table has to be created with local secondary index.
Reproduction steps

schema.graphql

# This "input" configures a global authorization rule to enable public access to
# all models in this schema. Learn more about authorization rules here: https://docs.amplify.aws/cli/graphql/authorization-rules
input AMPLIFY { globalAuthRule: AuthRule = { allow: public } } # FOR TESTING ONLY!

type Todo @model {
  userID: String! @primaryKey(sortKeyFields: ["postDate", "postStage", "title"])
  postDate: AWSDateTime!
  postStage: String!
  title: String!
  description: String
}

type ToDoShare @model {
  userID: String! @primaryKey(sortKeyFields: ["postDate", "like_num", "bookmark_num", "comment_num", "tryon_num"])
  postDate: AWSDateTime!
  like_num: Int!
  bookmark_num: Int!
  comment_num: Int!
  tryon_num: Int!
  title: String
  description: String
}

type ToDoAgain @model {
  userID: String! @primaryKey(sortKeyFields: ["postDate"])
  postDate: AWSDateTime!
  version: Int!
  like_num: Int!
  bookmark_num: Int!
  postStage: String
  title: String
  description: String
}

type ToDoLast @model {
  userID: String! @primaryKey(sortKeyFields: ["postDate"])
  postDate: String!
  version: Int!
  title: String
  description: String
}

First Try of 'override.ts' code

import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
    resources.models["ToDoLast"].modelDDBTable.attributeDefinitions = [
        {
            attributeName: "userID",
            attributeType: "S"
        },
        {
            attributeName: "postDate",
            attributeType: "S"
        },
        {
            attributeName: "version",
            attributeType: "N"
        },
        {
            attributeName: "title",
            attributeType: "S"
        },
        {
            attributeName: "description",
            attributeType: "S"
        }
    ]
    resources.models["ToDoLast"].modelDDBTable.keySchema = [
        {
            attributeName: "userID",
            keyType: "HASH"
        },
        {
            attributeName: "postDate",
            keyType: "RANGE"
        }
    ]
    resources.models["ToDoLast"].modelDDBTable.localSecondaryIndexes = [
        {
            indexName: "byVersion",
            keySchema: [
                {
                    attributeName: "userID",
                    keyType: "HASH"
                },
                {
                    attributeName: "version",
                    keyType: "RANGE"
                }
            ],
            projection: {
                nonKeyAttributes: [
                    "postDate",
                    "title",
                    "description"
                ],
                projectionType: "INCLUDE"
            }
        }
    ];
}

Second Try of 'override.ts' code

import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
    resources.models["ToDoLast"].modelDDBTable.localSecondaryIndexes = [
        {
            indexName: "byVersion",
            keySchema: [
                {
                    attributeName: "userID",
                    keyType: "HASH"
                },
                {
                    attributeName: "version",
                    keyType: "RANGE"
                }
            ],
            projection: {
                nonKeyAttributes: [
                    "postDate",
                    "title",
                    "description"
                ],
                projectionType: "INCLUDE"
            }
        }
    ];
}

Third Try of 'override.ts' code

import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
    resources.models["ToDoLast"].modelDDBTable.keySchema = [
        {
            attributeName: "userID",
            keyType: "HASH"
        },
        {
            attributeName: "postDate",
            keyType: "RANGE"
        }
    ]
    resources.models["ToDoLast"].modelDDBTable.localSecondaryIndexes = [
        {
            indexName: "byVersion",
            keySchema: [
                {
                    attributeName: "userID",
                    keyType: "HASH"
                },
                {
                    attributeName: "version",
                    keyType: "RANGE"
                }
            ],
            projection: {
                nonKeyAttributes: [
                    "postDate",
                    "title",
                    "description"
                ],
                projectionType: "INCLUDE"
            }
        }
    ];
}

Last Try of 'override.ts' code

import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
    resources.models["ToDoLast"].modelDDBTable.attributeDefinitions = [
        {
            attributeName: "userID",
            attributeType: "S"
        },
        {
            attributeName: "postDate",
            attributeType: "S"
        }
    ]
    resources.models["ToDoLast"].modelDDBTable.keySchema = [
        {
            attributeName: "userID",
            keyType: "HASH"
        },
        {
            attributeName: "postDate",
            keyType: "RANGE"
        }
    ]
    resources.models["ToDoLast"].modelDDBTable.localSecondaryIndexes = [
        {
            indexName: "byVersion",
            keySchema: [
                {
                    attributeName: "userID",
                    keyType: "HASH"
                },
                {
                    attributeName: "version",
                    keyType: "RANGE"
                }
            ],
            projection: {
                nonKeyAttributes: [
                    "postDate",
                    "title",
                    "description"
                ],
                projectionType: "INCLUDE"
            }
        }
    ];
}

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Because, The usage of secondary indexes was for same partition key. I don't need to use global secondary indexes, and I heard that LSI is better than GSI in performance for querying and managing RCU/WCU consumes. Anyway for right now, I figured out secondary indexes can be assigned only for attributes that have mandatory values and gave up to use secondary indexes because of single-table design which the target attributes for secondary indexes are not mandatory values.

Hey @jangminhyeok, Please provide us the model stack generated after executing the override? it will help us understand the issue better.

Hey @jangminhyeok, Are you still experiencing the issue?

Hey 👋 , This issue is being closed due to inactivity. If you are still experiencing the same problem and need further assistance, please feel free to leave a comment. This will enable us to reopen the issue and provide you with the necessary support.


which riscV mcu ?

why did you work so hard to remove gd32vf103 reference in the documentation ?
It's pretty clear the riscV mcu is this one on your SOC.

Bouffalo makes SOCs that are similar to the GD32V. They don't use the same peripheral sets. They don't even license the RISC-V cores from the same vendor (Nuclei vs. SiFive). (I've programmed both.) Lots of the middleware (e.g. FatFS, LGVL, etc.) will run on any part of this class, including ARM cores, so that code will look familiar.
I don't know if you think that Bouffalo is rebadging or even cloning the GD32V parts, but they're just not.  The libfat they're providing doesn't include code for the GD32V, but that doesn't mean they removed it - it came from the same beginnings and both vendors acknowledge that. The ChaN code is a good implementation that shows up on mcus everywhere from 8051 up.
It's clear that the Boufallo parts are RISC-V, just like the GD32V is.  There is no "aha!" Scooby-Doo reveal there. That's clearly stated by both of them. It's an open source IP and there are dozens or maybe even hundreds of implementations of RISC-V at this point.
What are you talking about?
If we take the title literally and try to answer that, we can answer that from the first sentence in README.md even without speaking any Chinese. This repository contains code for the BL602, BL604, BL702, BL704, and BL706 by Bouffalo Labs - the "BL" in those numbers.  Bouffalo isn't a well known company in the Western world - hell, I don't know if they're well known within China - but they're an actual company that makes actual chips. https://www.cnx-software.com/2020/10/24/bl602-bl604-risc-v-wifi-bluetooth-5-0-le-soc-will-sell-at-esp8266-price-point/ BL60x has been shipping for a while and is in many products that would have probably otherwise had ESP8266-class devices while BL70x is pretty new.
Be more respectful if you're asking an actual question. Have actual evidence if you're making an accusation. Your post is like asking StarFive or Kendryte why they removed GD32V references.

Thank you Very much Robert for your answer. I was a little rude i admit.


Handling of integrable singularities with quad_vec

Is your feature request related to a problem? Please describe.
I noticed that integrate.quad() correctly solves integrable singularities, whereas integrate.quad_vec() cannot. Here is a minimal working example:
from scipy.integrate import quad_vec, quad

def f(x):
    return 1/pow(1-x, 0.5)

x0, x1 = 0, 1

yl, err = quad(f, x0, x1)

yv, errv = quad_vec(f, x0, x1)

Within the numerical error, yl equals the analytic result 2. However, quad_vec() throws a ZeroDivisionError, which comes from line 483 of _quad_vec.py . Both results agree when the upper limit x1 is less than 1.
Describe the solution you'd like
Have quad_vec solve integrable singularities.
Describe alternatives you've considered
In my real problem, I use quad() and loop over the array elements. But this is slow, which is why I wanted to use quad_vec in the first place.
Additional context (e.g. screenshots)
By using numpy.sqrt() instead of pow, I get a RuntimeWarning: division by zero instead of an error, and the result is np.inf.

This is I think the Wynn epsilon part that was not implemented in quad_vec.
The Pade extrapolant computation is here https://gist.github.com/pv/756d55c3ceed89291a2ab68302ffee51, the missing part is hooking it up in the proper way in the subdivision loop.


[Snyk] Security upgrade zipp from 3.15.0 to 3.19.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements/dev.txt

⚠ Warning
```
Chameleon 4.3.0 requires importlib.resources, which is not installed.
Chameleon 4.3.0 requires importlib.metadata, which is not installed.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiJhY2JmZGE5OS02ZjViLTQ0NWEtODQ5Yi00YjQ4ODNjNTE3NzgiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6ImFjYmZkYTk5LTZmNWItNDQ1YS04NDliLTRiNDg4M2M1MTc3OCJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/arkid15r/project/645ed7a6-2a3a-4db7-b0e3-a216827cd4c5?utm_source=github&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/arkid15r/project/645ed7a6-2a3a-4db7-b0e3-a216827cd4c5?utm_source=github&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Learn about vulnerability in an interactive lesson of Snyk Learn.](https://learn.snyk.io/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"zipp","from":"3.15.0","to":"3.19.1"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-ZIPP-7430899","priority_score":738,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Infinite loop"}],"prId":"acbfda99-6f5b-445a-849b-4b4883c51778","prPublicId":"acbfda99-6f5b-445a-849b-4b4883c51778","packageManager":"pip","priorityScoreList":[738],"projectPublicId":"645ed7a6-2a3a-4db7-b0e3-a216827cd4c5","projectUrl":"https://app.snyk.io/org/arkid15r/project/645ed7a6-2a3a-4db7-b0e3-a216827cd4c5?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-ZIPP-7430899"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

A duplicate PR with no changed files.


no such container error

something went wrong, the error message is
no such container
404
no such id: jovial_carson

@marcomarasco What image did you try to pull? Can you provide steps to repeat?

merging with #1030


joystick node guidance by wire

Joystick node needs to be published to setpoints for the reference model.

open loop
depth hold
pose hold
depth heading hold

that modes need to be bounded for control buttons.

Make sure to keep integral action always off for the dp controller

Here is the proposed structure for solving this issue.

Pull #152 implements the structural changes to the joystick system, minus the control mode select. (This will come when the joystick is to be integrated with the controller

Start in AUV-mode by default, but possible to switch to ROV-mode via the joystick. I have written some code, but not published as of yet

Especially for Tau, I think having the ability to intervene at any point is good. We might have to add something in the current state machine that exists whatever it is currently doing when the joystick is activated? For the potential robosub stuff, we won't have a tether/joystick connected, so it seems safe to have it this way 👍

Closing this now that #162 has been merged


Question about filesystem

Its can write i  file system? Or run server code? Thanks you

It can generate the scripts but it doesn't have the skill yet to execute them or write to file system yet. But it is a good idea for making a pull request


Comment what local_assigns a partial can take

For ease of development, have comments near the top of a partial to explain what local_assigns it takes and how it makes use of them. This will also help people new to the code figure out what it does.
(Also should be documented, if distinct: locals.)

Rails 7.1 has some good support here :D
See "Allow templates to set strict locals." on https://github.com/rails/rails/blob/7-1-stable/actionview/CHANGELOG.md#rails-710beta1-september-13-2023


Bump dcos-net

High-level description

Add lashup metrics
Upgrade OTP version to 21.3
Add dns worker metrics

Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-4756 Add lashup metrics.
DCOS_OSS-4902 Upgrade OTP version to 21.3.
DCOS-49018 Add dcos-dns metrics for workers.

Related tickets (optional)
Other tickets related to this change:
Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: none
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]
https://github.com/dcos/dcos-net/compare/03975850c8240821a6dbe2e00750fb0fcc568e2a...9879b485f870c9c8fdc04a7f4cf6e1185fa5ff24
https://github.com/dcos/dcos-net/pull/121
https://github.com/dcos/dcos-net/pull/124
https://circleci.com/gh/dcos/dcos-net/942
https://codecov.io/github/dcos/dcos-net/commit/9879b485f870c9c8fdc04a7f4cf6e1185fa5ff24
https://github.com/erlang/otp/compare/377f19f25aeec6939a6728bd0c4910086c22ccdc...620ac3e68c5bc8b36143965fcf2892a07dc005c4

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label [Ready For Review]


Support classes for fixture element

Hello,
This PR allow to add class to content editor components that use fixtures, exactly like adding class to a normal HTML element. something like this :
{% component 'contenteditor' file="test" fixture="p" class="special" %}

It will output something like this
<p data-fixture="" data-ce-tag="p" data-component="contenteditor::onSave" data-file="test" data-tools="" class="special">

See this issue

This is cool! Can you please add that class support in
https://github.com/Samuell1/contenteditor/blob/master/components/contenteditor/render.htm ?

Commit added for render.htm

I forgot to say PRs needs to be targeted to develop branch. Do you think you can change it to develop?

Switched the PR to the develop branch !


ERROR: cannot-start when afm-util start agl-service-helloworld

Hello,
I got an error as below. Could anyone help me solving this problem and running helloworld sevice please?
raspberrypi3:~# afm-util install /tmp/agl-service-helloworld.wgt
{
<EMAIL_ADDRESS>}
raspberrypi3:~# afm-util start<EMAIL_ADDRESS>ERROR:  cannot-start
My environment to build helloworld service is described below.

Used xds-server on Docker container (docker_agl_worker-xds-<IP_ADDRESS>) to make agl-service-helloworld.wgt. Built as below
[07:31:45] devel@agl-xds-ic-LM-iHS320-0-ic:~/xds-workspace/helloworld-service(flounder)$ . /xdt/sdk/poky-agl/6.99.2/armv7vehf-neon-vfpv4/15c5b0191a303c615c371f62bfb6a66e/environment-setup-armv7vehf-neon-vfpv4-agl-linux-gnueabi
[07:32:13] devel@agl-xds-ic-LM-iHS320-0-ic:~/xds-workspace/helloworld-service(flounder)$ ./conf.d/autobuild/agl/autobuild package
Target board is Raspberrypi3
SDK is AGL-release-guppy-6.99.2-raspberrypi3, installed from XDS Dashboard.
The target is running with Flounder 6.0.2, I built as below
cd $AGL_TOP
source meta-agl/scripts/aglsetup.sh -m raspberrypi3 agl-demo agl-netboot agl-appfw-smack agl-devel
bitbake agl-demo-platform

I basically followed flounder version of the document site to got started.
I am a real newbie in open source project including AGL. So, please let me know if any information is missing in this post.

Hello @thisamatsu,
the commit https://github.com/iotbzh/helloworld-service/commit/9f3e302ec7e8548d4d1d35e1dc1b2e7ca78e993b should resolve the issue you found. Be free to test and confirm if the issue is resolved on your side too.
Thanks.

Hello pmarzin,
Thank you for your comment. The sample works with that change.
Thanks,

Thanks for your feedback @thisamatsu.
I then close the issue.
Regards.


Vault - Expand HKDF common test cases

Need more test cases added to vault common HKDF HMAC-SHA256 tests

https://tools.ietf.org/html/rfc5869#page-10
https://github.com/google/wycheproof/blob/master/testvectors/hkdf_sha256_test.json


Design v2 API

Need to design new client APIs for the upcoming LE v2 endpoint.
V1 APIs should remain intact as long as LE supports v1 endpoint.

Thanks for the help and clarification. The API doc is updated to match implementation now.


expose linked map info to metadata

Make https://github.com/polkadot-js/api/issues/784 possible
I have three ways to do this:
1ce4b94e632d6832fcc042097303a08c9650f9e4: Prefer way to me but also requires bumping metadata version
806df3a3e526d6918b15431c9f7a6122ed9ab56b: The easy way in the most backward compatible manner. Still going to cause issue for old SDK version doesn't understand LinkedMap, but less destructive error.
bc902ad2fd3b65a8a11583c465c0b5395b414062: Add new variant to enum is a parity-codec compatible change. Still going to cause crashing issue for old SDK version that doesn't handle the new enum variant. This going to make DoubleMap support more smooth as well.
@jacogr Any other better ideas to do this without bumping metadata version?

That will be a metadata breaking change and will require a metadata version bump.

I believe we should be going for the simplest, cleanest and most long-term viable approach here - the version is there so clients can know that there were breaking changes, so we may as well use it is that is the cleanest. (Even if that means some more work for SDKs)
I'm partial to either the first or second option, probably @bkchr and @gavofyork should weigh in here as well

option 1 fine for me.


Add Honey Pie by JAWNY

Song Name
Honey Pie
Artist Name
JAWNY
Source
Youtube
Youtube Link
pM-x6vOBGY0qrHMB
Direct File Link
No response
Song ID
123455
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 108230313

reject
invalid YouTube link


Web terminal: increase test coverage in preparation for release review

What will this achieve?
This component needs a few more tests in order to reach 100% test coverage and so we can start the release review.
How will success be measured?
When the component has 100% test coverage.
Current coverage

Additional information

Designs
Existing code
etc

Complete


MRG: Add NIRS support to animate topomap

What does this implement/fix?
Add support for NIRS types to animate_topomap.
Additional information
I am not happy with the test I have added. See below. Suggestions welcomed.

@rob-luke is it worth adding or updating some latest.inc entry? Maybe just tack your name and the :func: or :meth: onto https://github.com/mne-tools/mne-python/blob/master/doc/changes/latest.inc#L38 ? That way all the viz stuff can be grouped together in one entry

#7057

@rob-luke is it worth adding or updating some latest.inc entry? Maybe just tack your name and the :func: or :meth: onto https://github.com/mne-tools/mne-python/blob/master/doc/changes/latest.inc#L38 ? That way all the viz stuff can be grouped together in one entry

Thanks

@larsoner merge when happy

Thanks @rob-luke


Section Delegation

I don't remember reading about delegation, but if it's already in, please close this request.
But this section is really more for when the current leader retires and needs to pass on the leadership on to the next person who is ready.  Of course in other places where delegation is needed.
If possible, I'm willing to make a pull request to add my sustaining-it  branch to the main repo where these experimental sections can be written.  Or I can work with those who want to write these in my own fork and then make the requests myself.

@belkinsa hi! thanks for opening this issue. there's a small section on "distributed leadership" here, in the building communities section.  If I remember correctly I think we avoided the term "delegation" because it felt a little too much like a manager-term, but that's up for debate.  What would you add, or change, and where do you think this section should live?

That's the section that I was thinking of, thank you!  And I think it's fine the way it is.