Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #354 +/- ##
==========================================
+ Coverage 79.55% 79.73% +0.18%
==========================================
Files 17 17
Lines 1956 1984 +28
Branches 331 335 +4
==========================================
+ Hits 1556 1582 +26
- Misses 279 280 +1
- Partials 121 122 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
5cb9844 to
230527c
Compare
|
|
|
FAILED tests/test_babs_workflow.py::test_babs_init_raw_bids[session] - Failed: Timeout >300.0s But that one seems flakey, seen it time out a handful of times before. |
|
|
14092b8 to
7b01181
Compare
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Polls until all submitted jobs complete or fail, then exits. Exit code 0 if all succeeded, 1 if any failed or none submitted, 130 on Ctrl-C. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
After rebasing on the modified |
|
hit 2 smallish issues, updated description with TODO |
Add
--waitflag tobabs statusProblem
babs statusis a snapshot — it prints the current state and exits.Automation needs to wait for jobs to finish before running
babs merge, so end-to-end users must write wrapper scripts that pollsqueue.Workarounds exist, such as the walkthrough-tests.sh using an squeue loop, but users shouldn't have to implement that themselves, and that approach doesn't update
job_status.csv.Changes
Add
--waitand--wait-intervalflags tobabs status:Behavior:
babs status)Test changes
babs_status_waitcovering success, failure, mixed results, polling loop, no-submitted-jobs, report-each-iteration, and Ctrl-C handlingbabs status --waitinstead of manualsqueuepolling loopsjob_status.csvafterbabs merge— every submitted job must havehas_results=Trueandis_failed=FalseHousekeeping
pytest-timeoutfrom conda environment files topyproject.toml(separate commit)TODO:
Test plan
pytest tests/test_interaction.py)