Skip to content

odl-bench: per-release benchmark job with regression gate (#960) - #3

Open
jasperdew wants to merge 2 commits into
mainfrom
feat/odl-bench-job-960-v2
Open

jasperdew wants to merge 2 commits into
mainfrom
feat/odl-bench-job-960-v2

Conversation

@jasperdew

Copy link
Copy Markdown
Member

Refs pdfluent-internal#960, #969, #968. Replaces #2 (closed unmerged).

python -m bench.odl plus odl-bench/: versioned recipes (printed into every result), manifest hashes of the 200 PDFs and ground-truth files at the pinned commit (corpus cloned at run time, never copied), scoreless tune/holdout split (split-v1.json, with its SHA-256), per-metric scores per half, baseline gate with tolerance 0.005 and canaries, SKIPPED (not a pass) (exit 3) when models are absent, tests in CI, plus the contamination matcher (#968).

Baselines and the full class labels (#969) carry numbers and stay private: the engine CI passes --baseline and --labels, and runs the labels checks with ODL_LABELS.

Known and not from this PR: tests/test_workflows.py has two failures on main since the identity-check workflow landed (not self-hosted); tests/test_round.py has four failures that need the docker images.
Not done: competitor rerun with paired bootstrap, the 19-axes holdout call, the engine-side CI job.

…ression gate

Recipes (versioned, printed into every result), manifest hashes of the 200
PDFs and ground-truth files, a scoreless tune/holdout split file, per-metric
scores per half, a baseline gate (tolerance 0.005 and per-document canaries),
a loud skip when the models are absent, and the tests for all of it, run in CI.
The full class labels (#969) stay private and are passed with --labels.
Refs pdfluent-internal#960 and #969.

Signed-off-by: jasperdew <10383561+jasperdew@users.noreply.github.com>
…P-Bench fingerprints (#968)

Signed-off-by: jasperdew <10383561+jasperdew@users.noreply.github.com>
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