-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
add scripts/downloads.py #735
Conversation
@KIRA009 any idea why the number reported by this script (151) does not match the sum on the landing page (214)? ![]() |
Think this is the same thing that we found in the website implementation. The releases contain files other than the two apps, so we have to filter them out - see https://github.com/OpenAdaptAI/OpenAdapt.web/blob/main/utils/githubStats.js |
@KIRA009 I don't think that's it. I've updated the PR description to include the asset names, it only selects the relevant assets by default. In addition, the number produced by this script is lower than the number on the website, which means it's impossible for it to be over counting compared to the website. |
Ah, it looks like you are only getting results from 1 page. There are three pages of releases in the api - https://api.github.com/repos/OpenAdaptAI/OpenAdapt/releases?page=3. In the website implementation, I increment the page number after every response until I get a response with no releases |
|
Thank you @KIRA009 ! 🙏 ![]() ![]() |
* add scripts/downloads.py * add total cumulative downloads to title * add module docstring * pagination * add datetime * accumulate by release_date * black/flake8
Adds a script at
scripts/downloads.py
that produces the following:Previously:
c.f.