Skip to content

Include source_id and variant_label in batch report - #605

Merged
rbeucher merged 1 commit into
mainfrom
add-source-id-variant-label-to-batch-report
Aug 10, 2026
Merged

Include source_id and variant_label in batch report#605
rbeucher merged 1 commit into
mainfrom
add-source-id-variant-label-to-batch-report

Conversation

@rbeucher

Copy link
Copy Markdown
Member

Summary

  • Add source_id and variant_label to the top-level moppy_batch_report.json output, sourced from batch_config.yml the same way experiment_id already is.
  • This lets downstream tooling (e.g. the access-moppy-progress dashboard registry's manage-submission command) resolve model/member identifiers directly from the report instead of requiring them to be typed manually on every submission.

Test plan

  • pixi run -e test pytest tests/unit/test_batch_report.py -q — 27 passed

Downstream tooling (access-moppy-progress) needs model and member
identifiers to file a batch report without requiring them to be typed
manually, mirroring the existing experiment_id field.
@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.1%. Comparing base (16227e6) to head (3590fff).

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #605   +/-   ##
=====================================
  Coverage   75.1%   75.1%           
=====================================
  Files         38      38           
  Lines       8070    8070           
  Branches    1517    1517           
=====================================
  Hits        6061    6061           
  Misses      1688    1688           
  Partials     321     321           
Flag Coverage Δ
unit 75.1% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rbeucher
rbeucher merged commit 2b0d546 into main Aug 10, 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.

1 participant