[PR #5465] parallelise contributor points calculations #13004

Closed
opened 2026-04-10 21:44:35 -05:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/actualbudget/actual/pull/5465

State: closed
Merged: Yes


Before:

% time node .github/scripts/count-points.mjs
0.95s user 0.10s system 0% cpu 2:21.68 total

After:

% time node .github/scripts/count-points.mjs
0.88s user 0.08s system 14% cpu 6.702 total

Down from 2m21 to 6s - a nice improvement!

Rate limit info here: https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28#about-secondary-rate-limits

No more than 100 concurrent requests are allowed. This limit is shared across the REST API and GraphQL API.

90 should be safe here, obviously if you spam the command you'll hit the per-minute rate limit a bit faster now.

**Original Pull Request:** https://github.com/actualbudget/actual/pull/5465 **State:** closed **Merged:** Yes --- Before: ```sh % time node .github/scripts/count-points.mjs 0.95s user 0.10s system 0% cpu 2:21.68 total ``` After: ```sh % time node .github/scripts/count-points.mjs 0.88s user 0.08s system 14% cpu 6.702 total ``` Down from 2m21 to 6s - a nice improvement! Rate limit info here: https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28#about-secondary-rate-limits > No more than 100 concurrent requests are allowed. This limit is shared across the REST API and GraphQL API. 90 should be safe here, obviously if you spam the command you'll hit the per-minute rate limit a bit faster now.
GiteaMirror added the pull-request label 2026-04-10 21:44:35 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#13004