A2DP not working with Car Stereo (IDFGH-11166)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
Latest version installed with fresh Arduino installation
Espressif SoC revision.
VID: 0x1c04 PID 0xEA60
Operating System used.
Windows
How did you build your project?
Command line with Make
If you are using Windows, please specify command line type.
None
Development Kit.
ESP wroom 32
Power Supply used.
USB
What is the expected behavior?
As the A2DP example ESP32_A2DP_example_sender_8bit works perfectly fine with BT speakers which are not having any pin.
But Car stereo's which we call as Android Headup Display have a default pin "0000".
I am unable to connect the ESP32_A2DP_example_sender_8bit example project to the Android Head up Display.
My setup function and isValid funciton is as mentioned below rest is unchanged.
`bool isValid(const char* ssid, esp_bd_addr_t address, int rssi){
Serial.print("available SSID: ");
Serial.println(ssid);
return true;
}
void setup() {
Serial.begin(115200);
a2dp_source.set_ssid_callback(isValid);
a2dp_source.set_pin_code("0000",ESP_BT_PIN_TYPE_VARIABLE);
a2dp_source.set_ssp_enabled(1);
a2dp_source.set_auto_reconnect(false);
a2dp_source.start(get_data_frames);
a2dp_source.set_volume(30);`
What is the actual behavior?
Here I am explaining just what happens when we connect the Android Headup Display ( AHUD )with our Android Phone.
When the search for new bluetooth device is started, and the device (AHUD) is clicked upon a pop comes for entering password.
I am guessing this problem is coming.
In ESP32, I am unable to change the program to work with AHUD as it throws a pin pop on Android device whereas in ESP32 i am unaware of this function.
I have enable SSP in the code and set the pin but still it deosn;t connect.
Steps to reproduce.

I used A2DP example provided in Arduino Iibrary bt_music_sender_ssid_callback
Then I added following code to the setup function.
  a2dp_source.set_pin_code("0000",ESP_BT_PIN_TYPE_VARIABLE); a2dp_source.set_ssp_enabled(1);
The code works on Bluetooth speakers and Android Auto as these devices doesn't have any pin.
But Car Audio systems which we call as Android Headup display , , generally have default password "0000"
The ESP-32 is not able to connect to the Android Head Display .
...

Debug Logs.
[ 11144][D][BluetoothA2DPSource.cpp:458] bt_app_gap_callback(): [BT_AV] bt_app_gap_callback evt 0
[ 11144][I][BluetoothA2DPSource.cpp:385] filter_inquiry_scan_result(): [BT_AV] Scanned device: 00:58:76:c7:18:07
[ 11151][I][BluetoothA2DPSource.cpp:391] filter_inquiry_scan_result(): [BT_AV] --Class of Device: 0x240404
[ 11161][I][BluetoothA2DPSource.cpp:395] filter_inquiry_scan_result(): [BT_AV] --RSSI: -95
[ 11169][I][BluetoothA2DPSource.cpp:411] filter_inquiry_scan_result(): [BT_AV] --Compatiblity: Compatible
[ 11178][I][BluetoothA2DPSource.cpp:413] filter_inquiry_scan_result(): [BT_AV] --Name: CarKit_blink
[ 11187][I][BluetoothA2DPSource.cpp:433] filter_inquiry_scan_result(): [BT_AV] --Result: Target device found
[ 11196][D][BluetoothA2DPCommon.cpp:177] set_last_connection(): [BT_AV] set_last_connection: 00:58:76:c7:18:07
[ 11206][D][BluetoothA2DPCommon.cpp:181] set_last_connection(): [BT_AV] no change!
[ 11213][I][BluetoothA2DPSource.cpp:437] filter_inquiry_scan_result(): [BT_AV] Cancel device discovery ...
[ 11224][D][BluetoothA2DPSource.cpp:458] bt_app_gap_callback(): [BT_AV] bt_app_gap_callback evt 1
[ 11231][I][BluetoothA2DPSource.cpp:468] bt_app_gap_callback(): [BT_AV] Device discovery stopped.
[ 11240][I][BluetoothA2DPSource.cpp:469] bt_app_gap_callback(): [BT_AV] a2dp connecting to peer: CarKit_blink
[ 11249][W][BluetoothA2DPCommon.cpp:41] connect_to(): [BT_AV] connect_to to 00:58:76:c7:18:07
[ 11258][I][BluetoothA2DPCommon.cpp:277] set_scan_mode_connectable(): [BT_AV] set_scan_mode_connectable true
[ 11370][D][BluetoothA2DPSource.cpp:250] bt_app_work_dispatch(): [BT_API] bt_app_work_dispatch event 0x0, param len 16
[ 11371][D][BluetoothA2DPSource.cpp:301] bt_app_task_handler(): [BT_API] bt_app_task_handler, sig 0x1, 0x0
[ 11379][I][BluetoothA2DPSource.cpp:631] bt_app_av_sm_hdlr(): [BT_AV] bt_app_av_sm_hdlr state 4, evt 0x0
[ 11388][D][BluetoothA2DPSource.cpp:599] process_user_state_callbacks(): [BT_AV] process_user_state_callbacks
[ 11397][D][BluetoothA2DPSource.cpp:607] process_user_state_callbacks(): [BT_AV] process_user_state_callbacks ESP_A2D_CONNECTION_STATE_EVT: Connecting
[ 11411][D][BluetoothA2DPSource.cpp:696] bt_app_av_state_connecting(): [BT_AV] bt_app_av_state_connecting evt 0
[ 16497][D][BluetoothA2DPSource.cpp:458] bt_app_gap_callback(): [BT_AV] bt_app_gap_callback evt 16
[ 16498][I][BluetoothA2DPSource.cpp:526] bt_app_gap_callback(): [BT_AV] event: 16
[ 16502][D][BluetoothA2DPSource.cpp:250] bt_app_work_dispatch(): [BT_API] bt_app_work_dispatch event 0x0, param len 16
[ 16513][D][BluetoothA2DPSource.cpp:301] bt_app_task_handler(): [BT_API] bt_app_task_handler, sig 0x1, 0x0
[ 16522][I][BluetoothA2DPSource.cpp:631] bt_app_av_sm_hdlr(): [BT_AV] bt_app_av_sm_hdlr state 4, evt 0x0
[ 16531][D][BluetoothA2DPSource.cpp:599] process_user_state_callbacks(): [BT_AV] process_user_state_callbacks

More Information.

I used A2DP example provided in Arduino Iibrary bt_music_sender_ssid_callback
Then I added following code to the setup function.
  a2dp_source.set_pin_code("0000",ESP_BT_PIN_TYPE_VARIABLE); a2dp_source.set_ssp_enabled(1);
The code works on Bluetooth speakers and Android Auto as these devices doesn't have any pin.
But Car Audio systems which we call as Android Headup display , , generally have default password "0000"
The ESP-32 is not able to connect to the Android Head Display .

Hi,
I searched online that ESP-IDF on arduino may not work as expected hence I installed ESP-32 IDF latest version on ubuntu.
Following error is coming when i am trying to connect the device using ESP-32 IDF using the default program of A2DP_source
`I (663) main_task: Calling app_main()
I (683) BTDM_INIT: BT controller compile version [153f120]
I (683) BTDM_INIT: Bluetooth MAC: cc:db:a7:14:c7:d6
I (683) phy_init: phy_version 4771,450c73b,Aug 16 2023,11:03:10
I (1363) BT_AV: Starting device discovery...
W (1363) BT_BTC: btc_avrc_tg_set_rn_supported_evt failed: AVRC TG not yet initialized
W (1373) BT_BTC: A2DP Enable with AVRC
W (1373) BT_AVRC: For AVRCP Target Cateory 1 and 3, SDP record shall contain additional protocol list
I (1393) BT_AV: bt_app_av_sm_hdlr state: 1, event: 0x4
I (1393) BT_AV: event: 10
I (1403) BT_AV: event: 10
I (1403) BT_AV: event: 10
I (1403) main_task: Returned from app_main()
I (1413) BT_AV: Discovery started.
I (5613) BT_AV: Scanned device: 00:58:76:c7:18:07
I (5613) BT_AV: --Class of Device: 0x240404
I (5613) BT_AV: --RSSI: -75
I (6023) BT_AV: Scanned device: 00:58:76:c7:18:07
I (6023) BT_AV: --Class of Device: 0x240404
I (6023) BT_AV: --RSSI: -76
I (6223) BT_AV: Scanned device: 00:58:76:c7:18:07
I (6223) BT_AV: --Class of Device: 0x240404
I (6223) BT_AV: --RSSI: -77
`

@divyansh Thank you for your feedback. These warnings do exist for certain reasons, but theoretically they will not affect the establishment of the connection. Is it still not possible to connect to your device Using ESP-IDF A2DP_SOURCE?

Hi,
Yes I am finding difficulty in connecting the device.

@divyansh Sorry for late reply. Please attach the whole logs on ESP32 side. Thanks!


Embedded Image (max) scaling

