Is it possible to drag and drop between different tree?

If it is possible, is there a guide on how to achieve this ?

At the moment, drag-and-drop between different trees is not possible.
We are considering supporting this in the future, but I don't think it will be possible anytime soon.

https://user-images.githubusercontent.com/43137033/151507638-84a888cd-0639-4e56-a522-1e62c46272d9.mp4
Em... I think it is possible. But it will required advanced skills of React and deep understanding of your package. Surely you can make it easier for user to implement this.
mechanism
The mechanism of determining whether the item can be dropped to a tree is based on a function canDrop of the Tree component. The default implementation is to search the item in the tree and see if the item can be found in the tree. The item can be dropped if it is in the same tree. Therefore, if you have two tree with same tree data, you can make the cross dropping work.
The first step to make cross-tree dropping work is to pass your own canDrop property to the tree. Then you will see a lot of error because the items are not in same tree. But you can make that works if you solve all the errors.
btw
Thanks for this lib. It is very beautiful and the code is cleanly written. ✨

@HananoshikaYomaru I was trying to implement the same on my current project, and I found a workaround that might work for some people while this lib does not implemente the feature.
What I did was to provide the items of both trees to both trees, and to set a different rootId as the parent of the items that did not belong to the given tree. This makes the <Tree> hide the items with the different root id, but keep them on the tree data, so the validations work fine.
Something like this:
const rootIdOfTree1 = 1;
const rootIdOfTree2 = 2;
const itemsOfTree1 = originalItemsOfTree1.map(item => ({ ...item, parent: rootIdOfTree1 });
const itemsOfTree2 = originalItemsOfTree2.map(item => ({ ...item, parent: rootIdOfTree2 });

// When rendering tree1:
<Tree
  tree={[...itemsOfTree1, ...itemsOfTree2]}
  rootId={rootIdOfTree1}
/>

// When rendering tree2:
<Tree
  tree={[...itemsOfTree1, ...itemsOfTree2]}
  rootId={rootIdOfTree2}
/>

This is, of course, not a proper solution, just a workaround. But at least you might be able to keep using this lib and have the DnD between trees. I hope it works for you!
BTW, I second what you said: this is indeed a really nice lib, with a great API and very clean code. Congrats, @minop1205 , and thank you!

This looks much simpler than my implementation. But to show the dropped node from another tree, we need to change the root id of the dropped node, right?

This looks much simpler than my implementation. But to show the dropped node from another tree, we need to change the root id of the dropped node, right?

Do you mean "to show the placeholder while dragging the node to the other tree"? Or "to show the node on the tree after the node was dropped"?
On the first case you might need to change the canDrop, although I believe you won't need to on most of the scenarios; on the second case: yes, you do need to change the root id of the dropped node on the onDrop for the node to appear on the tree.

Yes, eventually the canDrop and onDrop will determine the cross-tree drag and drop behaviour. But I do think making every tree store every data will be easier for people to implement. But if you can perfectly control your canDrop, whether which tree hold what data will not be a big issue.


Command palette should score shorter commands higher

When you jump into the command pallet and type >install the default option is to Disable All Extensions. I've just done this by accident assuming that the most matched command would be to Install Extensions I now have the task of going through and enabling all my extensions again :(

VSCode Version: 1.9.1
OS Version: MacOS 10.12.3

Steps to Reproduce:

hit cmd + p
type >install and hit enter

Insiders is better but still not ideal:

https://github.com/Microsoft/vscode/issues/14727


oauth2: added refresh token generation for password grant type

I'm currently porting over some old authentication services with fosite and have ran into the issue specified here: https://github.com/ory-am/fosite/issues/78
Where internal clients will make use of the password grant type along with the access / refresh tokens.
There was a comment suggesting that this particular feature be added as a separate PR than the issue raised; and since it's been around a month with no updates I decided to give it a go.
Please let me know if anything doesn't meet the RFC or needs changes.

Awesome, thanks for this! I will take a closer look tonight!

Very cool indeed!
AFAIK, the documentation does not explicitly mention that the "offline" scope is used to determine that a client can use refresh tokens. Could you guys maybe add some documentation about how the mechanism is working in fosite?

this could be the right place for it, do you want to create a PR @256dpi ? :)

@arekkas I would love to, unfortunately I'm not aware of all the details of the implementation. Therefore someone that already read into that particular code section might be better suited for the job. ;)

Added: https://github.com/ory-am/fosite/commit/60a767221625d0f6541f203e41a7ef20a1782eb0
I promise to take the time at some point to write some docs on gitbook.

Thanks!


Slow docker build times on master

Problem
We are currently occuring around 2.5 hours of build times on docker. This makes integrated CI/CD workflows like canary deployments very lon running - not ideal for fast paced development
Possible solutions
I played around with this on my fork and here are some possible solutions:

Split the build and push action per plattform (amd64, arm64) (via a matrix build) -> Faster availability of the images that are already done
Create a custom Pypi index with all the missing wheels (like sentry for example) -> Wheel builds are one of the mayor hold ups
Switch to python 3.11 or 3.12 for the image (breaking!) -> better availabilty of new wheels, performance improvements
Use Arm-based builders (either GitHubs MacOS machines, 3rd party machines or self-hosted runners on our sponsor DigitalOcean)

Use Arm-based builders (either GitHubs MacOS machines, 3rd party machines or self-hosted runners on our sponsor DigitalOcean)

Hi @matmair
I apologize if I'm intruding here, but if you're considering MacOS runners, may I suggest giving FlyCI a try? We offer MacOS M1 and M2 runners that are faster than GitHub's. For public repos, we offer 500 mins/month of free M1 usage (4 vCPUs, 7 GB RAM, 28 GB storage).
We're a startup that just launched our public Beta, so you're able to experiment with all of our runners for free until we enable billing (within a few weeks). After that, the standard free tier applies.
Do you think this might be a good option for InvenTree?
Web: flyci.net | Twitter: flyciapp | LinkedIn: FlyCI | Discord: FlyCI

I will test it, we might need to optimize the pipeline to cut down on cycles enough to only need 500 cycles

@wolflu05 I would be intrested in your view of this as you already proofed to be knowledgeable with your work on docker/infrastructure things

@matmair I think all your possible solutions make sense, but we should start with the easier ones. Bumping the python versions also makes sense due to the performance improvements which came with python 3.11. And if there are more wheels for arm64 available, we could use them. One other thing we could do is using the gh actions cache for wheels caching, that woudnt require us to host some extra infrastructure (We could even try out caching docker image layers, but the gh actions cache is limited to 10GB/repo in the free variant, so we would have to first check if that would exceed that size.).

@kgantchev we have reviewed your offering and discussed it among devs. TBH there is far to less info about your company, your security / trust setup and track record online to use you for a critical application like CI/CD. Thank you for the generous offer though.

@matmair no problem. I understand your concern. We recently launched our startup so we haven't had enough time to provide all of that information. I presume your team is interested in knowing our security certificates. I'll circle back around when all of that information is available (if that's OK with you).

My main concern is with infrastructure attestation and visibility into security proesses/culture (Trust center with ISO 27001 or SOC2, visible security staff and following standards like security.txt). Feel free to ping me here - options are good for a competetive market.

@matmair will do! Thanks for sharing the info.


Support for New Architecture

Are you planning to add support for new architecture ?

This is a pure js library, nothing needs to be done to support the new architecture, are there any difficulties in using the new architecture?


Make download process distributed

We should rely on external processes for data download to make it more easy to scale, we could for instance rely on https://kalisio.github.io/krawler/how-to-use-it/hooks.html#numerical-weather-prediction-source.
In order to synchronize distributed processes with weacast we should rely on https://github.com/feathersjs-ecosystem/feathers-sync or https://github.com/scttnlsn/mubsub.

Started working on https://github.com/weacast/weacast-loader.


User type benchmarks

I want to merge this change because it adds benchmark tests for User type.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3691 Optimize User type resolvers


Change pod creation error to cause the AdaptDL job to fail instead of the scheduler pod

Fixes #84
Also makes make delete delete the registry even if AdaptDL hasn't been deployed/fails to be deleted.

Sampled failed job output:
Name:         linear-regression-7h5d7
Namespace:    default
Labels:       <none>
Annotations:  <none>
API Version:  adaptdl.petuum.com/v1
Kind:         AdaptDLJob
Metadata:
  Creation Timestamp:  2021-01-11T16:17:24Z
  Generate Name:       linear-regression-
  Generation:          1
  Resource Version:    25124
  Self Link:           /apis/adaptdl.petuum.com/v1/namespaces/default/adaptdljobs/linear-regression-7h5d7
  UID:                 f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b
Spec:
  Template:
    Spec:
      Containers:
        Command:
          python3
          /source/examples/linear_regression/main.py
          --epochs=5
        Env:
          Name:   PYTHONUNBUFFERED
          Value:  true
        Name:     main
        Resources:
          Limits:
            Cpu:  100m
          Requests:
            Cpu:  100m
        Volume Mounts:
          Mount Path:  /adaptdl/checkpoint
          Name:        adaptdl-pvc
          Sub Path:    adaptdl/checkpoint
          Mount Path:  /adaptdl/share
          Name:        adaptdl-pvc
          Sub Path:    adaptdl/share
          Mount Path:  /source
          Name:        adaptdl-source
      Volumes:
        Name:  adaptdl-pvc
        Persistent Volume Claim:
          Claim Name:  adaptdl-pvc-local
        Host Path:
          Path:  /source
        Name:    adaptdl-source
Status:
  Group:    0
  Message:  (422)
Reason: Unprocessable Entity
HTTP response headers: <CIMultiDictProxy('Audit-Id': '9507b1bf-0816-4e04-bb8d-bcd59c5db3c1', 'Cache-Control': 'no-cache, private', 'Content-Type': 'application/json', 'Date': 'Mon, 11 Jan 2021 16:17:24 GMT', 'Content-Length': '441')>
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Pod \"linear-regression-7h5d7-f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b-0-0\" is invalid: spec.containers[0].image: Required value","reason":"Invalid","details":{"name":"linear-regression-7h5d7-f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b-0-0","kind":"Pod","causes":[{"reason":"FieldValueRequired","message":"Required value","field":"spec.containers[0].image"}]},"code":422}

  Phase:   Failed
  Reason:  PodCreationError
Events:    <none>

Sampled failed job output:
Name:         linear-regression-7h5d7
Namespace:    default
Labels:       <none>
Annotations:  <none>
API Version:  adaptdl.petuum.com/v1
Kind:         AdaptDLJob
Metadata:
  Creation Timestamp:  2021-01-11T16:17:24Z
  Generate Name:       linear-regression-
  Generation:          1
  Resource Version:    25124
  Self Link:           /apis/adaptdl.petuum.com/v1/namespaces/default/adaptdljobs/linear-regression-7h5d7
  UID:                 f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b
Spec:
  Template:
    Spec:
      Containers:
        Command:
          python3
          /source/examples/linear_regression/main.py
          --epochs=5
        Env:
          Name:   PYTHONUNBUFFERED
          Value:  true
        Name:     main
        Resources:
          Limits:
            Cpu:  100m
          Requests:
            Cpu:  100m
        Volume Mounts:
          Mount Path:  /adaptdl/checkpoint
          Name:        adaptdl-pvc
          Sub Path:    adaptdl/checkpoint
          Mount Path:  /adaptdl/share
          Name:        adaptdl-pvc
          Sub Path:    adaptdl/share
          Mount Path:  /source
          Name:        adaptdl-source
      Volumes:
        Name:  adaptdl-pvc
        Persistent Volume Claim:
          Claim Name:  adaptdl-pvc-local
        Host Path:
          Path:  /source
        Name:    adaptdl-source
Status:
  Group:    0
  Message:  (422)
Reason: Unprocessable Entity
HTTP response headers: <CIMultiDictProxy('Audit-Id': '9507b1bf-0816-4e04-bb8d-bcd59c5db3c1', 'Cache-Control': 'no-cache, private', 'Content-Type': 'application/json', 'Date': 'Mon, 11 Jan 2021 16:17:24 GMT', 'Content-Length': '441')>
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Pod \"linear-regression-7h5d7-f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b-0-0\" is invalid: spec.containers[0].image: Required value","reason":"Invalid","details":{"name":"linear-regression-7h5d7-f23a6f03-b9f3-42a8-973e-bfdc4b1efd3b-0-0","kind":"Pod","causes":[{"reason":"FieldValueRequired","message":"Required value","field":"spec.containers[0].image"}]},"code":422}

  Phase:   Failed
  Reason:  PodCreationError
