Skip to content

Add CI test workflow with Codecov coverage#33

Merged
cweniger merged 7 commits intomainfrom
ci/add-tests-workflow
Feb 7, 2026
Merged

Add CI test workflow with Codecov coverage#33
cweniger merged 7 commits intomainfrom
ci/add-tests-workflow

Conversation

@cweniger
Copy link
Owner

@cweniger cweniger commented Feb 7, 2026

Summary

  • Add .github/workflows/tests.yml with two jobs:
    • unit: runs pytest -m "not slow" with coverage, uploads to Codecov
    • smoke: runs pytest -m slow (end-to-end example tests, 15min timeout)
  • Both trigger on push to main and PRs targeting main
  • Add Tests and Codecov badges to README

Setup required

  • Enable Codecov on the repo at codecov.io
  • Add CODECOV_TOKEN as a GitHub repo secret (Settings → Secrets → Actions)

Test plan

  • Verify unit job passes on this PR
  • Verify smoke job passes on this PR
  • After merging + Codecov setup, verify coverage badge renders

🤖 Generated with Claude Code

cweniger and others added 3 commits February 7, 2026 00:23
…, embeddings

Replace all falcon.contrib references with new package paths (falcon.estimators,
falcon.priors, falcon.embeddings). Consolidate SNPE_A docs into Flow, add new
pages for Gaussian, Product, FlowDensity, and Embeddings. Fix embedding config
to be sibling of network. Update mkdocs nav structure.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add GitHub Actions workflow that runs unit tests and smoke tests on
push to main and PRs. Unit tests upload coverage to Codecov. Add
test and coverage badges to README.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

cweniger and others added 4 commits February 7, 2026 01:30
Run each example as a separate CI job so they execute concurrently
instead of sequentially (~2min vs ~4min wall time).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add timm to install_requires (needed by 03_composite example).
Increase subprocess timeout from 3 to 10 minutes for CI reliability.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
03_composite needs >7GB RAM (two ResNet18 + Ray actors).
05_linear_regression requires a GPU. Both are auto-skipped
via CI env var detection and removed from the CI matrix.
They still run locally with `pytest -m slow`.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
timm is only needed by 03_composite which is skipped in CI.
Restore original 3 minute subprocess timeout.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cweniger cweniger merged commit a26d0a4 into main Feb 7, 2026
4 checks passed
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.

2 participants