Hi,
I've just integrated flexmark on our site, and I'm very happy with it.
I'd need a way to control the scaling of inline images. For one, I want to set the maximum dimension as an option and I'd also like to allow the user to specify the dimensions (within that limit) himself.
Is this a planned feature? Or is this already possible somehow and I missed it? Would this something to implement in the core or would you have to create an extension for this (even though it's an extension of a standard Markdown element)?

@Grestorn, you need to implement a custom attribute provider. See: AttributeProviderSample2.java
You can use the same provider to add/remove/change attributes for any node being rendered.
To allow attributes to be modified in markdown you can enable the AttributesExtension which adds ability to add any HTML attribute to any element. See: Extensions, described in Attributes-Extension
However, to implement limiting the user provided attribute within the default maximum dimension will require to create a custom node renderer which will examine the actual size attribute to the default maximum and change it if it exceeds the maximum. For an example of a custom renderer see  NodeRendererSample.java

Thanks for those pointers, I'm going to look them up!


try-runtime-cli: parallel key scraping

Motivation
The current key scraping implementation is is quite inefficient, fetching 1k keys at a time, waiting for each batch of 1k keys to be returned before asking for the next one
https://github.com/paritytech/substrate/blob/master/utils/frame/remote-externalities/src/lib.rs#L348-L392
This is not so bad when fetching keys from a local node where latency is not a big factor, but can slow things down very significantly when using a remote node when most of the time spend fetching keys becomes latency rather than the node doing work.
Unfortunately, most nodes seem to have a hard limit imposed preventing more than 1k keys being fetched in a single request.
Suggested solution
Rather than fetching all keys for the prefix in a single rpc_get_keys_paged request here https://github.com/paritytech/substrate/blob/master/utils/frame/remote-externalities/src/lib.rs#L528-L533, split the prefix up into N parts and run the rpc_get_keys_paged tasks in parallel using async/await.
e.g. two parallel tasks:
start_keys: 0x000000 end_keys: 0x777777
start_keys: 0x777777 end_keys: 0xffffff
Bonus (probably for a seperate issue/PR once this one is closed)
Fetch storage values for keys as they're pulled from the node, instead of waiting for all keys to be pulled before fetching values.
Notes
This is not high priority, but worth doing once try-runtime-cli is in a more complete stable state to improve the experience pulling state from a remote node.

https://github.com/paritytech/substrate/issues/12236#issuecomment-1246997128 could also help here to increase the page size so that it downloads more than 1k per page.

@liamaharon I believe that parallel scraping would be a great improvement in the try-runtime-cli user experience. As someone who was lately using the try-runtime-cli to run the try_state over the state of Polkadot and Kusama, I can definitely tell that it took quite some time to scrape all the keys.
I would like to implement parallel key scraping if no one is working on this currently.

@Szegoo have you tried downloading once to a snapshot and then loading from it each time thereafter? Loaded from a snapshot, the entire Polkadot state takes <1s

@liamaharon When testing I had a node running in the background which synced with Polkadot/Kusama, so no I did not download a snapshot once, I was always running try-runtime on the latest state. In another terminal, I was executing try-runtime commands, but as long as not scraping all the keys it won't take a lot of time(I was only scraping keys from a single pallet when doing try-state checks). The only problem is when scraping all the keys so that is when parallel key scraping would come really handy.

Not really, you are right having a snapshot is more efficient.

WDYT about adding a log when people use live state for something other than creating a snapshot, that they probably want to create a snapshot if they'll be running it repeatedly? I could imagine some devs just being unaware the snapshot feature exists, or how to use it.

WDYT about adding a log when people use live state for something other than creating a snapshot, letting them know they probably want to create a snapshot if they'll be running it repeatedly?

Might be a good idea. Did you imagine having this log whenever the --sync warp option is used?

I was thinking when someone executes a command like on-runtime-upgrade live ws://..., they probably want to use snap instead of live. Not sure why the sync mode would matter, maybe I misunderstand?

I misunderstood you. Yes, it definitely makes sense to warn a user in that case.


Optional route parameter constraint

I am unsure if this is supported or I'm missing some configuration to have it working. Below is an example of my route:
[HttpTrigger(AuthorizationLevel.Anonymous, "get", Route = "v1/product/search/{productNumber:int}/{productVersion:int?}")] HttpRequest request,

Notice the ? which indicates that the parameter is optional. Below is what swagger generate:

It seems to not end the trailing } in the title and it shows that the productVersion parameter is required when really it's optional.
I am using version 3.1.6.

Hi, @ducman
Unfortunately, it does not supported.

If the parameter is in "path", this property is required and its value MUST be true.

https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#fixed-fields-7
Similar issue =>
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/406


Feature not available

Type: Feature Request
Run button on visible on top even after installing code runner extension
VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: Yes

/causedByExtension
Please consult the extension author.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Remove unused common-core dep from JMX

Windows Build 907 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Linux Build 1218 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Core - Full Integration Build 485 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Windows Build 907 outcome was FAILURE using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Tests failed: 6 (6 new), passed: 3015, ignored: 95 Build time: 0:44:41
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.respawn.RespawnTestCase.testStartKilledServer: java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)

org.jboss.as.test.integration.respawn.RespawnTestCase.testReloadHcButNotServersWithFailedServer: java.lang.AssertionError: Did not have all running processes [Process{id=6828, process=Host Controller}, Process{id=2944, process=respawn-two}]
	at org.junit.Assert.fail(Assert.java:88)
	at org.jboss.as.test.integration.respawn.RespawnTestCase.waitForProcesses(RespawnTestCase.java:516)

org.jboss.as.test.integration.respawn.RespawnTestCase.testDomainRespawn: java.lang.AssertionError: Did not have all running processes [Process{id=6828, process=Host Controller}, Process{id=2944, process=respawn-two}]
	at org.junit.Assert.fail(Assert.java:88)
	at org.jboss.as.test.integration.respawn.RespawnTestCase.waitForProcesses(RespawnTestCase.java:516)

org.jboss.as.test.integration.respawn.RespawnTestCase.testReloadHc: java.lang.AssertionError: Did not have all running processes [Process{id=6828, process=Host Controller}, Process{id=2944, process=respawn-two}]
	at org.junit.Assert.fail(Assert.java:88)
	at org.jboss.as.test.integration.respawn.RespawnTestCase.waitForProcesses(RespawnTestCase.java:516)

org.jboss.as.test.integration.respawn.RespawnTestCase.testReloadHcButNotServers: java.lang.AssertionError: Did not have all running processes [Process{id=6828, process=Host Controller}, Process{id=2944, process=respawn-two}]
	at org.junit.Assert.fail(Assert.java:88)
	at org.jboss.as.test.integration.respawn.RespawnTestCase.waitForProcesses(RespawnTestCase.java:516)

org.jboss.as.test.manualmode.vault.VaultPasswordsInCLITestCase: java.lang.RuntimeException: Could not stop container
	at sun.nio.ch.SocketChannelImpl.checkConnect(Native Method)
	at sun.nio.ch.SocketChannelImpl.finishConnect(SocketChannelImpl.java:708)

Linux Build 1218 outcome was FAILURE using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Execution timeout (new); tests passed: 2677, ignored: 80; org.wildfly.core:wildfly-core-testsuite-domain Build time: 0:45:01
Build problems:
Execution timeout
Process exited with code 137

Core - Full Integration Build 485 outcome was SUCCESS using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Tests passed: 2859, ignored: 378 Build time: 0:48:58

retest this please

Core - Full Integration Build 487 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Linux Build 1220 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Windows Build 909 is now running using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849

Windows Build 909 outcome was SUCCESS using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Tests passed: 3020, ignored: 95 Build time: 0:18:36

Linux Build 1220 outcome was SUCCESS using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Tests passed: 3020, ignored: 95 Build time: 0:19:45

Core - Full Integration Build 487 outcome was SUCCESS using a merge of e6e61df54879f5a306eb04d7202be5b2bddba849
Summary: Tests passed: 2859, ignored: 378 Build time: 0:48:19


Implement learning rate schedule

I was able to get better results with SGD on my own dataset with this library, if I use SGD and half the learning rate after each epoch, than using Eddy without such a learning rate schedule.
It would be nice if this was an official feature. I would make a pull request, but I am not sure what the best way is to implement it.

Mind posting your unofficial working code?  I would be very excited to see what you have, and perhaps, it would give us a starting point for making it an official feature.
There's definitely already a great selection of update rules in Passage, I'm wondering if you've tried any of them for your task in substitution for scheduled-halving-SGD


Post-usability testing fix: work out solution for prompting people to save marked regions before leaving the image

This one needs some thought.

Duplicate of #124


Add Support to run tests in docker

What is the motivation for adding / enhancing this feature?
The docker container does not include the needed dependencies for cypress. So, no tests can be run at this point.
What are the acceptance criteria

When we can use cypress to run tests in the docker container.

Can you complete this feature request by yourself?
Yes
Additional information
I think switching to jessie instead of alpine would be a solution and then update the docker file to install the dependencies for cypress, using apt, which can be found here: https://docs.cypress.io/guides/guides/continuous-integration.html#Advanced-setup

Why would jessie be better than alpine?

If I remember correctly the dependencies for cypress was not available for alpine.

Alpine indeed does not have all the dependencies recommended by the Cypress documentation:
RUN apk add xvfb libgtk2.0-0 libnotify-dev libgconf-2-4 libnss3 libxss1 libasound2
 ---> Running in 1712070bdd41
fetch http://dl-cdn.alpinelinux.org/alpine/v3.9/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.9/community/x86_64/APKINDEX.tar.gz
ERROR: unsatisfiable constraints:
  libasound2 (missing):
    required by: world[libasound2]
  libgconf-2-4 (missing):
    required by: world[libgconf-2-4]
  libgtk2.0-0 (missing):
    required by: world[libgtk2.0-0]
  libnss3 (missing):
    required by: world[libnss3]
  libxss1 (missing):
    required by: world[libxss1]

I was able to create an experimental Docker image on top of debian:jessie, and successfully run the tests in it:

But now I'm wondering whether there is any point in running e2e tests unless we also have an instance of both Vue Storefront API and Magento running within the same container (or somewhere else as long as the container has access to them?).
Any ideas?

The workaround in my project is currently that only the VSF branches are being run inside containers, and they all connect to a separate external VSF-API instance.


rhel 9: Regression tests are broken

Are we even supposed to be running them on RHEL 9?
See https://gitlab.com/osbuild/ci/osbuild-composer/-/jobs/1477814548

We decided that we want to run the tests on RHEL-9 as well, assigning to myself.

Fixed in https://github.com/osbuild/osbuild-composer/pull/1748


v5.5.0

Fix typo
Added Go modules support

Pull Request Test Coverage Report for Build 146

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 95.142%

Totals

Change from base Build 140:
0.02%

Covered Lines:
235

Relevant Lines:
247

💛  - Coveralls


Expose DWN PermissionsGrant Message

Minimum permissions interface to expose to unblock delegated permissions grants.
dwn.permissions.send() will be extended in a future PR to to send other Permissions interface messages.

@diehuxx Reviewed the PR and had one question that I was hoping to get your insight on:
The approach to sending a PermissionsGrant is different than how both Record and ProtocolsConfigure messages are sent to a remote DWN.  What was the motivation for introducing a new message property to the send request object?


Loading screen interface and implementation