Events:    <none>

Codecov Report

Merging #89 (e616901) into master (42d00c4) will decrease coverage by 0.15%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   62.59%   62.44%   -0.16%
==========================================
  Files          31       31
  Lines        2390     2399       +9
  Branches      355      355
==========================================
+ Hits         1496     1498       +2
- Misses        821      829       +8
+ Partials       73       72       -1

Impacted Files
Coverage Δ

sched/adaptdl_sched/controller.py
0.00% <0.00%> (ø)

adaptdl/adaptdl/reducer.py
90.00% <0.00%> (+2.00%)
: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 42d00c4...22858c4. Read the comment docs.

Codecov Report

Merging #89 (e616901) into master (42d00c4) will decrease coverage by 0.15%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   62.59%   62.44%   -0.16%
==========================================
  Files          31       31
  Lines        2390     2399       +9
  Branches      355      355
==========================================
+ Hits         1496     1498       +2
- Misses        821      829       +8
+ Partials       73       72       -1

Impacted Files
Coverage Δ

sched/adaptdl_sched/controller.py
0.00% <0.00%> (ø)

adaptdl/adaptdl/reducer.py
90.00% <0.00%> (+2.00%)
: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 42d00c4...22858c4. Read the comment docs.


fix: escaping in options

FIXES: #297

All configs in https://github.com/vdbe/catppuccin-tmux-dev have no regression from this change.


Item icons with parens in their image filename don't display in certain cases

Originally in GitLab by @unsoluble1
(0.6.6 / 0.96)
This might actually be a core Foundry thing, but I figure you can sort that out. :)
If an item's image has parentheses in its filename, the image won't display when in lists on the character sheet (like in Weapons or Features etc.). It'll show fine in the item's own app window though.

(Screenshot has a module in effect, but it doesn't matter in this case. Also note that it's the filename that breaks things here, not the item name.)

@aaclayton Looks like this one is back, but in a slightly different form — now it's apostrophes anywhere in the filepath that cause the sheet icon to fail to render (and perhaps other special characters, haven't had a chance to test more).
Looks like it's affecting any game system that was (I'm guessing) originally forked from dnd5e — same results in pf2e and ose, but things render fine in pf1e. No console errors.
dnd5e:

pf2e:

pf1e:

I'm guessing the same issue as https://github.com/foundryvtt/foundryvtt/issues/8222

This seems to have been addressed by core changes now in v10 which urlencode apostrophe characters in filenames.


Version of shipkit upgraded to 0.9.57

This pull request was automatically created by Shipkit's 'org.shipkit.upgrade-downstream' Gradle plugin (http://shipkit.org). Please merge it so that you are using fresh version of 'shipkit' dependency.

We don't need it any more. We bumped to bigger version since.


chore: update UI5 from v1 to v2

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)
Closes #3422

Definition of done

[ ] The PR's title starts with one of the following prefixes:

feat: A new feature
fix: A bug fix
docs: Documentation only changes
refactor: A code change that neither fixes a bug nor adds a feature
test: Adding tests
chore: Maintainance changes to the build process or auxiliary tools, libraries, workflows, etc.

[ ] Related issues are linked. To link internal trackers, use the issue IDs like backlog#4567
[ ] Explain clearly why you created the PR and what changes it introduces
[ ] All necessary steps are delivered, for example, tests, documentation, merging

/hold
Hold this PR till the end of the year.


Add Host GPIO function for Ibiza

change the GPIO mapping according to Ibiza board design and GPIO structure size.

