Skip to content

Run the darwin reference tests in CI - #32

Merged
alpae merged 2 commits into
DessimozLab:masterfrom
alpae:darwin-tests-ci
Oct 29, 2026
Merged

alpae merged 2 commits into
DessimozLab:masterfrom
alpae:darwin-tests-ci

Conversation

@alpae

@alpae alpae commented Oct 29, 2026

Copy link
Copy Markdown
Member

test_darwin.AlignTest.test_align compares the alignment scores, aligned strings and estimate_pam results with the 35,156 reference results from darwin, but it was skipped because it takes about 20 minutes.

  • By default it now runs a fixed subsample: every 20th result with an alignment (these also check align_strings and estimate_pam) and every 200th of the others. That's 335 results in about 12 s, so it runs in every CI job and in the wheel tests.
  • PYOPA_TEST_FULL=1 runs all results. Locally, the full run passes in 20 min on current master.
  • A new workflow, full-tests.yml, runs the full comparison on pushes to master, weekly (Monday 03:00 UTC) and on demand (workflow_dispatch), on Ubuntu with Python 3.14.
  • Failure messages now report the row of the reference results instead of a running counter.

Most of the time goes to the scalar reference alignment (about 65 %), which is the reference check itself, so the speedup comes from the subsample.

alpae added 2 commits October 29, 2026 22:32
test_align compared pyopa with the darwin reference results, but was
skipped as it takes about 20 minutes. Run a fixed subsample by default
(every 20th result with an alignment, which also checks estimate_pam,
and every 200th of the others; about 12 s), and all results if
PYOPA_TEST_FULL is set. Failure messages give the row of the reference
results.

A new workflow runs the full comparison on pushes to master, weekly and
on demand.
@alpae
alpae merged commit 62d6258 into DessimozLab:master Oct 29, 2026
20 checks passed
@alpae
alpae deleted the darwin-tests-ci branch October 30, 2026 06:41
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