It is now possible to create a new implementation of the loading screen and set it in the engine.
The engine now includes a loadingScreen variable that is per default the native loading screen but can be set with a new loading screen in accordance to the ILoadingScreen interface.
Breaking changes
the engine's set loading text and set background color were removed as they are no longer conform with the new interface!

Hello I think this is great but I would like to keep the set loading text and background color on the engine. This then will be sent to the interface. If someone wants to have a loader which does not take this in account that's fine but I think this is a good first level of customization without having to completely rewrite a loader

Will make the changes and commit soon

Interface and engine updated (including a small and silly bug fix).

Another small ask :) can you update the file list here:https://github.com/BabylonJS/Documentation/blob/master/content/generals/General/Creating_the_Mini-fied_Version.md

updating the documentation now.

Lol this is fast:) thank you!

I'm only fast when I am sitting in front of my computer :smile:

I hope so :D


[MTSRE-272] Adding the AddonOperator CR to the OLM artifacts

Signed-off-by: Amador Pahim<EMAIL_ADDRESS>

lgtm


doc: fix xfr example.

The currently example code has a data race, put in the proper code.
Fixes: #1061

Codecov Report

Merging #1062 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1062   +/-   ##
=======================================
  Coverage   54.93%   54.93%
=======================================
  Files          41       41
  Lines        9878     9878
=======================================
  Hits         5426     5426
  Misses       3430     3430
  Partials     1022     1022

Impacted Files
Coverage Δ

xfr.go
32.62% <ø> (ø)
: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 bfd8601...0b9f589. Read the comment docs.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] doc: fix xfr exampl..." ]

Missing ().

thanks. Wish we could test these code snippets.


"Chaves" instead of "teclas" in portuguese

Microsoft PowerToys version
0.61.1
Utility with translation issue
Keyboard Manager
🌐 Language affected
Portuguese
❌ Actual phrase(s)
Remapear uma chave
✔ Expected phrase(s)
Remapear uma tecla
ℹ Why is the current translation wrong
It doesn't make sense, because in other occurrencies in the same context, it is "teclas".

https://microsoft.visualstudio.com/OS/_workitems/edit/42763742 /loc

This should be fixed in a newer build such as .70.1, can you please verify? /needinfo


Fixed issue #875 - Added eightify to ai

Related Issue
Closes #875

Description

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[ ] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Hello @saiyyamkochar-29, this issue is assigned to me. Afterwards if you find any issue in that then you may raise issue. Please delete your PR.

Hello, contributors! This is Jason speaking. Due to some workflow issues, we will be closing all the PRs today. Therefore, I kindly request all of you to create new PRs tomorrow, as the currently open ones will be closed by our team. I apologize for any inconvenience caused. Also, please make sure to check the titles when creating the PRs. You can refer to the README.md for more details. Thank you.


No database

could not able to find your database file pms

its not public, if you want i will give you no problem @mjayaraj3


Add support for multiple Canadian retailers

Add support for CanadaComputers, EBGames, ToysRUs, Costco Canada.
@EricJMarti
I wanted to add support for the following retailers but faced some issues:

Amazon.ca: I get a float error as the currency field has text (CDN$ 99.99 for example)
MemoryExpress: I get a float error as MemoryExpress has the word "Only" before the number
Walmart.ca: Gets detected as a bot instantly, even when using selenium

Do you know if the Costco scraper works on the US site?
I haven't found a good way of handling prices yet, so I may change the code to simply look for a number in the price string and ignore any currency symbols. Fortunately, I don't think the price conversion bug actually prevents alerts from going out.
Also, can you take a look at the unit test suite I put together for scrapers? Basically, when you run a scraper, it dumps the html to inventory-hunter/data/${container_name}. From there, you can copy the contents into the tests directory and spin up a unit test for the parsing logic. You can use the run_tests.bash script to run the test suite.

Do you know if the Costco scraper works on the US site?
I haven't found a good way of handling prices yet, so I may change the code to simply look for a number in the price string and ignore any currency symbols. Fortunately, I don't think the price conversion bug actually prevents alerts from going out.
Also, can you take a look at the unit test suite I put together for scrapers? Basically, when you run a scraper, it dumps the html to inventory-hunter/data/${container_name}. From there, you can copy the contents into the tests directory and spin up a unit test for the parsing logic. You can use the run_tests.bash script to run the test suite.

I did not try with Costco.com as the PS5 they sell is a member only item and I guess that means there is no way to know when it is in stock
That's correct, the script can ignore the price but that mean it will send alerts for items sold by scalpers incase of Amazon.ca for example. The alternative option is to ignore add to cart and look for sold and shipped by Amazon.ca instead. That is something I plan to do for my personal use.
I will take a look at the test suite. Thanks 👍🏻

Also, interesting find regarding walmart.ca. It seems the US site doesn't have the same level of protection; I get a reCAPTCHA when I try scraping the Canadian site.

Also, interesting find regarding walmart.ca. It seems the US site doesn't have the same level of protection; I get a reCAPTCHA when I try scraping the Canadian site.

They are taking it so serious. Not just reCAPTCHA. They also show all products as unavailable if you connect using a known VPN provider/data center range.

Can you try Amazon.ca again with the latest image? I recently committed a change that was intended to ignore currency symbols.

Can you try Amazon.ca again with the latest image? I recently committed a change that was intended to ignore currency symbols.

@EricJMarti It works now with Amazon.ca, MemoryExpress and eBuyer (UK).
I have tried adding so many other sites over the week and had success with most but a few:
Amazon.de/.nl do not seem to work. Error: "unable to find price in string: "€ 624,02"
AntOnline/ZotacStore have false positives and no matter what I change they always send false positives.

I just checked in a change that should fix amazon.de and similar sites: https://github.com/EricJMarti/inventory-hunter/commit/171f261a39f5c463caff921efcd8f9d36f5bc681
Can you try these sites again? I'm really hoping this commit fixes a majority of the international price conversion issues we're seeing.

@MRizkBV Thanks for your contribution!
Also, I just put together this wiki page: https://github.com/EricJMarti/inventory-hunter/wiki/Retailer-Support-Matrix
I'll add these retailers to the matrix.


python311Packages.aiounifi: 65 -> 66

Diff: https://github.com/Kane610/aiounifi/compare/refs/tags/v65...v66
Changelog: https://github.com/Kane610/aiounifi/releases/tag/v66
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

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

3 packages built:

home-assistant-component-tests.unifi
python311Packages.aiounifi
python311Packages.aiounifi.dist


dev: tooling for developers to bring up local nodes

Updates local net script. Adds docker-compose and capability to bring up single and multiple nodes locally.

Instead of commenting out doc.go, should it be deleted?

Instead of commenting out doc.go, should it be deleted?

i wanted to leave it in as an example for us


Enhancements for Carrier Billing v0.2 - Part II

What type of PR is this?
Add one of the following kinds:

correction
enhancement/feature
documentation

What this PR does / why we need it:
This PR deals with some Issues whose output is already aligned/agreed:

Issue#109 [COVERED]

Enhancements in filtering for retrievePayments

Issue#110 [COVERED]

Aligments in transitionOperationStatus
New Exception 403 "CARRIER_BILLING.PAYMENT_DENIED"
New event payment-denied
Enhancements in documentation.

Issue#111 [COVERED]

Clarifications for retrievePayments and retrievePayment endpoints
New Exception 403 "CARRIER_BILLING.INVALID_TOKEN_CONTEXT" for these endpoints

Issue#116 [COVERED]

New endpoint (optional in 2-step) validatePayment
New event payment-pending-validation
New transationOperationStatus pending_validation
Enhancements in documentation.

Issue#117 [COVERED]

Implicitly covered with Issue#110

Other enhancements/corrections:

Fix scopes according to model proposed in Commonalities/PR#57
Fix typo in transactionOperationStatus query_param enum value "defined"
Examples values enhancement

Which issue(s) this PR fixes:
Fixes #109
Fixes #110
Fixes #111
Fixes #116
Fixes #117
Other enhancements/corrections:

Fix scopes according to model proposed in Commonalities/PR#57
Fix typo in transactionOperationStatus query_param enum value "defined"
Examples values enhancement

Special notes for reviewers:
N/A
Changelog input
N/A
Additional documentation
N/A

Merging this PR and I generate new Issue with related PR to set this proposal as Release Candidate


A few fixes to the sql syntax and fixed some issues w/ foreign key declarations.

Had some issues sourcing the sql file in MySQL and had to make the following changes in order for it to import into a clean database.
Thanks for creating this repo it was helpful for testing out queries from the book!

Hello @MilkyC ,
Thanks for your contribution.
I dont have mysql on my computer now so I cant test it. However I tried with online mysql terminal and it was working well. I hope people will benefit from it.
Thanks!


docker container fails with --extended-transactions

