Skip to content

Instrument and expand the 50k download goal - #81

Open
dnncha wants to merge 87 commits into
mainfrom
codex/dotmatch-50k-downloads
Open

Instrument and expand the 50k download goal#81
dnncha wants to merge 87 commits into
mainfrom
codex/dotmatch-50k-downloads

Conversation

@dnncha

@dnncha dnncha commented Aug 22, 2026

Copy link
Copy Markdown
Owner

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.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment thread scripts/check_download_metrics.py Outdated
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)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

@dnncha

dnncha commented Aug 22, 2026

Copy link
Copy Markdown
Owner Author

Current head 13febce has all substantive repository checks passing: Linux/macOS build and tests, Python 3.9–3.12 compatibility, workflow-ecosystem smoke tests, CodeQL, C/C++ and TypeScript analysis, GitGuardian, and the reviewer reproducibility packet. The only failing status is the unrelated Vercel integration, which reports Account is blocked.

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.

@dnncha

dnncha commented Aug 22, 2026

Copy link
Copy Markdown
Owner Author

Follow-up: the metrics-only audit refresh is now b3c7f98, and its complete CI/analysis suite is green as well. The Vercel account-blocked status remains the sole failing check; the measured aggregate remains 5,215 retrieval events.

@dnncha

dnncha commented Aug 22, 2026

Copy link
Copy Markdown
Owner Author

The current head is now 9572f20, which only corrects the Galaxy submission evidence to the actual validated head b547330d and CI run 32588612106. The complete substantive check suite remains green on this exact head; Vercel is still the sole external account-blocked failure.

@dnncha

dnncha commented Aug 22, 2026

Copy link
Copy Markdown
Owner Author

Latest candidate head 01c2524 is now fully checked: Linux/macOS build and tests, Python 3.9–3.12 compatibility, workflow ecosystem smoke tests, CodeQL, C/C++ and TypeScript analysis, GitGuardian, and the reviewer reproducibility packet all pass. The only failing status is the unrelated Vercel integration, which reports Account is blocked.

The provider ledger remains 5,215 aggregate retrieval events, with 44,785 remaining to the 50,000 target. The snapshot keeps package retrievals separate from GitHub discovery traffic and does not claim distinct users. I also asked the Bioconda/conda-forge maintainers to resolve the duplicate-name policy for staged-recipe PR #34608.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant