Instrument and expand the 50k download goal - #81
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ba03b5209e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| nfcore = fetch_optional_json(NFCORE_PR_URL) | ||
| anaconda_breakdown = aggregate_files(anaconda_files) | ||
| anaconda_downloads = integer(anaconda_package.get("download_count")) | ||
| pypi_downloads = aggregate_pypi_downloads(pypistats_overall) |
There was a problem hiding this comment.
Preserve PyPI downloads beyond the API window
When the package becomes older than PyPI Stats' default 180-day reporting window, summing the current overall response discards downloads as their dates age out. The weekly job then overwrites the snapshot with this rolling-window sum, so current_reported_downloads can decrease and the documented “cumulative” 50k goal may never be reached even when lifetime retrievals exceed it. Persist and merge the prior daily ledger, or explicitly retrieve all historical date ranges, before calculating this total.
Useful? React with 👍 / 👎.
|
Current head This candidate also contains the public Binder/Colab conversion path and the scheduled auditable download-metrics refresh. The measured ledger remains 5,215 aggregate retrieval events, not distinct users or adoption. |
|
Follow-up: the metrics-only audit refresh is now |
|
The current head is now |
|
Latest candidate head The provider ledger remains |
Adds an auditable Anaconda/PyPI download ledger with preserved daily series, provider-outage handling, version/platform breakdowns, and a scheduled GitHub updater. Refreshes install/discovery surfaces and records GHCR/BioContainers as public but currently unmeasured channels. Verification completed locally: focused metrics tests, package checks, app lint/build, and distribution checks.