Describe the bug
When running mintapi from the provided docker image with --extended-transactions it fails.
What version of MintAPI are you using?
ghcr.io/mintapi/mintapi                                 latest           0e29c480113c   3 weeks ago      1.56GB
What command[s] did you run / steps to reproduce?
Stacktrace/error received:
docker run --rm --shm-size=2g ghcr.io/mintapi/mintapi mintapi $username $password --headless --use-chromedriver-on-path --extended-transactions Traceback (most recent call last):   File "/home/seluser/.local/bin/mintapi", line 8, in <module>     sys.exit(main())   File "/home/seluser/.local/lib/python3.8/site-packages/mintapi/cli.py", line 507, in main     data = mint.get_detailed_transactions(   File "/home/seluser/.local/lib/python3.8/site-packages/mintapi/api.py", line 472, in get_detailed_transactions     result = self.get_transactions_json(   File "/home/seluser/.local/lib/python3.8/site-packages/mintapi/api.py", line 429, in get_transactions_json     if __include_investments_with_transactions(id, include_investment): NameError: name '_Mint__include_investments_with_transactions' is not defined
What did you expect to happen?

What actually happened?
Application errored out.
Additional context
N/A

If it helps, regular -t doesn't work either.  With or without the start date.

Using the latest docker image and upgrading selenium changes the behaviour:
+ /home/seluser/.local/bin/mintapi $username $password --use-chromedriver-on-path --headless --extended-transactions --start-date 03/05/22 Traceback (most recent call last):   File "/home/seluser/.local/bin/mintapi", line 8, in <module>     sys.exit(main())   File "/home/seluser/.local/lib/python3.8/site-packages/mintapi/cli.py", line 507, in main     data = mint.get_detailed_transactions(   File "/home/seluser/.local/lib/python3.8/site-packages/mintapi/api.py", line 483, in get_detailed_transactions     df.amount = df.apply(reverse_credit_amount, axis=1)   File "/home/seluser/.local/lib/python3.8/site-packages/pandas/core/generic.py", line 5612, in __setattr__     self[name] = value   File "/home/seluser/.local/lib/python3.8/site-packages/pandas/core/frame.py", line 3645, in __setitem__     self._set_item_frame_value(key, value)   File "/home/seluser/.local/lib/python3.8/site-packages/pandas/core/frame.py", line 3775, in _set_item_frame_value     raise ValueError("Columns must be same length as key") ValueError: Columns must be same length as key
But it works without specifying the start date.  Seems like there is a problem with comparing the start date.  Weird that this doesn't happen on my non-docker system.

I guess this should be closed because --extended-transactions is no longer supported.

Thanks! Let me know if https://github.com/mintapi/mintapi/pull/477 addresses any docs concerns.


Fix hawkular provider refresh issue due to new resource type naming

The hawkular-agent-defined operating system resource types may no longer be identified with Id  equals 'Operating System'.  Instead, the more unique Ids are guaranteed only to contain the 'Operating System' substring.  This PR is required for use with hawkular server version 0.0.11 and later.  It is backward compatible with earlier server versions.
The PR also adds some middleware specific logging that would have helped immensely in finding this issue.
Links

https://issues.jboss.org/browse/HWKAGENT-133

@miq-bot add-label providers/hawkular,bug
@abonas please review and fast track as this is required for services 0.0.11 compatibility.

@miq-bot assign @Jiri-Kremser
@Jiri-Kremser please review

@jshaughn I am wondering if this should be categorized as a bug since this is caused by a change of behavior on the agent side.

lgtm,
btw. .include? is good because it won't break the backward compatibility with old h-services

@Jiri-Kremser thanks
@miq-bot assign @chessbyte


feat: add health liveness and readiness endpoints to go buildpack

This commit adds /health/readiness and /health/liveness endpoints
to the Go buildpack for both HTTP and CloudEvent functions.
See: https://github.com/boson-project/faas/issues/169
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

LGTM


bug: Particle Network revert

Bug Report
Current behavior:
Calling
curl --location 'https://sepolia-rpc.kakarot.org' \
--header 'Content-Type: application/json' \
--data '{
    "jsonrpc": "2.0",
    "id": 1,
    "method": "eth_estimateGas",
    "params": [
        {
            "from": "0xAC16076B48e197D7Dbc733aB7C5DC1002108720e",
            "to": "0x5FF137D4b0FDCD49DcA30c7CF57E578a026d2789",
            "data": "0xd6383f940000000000000000000000000000000000000000000000000000000000000060000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000004400000000000000000000000003ca0c1b9ad97e001a31ef1bf6de058fdf6d4b09e0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000016000000000000000000000000000000000000000000000000000000000000001e0000000000000000000000000000000000000000000000000000000000098968000000000000000000000000000000000000000000000000000000000000f4240000000000000000000000000000000000000000000000000000000000000c93c0000000000000000000000000000000000000000000000000000000000000001000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000002a000000000000000000000000000000000000000000000000000000000000003600000000000000000000000000000000000000000000000000000000000000058decb4780f435bb1194835e15cc964b1d9c8c79f25fbfb9cf000000000000000000000000a60123a1056e9d38b64c4993615f27cce9a9e8d5000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000084b61d27f6000000000000000000000000ac6a87c681a5ed4cb58bc4fa7bf81a83b928c83c000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000000600000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000095ba6ab55f69ba50b4c0ad96795a40b98e9b5f405e000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000003054659b5e29460a8f3ac9afc3d5fcbe4b76f92aed454b944e9b29e55d80fde807716530b739540e95cfa4880d69f710a9d45910f2951a227675dc1fb0fdf2c71c000000000000000000000000000000000000000000000000000000000000000000000000000000000000413054659b5e29460a8f3ac9afc3d5fcbe4b76f92aed454b944e9b29e55d80fde807716530b739540e95cfa4880d69f710a9d45910f2951a227675dc1fb0fdf2c71c000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000"
        }
    ]
}'

returns
{"jsonrpc":"2.0","error":{"code":3,"message":"execution reverted","data":"0x8b7ac980000000000000000000000000000000000000000000000000000000000005c519000000000000000000000000000000000000000000000000000000000005f21000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000ffffffffffff000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000c00000000000000000000000000000000000000000000000000000000000000000"},"id":1}

Steps to reproduce
https://stackblitz.com/edit/stackblitz-starters-uz6yqz?file=index.ts
Expected behavior:
Call should revert with revert ExecutionResult(opInfo.preOpGas, paid, data.validAfter, data.validUntil, targetSuccess, targetResult);

So the issue came from the fact that Particle Network was making a eth_call to the entrypoing contract, which included a gas limit for the call of 10M gas. However, during the call, there was a check to verify that gasleft() < callGasLimit + mUserOp.verificationGasLimit + 5000. Since when performing the eth_call, no gas limit was used as a parameter, we defaulted to 7 million gas, which caused the aforementioned check to fail.


Upgrading from CF to SAM - Stack already exists error

Description:
I'm transitioning from a currently deployed pure CloudFormation stack to a SAM template. This will be the first time deploying to the stack with SAM. I configured my samconfig.toml to use the already deployed stack's name. When I do sam deploy, I get the following error:
Error: Failed to create changeset for the stack: my-backend-infra,
An error occurred (ValidationError) when calling the CreateChangeSet operation:
Stack [my-backend-infra] already exists and cannot be created again with the changeSet [samcli-deploy1593732248].

Steps to reproduce the issue:

Deploy pure CloudFormation template with the name my-backend-infra
Change (some) resources to SAM
sam deploy --stack-name my-backend-infra

Observed result:
SAM failed to update/deploy the my-backend-infra stack due to it already existing.
Expected result:
SAM updates the my-backend-infra stack in-place with the new resources.
In the sam deploy docs it says that the --stack-name flag will update the existing stack if it exists. I can not delete the current stack due to it containing other production resources.

Just a workaround: take a look on CloudFormation "DeletionPolicy" : "Retain" and CloudFormation imports. You can deplete the current stack but without deleting the production relevant resources and then import them into the new stack.

I found the root of the problem. It was very helpful to add the --debug flag to sam deploy as it prints out the issue much more descriptively. Turns out that the user who I used to deploy the template with, did not have some cloudformation: permissions that were required. I tested with a user who had admin rights and it went through. It was not a problem with the stack already existing, but a policy problem that it had AccessDenied to the currently deployed stack. I was able to migrate to SAM and update this stack in-place without any downtime.


Unable to change debug level of the console output

I am sorry if this is not the right forum to ask these questions.
I have checked out the 249-SNAPSHOT branch and build it locally with my own outputWriters implementations.
I start the jmxtrans from command line using the following:
<java -Djmxtrans.log.level=10 -jar jmxtrans-249-SNAPSHOT-all.jar -e -f D:\Templates\AyusmanMultiServerConfig1.json -s 5 > D:\JmxTrans\logs\Set6\_Console.log>
This produces a lot of debug messages in the console:
<15:04:51.567 [Thread-2] DEBUG c.googlecode.jmxtrans.JmxTransformer - Stopped writer: SimpleCsvWriterForMemory for query: Query [obj=java.lang:type=Memory, resultAlias=null, attr=[HeapMemoryUsage, NonHeapMemoryUsage]] 15:04:51.567 [Thread-2] DEBUG c.googlecode.jmxtrans.JmxTransformer - Stopped writer: SimpleCsvWriter for query: Query [obj=java.lang:type=OperatingSystem, resultAlias=null, attr=[SystemLoadAverage, SystemCpuLoad, ProcessCpuLoad]] 15:04:51.567 [Thread-2] DEBUG c.googlecode.jmxtrans.JmxTransformer - Stopped writer: SimpleCsvWriterForMemory for query: Query [obj=java.lang:type=MemoryPool,name=PS*, resultAlias=, attr=[Name, Usage]]>
I tried changing the command line java parameter -Djmxtrans.log.level=10 to each of the following
< -Djmxtrans.log.level=0>
< -Djmxtrans.log.level=ERROR >
< -Djmxtrans.log.level=1 >
But the STDOUT log remains at DEBUG level. Is it possible to configure log level in JMXTRANS by commandline parameters?

Yes, logging is a mess. #215 is opened to track progress on this front. More specifically, #288 should be merged soon and should fix your specific issue.
I'll close this issue for the moment as it is tracked in #215 and #288.


Mac: Black screen after maximizing LBRY and then closing

The Issue
On Mac, if you maximize the LBRY app window and then click the close button, you end up with a black screen with no way of bringing LBRY back up.
Possibly related: https://github.com/electron/electron/issues/6033
Steps to reproduce

Maximize Window
Close the window
Blackscreen

Expected behaviour
Window show close to tray
Actual behaviour
Black screen
System Configuration

LBRY Daemon version: 0.19.1
LBRY App version: 0.21.2
LBRY Installation ID:
Operating system: Mac

Anything Else

Screenshots

One solution would be to disable the close action (cmd+W) when the window is in fullscreen mode. At least Discord Mac app behaves like this as well.
The code to be changed would be in src/main/createWindow.js in window.on('close'... part.
 if (!window.isFullScreen()) {   window.hide(); }

There was a better solution, and a pull request (#1235) was made.


Add variable for default darkmode and default preview

Pull request to solve #92, adds an option to set a variable called org-roam-server-default-darkmode, which if set to t will always toggle darkmode when the server starts, regardless of cookies set.
The second pull commit adds the option to set a variable called org-roam-server-default-preview, which if set to t will enable preview by default, to solve #134.
The latter does work a bit janky though, as it does not yet change the text or state of the preview button, but this is remedied when the Enable Preview button is pressed once.
Also they are two different AJAX calls and functions at the moment, ideally all these settings would only require a single one of course.
Apologies for the possibly somewhat sloppy code, I'm rather new to elisp.

Hello, thanks for the pull request, I will try it when my exams are done and I have some time.


Add pytest-repeat

Allows repeat runs for testing case stability using the --count option.
https://github.com/longhorn/longhorn/issues/2379

Test regression for test_hosts 5 times Build#137.


cmd/format: prevend reusing object storage when formating a volume

Codecov Report

Merging #1420 (ef08f74) into main (136ae2a) will decrease coverage by 0.07%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #1420      +/-   ##
==========================================
- Coverage   59.63%   59.56%   -0.08%
==========================================
  Files         129      129
  Lines       21746    21754       +8
==========================================
- Hits        12969    12957      -12
- Misses       7231     7248      +17
- Partials     1546     1549       +3

Impacted Files
Coverage Δ

cmd/format.go
66.36% <50.00%> (-0.61%)
:arrow_down:

pkg/meta/sql_lock.go
72.84% <0.00%> (-2.65%)
:arrow_down:

pkg/meta/tkv.go
75.27% <0.00%> (-0.77%)
:arrow_down:

pkg/chunk/cached_store.go
73.52% <0.00%> (-0.70%)
:arrow_down:

pkg/meta/sql.go
68.25% <0.00%> (+0.10%)
:arrow_up:

pkg/chunk/disk_cache.go
80.29% <0.00%> (+0.49%)
:arrow_up:

pkg/vfs/handle.go
86.44% <0.00%> (+0.84%)
: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 136ae2a...ef08f74. Read the comment docs.


model_head.pkl not loaded when HF repo is made private

Hello.
Whenever I use SetFitModel.from_pretrained(model_id, use_auth_token=...) to load a model from a private repo, the model_head.pkl is  not found. When the repo is made public, the head is found and loaded as normal. My repo contains both model_head.pkl and pytorch_model.bin files.
Please check this please.

Hi @Ioannis-Pikoulis, please see @tomaarsen's comment at https://github.com/huggingface/setfit/issues/409#issuecomment-1683762555

Hello!
I'm unable to reproduce this, I'm afraid. The use_auth_token is also correctly passed to the hf_hub_download method that is responsible for downloading this file.

Tom Aarsen


DryIoc new URL

Need to update the readme to reflect DryIoc's new location. Quoting the bitbucket site:

"The new DryIoc home is on GitHub: https://github.com/dadhi/DryIoc"

Opened #39 to fix.


Feat: add background worker and job scheduler

Describe your changes

Added rq (Redis Queue) and APScheduler libraries
Added initial setup for Redis Queue Background Task in services/worker.py
Added initial setup for APScheduler job scheduler in services/clock.py
Added both processes to the Procfile for Heroku

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] All tests passing
[x] Server runs locally
[x] Endpoints look correct locally
[x] Admin dashboard works locally
[] No remaining known bugs

I finally dug deeper into the background worker. I think I got it possibly working, but I am not going to be able to confirm since I did not setup the heroku CLI and connect to our server with it to be able to deploy this right now and see if it works. So I am going to add the code that I have and pass along the info that I have to you. It seems like you should be able to finish figuring this out relatively easily if you have time this weekend (or even just for later after the presentation as well). It looks very doable.
So as I mentioned before, this appears to be the best place to start looking up this info: https://devcenter.heroku.com/categories/background-jobs-in-python
That page presents us with 3 different options:

Redis Queue Background Task
Scheduled Jobs with APScheduler
Celery

I looked at all 3 options and learned some things that I think gave me greater clarity on what we would have to do. The big thing that I learned is that there is a difference between a job scheduler and a background worker:
Job scheduling is like a cron job - it schedules code to be run based on a set time or time interval. It is only for queuing background jobs - not executing them.
A background worker is what actually executes the scheduled jobs. It is just like adding asynchronous code. Asynchronous code is something that is more common in JavaScript. If you are familiar with JavaScript at all (or frontend will be familiar with asynchronous JavaScript), JavaScript has the ability to run code asynchronously via browser APIs. It is still a single threaded execution environment, but it is using a queue so that whenever there is downtime on the server, it will check the job queue and start running any tasks from the job queue. This is a popular video that explains this in JavaScript. This is useful for making certain code asynchronous so that it does not block the execution of the rest of your server (say for a possibly slow/unreliable network request).
What we need is a combination of both of these things - both a job scheduler, and a background worker. The job scheduler will add the task (in our case - your generate_random_world function I believe) to a job queue. And the background worker will execute the task when the server is idle.
The reason we need both rather than only the background worker is that the code we want to run is not technically part of the application - it is non-blocking code, i.e. it isn't part of our routes/views, etc., therefore nothing will trigger it to be executed otherwise. So we want to run our application like normal, but we want to have these jobs added to the queue at some time interval (maybe 10-15 minutes - can test around with it) for the background worker to start working on whenever the server is idle (i.e. when our site isn't getting traffic, which should be most of the time). Also, if we tried to somehow run this code on a time interval without a queue, then it could interrupt/conflict with current processes (i.e. existing traffic) causing one or both to fail. So we need both pieces.
So going back to the 3 options they give us on that page, this makes the solution more clear:

Redis Queue background task - this is a lightweight background worker, not a job scheduler. So we need this as the background worker piece.
Scheduled Jobs with APScheduler - this appears to be just like a cron job, but as Heroku explains - a cron job doesn't work for distributed environments like Heroku (or anything in the cloud), so this tool is specifically for doing cron jobs in such distributed/cloud environments. So this is a lightweight job scheduler only (i.e. not a background worker) which is the other piece that we also need. There is another relevant page on this topic that you would probably need to look at as well: Scheduled Jobs and Custom Clock Processes.
Celery - This appears to be a combination of both a job scheduler and a background worker. It is a heavier weight solution though, and the Heroku page seems to only discuss it as a background worker and nothing about job scheduling. So you probably could do it this way as well, but its going to be more complicated to setup and take longer to dig through the Celery documentation for it directly. Therefore, I ruled this out for now.

There is also another option for a job scheduler that they mention, which uses a Heroku add-on called Heroku Scheduler. I am not familiar with Heroku add-ons, but I presume they are plugins that can be added to your deployed servers on Heroku. If you run into difficulties with APScheduler, then this seems like another backup option worth exploring as well.

I think I got it close to working based on Heroku's instructions, but I was not able to test this locally because I don't have Redis setup locally. You have to run a Redis server for the Redis Queue background worker to work. I believe Heroku sets this up for you via a heroku-redis addon that you have to add via the Heroku CLI. However, I did not setup the Heroku CLI locally, and never connected to our server with it, so I am unable to finish it. Look through the instructions and my code. There are a few commands to run via the Heroku CLI:
heroku addons:create heroku-redis
git push heroku main
heroku scale worker=1
heroku ps:scale clock=1

And some additional relevant commands (for seeing logs/troubleshooting and manually running the worker, respectively):
heroku logs -t -p worker
heroku run worker

Check the logs after deploying this to see what it is doing / if it is working. I would be shocked if it all worked on the first try as-is, but also I did follow all their instructions correctly I believe, so it should be close to working at the very least.


softDelete with a date instead of boolean

Currently my softDelete are done with a value deletedAt which is the date of the deletion, is it possible to modify softDelete to reproduce this behavior when it is wanted?
For the moment softDelete only allows to add a true value to the selected column.

And for some mysql tables I have 1/0 instead of true/false.

https://github.com/feathers-plus/feathers-hooks-common/issues/130

We need a new option for softDelete: boolean, 0/1, date. I'll add this next time I address the repo.

softDelete2 is a notable improvement over softDelete. It only supports a deletedAt date of Date.now().
softDelete will be released by the end of this week once its docs are done.

Published as v4.15.0


nosurf breaks MultipartReader()

First, thank you for fixing enctype="multipart/form-data".
However, now processing a form with reader, err := r.MultipartReader() results in the following error: http: multipart handled by ParseMultipartForm.
Using MultipartReader() is advantageous for my use case because it allows me to process the request body (most importantly the file uploads) as a stream.

I'm kind of stuck on how to get the both sides happy here. :(

I wondered if using MultipartReader() would cause problems because it is trying to stream in the data.
Would it be possible to expose verifyToken()? Then I can mark my route as exempt, and process the CSRF token outside of your middleware.

I'm kind of stuck on how to get the both sides happy here. :(

What's the other side?

@shurcooL if nosurf parses multipart forms, they cannot be parsed in alternative ways by the end-user. If nosurf doesn't, it will not be able to verify multipart requests.

What about this. Make a copy of the Request Body, then parse it. But restore the Body before calling the success handler. Similar to how httputil.DumpRequest does it (see its source code).

To play devil's advocate: copying the entire request body could/can consume
a non-negligible amount of memory (and therefore garbage).
On Tue, 4 Aug 2015 at 5:20 pm Dmitri Shuralyov<EMAIL_ADDRESS>wrote:

What about this. Make a copy of the Request Body, then parse it. But
restore the Body before calling the success handler. Similar to how
httputil.DumpRequest https://godoc.org/net/http/httputil#DumpRequest
does it (see its source code
http://gotools.org/net/http/httputil#dump-go-L177-L193).
—
Reply to this email directly or view it on GitHub
https://github.com/justinas/nosurf/issues/27#issuecomment-127539133.

@elithrar That is a good point, making a copies of the request body can be detrimental to performance.
I wonder if it can be optimized by copying a part of the body until the CSRF token is found (and suggesting making it the first POST parameter), for example. Or some other means.
On another note, unless something here changes, I think the current documentation can be improved. It says:

... acts like a middleware and therefore is compatible with basically any Go HTTP application.

Supports any http.Handler (frameworks, your own handlers, etc.) and acts like one itself.

It should be noted that this middleware may call ParseMultipartForm and ParseForm, therefore populating request's Form, PostForm, and MultipartForm fields, but consuming the body in the process.
Therefore, it will not work with http.Handlers that expect the original request body to be there unmodified.

That might work, but it's a pretty sharp edge: if the token is after the other form values (e.g. a large file upload) that's a ton of implicit copying, especially for existing users.
Like @justinas said, I'm not quite sure how we can address this (my own CSRF lib has the same problem) and make both sides happy. Perhaps an option (not a huge fan of providing more options) to just do the copy operation (copy -> restore body) when parsing the multipart form?

I'm not quite sure how we can address this

What do you think about my suggestion in https://github.com/justinas/nosurf/issues/27#issuecomment-127715684 to improve documentation?

Agree with improving the docs. I also think providing an example "how to" for package users (i.e. how to do the copy themselves) is worthwhile though: it can be a real pain.

@bryanjeal Now that manual token verification is available, do you think it serves as a decent workaround to use when parsing multipart forms manually?

@justinas That is exactly what I was hoping for.
Thank you very much.
Closing this issue.


Is this going to return?

I noticed that the page was completely wiped since September.
Are there any chances of it returning?

Unfortunately, heroku decided the app breached its terms of service in some manner (no further information was provided), so at the moment the hosted version of this site is unlikely to return.


Wayland support

Wayland support would be really nice, since I'm using a Wayland compositor half the time, which currently runs all Xwayland apps at a fixed scale of 1.0, regardless of the scale factor of the current display.
As a result of the current design, it ignores the GDK_SCALE environment variable I try to pass a value of 1 to, and scales the app to 2.0 according to the factor of my primary display, resulting in the app being too large on both displays.
I am using Arch Linux, and running wayfire-git, self built with several patches which would not affect this particular bug.

Wayland support will definitely be added at some point, but as it will probably be a bigger project, it cannot be done shortly.
That said, if anyone out there would like to contribute Wayland support, I'll be more than happy to adopt it.
In the meantime, try starting your app with the --scale:n.n argument, where n.n is the scale factor. E.g. freac --scale:1.75 for 1.75x scaling (relative to the internal 8pt font size baseline).


Introduce graceful shutdown in sticky-cluster

Hi @uqee ,
I am working with @gboysko on the better shutdown handling. We try to maintain a map to record all the incoming connections and destroy them before exiting so that we could achieve the "graceful shutdown". Could you please help review this? Thanks.

Glad to see you, @xiaolongguan, thanks for the contribution! I'll look at it first thing in the morning.

Everything's fine so far. Just two more wishes:

bump package version in package.json,
describe new functionality in the readme (esp. in accepted options and changelog).

Hi @uqee , thanks for the review.
I have bumped the version to 0.3.2 and updated the readme accordingly, could you please take a look again? Thanks. cc @gboysko


Authentication tests with postman examples.

Authentication tests with postman examples.

[170] - https://transfer.sh/oQryB/jdi-dark-tests.tar.gz

[170] - https://transfer.sh/N7zOD/jdi-dark-bdd-tests.tar.gz

[170] - Allure report on Netlify: https://5e37c090e9be2030e0746884--jdi-dark.netlify.com

[172] - https://transfer.sh/4jkos/jdi-dark-tests.tar.gz

[172] - https://transfer.sh/QMyms/jdi-dark-bdd-tests.tar.gz

[172] - Allure report on Netlify: https://5e37c39149a8d8513201037f--jdi-dark.netlify.com

[177] - https://transfer.sh/xbWAy/jdi-dark-tests.tar.gz

[177] - https://transfer.sh/kn9OX/jdi-dark-bdd-tests.tar.gz

[177] - Allure report on Netlify: https://5e37e8749fec2e93eac4a3ed--jdi-dark.netlify.com


Support for np.zeros in nopython mode

I'm just starting playing around with the support for allocating arrays in no-python mode (using master). It's very nice work!
The first thing I did was write my one equivalent of np.zeros to allocate and zero-out an array. It would be nice if numba came with this built in.

Agreed.  I think we can add the rest of the usual numpy array allocators now that the main PR has merged.

This has been merged in.


[PkgEval] DataStructures may have a testing issue on Julia 0.4 (2014-09-29)

PackageEvaluator.jl is a script that runs nightly. It attempts to load all Julia packages and run their tests (if available) on both the stable version of Julia (0.3) and the nightly build of the unstable version (0.4). The results of this script are used to generate a package listing enhanced with testing results.
On Julia 0.4

On 2014-09-28 the testing status was Tests pass.
On 2014-09-29 the testing status changed to Tests fail, but package loads.

Tests pass. means that PackageEvaluator found the tests for your package, executed them, and they all passed.
Tests fail, but package loads. means that PackageEvaluator found the tests for your package, executed them, and they didn't pass. However, trying to load your package with using worked.
This error on Julia 0.4 is possibly due to recently merged pull request https://github.com/JuliaLang/julia/pull/8420.
This issue was filed because your testing status became worse. No additional issues will be filed if your package remains in this state, and no issue will be filed if it improves. If you'd like to opt-out of these status-change messages, reply to this message saying you'd like to and @IainNZ will add an exception. If you'd like to discuss PackageEvaluator.jl please file an issue at the repository. For example, your package may be untestable on the test machine due to a dependency - an exception can be added.
Test log:
>>> 'Pkg.add("DataStructures")' log
INFO: Installing DataStructures v0.3.2
INFO: Package database updated

>>> 'using DataStructures' log
Julia Version 0.4.0-dev+842
Commit e5d8c1a (2014-09-29 06:50 UTC)
Platform Info:
  System: Linux (x86_64-unknown-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2650 0 @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Sandybridge)
  LAPACK: libopenblas
  LIBM: libopenlibm
  LLVM: libLLVM-3.3

>>> test log
ERROR: test error in expression: d['c'] == 1
InexactError()
 in ht_keyindex2 at /home/idunning/pkgtest/.julia/v0.4/DataStructures/src/hashdict.jl:296
 in get! at /home/idunning/pkgtest/.julia/v0.4/DataStructures/src/hashdict.jl:385
 in getindex at /home/idunning/pkgtest/.julia/v0.4/DataStructures/src/delegate.jl:11
 in anonymous at test.jl:83
 in do_test at test.jl:47
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
 in anonymous at no file:17
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
 in _start_3B_3625 at /home/idunning/julia04/usr/bin/../lib/julia/sys.so
while loading /home/idunning/pkgtest/.julia/v0.4/DataStructures/test/test_defaultdict.jl, in expression starting on line 16
while loading /home/idunning/pkgtest/.julia/v0.4/DataStructures/runtests.jl, in expression starting on line 14
test/test_deque.jl ...
test/test_stack_and_queue.jl ...
test/test_accumulator.jl ...
test/test_classifiedcollections.jl ...
test/test_disjoint_set.jl ...
test/test_binheap.jl ...
test/test_mutable_binheap.jl ...
test/test_defaultdict.jl ...

>>> end of log

Confirming again that this has been fixed.


[ODS-5835] Promote Packages workflow promotes with Standard Version nuget packages

Verify all packages under Current packages for Suite 3 v6.1+:
are deployed with prerelease and also verify that it has 4.0.0 as well for many packages based on configuration packages json
https://techdocs.ed-fi.org/pages/viewpage.action?pageId=58721615
you can find packages list here https://github.com/Ed-Fi-Alliance-OSS/Ed-Fi-ODS-Implementation/actions/runs/5017187991/jobs/8994999884#step:5:123

@simpat-jesus  ackageName is missing 'Standard.' before '{StandardVersion}' - Fixed it in new commit . Ready for review


Listing Screen

@Thiendekaco please list all screen with screen to this issue for UI re-design purponse

List all scree :
Connect screen

When generate qrCode :

When not generate qrCode:

When connecting the device is phone or short width:

Connecting screen:

Reject connect screen:

Connected screen:

Account center

When multiple accounts

When open option

3.Notifications appear when performing tasks such as sign message, switch chain

Notification


Ctrl-DELETE is not recognised

I am running a loop as shown in the examples:
with Input() as keys:
    for key in keys:
        print(key)

DELETE is recognised as expected but Ctrl-DELETE is not printed until the next character when it is shown as ^[[3;5~.
Adding the tuple (b"\x1b[3;5~", u'<Ctrl-DELETE>') to curtsieskeys.py fixes the problem for me but I'm not sure whether it'd cause problems elsewhere.
Let me know if there's a better way of handling this.
Thanks

whoops tab took me to the wrong button first

Should be fixed by #106


Client-side validation can silently fail on name collision

Describe the bug and the expected behavior
In certain cases, form validation can fail with no apparent error message.
In this contrived example, I've added a read-only input with the same name as a text input:
<input {...conform.input(field)} />
<input type="text" name="field" value="bar" readOnly />

When the form is parsed against this schema:
const schema = z.object({
  field: z.string(),
});

Validation fails, but no error is reported.
If you switch the order of the inputs, you will see the expected error message.
Conform version
v0.9.1
Steps to Reproduce the Bug or Issue
StackBlitz reproduction
Note the two routes, the only difference between them is the order of the fields.
What browsers are you seeing the problem on?
No response
Screenshots or Videos
No response
Additional context
I ran into this organically and couldn't figure out what was happening. It turned out multiple values were colliding, causing client-side validation to fail; but no error was being reported at either the form or field levels.
I have no idea why the order of the fields matters, because the resulting FormData seems like it should be identical.

I ran into this and wasn't sure if it was zod or conform.

This is because of the use of setCustomValidity() API in the current version and will be resolved with the new archiecture.

Confirming this is resolved in v1.0.0-rc.0:
https://stackblitz.com/edit/remix-run-remix-auu2yj?file=app%2Froutes%2Fexpected.tsx
Thank you!

This should be resolved in v1.0.0.


Missing space

Hey,
just improving the readability a bit - missing a space.
Cheers!

Thanks!


Improve startup time

During a chat on Gitter fish-shell/fish-shell , Kurtis Rader - one of the core contributors - suggested that the startup time of this plugin could be improved by modifying this line:
 set -l __z_marks (cat $Z_DATA | sed -e "s/|.*//" -e "s/^.*\$/\"&\"/g" | tr '\n' ' ')

Looking at your profile the most expensive item is the (cat $Z_DATA | sed -e "s/|.//" -e "s/^.$/"&"/g" | tr '\n' ' ') subcommand. You can speed that up by 1) removing the useless use of cat, and 2) switching the sed and tr to the string builtin.

Definitely room for improvement there, here is a start:
set -l __z_marks (awk '

' < $Z_DATA)

Agreed. I'll take a look at it at the start of May, once my exams are over. Feel free to send a PR though. 😄

Just pushed a change, should hopefully solve this issue: https://github.com/fisherman/z/commit/748fc0fd8119515f65e073cc08dba5d3db829eb5

Nice. did you measure the time difference? is awk faster than the string builtin which kurtis suggested?

@aristidesfl The string builtin is supposed to be faster than awk for most simple to "intermediate" tasks. But as soon as you mix it with any fish construct, if/while, etc., expect a dramatic performance drop.

Also, we're dealing with structured data here, so awk fits the task really well.

Closing as resolved.

@jbucaran implementation using string:
string replace -r '(^[^|]*)\|.*$' '"$1"' < $Z_DATA | string join ' '

Do yo even need to remove the newlines? https://github.com/fisherman/z/commit/748fc0fd8119515f65e073cc08dba5d3db829eb5#diff-8acb27c0d8c9f82638db3460675b73c7R2 produces a different result than the original command doesn't it?

Re newlines, not needed; turns out complete handles newlines as well.
Awk version:
0.07user 0.00system 0:00.09elapsed 82%CPU (0avgtext+0avgdata 5844maxresident)k
0inputs+0outputs (0major+5078minor)pagefaults 0swaps

string version:
0.07user 0.00system 0:00.09elapsed 82%CPU (0avgtext+0avgdata 6000maxresident)k
0inputs+0outputs (0major+4834minor)pagefaults 0swaps

I actually have no idea how to interpret this data , but it seems that they're practically equivalent.

Maybe time is not precise enough to measure this?

@jethrokuan does it also handle unescaped paths with spaces and other strange characters?

It turns out current and previous versions are broken if any path contains a " character. Previous version might support single quote characters '.
Opened a PR with fix using string builtin.


[Feature Request] Beautify XML on response

Overview

Insomnia Version: 5.6.3
Operating System: Windows / OSX
Summary: "Beautify XML" on response

Working with SOAP-based services, it is very useful to have the same Beautify XML feature available on response as it is already available on request.

Thanks!

@vision8tech Insomnia currently does this already, but relies on Content-Type: application/xml to  to tell whether or not it's XML. Maybe a solution could be to also look for <?xml version="1.0" encoding="UTF-8"?> in the document if the Content-Type header is missing?


[Bug]: Nullable Generic Scalar generated as object

Describe the bug
When using a scalar as a generic, the OpenAPI output sometimes is defined as type object even though the file is not an object.
Using typespec v0.62.0
Reproduction
Typespec
scalar ISODate extends string;

alias RangeUnbound = null;

model Range<Inner> {
  from: Inner | RangeUnbound;
  to: Inner | RangeUnbound;
  lowerInclusive: boolean;
  upperInclusive: boolean;
}

model ISODateRange is Range<ISODate> {}

OpenAPI Output
ISODateRange:
  type: object
  required:
    - from
    - to
    - lowerInclusive
    - upperInclusive
  properties:
    from:
      type: object
      allOf:
        - $ref: '#/components/schemas/ISODate'
      nullable: true
    to:
      type: object
      allOf:
        - $ref: '#/components/schemas/ISODate'
      nullable: true
    lowerInclusive:
      type: boolean
    upperInclusive:
      type: boolean

Expected
RangeUnbound:
  type: null
ISODateRange:
  type: object
  required:
    - from
    - to
    - lowerInclusive
    - upperInclusive
  properties:
    from:
      oneOf:
        - $ref: '#/components/schemas/ISODate'
        - $ref: '#/components/schemas/RangeUnbound'
    to:
      oneOf:
        - $ref: '#/components/schemas/ISODate'
        - $ref: '#/components/schemas/RangeUnbound'
    lowerInclusive:
      type: boolean
    upperInclusive:
      type: boolean

or
ISODateRange:
  type: object
  required:
    - from
    - to
    - lowerInclusive
    - upperInclusive
  properties:
    from:
      type: string
      nullable: true
    to:
      type: string
      nullable: true
    lowerInclusive:
      type: boolean
    upperInclusive:
      type: boolean

Checklist

[x] Follow our Code of Conduct
[x] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[x] The provided reproduction is a minimal reproducible example of the bug.

@skywing918


Dockerhub images referenced in the documentation don't exist

The following images, referenced in the docs, don't seem to be available on dockerhub:
localai/localai:v2.10.1-aio-cpu
localai/localai:v2.10.1-aio-gpu-cuda-11
localai/localai:v2.10.1-aio-gpu-cuda-12
localai/localai:latest-aio-cpu
localai/localai:latest-aio-gpu-cuda12

Confirmed, tried multiple commands to deploy and asked Copilot for a sanity check.

Definitely some doc updates needed; if you're looking around in the mean time, here are the images currently available related to the 2.11.0 AIO release:
https://hub.docker.com/r/localai/localai/tags?page=1&name=v2.11.0-aio

Closing as this is solved by now and all CI pipelines seems to be functional


Strange StringIndexOutOfBoundsException if using KC_HTTP_RELATIVE_PATH=/auth

Description
I'm facing an StringIndexOutOfBoundsException in ResourceExtractor .
The problem occurs only in case of KC_HTTP_RELATIVE_PATH and only if the path was entered without the trailing /.
Expected Behavior
Should work as excpected also if KC_HTTP_RELATIVE_PATH is configured.
Actual Behavior
Eg: localhost/auth causes the exception while localhost/auth/ looks good. Internally redirects keycloak from /auth to /auth/ but with the MetricsFilter enabled it ends up in an exception.
We could redirect /auth$ to /auth/ on a reverse proxy, but it would be a strange workaround.
Without KC_HTTP_RELATIVE_PATH the filter works as expected (at least there is no exception).
Environment
Latest Keycloak: 23.0.5
keycloak-metrics-spi: 5.0.0
Configured via docker-compose
Steps to reproduce

Start the keycloak with KC_HTTP_RELATIVE_PATH=/auth and keycloak-metrics-spi enabled
Navigate to localhost:8080/auth - an error should occur

The error occurs within the class ResteasyReactiveRequestContext here


Calling visitChildren on a _RenderLayoutBuilder does nothing

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Create a RenderObject that calls visitChildren in a _RenderLayoutBuilder (like in the code sample).
In the given code example, we call it for either a SizedBox and a LayoutBuilder. The former works, while the later doesn't.
Expected results
The visitor should be called for the _RenderLayoutBuilder child
Actual results
The visitor is not called for the _RenderLayoutBuilder child
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(
    const ChildPrinterBox(
      child: SizedBox(
        child: SizedBox.shrink(),
      ),
    ),
  );

  runApp(
    ChildPrinterBox(
      child: LayoutBuilder(
        builder: (context, constraints) => const SizedBox.shrink(),
      ),
    ),
  );
}

final class ChildPrinterBox extends SingleChildRenderObjectWidget {
  const ChildPrinterBox({super.key, super.child});

  @override
  RenderObject createRenderObject(BuildContext context) =>
      RenderChildPrinterProxyBox();
}

final class RenderChildPrinterProxyBox extends RenderProxyBox {
  @override
  void performLayout() {
    if (child case final RenderBox child) {
      child.visitChildren(print);
    }

    super.performLayout();
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.5.2 22G91 darwin-x64, locale en-BR)
    • Flutter version 3.10.6 on channel stable at /Users/pintome/fvm/versions/3.10.6
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (9 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/pintome/Library/Android
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/pintome/Library/Android/
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment Temurin-11.0.14+9 (build 11.0.14+9)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio
    • Android Studio at /Users/pintome/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 13.5.2 22G91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.187

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Hi @mateusfccp. Thanks for filing this. Not sure i fully understand this. I can see same behavior both with LayoutBuilder and SizedBox, can you help with more information on the expected behavior and the actual behavior.
I also see you are on an older version of flutter, can you upgrade to the latest versions to see if the issue you are experiencing still persists?

@dam-ease
By running the provided code in DartPad (3.13.0), the problem is evident.

As you can see in the console, the print is only executed once. Commenting the SizedBox makes nothing be printed, showing that the print comes from the SizedBox, and not from the LayoutBuilder.

Thanks for your response. I can reproduce this on the latest master and stable channels.
stable, master flutter doctor -v

[✓] Flutter (Channel stable, 3.13.3, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.13.3 on channel stable at /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (7 days ago), 2023-09-06 14:32:31 -0700
    • Engine revision b8d35810e9
    • Dart version 3.1.1
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.82.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (3 available)
    • iPhone 13 Pro (mobile) • 14B6FEE4-2F1C-401F-8804-A5115D7A6DA2 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   •
      macOS 13.0 22A380 darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript •
      Google Chrome 116.0.5845.187

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[✓] Flutter (Channel master, 3.14.0-14.0.pre.266, on macOS 13.0 22A380 darwin-arm64, locale en-NG)
    • Flutter version 3.14.0-14.0.pre.266 on channel master at /Users/damilolaalimi/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d3bc797758 (2 hours ago), 2023-09-13 03:13:39 -0400
    • Engine revision 8611cb0ce3
    • Dart version 3.2.0 (build 3.2.0-140.0.dev)
    • DevTools version 2.27.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.82.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (3 available)
    • iPhone 13 Pro (mobile) • 14B6FEE4-2F1C-401F-8804-A5115D7A6DA2 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 116.0.5845.187

[✓] Network resources
    • All expected network resources are available.

• No issues found!

@dam-ease Are you sure f: material design is a good tag for this issue? This is not really related to Material, specifically, but to the implementation of LayoutBuilder, which is agnostic to any design system.

  @override
  void performLayout() {
    if (child case final RenderBox child) {
      child.visitChildren(print);
    }

    super.performLayout();
  }

LayoutBuilder doesn't inflate its child widget tree until performLayout is called, since the child widget subtree depends on the incoming constraints. I think that's expected behavior.

  @override
  void performLayout() {
    if (child case final RenderBox child) {
      child.visitChildren(print);
    }

    super.performLayout();
  }

LayoutBuilder doesn't inflate its child widget tree until performLayout is called, since the child widget subtree depends on the incoming constraints. I think that's expected behavior.

Sure, reverting the order does not solve the issue. I am going to update the issue so it's clearer.

What does the example code give you if you call visitChildren after performLayout? RenderLayoutBuilder's superclass should provide the right visitChildren implementation. Many other things depend on a correct visitChildren implementation (RenderObject.attach for example) and if the implementation isn't correct I'd expect LayoutBuilder to be not usable at all.

It seems you are right.


(Feat) : Added Cancel button when editing project details from Dashboard

Summary of Changes
This pull request aims to enhance the user experience when editing project details in StatWrap by adding a "Cancel" button that allows users to discard any changes made and revert to the original view.
Changes:

Added a "Cancel" button to the right of the "Save Details" button.
Implemented functionality to discard any edits when the "Cancel" button is clicked.
Added a prompt to alert users if they have unsaved changes when they click "Cancel", asking "You have made changes. Do you wish to discard those changes?"
Ensured that clicking "Cancel" exits the editing interface and returns the user to the Dashboard view.

Related Issue
Closes #167
Checklist

[x] My code is well-documented, and I've updated relevant documentation.
[x] I have tested these changes on my local environment.
[x] I have reviewed and proofread my code and the changes.
[x] The branch is up-to-date with the base branch.

Additional Context
https://github.com/StatTag/StatWrap/assets/64387054/7838c50d-856d-4f84-83ef-87dcae5527ef
Reviewer(s)
@lrasmus

Just reviewed the changes, and they all seem quite relevant.

@Abhijay007 - thank you so much for the work on this!  Please see the two issues I noted: one related to layout, the other one more about functionality.  Thank you!

@Abhijay007 - thank you so much for the work on this! Please see the two issues I noted: one related to layout, the other one more about functionality. Thank you!

Thanks for the review @lrasmus I will look into the suggested points and make the changes accordingly

Thanks for the review @lrasmus I will look into the suggested points and make the changes accordingly

Thank you!!!  Sorry I didn't have time to dig in more myself.


Can we make custom divider ?

Hey,
It's a nice lib, thank you. I have a question can we make custom divider instead of the divider can we replace it with some circular view.

Hi @shashank26ashu,
It should be possible but unfortunately it's not supported yet. 😢

Hey,
It's a nice lib, thank you. I have a question can we make custom divider instead of the divider can we replace it with some circular view.

Hi, I suggested using the drawable resources as a divider in a pull request here
https://github.com/ShawnLin013/NumberPicker/pull/196
Hope it gets considered.


Session watcher only working in the first attached VScode folder

Describe the bug
In a VScode workspace I am only able get the R session watcher working in the first attached folder.
To Reproduce

Create the following folder structure

Attach folders in VScode (File>Add Folder to Workspace...) to obtain the following:

In the setup above start an R terminal under project_1. The R session ID should be properly shown in the VScode status bar

Now do the same but start a new R terminal under project_2 instead. The VScode status bar will stay with R: (not attached)

Note: if the project_2 folder were attached first to VScode (i.e. shown first in the side bar list) then the Rsession watcher would work for project_2 but not for project_1.
setting.json
    "r.alwaysUseActiveTerminal": true,
    "r.bracketedPaste": true,
    "r.rterm.mac": "/usr/local/bin/radian"
    "r.sessionWatcher": true

Expected behavior
R session watcher should be able to work no matter from which folder it has been attached.
Environment:

OS: macOS
VSCode Version: 1.42.1
R Version: 3.6.2
vscode-R version: 1.2.3

Additional info
This could be an extension of #196

Thanks for reporting!
Multi-root workspace is not supported yet. We may consider how it should be supported.

@renkun-ken thanks for the feedback, I do not know how things work internally but it seems to me that it is almost there i.e. the .vscode/vscode-R/<session ID> are properly created it' may "just" be a matter of attaching them to VScode.

Multi-root workspace has some complexity that may require everything to be an array, e.g. file watchers, managed terminals, etc.

Single root workspace, single terminal (the simplest case, works)
Single root workspace, multiple terminals (works but only tracks last created terminal #200)
Multi-root workspace, single terminal (only starting R session from first workspace folder works, and session info are shared among all workspaces)
Multi-root workspace, multiple terminals (multiple .vscode/vscode-R, multiple response.log)

Do you expect the behavior when you are editing a file from workspace 1, session watcher uses the session data from workspace 1's attached session, and when you are editing a file from workspace 2, session watcher uses the session data from workspace 2's attached session? This allows each workspace has an attached session.

In the case I described above (4^th^ item in your list), my expectations would be that the workspaces should be independent from each others.


Feedback in terminal

Description

Add option feedback as boolean

Describe the solution you'd like

If feedback is true, then display in terminal something like:

Uploading source map ⬆
Source map upload ✅

Additional context
It will be great to know what is doing the process.

I just see BuildReporterConfig has the option to set the logLevel
logLevel: 'debug'

There is no 'logLevel' option in the SourceMapUploaderConfig. ☹


Incompatibility with node v21.7.1

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
<EMAIL_ADDRESS>What version of Remix are you using?
2.0.3
Steps to Reproduce
Run:
npm create @shopify/hydrogen@latest
cd my-project
npm run dev
Expected Behavior
The dev server should not crash on node 21.7.1
Actual Behavior
The dev server crashes if you visit the localhost.
TypeError [ERR_INVALID_ARG_TYPE]: The "strategy" argument must be of type object. Received type number (0)
    at new ReadableStream (node:internal/webstreams/readablestream:254:5)
    at safeReadableStreamFrom (/Users/jakob/coding/my-project/node_modules/miniflare/dist/src/index.js:7975:10)
    at #handleLoopback (/Users/jakob/coding/my-project/node_modules/miniflare/dist/src/index.js:8137:36)
    at Server.emit (node:events:531:35)
    at Server.emit (node:domain:488:12)
    at parserOnIncoming (node:_http_server:1137:12)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)

This looks like it is related to this https://github.com/cloudflare/workers-sdk/issues/5190. Perhaps updating deps to miniflare could fix the issue.
I can also confirm that if you downgrade node to 20.12.1 it works.

Closing as duplicate: https://github.com/Shopify/hydrogen/issues/1833
This was fixed in https://github.com/Shopify/hydrogen/pull/1866, to be released.


Editor doesn't see list

Hello, when I use a custom toolbar list (ordered or bullet). The editor can't see the changes.
How it looks like
I have preview and edit mode for text. And in edit mode, I want to create a list. When I create a list everything is okay. But when I save and reopen edit mode my list is gone. Not displayed for some reason.

This screen is a preview:

This screen is Edit mode:

As you can see, empty space. How can I fix it? Thank you!

Update: The issue happened only when I used more than one tag . But as you now in lists its happed very often)


Remove blockchains from homepage UI when syncing is disabled in server settings

Update UI to hide blockchains when

add core icon. frontend flow documentation for support a new blockchain.

Support for currency abbreviation. Astar = ASTR. Base = bETH etc


Cleanup README

add installation section
move low-level examples from README to examples dir

Codecov Report
Base: 87.55% // Head: 87.55% // No change to project coverage :thumbsup:

Coverage data is based on head (d50632e) compared to base (42cd062).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #81   +/-   ##
=======================================
  Coverage   87.55%   87.55%
=======================================
  Files          33       33
  Lines        2041     2041
  Branches      234      216   -18
=======================================
  Hits         1787     1787
  Misses        188      188
  Partials       66       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Where can we use the interface for social scientists?

In the Todos section, you guys mentioned there is an interface for social scientists, I'm wondering how can I access it?

Yeah we are in the process of building huggingface spaces. Will notify once done


Fix to multio linkage from IFS CY49R1

This cross-ports a fix from IFS CY49R1 that was missing and its absence here was causing MULTIO configuration to fail in the standalone WAM experiment in the IFS develop CI.

adapt CMakeLists.txt to latest multio-fapi

Do you need this fix in a tag that is bitid to 49R1? If so, this should go in before #27 and should be tagged before that is merged.

Yes, that is required in order to be able to get the standalone WAM CI test running as part of the IFS "develop" CI, as "develop" is only intended to take bit-identical changes relative to the current cycle - that's why I targeted it at develop-1.3 rather than develop.

All comments hopefully addressed in latest push, which I can confirm is still working in the IFS context I was trying to fix.


Remove unnecessary SendGridUser field from Identity SendGrid example code

The SendGrid client API doesn't need (or use) the 'user' field, so including it in the sample code and stored secrets/keys, etc, is just unnecessary and confusing. It should be removed; the only field that's required is the API Key.1
The SendGrid API example demonstrates that only the API key is required: https://app.sendgrid.com/guide/integrate/langs/csharp
If you want to add a second config field, it would make more sense to put the 'from address' into config, as that is directly associated with the API key, and if they don't match, SendGrid will return a 400 response, i.e, change:
   From = new<EMAIL_ADDRESS>Options.SendGridUser),

to
    From = new EmailAddress(Options.SendGridFromAddres, "Password Recovery System"),

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

ID: fe1bea60-d173-f6d2-3559-b147699e295a
Version Independent ID: 98e7aacb-2a01-d581-9b03-bfef2893163e
Content: Account confirmation and password recovery in ASP.NET Core
Content Source: aspnetcore/security/authentication/accconfirm.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The SendGrid API example demonstrates that only the API key is required: https://app.sendgrid.com/guide/integrate/langs/csharp

Any chance you could PR this? At the top right of the article, select the Edit pen icon:

Select the pen icon again. Save.

Hi Rick, Has this been worked on? Can I make a PR with the correction?

I've not had a chance to do it so be my guest if Rick hasn't either!

@Maniizzle  yes, a PR would be great.

@Rick-Anderson Since we are removing the sendgrid user information. I think only the key would be stored as user secrets as the "FromEmail" is not a confidential information. Right?

Correct

the only field that's required is the API Key.


api: remove GraphQL prefix from type names

Fixes #101.

I don't think you should change the derive names as they aren't imported explicitly and  things like "Query" in large codebases are hard to grep for.

@LegNeato Makes sense. Updated.

There seems to be a rough consensus in favour of renaming so let's merge this :)