chaoss / chaoss/CollectOSS

process contributors could be more efficient with API calls

Open
#423 0 comments 0 reactions 0 assignees View on GitHub
disclosed-ai task:core tech debt
Dominant language
Python
Stars
13
Forks
17
Avg merge
6h 59m
Merged PRs (30d)
1

Description

We might be able to squeeze a lot more performance from our api calls in the process_contributors task

Sonnet 4.6 has identified that we are not being efficient for large repos:

https://github.com/chaoss/CollectOSS/blob/96adf3a4d68725db21622673ee6613693c0f5ace/collectoss/tasks/github/contributors.py#L51-L62

> loads up to 500 contributors missing cntrb_created_at, then hits GET /users/{login} one at a time. This is essentially the same pattern as the comment collection win — it could be replaced with batched GraphQL user lookups (up to ~50 logins per query with aliases).

this batching behavior is already present here:

https://github.com/chaoss/CollectOSS/blob/96adf3a4d68725db21622673ee6613693c0f5ace/collectoss/tasks/frontend.py#L207-L212

(note: this issue is very similar to several others. be careful to make sure you are talking about the same issue in the comments)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.