-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rate limits? #4
Comments
@phiwut Did a dialog show up telling you rate limits was exceeded and you could provide an access token to get over it? |
Even with extended rate limits it is hard to use hubble for medium or big project. I would suggest to use the latest number of stars and count backwards from there to draw the chart. So a user would at least get the latest results. You can even use the current starcount as a test to decide wich way to go... |
@indus Good advice. Yes this way user can see latest results. However, the user that requests this race feature #3 wants to see them "race each other", and I feel it means a process from the beginning. 😂 So I should probably focus on controlling the request rate, it’s the key problem itself after all. |
@SevenOutman Then maybe there is some way to set up a cronjob on a pc or server that fetches old star data for repos with over 1000 stars https://github.com/js-org/stats.js.org/tree/master/_data once a week or so that gets pushed in this repo and that gets used a primary source keeping API requests to a mimimum. |
@indus I see. Disadvantage is that the repo is then getting too big. But I'll still consider it as an alternative. |
Github's abuse detection mechanism gets triggered in the racing mode (Link). I think that you should reduce the number of requests per second.
The Requests return the following message:
The text was updated successfully, but these errors were encountered: