Skip to content

Run-herg-qc output, not for merging just for testing#112

Closed
MichaelClerx wants to merge 7 commits intomainfrom
compare-output-org
Closed

Run-herg-qc output, not for merging just for testing#112
MichaelClerx wants to merge 7 commits intomainfrom
compare-output-org

Conversation

@MichaelClerx
Copy link
Member

For well A03, wells A01+A02+A03, and for all wells in the test data

Will be using to check initial rewrites don't change run_herg_qc output, then deleting

On my laptop, the full run takes

real	45m13.673s
user	62m14.865s
sys	0m14.662s

with 1 cpu

@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.73%. Comparing base (36d3eac) to head (1334aee).
⚠️ Report is 25 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   91.73%   91.73%           
=======================================
  Files          11       11           
  Lines         871      871           
=======================================
  Hits          799      799           
  Misses         72       72           

☔ View full report in Codecov by Sentry.
📢 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.

@MichaelClerx
Copy link
Member Author

A reasonable test set might be A03 (full pass), A20+D16 (a full* set of failures)

No cells in the test data fail QC.Erev.all_protocols, QC.Erev.spread, QC1.all_protocols, QC4.all_protocols, QC6.all_protocols

@MichaelClerx
Copy link
Member Author

Time for 3 cells is 1m45s

@MichaelClerx MichaelClerx deleted the compare-output-org branch November 12, 2025 18:17
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