Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

🔨 FixFastAPI People workflow#14951

Open
YuriiMotov wants to merge 3 commits intomasterfrom
fix-people-workflow
Open

🔨 FixFastAPI People workflow#14951
YuriiMotov wants to merge 3 commits intomasterfrom
fix-people-workflow

Conversation

@YuriiMotov
Copy link
Member

CurrentlyFastAPI People workflow fails due to hitting GitHub API rate limits.

After some experiments and googling I found out that the reason of hitting rate limits is in requests per query.
Since September rate limit rules became more complex and it's not recommended to make heavy queries (that return thousands of records).

To avoid hitting rate limits I:

  • reduced the number of records fetched by query (30 discussions instead of 100)
  • added a rate limiter that respects different types of limits:
    • pauses if the number of remaining points is less than needed for request
    • respects safe time intervals between requests (interval time is calculated from the limitation of 5000 points per hour and the number of points used by each request)

This way script will take more time to fetch data, but we will likely avoid hitting rate limits


If we need it, we can experiment with the speed by settingspeed_multiplier ENV variable (defaults to 1, that is the spread with which we will spend 84 points per minute (5000 points per hour divided by 60 minutes)).


github_graphql_url = "https://api.github.com/graphql"
questions_category_id = "MDE4OkRpc2N1c3Npb25DYXRlZ29yeTMyMDAxNDM0"
questions_category_id = "DIC_kwDOCZduT84B6E2a"
Copy link
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Old value is "global id" and it's deprecated (produced deprecation warning with the recommendation to use this new ID)

@YuriiMotovYuriiMotov changed the title🔨 Fix people workflow🔨 FixFastAPI People workflowFeb 20, 2026
@codspeed-hq
Copy link

codspeed-hqbot commentedFeb 20, 2026
edited
Loading

Merging this PR willnot alter performance

✅ 20 untouched benchmarks


Comparingfix-people-workflow (e082d67) withmaster (cf05823)1

Open in CodSpeed

Footnotes

  1. No successful run was found onmaster (d2c17b6) during the generation of this report, so cf05823 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@YuriiMotov

[8]ページ先頭

©2009-2026 Movatter.jp