you need to expand the .c file in the review.   git only shows a few lines around each change.  there are 12 elements in the struct.
From: Wen Wang @.>
Sent: Friday, August 2, 2024 4:03 PM
To: silicom-ltd/ecfw-zephyr @.>
Cc: Jeff Daly @.>; Review requested @.>
Subject: Re: [silicom-ltd/ecfw-zephyr] Add Host GPIO function for Ibiza (PR #2)
Caution: This is an external email. Please take care when clicking links or opening attachments.
@wwang22 commented on this pull request.
mecc172x_cfg_host[] defined in adl_n_mec172x.c has 8 elements, extern mecc172x_cfg_host[12] is hardcoded to 12 in gpiomgmt.c. These two places are not consistent.
—
Reply to this email directly, view it on GitHubhttps://github.com/silicom-ltd/ecfw-zephyr/pull/2#pullrequestreview-2216323019, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASDWUUAHHRO7ITEKRIL6V6TZPPQWBAVCNFSM6AAAAABLYUQRYWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMJWGMZDGMBRHE.
You are receiving this because your review was requested.Message ID<EMAIL_ADDRESS>


Adds support for Access CA certificates

This introduces support for managing Access CA certificates (often
referenced as short lived certificates) via the API.
I opted to call this "Access CA certificates" as that is what the functionality
technically is. Should this instead be "Access Short Lived certificates" to
line up with documentation?
Documentation: https://api.cloudflare.com/#access-short-lived-certificates-properties
Note: When I raised this PR, the create new certificate + list single
certificate details were missing the response payload in the
documentation. I instead made actual calls to confirm the correctness of
these tests and updated a Cloudflare support ticket to get this fixed
for others.

I think we can leave "CA" here if we clearly state that they mean "short-lived certificates" as a product.


Hybrid connectivity

[x] Existing Review Checklist containing info?

https://github.com/Azure/review-checklists/blob/main/checklists/alz_checklist.en.json

[ ] Mindmap info extracted to JSON checklist
[x] Decision of whether copying file over, or whether adding checks to existing checklist: add items to existing checklist.
[ ] Items available in Checklists repo.

Checks from mindmap:

VPN Gateway and Express Route DR Best Practices will apply to Hub & Spoke and VWan Architectures
VPN Gateways

[x] Deploy Gateways in to multiple Availability Zones for resiliency: D4.9
[x] Redundancy in Azure is only half of the consideration - the on-prem side of the VPN needs to also be resilient.

ExpressRoute

Each ExpressRoute Circuit contains two connections
[x] Having multiple ExpressRoute Circuits add additional redundant connection pairs: D4.14
[x] You can use VPN as a backup for ExpressRoute.
Routing for Multiple Circuits:

Be aware that you can leverage Connection Weights. You can leverage Connection Weights to ensure their is traffic symmetry.
Gateways:

[x] AZs: D4.6

Data Gateway: not applicable

Leverage data gateway cluster to avoid single points of failure and to load balance traffic across gateways in a cluster


Prefetch sticks at bottom

If you have set limit (endless scrolling enabled) and scroll to the bottom of the page new items are loaded and inserted but the scrollposition stays at bottom so you stick to the bottom and don't really recognize that new items are loaded.

I close this bug because I cannot reproduce it.


Ai-M61-32s dev kit issues

Hi,
Thank you for your support for Bouffalo  MCUs in Arduino IDE
I got this dev kit from AiThinker. I get the Done Uploading message when uploading any sketch, but the board isn't doing anything.
I have the latest version of your package. The PC sees the board. But not a simple Blink blinks the LED. This is the serial output:

Sketch uses 201304 bytes (4%) of program storage space. Maximum is 4190208 bytes.
Global variables use 18464 bytes (9%) of dynamic memory, leaving 178144 bytes for local variables. Maximum is 196608 bytes.
[10:06:05.120] - Serial port is COM5
[10:06:05.120] - ==================================================
ConfigParser Error: [Errno 13] Permission denied: 'C:\Program Files (x86)\Arduino\config.ini'
[10:06:05.121] - Program Start
[10:06:05.121] - ========= eflash loader cmd arguments =========
[10:06:05.123] - serial port is COM5
[10:06:05.123] - chiptype: bl616
[10:06:05.123] - cpu_reset=False
[10:06:05.137] - com speed: 2000000
[10:06:05.137] - ========= Interface is uart =========
[10:06:05.137] - Bootrom load
[10:06:05.137] - ========= get_boot_info =========
[10:06:05.137] - ========= image get bootinfo =========
[10:06:05.415] - tx rx and power off, press the machine!
[10:06:05.415] - cutoff time is 0.05
[10:06:05.479] - power on tx and rx
[10:06:06.469] - reset cnt: 0, reset hold: 0.05, shake hand delay: 0.1
[10:06:06.469] - clean buf
[10:06:06.470] - send sync
[10:06:06.699] - ack is b'4f4b'
[10:06:06.700] - shake hand success
[10:06:07.212] - data read is b'010016060000010027128901db002bcf0eb499000f758010'
[10:06:07.212] - ========= ChipID: b40ecf2b00db =========
[10:06:07.212] - Get bootinfo time cost(ms): 2074.920654296875
[10:06:07.212] - change bdrate: 2000000
[10:06:07.212] - Clock PLL set
[10:06:07.214] - Set clock time cost(ms): 2.004638671875
[10:06:07.347] - Read mac addr
[10:06:07.349] - MACADDR: b40ecf2b00db
[10:06:07.350] - flash set para
[10:06:07.350] - get flash pin cfg from bootinfo: 0x24
[10:06:07.350] - set flash cfg: 1014124
[10:06:07.350] - Set flash config
[10:06:07.353] - Set para time cost(ms): 2.970947265625
[10:06:07.353] - ========= flash read jedec ID =========
[10:06:07.356] - Read flash jedec ID
[10:06:07.356] - readdata:
[10:06:07.356] - b'5e401700'
[10:06:07.356] - Finished
[10:06:07.358] - flash config Not found,use default
[10:06:07.358] - jedec_id:5e4017
[10:06:07.358] - capacity_id:23
[10:06:07.358] - capacity:8.0M
[10:06:07.358] - get flash size: 0x00800000
[10:06:07.358] - Program operation
[10:06:07.359] - Warning: No input file to program to flash
[10:06:07.359] - All time cost(ms): 2238.60205078125
[10:06:07.473] - close interface
[10:06:07.473] - [All Success]

int led = 29; //cold LED is on IO29, tried RGB also
void setup() {
  pinMode(led, OUTPUT);
}
void loop() {
  digitalWrite(led, HIGH);
  delay(1000);
  digitalWrite(led, LOW);
  delay(1000);
}

Any help would be appreciated.
Regards

@ITstreet1
Hi, I noticed some errors in your log.
This error.
ConfigParser Error: [Errno 13] Permission denied: 'C:\Program Files (x86)\Arduino\config.ini'
It looks like the program tool failed to create some configuration! So you downloaded a NULL bin.
You can try starting the Arduino IDE with administrator privileges and then programming it.
This appears to be a compatibility issue with our programming tools on Windows. He seems to be running fine on my computer.
Anyway thanks for the test.

Hi,
Thank you for your fast response.
This did the trick.
Any idea how to solve this issue and not start IDE every time as administrator?
No matter, I will try a few examples to check it out. So far, so good.

@ITstreet1 you can force launching IDE as administrator always, by C:\Program Files (x86)\Arduino\Arduino.exe , right click, properties, and enable Run this program as administrator:

About auto-upload circuit, there are multiple auto-upload circuits. I see this circuit for BL616 for first time, but previous Bouffalo chips, as BL602 etc. used another circuit, just directly connecting the RTS and DTS to boot pin and reset pin.

I am sure this reset circuit is implemented in Bouffalo DevCube

What do you recommend for an auto-upload circuit? Direct connection of RTS and DTS pins to EN and IO8?
And btw, here the boot is on IO2.
While looking at the schematic, I can see that the button IS NOT connected to the IO2, lol. I checked it with a multimeter and it is not connected.
But while pressing the EN and IO2 buttons, I can enter the bootloader mode and upload the sketch.

As I mentioned, the circuit is for BL602, which have different pin specified for boot. And not sure, best would be ask Bouffalo engineers.

Yes, the defualt BOOT pin of BL616/8 is IO2. At present, I don't have an Ai-M61-32s dev kit, so I can't test it yet, but IO2 seems to be NC on the schematic you shared.

Ok, I made a few further GPIO tests, and here is what I got.
This board has two LEDs on GPIO27 and GPIO29. It also has an RGB LED on GPIO12, 14, and 15. With a simple Blink sketch on this board only LEDs on GPIO29 and 15 work. There is nothing on those pins, either.
To enter the bootloader mode I must press the buttons combination when it starts uploading. If I press it earlier, the upload will fail.

Meanwhile, the Ai-M62-12f kit has arrived. It uses the same BL616 IC. The board is smaller than the Ai-M61-32s but the components look the same. It also has two LEDs and an RGB LED, but the pins are a little bit different.
Firstly, I have a hard time uploading a sketch. It is hard to get the right timing. But when I uploaded it, none of the LEDs worked.
This is the spec doc for the Ai-M62-12f kit
https://docs.ai-thinker.com/_media/ai-m62-12f-kit_v1.0.0_specification.pdf

Hi,
Thank you for your support for Bouffalo MCUs in Arduino IDE
I got this dev kit from AiThinker. I get the Done Uploading message when uploading any sketch, but the board isn't doing anything.
I have the latest version of your package. The PC sees the board. But not a simple Blink blinks the LED. This is the serial output:

Sketch uses 201304 bytes (4%) of program storage space. Maximum is 4190208 bytes.
Global variables use 18464 bytes (9%) of dynamic memory, leaving 178144 bytes for local variables. Maximum is 196608 bytes.
[10:06:05.120] - Serial port is COM5
[10:06:05.120] - ==================================================
ConfigParser Error: [Errno 13] Permission denied: 'C:\Program Files (x86)\Arduino\config.ini'
[10:06:05.121] - Program Start
[10:06:05.121] - ========= eflash loader cmd arguments =========
[10:06:05.123] - serial port is COM5
[10:06:05.123] - chiptype: bl616
[10:06:05.123] - cpu_reset=False
[10:06:05.137] - com speed: 2000000
[10:06:05.137] - ========= Interface is uart =========
[10:06:05.137] - Bootrom load
[10:06:05.137] - ========= get_boot_info =========
[10:06:05.137] - ========= image get bootinfo =========
[10:06:05.415] - tx rx and power off, press the machine!
[10:06:05.415] - cutoff time is 0.05
[10:06:05.479] - power on tx and rx
[10:06:06.469] - reset cnt: 0, reset hold: 0.05, shake hand delay: 0.1
[10:06:06.469] - clean buf
[10:06:06.470] - send sync
[10:06:06.699] - ack is b'4f4b'
[10:06:06.700] - shake hand success
[10:06:07.212] - data read is b'010016060000010027128901db002bcf0eb499000f758010'
[10:06:07.212] - ========= ChipID: b40ecf2b00db =========
[10:06:07.212] - Get bootinfo time cost(ms): 2074.920654296875
[10:06:07.212] - change bdrate: 2000000
[10:06:07.212] - Clock PLL set
[10:06:07.214] - Set clock time cost(ms): 2.004638671875
[10:06:07.347] - Read mac addr
[10:06:07.349] - MACADDR: b40ecf2b00db
[10:06:07.350] - flash set para
[10:06:07.350] - get flash pin cfg from bootinfo: 0x24
[10:06:07.350] - set flash cfg: 1014124
[10:06:07.350] - Set flash config
[10:06:07.353] - Set para time cost(ms): 2.970947265625
[10:06:07.353] - ========= flash read jedec ID =========
[10:06:07.356] - Read flash jedec ID
[10:06:07.356] - readdata:
[10:06:07.356] - b'5e401700'
[10:06:07.356] - Finished
[10:06:07.358] - flash config Not found,use default
[10:06:07.358] - jedec_id:5e4017
[10:06:07.358] - capacity_id:23
[10:06:07.358] - capacity:8.0M
[10:06:07.358] - get flash size: 0x00800000
[10:06:07.358] - Program operation
[10:06:07.359] - Warning: No input file to program to flash
[10:06:07.359] - All time cost(ms): 2238.60205078125
[10:06:07.473] - close interface
[10:06:07.473] - [All Success]

int led = 29; //cold LED is on IO29, tried RGB also
void setup() {
  pinMode(led, OUTPUT);
}
void loop() {
  digitalWrite(led, HIGH);
  delay(1000);
  digitalWrite(led, LOW);
  delay(1000);
}

Any help would be appreciated.
Regards

On my Ai-M62-12f kit only BLUE led (IO15) and IO29 work


Suggestion: Append a space to tab-completions that can be assumed to form a self-contained argument

POSIX-like shells (but not cmd.exe) insert a space after a completion if the assumption is that the completion results in a self-contained argument (to which you won't want to append further characters directly).
Tab-completing a file path triggers appending a space (because a file path by definition ends there), whereas tab-completing a directory path does not (as you may want to extend the path to target a file / dir. inside).
PSReadline could employ the same logic and extend it to other token types, such as parameter names (where you almost always want a space to follow it).
This makes for a streamlined tab-completion experience.
Even though you could end up with an extra space at the end of the command that way, that is generally benign.
This behavior could be controlled with a new option, and - ideally - become the default behavior.
Environment data
PS version: 6.1.0-preview.4
PSReadline version: 2.0.0-beta2
os: 10.0.16299.15 (WinBuild.160101.0800)
PS file version: 6.1.0

Besides the default behavior that Bash, Zsh and other shells implement for this, note that their completion scripts often specify if a space shouldn't be added after a completion when needed. For example completing a long option such as --file=. In Bash this is done via the nospace completion option and in Zsh via -S '' though that is wrapped by a few layers of functions and the completion script often doesn't specify that directly.

@segevfiner - you should open an issue in the PowerShell repo for nospace - PSReadLine is just one client of PowerShell completions, PowerShell Editor Services could also use this feature.


Connecting remote server fails

I'm trying to connect to a remote indiserver, but this fails with AstraLite.

Remote server is running and I can connet with Kstars.
Can I provide additional information here?

Sorry, misconfiguration.


DEV Add new fields for Gerrit basic master integration

https://github.com/Shippable/pm/issues/11218

This is complete.
Closing.


Div doesn't show with mobile disable

This is:

Bug
Specifications

AOS version: 2.3.4
OS: Windows
Browser: Chrome

Expected Behavior
Remove animation for data-aos with config disable: mobile
Actual Behavior
Div with data-aos doesn't show if mobile is disabled
Steps to Reproduce the Problem
in nuxt plugin
 AOS.init({       disable: 'mobile})
in html

I have the same problem.
Using AOS.init({ disable: 'mobile}) makes that DIVs where animations are used don't display at all, even on desktop size. I have only white space there.

I have the same problem for elements loaded async after page loaded.

I have the same problem for elements loaded async after page loaded.

Hi Guys, I was having the same issue. It was working fine on Desktop but the animation was not working on mobile. (iPhone + Safari Browser) I was testing on it. After researching I found few solutions like adding mobile disabled and overflow for html or body, nothing worked for me. Following solution worked for me.
Instead of adding init function in a separate JS file, I had to add the following in the footer of my html file and it started working for me.

A bit late to the party by here's the workaround I'm using to disable AOS animation on smaller screen sizes only:
First, disable animation on the screen size of your choice in the AOS.init function:
AOS.init({
  disable: window.innerWidth < 768,
  ...
});

When you set the 'disable' prop it seems AOS will add pointer-events: none and opacity: 0 to your data-aos=* elements, so you need to overwrite the AOS styles in a media query.
For example, if you're using data-aos="fade-up", your custom CSS might look like this:
@media screen and (max-width: 768px) {
  [data-aos] {
    pointer-events: auto !important;
  }

  html:not(.no-js) [data-aos^=fade][data-aos^=fade] {
    opacity: 1 !important;
  }

  html:not(.no-js) [data-aos=fade-up] {
    transform: none !important;
  }
}

I face this mobile sections going black issue in react.js.
What I was doing was intializing aos animation in App.js file which is correct.
but to solve this problem for mobile I ended up initializing aos into separate components inside useEffect instead of App.js.
This might solve your problem of section going black in mobile.
do following:-
useEffect(() => {
AOS.init({
duration: 2400,
disable: "mobile",
});
}, [])

Hi Guys, I was having the same issue. It was working fine on Desktop but the animation was not working on mobile. (iPhone + Safari Browser) I was testing on it. After researching I found few solutions like adding mobile disabled and overflow for html or body, nothing worked for me. Following solution worked for me. Instead of adding init function in a separate JS file, I had to add the following in the footer of my html file and it started working for me.

Finally a stupid solution that worked

A bit late to the party but here's the workaround I'm using to disable AOS animation on smaller screen sizes only:
First, disable animation on the screen size of your choice in the AOS.init function:
AOS.init({
  disable: window.innerWidth < 768,
  ...
});

...

 Thanks!


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-11-25)
Features

file-manage: add option to edit files from the file directory and enabled folder uploader along with folder delete features (#363) (c2b7536)

Bug Fixes

Enable large file saving and no content files (#378) (23b8908)
Lint improvements, update on actual lint change and new debounce (#373) (f4832ed)
Reduce vertical space and add linter focus changes (#377) (86a0e5d)
Updated Docs (#376) (f3eec1f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Zero-True/zero-true/releases/tag/v0.5.0 :sunflower:


console suggest connect command which isn't working

After connect [URL] [user] [password] to and empty installation of 2.1-rc6, browse class clazz fails with
Error: com.orientechnologies.common.exception.OException: Database not selected. Use 'connect <database-name>' to connect to a database.

Running connect GreatfulDeadConcerts fails with
!Wrong syntax. If you're using a file make sure all commands are delimited by semicolon (;) or a linefeed (\n)

 Expected: connect <url> <user> [<password>]

Where:

* url            The url of the remote server or the database to connect to in the format '<mode>:<path>'
* user           User name
* password       User password

!Unrecognized command: 'connect GreatfulDeadConcerts'

That doesn't make sense.

console commands are documented here:
http://orientdb.com/docs/last/Console-Command-Connect.html

It's great that there are docs that show how to actually use the connect command, but the description of the connect command returned by this error is misleading. It leads new users to believe that they can switch to a different database on the same remote by just passing the name of the database to the connect command


HttpClient Extensions: GetToStreamSafeAsync() and GetToFileSafeAsync()

The idea here is to create common methods for downloading a URI's contents to a stream or file.
I've noticed that I've replicated the functionality a few times now so it's time to consolidate.

DONE for neonLIBRARY v2.11.0


IndexOutOfBoundsException

The function insertLiteralWord in EWAHCompressedBitmap32 should have its last two lines in an else block. Otherwise, we get an IndexOutOfBoundsException when the number of literal words are more than RunningLengthWord32.LARGEST_LITERAL_COUNT

Acknowledged. Will verify soon.

Fixed in release 1.1.1. Thanks for reporting the issue.


operation - R2- starting the webcam

User is confused as to how to start the webcam. Didnt know it ships with it off - need to show how to turn it on.

@Ximidar is this in the latest release? Is there any special instructions beside toggling it on and off from the screen?

@CobyKabili There is only a toggle on the LCD screen. I believed it was pushed to the users, but we will need @victorevector to verify. There is no other information on turning the webcam on and off

@Ximidar @victorevector what is the function that this button does? (I assume its just starting the webcam) Does the user need to refresh the browser once it is pressed?

@CobyKabili there are currently 3 official ways to turn the webcam on and off:

RoboLCD: Utilities > Options > Webcam

Web Dashboard: Top Right Menu > System > Start Webcam

iOS (BETA only): Webcam > toggle button


XG BOOST , ADA BOOST , GRADIENT BOOST

I am a GSSoC 21 participant. I can contribute to Decision Trees and Ada-Boost, Gradient Boost, and XgBoost algorithms (including Boosting Concept) with proper implementation and Documentation. Kindly assign this work to me @kritikaparmar-programmer @geekquad @MAJOR-BEAST
discord: diksha#6684

@dikshajoshi18 Pls make separate issues for all 3,that way I can start assigning issues to you

@geekquad we already have setup + issues for XGBOOST,ADBOOST.We would also kind of need a folder for Gradient Boost,if we are including it..

hi I am gssoc 2021 participant.I would like to contribute to as many algos as I can.Can u assign me with this task??

Hi,
I am GSSOC'21 participant
please assign me this.

Hi, can I get this task assigned?

@geekquad we already have setup + issues for XGBOOST,ADBOOST.We would also kind of need a folder for Gradient Boost,if we are including it..

Yes @ashwani-rathee.

Need to create another folder for Gradient Boost


Convert Zoloto output to metres

Slightly relevant: #346

Any reason why Zoloto outputs centimetres and not metres in the first place?

Any reason why Zoloto outputs centimetres and not metres in the first place?

Zoloto outputs in relative units. Unfortunately the function which sets those relative units can only return an int currently.


Question about the FirestoreAdapter DocumentSnapshot deserialized

Thanks for the codelab its fun to learn this coming from Firebase real time databas.
In the FirestoreAdapter  class javadoc I read:
/**
 * RecyclerView adapter for displaying the results of a Firestore {@link Query}.
 *
 * Note that this class forgoes some efficiency to gain simplicity. For example, the result of
 * {@link DocumentSnapshot#toObject(Class)} is not cached so the same object may be deserialized
 * many times as the user scrolls.
 */

First I was thinking that efficiency explanation should be in the  RestaurantAdapter but could you explain  some more how this works if this is some Cloud Firestore feature or RecyclerView?
What about that all objects are cashed on Android and never read twice unless they are changed? What does this mean "many times as the user scrolls." scroll happens in RestaurantAdapter which read the already loaded DocumentSnapshot´s objects or??

@erikswed thanks for the feedback.  Calling .toObject() uses reflection to turn the DocumentShapshot into your own custom model class.  If you call toObject on the same snapshot a few times, this same somewhat expensive operation is done each time even though it would be more efficient to save the result of toObject() somewhere rather than recreating it each time you need it.
The adapter in this codelab is a simple one that is useful for learning about Firestore.  If you want to use the most efficient Firestore RecyclerView adapter implementation, try the one we built in FirebaseUI:
https://github.com/firebase/firebaseui-android

Thanks grate info. What about in the Firebase real time databas like:
   @Override
    public void onDataChange(DataSnapshot dataSnapshot) {
        Post post = dataSnapshot.getValue(Post.class);
        System.out.println(post);
    }

Is this also an expensive operation like you describe?   What´s the differens under the hood?

@erikswed when we say "Cloud Firestore caches data that you are using" we mean that the DocumentSnapshot is cached, so we don't contact the server again unless we have to or you ask us to.
What is not cached is the conversion from DocumentSnapshot to your custom object classes.  That operation is repeated each time you call .toObject().


Drop support for py 3.7

Drops support for python3.7.
We've had long standing dependency issues due to pandas and awswrangler not supporting py 3.7 and it's a major blocker in https://github.com/moj-analytical-services/splink/pull/1554.
If this is an issue for users, we can either:

Undo this PR and reimplement support
Tell users to use Splink <=3.9.6. If this become a common theme, we may want to prioritise getting docs versioning sorted.

See here for some further reasoning behind this change - https://pythonspeed.com/articles/stop-using-python-3.7/.

This all makes sense to me - can you add a caveat about dropping support into the docs and I will approve?

This all makes sense to me - can you add a caveat about dropping support into the docs and I will approve?

E.g. in "getting started" we quote 3.7+


Bookmarks not in right order

I have merged 2 jwlibrary files with bookmarks not being present for one of the two. The one with bookmarks was the iPad and the bookmarks for my Bible look like this:

On my Pixel 2 XL, which was the source of the other backup file and where there were no bookmarks for my Bible, this is the result of restoring that merged backup:

So it looks like perhaps the way of calculating the slot number for the bookmarks needs a bit of tweaking as it almost seems random. The slot numbers seem ok in the iPad's backup:

Thanks @AntonyCorbett
I'll build and try it out later. I'm also working on adding option parameters to the cli. You accept PRs I assume?


module sharepoint

We have to transform translations xml to json.

We have to transform translations xml to json.

done


Means to view project-wide diagnostics?

In Sublime Text, I'd love to have a way to see ruff's findings for all of the files within my project. At the moment, I don't see such a feature (though maybe I'm missing it). Is such a thing supported? All I see is file-by-file results (as I'm viewing the file).
The SublimeLinter package (using flake8) allows doing this, and it's a feature I miss having.

You should report and discuss it in https://github.com/astral-sh/ruff-lsp.
(Though personally I would say this is more of a CI/commit hook job)

This project is just a wrapper around https://github.com/astral-sh/ruff-lsp. It's the ruff-lsp that would have to run that command and expose all diagnostics. Nothing can be done here.

Thanks for the clarification!


Update rips_registry_schema.json

Updated Schema To Provide More Useful Information

chain ID is better indicator than name
activation block useful for allowing addition to registry when activation is in future / better historical resource

also changed from chains_with_mainnet_support -> chains_with_support to allow for listing of major testnets


Does this project support for ros noetic and ubuntu 20.04?

Hi, thank you for sharing your LVI project. I wonder to know if this project support for ros noetic and ubuntu 20.04?

Hi, thank you for sharing your LVI project. I wonder to know if this project support for ros noetic and ubuntu 20.04?

Hi, this project has exactly the same dependencies as the original LVI-SAM. I think it can run on ubuntu 20.04 , but I haven't tested it.

Hi, thank you for sharing your LVI project. I wonder to know if this project support for ros noetic and ubuntu 20.04?

Hi, this project has exactly the same dependencies as the original LVI-SAM. I think it can run on ubuntu 20.04 , but I haven't tested it.

Hi, thank you for your quick reply. I will try it and tell you the test result.


code: 'SQLITE_CORRUPT'

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I have a similar problem as https://github.com/louislam/uptime-kuma/issues/51 but can't see the solution there. Or maybe I'm just not hardcore Docker man enough.
I get the following error:
errno: 11,
code: 'SQLITE_CORRUPT'
}
Start Uptime-Kuma failed due to issue patching the database
Patching the database failed!!! Restoring the backup
Whole error message can be seen here: https://pastebin.com/Zwmh6PBx
I've had uptime-kuma before, but deleted it again. Now I wanted to install it again, and got that error.
🐻 Uptime-Kuma Version
Last Update:	Mar 18, 2022
💻 Operating System and Arch
unRAID, Docker
🌐 Browser
Chrome
🐋 Docker Version
20.10.5
🟩 NodeJS Version
No response

How do you start your docker container?
Can you post the docker cli or yml file here?

are you using NFS for your volume mount?

I would like to, but how do that in unRAID?
Or can I start it, or see it in the CLI??

are you using NFS for your volume mount?

Maybe. I'm still a little new into this. Do you know how I see it?

you would know based on your docker-compose file where your volume mount is pointing

In unRAID it's all GUI. But I can open a CLI. Just don't know much to look for there....

I deleted the user folder and integration. Installed again, and it worked.


Theme Replace - Update the "Last Edited" Date

We have two themes on our production site. Once changes are made, we use themekit to automatically deploy to the currently inactive theme. Then once we are ready to go live with our changes, we simply publish the inactive theme that has the changes.
Currently, when we deploy the changes to the inactive theme the "Last Edited" column does not change. It simply shows the last date when we manually edited the theme via the admin portal. We would like this to show the last date that we automatically deployed changes to the theme via themekit as well.
Is there an already implemented solution for this? I've been looking through the docs and can't seem to find any configuration for this. It's not a big deal, but would be nice to have in the near future.

Hey @shampto3 this seems like it might be a bug in the Shopify API. I will contact them for this.

Thanks so much for your help. Please keep me posted on their response.

Okay so unfortunately I have clarified with the themes team and they said that this is on purpose. The Last Edited value displayed on the admin is only meant to reflect edits made within shopify Admin.

Thanks @tanema for looking into this for me. Sorry to hear about the response, but it's not the end of the world - more of a nice to have kind of thing.


Feat/documentation add how to guides section

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hello there,
Thank you, for taking the time to contribute. However, I do not believe this change would be beneficial, at this time.
Mostly, because the contributing link is not solely for developers, but translators, as well as moderators. As such, we would not want to reduce the likelyhood of someone coming across these changes.
Also, we are unlikely to take any links to external repositories, as these links could go dead, and leads to unnecessary maintenance.
Feel free to contribute to one of the open issues with a help wanted label.
Happy coding.


Make QA node improvements (add inquiry intent filter and add chat history functionality)

Also removes the while loop in Dialogue Development Node to prevent infinite GPT queries.

Looks like there are conflicts in this branch for this file:
ros/angel_system_nodes/angel_system_nodes/audio/question_answerer.py
@derkmed , do you want to resolve those or should I give it a go?

Looks like there are conflicts in this branch for this file:
ros/angel_system_nodes/angel_system_nodes/audio/question_answerer.py
@derkmed , do you want to resolve those or should I give it a go?

Should be hopefully resolved.


Add Edit Project when cluster has PSP enabled

We're missing the ability to edit the project if PSPs are enabled.
Here's the view in Ember for 2.5.x:

This is what's currently in Vue:

@codyrancher are you able to grab this this morning?


Karate tests are hanging

Karate tests are hanging at "waiting for parallel features to complete ..."
Please help me @ptrthomas , unable to proceed further

@krishnasomayajulu please follow this process to reopen: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


Auth Returning 405

I have been struggling with authenticating issue for the past few days.
I have this setup:
Setup:
'auth' => [
		'basic' => function ($app) {
			return new Dingo\Api\Auth\Provider\Basic($app['auth']);
		},

		'jwt' => function ($app) {
			return new Dingo\Api\Auth\JWTProvider($app['tymon.jwt.auth']);
		},
	],

My autthentcation route:
$api->get('authenticate/user', ['middleware' => 'api.auth', 'providers' => ['jwt'], function () {

		// This route requires authentication.
		$user = app('Dingo\Api\Auth\Auth')->user();
		return $user;

	}]);

POSTMAN error:

I don't know this makes any differences if I am running local PHP server.
I have confirmed the route is protecting correctly.

What am I doing wrong?
I am using version 0.10.0.

@JustinLien  the api/authenticate/user accepts only GET but your trying it using POST method update your postman to send GET request

@sunel I feel really stupid right about now. =X
I fixed that and it complains about "JWTProvider" not found.
Update that to new path:
'auth' => [
		'jwt' => function ($app) {
			// return new Dingo\Api\Auth\JWTProvider($app['tymon.jwt.auth']);
			return new Dingo\Api\Auth\Provider\JWT($app['tymon.jwt.auth']);
		},
	],

We're golden. Thanks sunel!!!

Note that in the newest version you can't use closures in your config file.
On Wed, 9 Sep 2015 03:51 Justin<EMAIL_ADDRESS>wrote:

Closed #634 https://github.com/dingo/api/issues/634.
—
Reply to this email directly or view it on GitHub
https://github.com/dingo/api/issues/634#event-403945869.

:+1:

Did you re-open this for a reason @JustinLien?

@jasonlewis sorry, no it was an accident. Must have happened when I was viewing it on the phone. Closing!
Thank you for an awesome package BTW!
Thanks for the heads up about closures in config file.

Some feedback: I wish there were some sort of feature list or upgrade history and more example (if not complete) in your documentation. (I can help with that if needed)
For an example, Facades in the installation section.
You gave example for the providers:
'providers' => [
    Dingo\Api\Provider\LaravelServiceProvider::class
]

But in your Facades section you gave just the class.
Dingo\Api\Facade\API
Dingo\Api\Facade\Route

For consistency and clarity I would use the same approach. (you can disagree with me)
'DingoApi' => Dingo\Api\Facade\API::class,
'DingoRoute' => Dingo\Api\Facade\Route::class,

Then perhaps followed by an example.

// App\Http\routes.php

DingoRoute::version('v1', , function ($api) {

   // routes here

});

Another area is the API Endpoints.
$api->version('v1', function ($api) {

});

Would be nice to provide example of what this route would be.
At first, I thought this would give us an URL of api/v1/your_route_here.
Sorry, it was my first time going through your documentation and those were some (not all) of the questions I had while I was reading them.
Again, awesome work here!


Mobile - Grid is flickering on horizontal scrolling

When using react-virtualized Grid on iphone (both Safari and Chrome), there is an issue with
horizontal scrolling.
You can have a look on the issue here:
https://www.youtube.com/watch?v=087MM98zF28
It seems that during one of the touch events, the list left offset is set to 0 when it's actually higher, which make the grid flicker.

@GulinSS I have the same issue,  I have a large list that also displays images it flickers when I allow images to load, I also noticed that when I dont allow images to load scroll works fine... I'll create a gif and attach it as soon as I can


Warn user when API v2 is not detected

Is your feature request related to a problem? Please describe.
Client works with API v1, this leads to errors happening later in the stack with them having misleading information regarding the source of the issue.
(from a user) For example, when trying to resolve a link (through @prismicio/react@2), the above is thrown when using API v1, which is not helpful regarding the source of the issue:

Describe the solution you'd like
Client could console.warn if v2 is not detected in the endpoint: https://github.com/prismicio/prismic-client/blob/v6/src/client.ts#L313
I don't really see use cases where this could be inconvenient 🤔
Describe alternatives you've considered
/
Additional context

Not 100% sure about it, but well, could help avoid that kind of error~

Good idea. Should be simple enough to check for /api/v2 in the endpoint pathname. We could make sure it only happens in development.
If someone uses a custom endpoint (e.g. through a proxy), I don't think we should do anything complex to detect API V1. In that case, I think we should just rely on documentation to state that all current JavaScript packages only support API V2.

Added in #203. It can be published in the next release.


Update links in extensible-admission-controllers.md

update link

/assign @kbhawkey

/assign @kbhawkey

Hello @Abirdcfly . Thanks for opening this pull request.
Page preview:
https://deploy-preview-26060--kubernetes-io-master-staging.netlify.app/docs/reference/access-authn-authz/extensible-admission-controllers/#write-an-admission-webhook-server

Hello @Abirdcfly . Thanks for opening this pull request.
Page preview:
https://deploy-preview-26060--kubernetes-io-master-staging.netlify.app/docs/reference/access-authn-authz/extensible-admission-controllers/#write-an-admission-webhook-server

/retitle Update links in extensible-admission-controllers.md

/retitle Update links in extensible-admission-controllers.md

/lgtm

/lgtm

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving  target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving  target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

It is common to pointing to source code. And If we want to explain clearly, it is necessary to show the source code. I have change master branch to param "githubbranch" like other pages do. 😊

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

It is common to pointing to source code. And If we want to explain clearly, it is necessary to show the source code. I have change master branch to param "githubbranch" like other pages do. 😊

/assign @kbhawkey

/assign @kbhawkey

ping @kbhawkey

ping @kbhawkey

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Hi @Abirdcfly . What do you think about updating the version in the links to the latest version, v1.20.0? Will that work?

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Hi @Abirdcfly . What do you think about updating the version in the links to the latest version, v1.20.0? Will that work?

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Hi @Abirdcfly . What do you think about updating the version in the links to the latest version, v1.20.0? Will that work?

It work. There are two choices:
1>. We point to v1.20.0 and when k8s v1.21.0 release, we update link to v1.21.0 ?
2>. We point to master ( param "githubbranch" like other pages ) and when source code changes, we update this link.
Based on the historical update frequency of these source code,, I think 2> is more appropriate.

Pointing to old versions of source code is not ideal. However, pointing to the "master" branch is not a good idea either because the "master" is a moving target.
Please think twice... do we really need this link to the source code? If we do, can we use a more recent tag (rather than branch) as the link target?

Hi @Abirdcfly . What do you think about updating the version in the links to the latest version, v1.20.0? Will that work?

It work. There are two choices:
1>. We point to v1.20.0 and when k8s v1.21.0 release, we update link to v1.21.0 ?
2>. We point to master ( param "githubbranch" like other pages ) and when source code changes, we update this link.
Based on the historical update frequency of these source code,, I think 2> is more appropriate.

Pointing to master/main is never a good choice to me. Please consider pointing to a tag for latest release.

Pointing to master/main is never a good choice to me. Please consider pointing to a tag for latest release.

/close


RF Builder:  Untranslated string in deletion confirmation

Might as well put some space between the ! and the header text while there.

Full message is now translated, and there's a bit of space between the ! icon and the text next to it.


fix: Fix lazy aliased connections type resolution

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release fixes an issue that would prevent using lazy aliased connections to
annotate a connection field.
For example, this should now work correctly:
# types.py

@strawberry.type
class Fruit: ...

FruitConnection: TypeAlias = ListConnection[Fruit]

# schema.py

@strawberry.type
class Query:
    fruits: Annotated["FruitConnection", strawberry.lazy("types")] = (
        strawberry.connection()
    )

Here's the tweet text:
🆕 Release (next) is out! Thanks to @_bellini666 for the PR 👏

Get it here 👉 https://strawberry.rocks/release/(next)


only one parameter not use @param, may not work for DynamicSqlSource

MyBatis version
3.4.6
Database vendor and version
5.7.20
i has a question in mybatis:3.4.6;
#the sql refer method
List selectOnlyIf(String entryDate);
when our sql has only one parameter, it't not must to add @Param annotation;
as follow, this run ok,
SELECT  * from tb_userInfo
 where entry_date = #{entryDate}
  order by id desc

is's because when runing,in MapperMethod.convertArgsToSqlCommandParam inner,
it jude, when don't have @Param and paramCount=1, it's return origin parameterObject;
but,when my SQL change as follow:
  SELECT *         from tb_userInfo         where         <if test="entryDate!=null">             entry_date = #{entryDate}         </if>  order by id desc
this sql will change to dynamic;
when DynamicSqlSource  => rootSqlNode.apply(context);
rootSqlNode while contain four SqlNode: StaticTextSqlNode,StaticTextSqlNode,IfSqlNode,StaticTextSqlNode
when IfSqlNode apply:
evaluator.evaluateBoolean(test, context.getBindings()))
here it real like: evaluator.evaluateBoolean("entryDate!=null", context.getBindings()))
DynamicContext has two key:
_parameter: ?
_databaseId: null
MetaObject parameterMetaObject,here,it's string;
OGNL.getValue, can't find any key for entryDate,will thrown error;
So, only one parameter not use @param can use for RawSqlSource,
but not apply for DynamicSqlSource,
this design, is a problem  or we use it in the wrong way ?

@xiaolongzhao2015
Could you try latest version (3.5.5)? Probably, this issues has been fixed since 3.5.2 via gh-1487.

If you cannot upgrade to 3.5.2+, you need to use the _parameter instead of argument name like as _parameter!=null. By the way, we have no plan that backport gh-1487 to 3.4.x line.
Thanks.

thank you very much for so rapid response;
i just pull the newest version 3.5.5, and look. and run test; it' ok;
and is there any new features introduce when every new version publish,
i want to follow and study,  because i study code just for version 3.4.6,
when new version, i can diff new features and to study

In latest version, there are many enhancements from 3.4.x.
I suggest to confirm a release note on GitHub about new features and enhancements.

https://github.com/mybatis/mybatis-3/releases


MM-12944: force "Delete" focus on showing delete post modal

Summary
Focus handling is a mess. This doesn't really make it much better, other than making the behaviour explicit and constrained to within the modal dialog in question, fixing a regression introduced by MM-12688.
Ticket Link
https://mattermost.atlassian.net/browse/MM-12944
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[ ] Needs to be implemented in mobile (link to PR or User Story)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[ ] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

@lieut-data

Focus handling is a mess

I coudn't agree more.


21 add api endpoints for crud for ngos

Description
**Adding endpoint for NGOs
Related to
#21
Fixes #1
Checklist:

[ ] My code follows the style guidelines of this project
[x] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation

@EneVladMihai This pull request is ready for review.


Update jsoniter-scala-core, ... to 2.17.5

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.13.39 to 2.17.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.github.plokhotnyuk.jsoniter-scala" }
}]

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

Superseded by #376.


feat: implement a long press action

Status
READY
Description
Implement a longTapOn action.
Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Closing in favor of #150 as this PR is outdated.


How do you generate the vscode patch?

Description
I am just wondering how you generate the vscode patch?
Is there a script ?

Related Issues

AFAIK you can generate your own via git patch or simply the patch command.

so, every time vscode updates, u have to update the vscode.patch file ...

Closing in favor of #417.


Histogram._samples() returns wrong values for buckets

Sample:
import prometheus_client
histogram = prometheus_client.Histogram('histogram', 'histogram', buckets=range(1,3))
histogram.observe(1)
histogram.observe(3)
print([m.get() for m in histogram._buckets])
print([v for k,l,v in histogram._samples() if k=='_bucket'])

expected results - printed lists should be same (1, 0, 1).
actual results:
[1.0, 0.0, 1.0]
[1.0, 1.0, 2.0]

That's an internal method, there's no guarantees about what it returns.

It's what shown in metrics!

What's the actual problem you're trying to solve here?

I'm trying to store and show histogram of file sizes.

You will either want to use a histogram in the usual fashion there, or write a custom collector.

Well, seems that I've missed something. What is "usual fashion"? Could you give me link to docs or example?

https://github.com/prometheus/client_python#histogram

I mean docs for usage. I.e. I want to receive alert on timeout. I thought that I should use something like function_execution_histogram{le="+Inf"}>0...

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Ok. Thank you.


Prepared HDF5 data for training: broken link

I would like to download the prepared HDF5 data for training but the link doesn't work. Could you fix it?
Do you have a link to download raw S3DIS data?
Thank you

Hi,  you can download the raw S3DIS dataset through this form.
The README will be updated.


Update dependencies for v1.1.2

Test new CLI build

Signed-off-by: Nicko Guyer<EMAIL_ADDRESS>

Codecov Report

Merging #1106 (029b39c) into main (530e09f) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 029b39c differs from pull request most recent head 6e6bfa7. Consider uploading reports for the commit 6e6bfa7 to get more accurate results

@@            Coverage Diff             @@
##             main    #1106      +/-   ##
==========================================
- Coverage   99.97%   99.94%   -0.03%
==========================================
  Files         309      308       -1
  Lines       20546    20460      -86
==========================================
- Hits        20540    20448      -92
- Misses          5        9       +4
- Partials        1        3       +2

Impacted Files
Coverage Δ

internal/events/webhooks/webhooks.go
97.23% <0.00%> (-2.77%)
:arrow_down:

pkg/core/message.go
100.00% <0.00%> (ø)

pkg/core/contracts.go
100.00% <0.00%> (ø)

pkg/core/tokenpool.go
100.00% <0.00%> (ø)

pkg/core/transaction.go
100.00% <0.00%> (ø)

internal/txcommon/txcommon.go
100.00% <0.00%> (ø)

internal/data/message_writer.go
100.00% <0.00%> (ø)

internal/events/persist_batch.go
100.00% <0.00%> (ø)

internal/database/sqlcommon/message_sql.go
100.00% <0.00%> (ø)

internal/database/sqlcommon/transaction_sql.go
100.00% <0.00%> (ø)

... and 1 more

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


Fix synchronization of global class name

This should fix the issues #76380, #75388 and #81615
This should manage these situations:

At startup, the global_script_class_cache.cfg is missing or incomplete but the filesystem cache is present
At startup, the global_script_class_cache.cfg and the system cache are missing
At startup, the global_script_class_cache.cfg is missing classes but are present on disk
At startup, the global_script_class_cache.cfg has a class that does not exists on disk
At startup, the path for class name in global_script_class_cache.cfg has changed
In the editor if the user duplicates a script with a class name and change the class name after
In the editor if the user changes the class name for an existing class name and change it again
Exporting project from command line that uses global class names in addons without or with an invalid global_script_class_cache.cfg

The global class name are now loaded from the scripts at the editor startup, always recreating the global_script_class_cache.cfg. When not in the editor, the global_script_class_cache.cfg is used for performance when the project runs.
Bugsquad edit:

Fix #76380
Fix #75388

Edited: Adding modifications for the issue #81615
Edited: Updating for the modifications loading class name at startup from scripts.

If this works, you will be a God amongst ots! :D

I added some modifications to fix the issue #81615
I tested with the MRP project from this issue when starting the projet in the editor and exporting from the command line.

Found a problem in the is_ready condition in editor_node. Last commit should fix that.

While I'm sure this change will be of help to many people, since it does seem to resolve the issue of scripts ending up in a perpetual state of error and needing to be resaved one at a time, it obviously does not change the actual order of global_script_class_cache.cfg being populated after the initial load of the autoloads, so you'll still see errors from that initial broken compilation of the autoloads that reference any class_name, although it will (mostly) only output to the command-line (i.e. stderr) with this change, since EditorLog has yet to initialize when autoloads are first loaded.
This does however mean that importing from the command-line using --headless, through something like a CI job, either explicitly through the new --import option or implicitly through --export-debug/--export-release, will still emit errors. I'm able to reproduce this by exporting the repro project in #75388 for example, even with the latest commit as of this comment.
There's also the issue of the more or less official plugin pattern where you have a @tool script that registers a potentially new autoload using add_autoload_singleton in its _enter_tree. In such a case add_autoload_singleton will trigger update_autoload, which kicks off another broken (assuming it depends on some other class_name) autoload compilation, which this time actually emits to the editor log, since EditorLog is up and running by that point. You'll see that in this repro that I just made: autoload-plugin-errors.zip
Both of these are of course largely "cosmetic" issues (as far as I can tell), since these errors seem to eventually get dealt with by your recompile_invalid_autoload_scripts, but any non-trivial project that uses class_name or a plugin autoload (like GodotLogger) in a widespread manner will still end up with hundreds/thousands of errors in these two scenarios if there's no valid global_script_class_cache.cfg.
Anyway, again, I'm not saying this change shouldn't be merged, and I shouldn't be the judge of that in the first place, but I figured I'd bring this up in case it hadn't been considered already, or if there's some discussion to be had about a more holistic solution.

Thanks @mihe for your detailed response and to take the time to test my modifications. I'll try your autoload-plugin-errors.zip and see if I can do something about the error messages.
If I understand correctly, with this fix, you have error messages in the console and output panel (and potentially a lot of them), but with the recompile_invalid_autoload_scripts, at some point they are working without having to resave them manualy?

A holistic solution is vital. If class_name is not immediately and always respected, then what is the point of class_name? All class_names, no matter their origins, must be known at all times.

I totally agree with that. Actually, the way I see it, the only way to really do that would be to remove completely the global_script_class_cache.cfg and reload all the class on startup from the code. I don't why it was not done like that from the beginning, maybe for performance issue?
Just today I think I saw 2-3 news issues linked to global script class.
I can try to implement it that way if everyone thinks that's a good idea? I'm a pretty new contributor, I usually try to limit the amount of code I touch, but I'm welling to try!

I don't know why there's a cache file either, but I guess it was for speed.
Perhaps moving it out of the dot-godot directory could be an answer; but that would involve merging it with other class caches that come from plugins and general project churn.
The simplest way feels like, on startup, grepping all the resource and gdscript files for 'class_name' and thence into a hash map.

I took your advice @mihe and @donn-xx and I’m pretty happy with the result.
This commit removes the need for global_script_class_cache.cfg when starting the editor. It loads class_name from the scripts on disk at startup and rewrites the global_script_class_cache.cfg. I kept the global_script_class_cache.cfg for exported project and when running the project to be sure that the start time is not affected. Doing it like that, I was able to remove the recompile_invalid_autoload_scripts after the first full file scan, preventing potential double error messages.
I tested with a project having 3000 gdscripts of 22ko each. At first, it took 10 sec to parse every file at startup. After a bit of debugging I found out that the complete file was parsed just to find the class_name and the icon. I added a new parameter to the GDScriptParser.parse to only parse the script header. After this modification, it took in a debug build only around 400ms to read all the 3000 files on my Intel i5-11600 3.9Ghz on Nvme. It’s not extensive testing but I think it’s good enough, it will append only on startup of the editor.
With this modification the test project from @mihe loads without any error or warning event if the .godot folder is missing or the
global_script_class_cache.cfg missing or invalid.

I also tested the MRP from #75388 and #92054 with these news modifications, and everything seems to work fine, even on fresh git clone (no .godot folder).

I am holding my breath! So excite!

While this fix is underway, it might be a good time to raise the issue of the classDB object not having access to these user scripted classes.
There is this convo about it: https://github.com/godotengine/godot/pull/71665 and this comment sums-up my feelings on it: https://github.com/godotengine/godot/pull/71665#issuecomment-1399369620

That's not relevant to this so please stay on topic

I also just tried #92054 with this PR, and I'm not seeing it fixed like you mentioned above, @Hilderin.
What seems to happen is that _update_scan_actions runs, where _update_script_classes does nothing, because no script classes have been queued up yet, then the script classes do get queued up due to being in scan_actions, and because one of them is added to the reloads list then resources_reload signal is fired at the end of _update_scan_actions which invokes EditorNode::_resources_changed which in turn causes a full parse of the file before any global class names have had a chance to be added.

I'll look into this, thanks for testing.

@mihe I just retested the project from #92054, I did not realized that you were running the editor when you do the git checkout. I'll look into it later today.

@KoBeWi I tested your addon. These are the error I get:

I don't see any error from missing global class. I think, these errors are due to addons being loaded before the resources. This PR should fix this problem: https://github.com/godotengine/godot/pull/86453
You probably need both fixes to make it work. I'll try later creating a branch with both fixes to test it.

@KoBeWi I merge my PR and the PR from https://github.com/godotengine/godot/pull/86453 and no, it does not fix the problem. The https://github.com/godotengine/godot/pull/86453 was only focused on addons script dans not autoload from what I understand. Do you have a issue open for that, maybe I can take a look in another PR.

Do you have a issue open for that, maybe I can take a look in another PR.

I opened https://github.com/godotengine/godot/issues/92493

This should also fix/close #83395

Rebased following merge conflicts.

I did a review and I am not very found of the approach. The manual filesystem scan in ScriptServer is likely overkill. EditorFileSystem is specially optimized to only scan files and directories that change, which allows for the editor to remain snappy even with very large projects. This PR goes against that.
The way I would solve this is directly inside EditorFileSystem by having a more layered way to scan the files. Probably something like:

Scan all changes
Instead of doing changes immediately, build a list of things that changed.
Pre process scripts first in the list.
Process everything else.

EditorFileSystem is specially optimized to only scan files and directories that change, which allows for the editor to remain snappy even with very large projects.

Eh, that's not entirely true. FileSystemDock rescans all files on pretty much every operation. It's nice that EditorFileSystem was designed this way, but it's not used this way.

@KoBeWi look again, it uses EditorFileSystem:
https://github.com/godotengine/godot/blob/e2fc6d38cb240ea23bd5700514f8a8ce7011e89c/editor/filesystem_dock.cpp#L230

@KoBeWi And in EditorFileSystem, only directories that change are scanned:
https://github.com/godotengine/godot/blob/e2fc6d38cb240ea23bd5700514f8a8ce7011e89c/editor/editor_file_system.cpp#L981

I only looked at what happens in the editor, and what happens is that in a big project renaming or doing anything with a file/folder makes FileSystemDock turn into scanning mode for few seconds as it processes every single file, so e.g. duplicating and renaming a file, instead of taking 2-3 seconds, takes 30. See #50576

@KoBeWi No idea about this, it may be an unrelated bug, but remember that renaming needs to change all dependencies to the file. When we complete the transition to using UIDs then this may not be needed anymore.

Just to be sure it's clear, the full scan of scripts to load the global class names is done only at the startup, not on very full scan (ex: renaming a file in the FileSystem Dcock). On large project, it could be very slow.

@Hilderin There is only full scan on first startup. Even for subsequent startups, EditorFileSystem keeps a file with a cache. This is why ideally, what you are trying to do probably should happen there.

Exactly!  The problem identified by the users is that the .godot folder is not always up to date or present on CI servers or on user computers. When multiple users are working on the same project. On pull or checkout from git, if Godot is not running, the global_script_class_cache.cfg is not updated, causing problems on startup.
I'll have to think about your suggestion:

Scan all changes
Instead of doing changes immediately, build a list of things that changed.
Pre process scripts first in the list.
Process everything else.

I like your suggestion to find only the changed scripts while scanning for changes in the first scan. I guess we will also need to remove global class names that no longer exist. My concern is the plugins that are loaded before the first scan. These plugins need the global class names to be loaded. If we do it, we will need to move the plugins loading after the Pre process of scripts (reading Global class names) and before the processing of everything else.
That seems a lot of modifications, what do you think?

I tried to implement the suggestion of @reduz but my I have serious problem:
The comparison of the files and the hash to validate the filesystem_cache8 on the first scan uses the importers. Importers can be customized by plugins. I don't think it's possible to accurately check if the file changed without validating the .import file. And revalidating the .import files without the plugins loaded seems not possible also. If I load the plugins before global class names, they could fail. I don't see right now how the suggestion "Instead of doing changes immediately, build a list of things that changed" could be done without the plugins loaded.
One approach could be:

First scan: load the new_filesystem for all the files but validate only the scripts and call _queue_update_script_class if modified
Update global class names
Create autoloads
Init plugins
Validate all the other file in new_filesystem and build scan_actions if modified
Execute _update_scan_actions as usual

I'll try that tomorrow!
There is a branch of a new draft: https://github.com/Hilderin/godot/tree/fix-synchronization-global-class-name-v2
Commit: https://github.com/Hilderin/godot/commit/a27547e15f680a948340ec7e1db018837451fa08

@Hilderin I would suggest something as follows:

Modify the scan filesystem function to instead just return a list of files that have to be checked/updated/scanned.
Once you have the list, try to go through it a couple of times by priority, such as:

Scan all scripts for class names
Check add-ons and load them. I think we could probably, worst case, add some API to add ons that they are adding importers? so they are parsed first.
If add-ons added new importers (We could kind of do this by checking the amount of them before and after doing this step), goto 1. (redo scanning)

I think this should work for the most part. As I mentioned, the main issue is how we handle importers, so this may need to change in the engine (maybe new API to expose those?).

Thanks for your feedback @reduz really appreciated. I'll check that tonight.

Thanks @Hilderin for working on this problem. It sounds thorny, but is totally essential!

I just pushed a new version for this PR following the recommendation of @reduz.
Modifications:

This new version reverts all the modifications in ScriptServer and puts the management of the global_script_class_cache.cfg back in ProjectSettings.
All scans are now done directly in EditorFileSystem as it should be.
The loading of autoloads and plugins have been moved to the first scan.
The first part of the first scan is done in the new method _first_scan_filesystem.
_scan_new_dir only discovers the directories and files and calculates the file types.
A new method _process_file_system validates the files and creates actions for updates/reimportations. It has two modes:

Script Only: Used in _first_scan_filesystem to process only scripts to load global class names.
All: Processes all the files. This mode is used after the global class names, autoloads, and plugins are loaded.

The progress bar in FileSystemDock has been fixed while processing files.

Algorithm:

On the first scan, call _first_scan_filesystem:

Discover all the files (_scan_new_dir).
Process all the scripts.
Load autoloads.
Load plugins.
If new importers have been added:

Rediscover all the files (_scan_new_dir): new extensions or resource types could have been added/updated from custom importers.
Process all the discovered files (_process_file_system).

On subsequent scans:

Discover all the files (_scan_new_dir).
Process all the discovered files (_process_file_system).

Test cases
It took my a while but I retested all these test cases with different MRP from issues:

At startup, the global_script_class_cache.cfg is missing or incomplete but the filesystem cache is present
At startup, the global_script_class_cache.cfg and the system cache are missing
At startup, the global_script_class_cache.cfg is missing classes but are present on disk
At startup, the global_script_class_cache.cfg has a class that does not exists on disk
At startup, the path for class name in global_script_class_cache.cfg has changed
In the editor if the user duplicates a script with a class name and change the class name after
In the editor if the user changes the class name for an existing class name and change it again
In the editor if the user deletes a folder containing a script with a class name that is used by another script
Exporting project from command line that uses global class names in addons without or with an invalid global_script_class_cache.cfg
Updating scripts from outside the Editor that uses new global class name

Just rebased from master after new conflicts.

@reduz
I think I finally managed to do only one disk scan and use it twice, the first time in _first_scan_process_scripts and the second time in _scan_filesystem on the first scan.
I just push some modifications:

_scan_new_dir loads all the files in a ScannedDirectory object.
_first_scan_process_scripts uses ScannedDirectory to process all the scripts. No needs to queue them with _queue_update_script_class, I call directly _register_global_class_script. The full process of scripts is done in _update_scan_actions.
_first_scan_filesystem does not need to check for new custom importers since it does not load a EditorFileSystemDirectory.  All scans are done with ScannedDirectory.
_process_file_system always process all files, so the parametre p_process_step (script only or full) has been removed.
_update_extensions have been moved after _first_scan_filesystem to be sure extensions are correctly loaded before _scan_filesystem.

Algorithm:

On the first scan, call _first_scan_filesystem:

Discover all the files (_scan_new_dir) and keep it in first_scan_root_dir.
Process all the scripts (_first_scan_process_scripts).
Load autoloads.
Load plugins.
Update extensions (_update_extensions).
Scan files (_scan_filesystem):

Use files in first_scan_root_dir.
Process all the discovered files (_process_file_system).

On subsequent full scans (ex: renaming files):

Discover all the files (_scan_new_dir).
Process all the discovered files (_process_file_system).

I moved the removal of invalid global class names in _first_scan_filesystem to fix the crash from #93410
Even if this fix the problem, I created another PR #93575 just to add a validation in GDScriptAnalyzer that cloud be merged more easily.

When opening a big project, the FileSystem dock changes into scanning mode, then stops at ~99% and the editor becomes unresponsive.

I compared it with the behavior without this PR and there is no scanning mode, however it has no impact on startup time, which is more or less the same. At least the progress bar gives some better feedback on what is happening. Though it would be nice if it stopped at 100% consistently 🤔 (does not need to be fixed now)

For the progression that does not go all away to 100%, I think it's because the scan is done before the main iteration can rerender the progress bar. After that, it's _update_scan_actions that is executed on the main thread that cause the editor to freeze.
Il worked on that on the PR #93064.
There are several missing progress popup to prevent the editor to freeze while loading.
At least, with this PR, we get the progress bar. I suggest I work on editor freezing and the progress bar in the PR #93064 when this one is merged.

@Hilderin Don't hesitate to come chat in the developers chat! Especially for such a big PR! :)

I've made some test builds to help validate this PR, in hope that some Godot users could try it out on their projects before we include it in the next 4.3 beta.
I built the editor for Windows, macOS, and Linux, with:

Current master branch (7907ef8) as point of comparison: https://downloads.tuxfamily.org/godotengine/testing/4.3-beta-7907ef8/
This PR (#92303) rebased on top of 7907ef8: https://downloads.tuxfamily.org/godotengine/testing/4.3-beta-pr92303/

To test and help validate this PR:

Make a copy of your Godot project(s) you want to test with
Delete .godot folder
Open the project in the 7907ef8 build, take note of potential errors that may not be present in the version you were using (to be reported separately as general 4.3 bugs). Try usual workarounds like restarting the editor once after the initial import, and check if your game works as intended.
Close editor
Delete .godot folder
Open the project in the #92303 build and see if it performs better, using the same steps as with the 7907ef8 build. Ideally there should be a lot less errors about class names not being found, though some may still appear if using addons (this will be tackled later in another PR).

If your project generally works well in current 4.3-beta / 7907ef8, try to use the build from #92303 to do some development work like creating new global classes, renaming some, etc., to find potential bugs.

Thanks! Amazing work :tada:

Thanks for your help and support!!

Sorry for being away. Have just done my first few tests for this fix and it's looking real good. I'll poke at my addon project and keep notes.


test(core): update visual regression breadcrumbs test

What/Why?
Updates the breadcrumbs test after @jorgemoya updated the styles in #861
Testing
See CI

Thank you 🙏 . Worst case is I remembered last night I needed to fix these, but this morning I forgot.


docs(scroll-bar): add props description

Codecov Report
Base: 62.50% // Head: 62.52% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (2897aaf) compared to base (bbf09ef).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4234      +/-   ##
==========================================
+ Coverage   62.50%   62.52%   +0.01%
==========================================
  Files        1069     1069
  Lines       22412    22420       +8
  Branches     5876     5882       +6
==========================================
+ Hits        14009    14017       +8
  Misses       6880     6880
  Partials     1523     1523

Impacted Files
Coverage Δ

src/data-table/src/interface.ts
100.00% <0.00%> (ø)

src/dynamic-input/src/PairPreset.tsx
72.72% <0.00%> (ø)

src/dynamic-input/src/InputPreset.tsx
100.00% <0.00%> (ø)

src/data-table/src/TableParts/Header.tsx
67.10% <0.00%> (+0.43%)
:arrow_up:

src/dynamic-input/src/DynamicInput.tsx
54.65% <0.00%> (+0.76%)
:arrow_up:

src/date-picker/src/panel/datetimerange.tsx
64.40% <0.00%> (+1.69%)
:arrow_up:

src/date-picker/src/panel/daterange.tsx
69.38% <0.00%> (+2.04%)
:arrow_up:

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.


Multiple schema support

For the issue:
https://github.com/timescale/pg_prometheus/issues/34

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: ahmad iftekhar:x: irummanahmad iftekhar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Bug]: Memory leak in NetworkEventManager causing crashes

Bug description
Using 18.0.5 but reproduced on a variety of versions with no resolution. It appears that navigating away from a page before it has finished loading causes request maps, such as #requestWillBeSentMap, to consume memory until the heap is exhausted.
const browser = await puppeteer.launch(options);
const page = await browser.newPage();

await page.setRequestInterception(true);

page.on(
    'request',
    request => {
        const resourceType = request.resourceType();

        // Simple hook to remove any images or fonts
        if (resourceType === 'image' || resourceType === 'font') {
            request.abort();
        } else {
            request.continue();
        }
    },
);

// Navigate to and from URLs that trigger larger network requests

This is especially prevalent with larger payloads and often leads to a hard crash within a few hours. You may need to load a page that sends many large payloads to reproduce effectively.
Puppeteer version
18.0.5
Node.js version
16.18.0
npm version
1.22.19 (Yarn)
What operating system are you seeing the problem on?
Linux, Windows

@Crunkle it'd be great if you could provide a test script that demonstrates the memory growth? The NetworkManager code indeed relies heavily on backend events and if events arrive in unexpected order that might lead to a memory leak.

Sure, I will try and get around to it this weekend. A simple page navigate loop should suffice.

@Crunkle thanks for the script! Unfortunately, I don't seem to be able to repro: SHOULD_MEMORY_LEAK = true my memory usage gets to the peak of 43.1 MB and stays the same (I ran the script for a few minutes). I am on Mac and Node 18.

@Crunkle do you run<EMAIL_ADDRESS>

@OrKoN I was originally on 18 but am able to reproduce on the latest 19 release under Windows. I have not tested recently but can confirm this was an issue under Linux as well, but perhaps Chromium has updated since my last test. You may need to marginally increase the timeout to ensure the fetch is executed before navigate.
See this video for a Windows demo.

Still not able to repro (tried different time outs up to 2000ms): in my case, if I look into the CDP log I get Network.loadingFailed events upon navigation that are handled by the network event manager and clears all local data. I wonder if you see those events in the CDP log too? ('env DEBUG=puppeteer:*')

I see one Network.loadingFailed per request being sent in the log, which I am guessing is the correct behavior. This event does not seem to trigger any clearing of the #requestWillBeSentMap though, at least for me, and the data is retained within. If you are still unsure then I can have a dig at the Puppeteer source myself to try and pinpoint the issue. I am pretty confident after testing in an isolated container that it is on the Puppeteer side, and I have a hunch #9283 is the same issue.

@Crunkle I wonder if Network.loadingFailed events are being only sometimes missed, causing the leak? It'd be hard to isolate the issue then. In general, copying the Network events in the order they arrive from the log into a test https://github.com/puppeteer/puppeteer/blob/main/test/src/NetworkManager.spec.ts#L39 should help to reproduce the issue.

Is there any update on this or any workarounds? I am also facing this issue.
I use the request interception feature for scope management in my project https://github.com/karthikuj/sasori
In a couple websites too many requests get stored in the heap and then it crashes with JavaScript heap out of memory error.
I am not sure why it only happens in a couple websites and not all of them :/

@karthikuj do you have an example to reproduce the leak? would you be able to add a test case to https://github.com/puppeteer/puppeteer/blob/9ec5f25ea6f9e60d250c7413e122a5c32faeb3f9/packages/puppeteer-core/src/cdp/NetworkManager.test.ts#L37 with the requests that you find leaking?

@OrKoN I don't have a clear understanding of why this is happening, and the website I found this on is a bit confidential, but I will try reproducing it with some testcase whenever I get the time.

Still not able to reproduce. If anyone is able to reproduce with a minimal example, please post it here or feel free to file a new issue.


fix(routes): update router order for congregations

:tada: This PR is included in version 2.71.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: