[WAVE 1] Test cost printing across uneven multiphase states - #1076
Conversation
|
Wave 1 preparation update: this PR remains intentionally isolated. Full CI is waiting behind the shared Matplotlib prerequisite #1081; it will be marked ready after that lands and checks are rerun. |
cf5b33d to
31527d0
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1076 +/- ##
=======================================
Coverage 77.26% 77.26%
=======================================
Files 196 196
Lines 21506 21506
=======================================
Hits 16616 16616
Misses 4890 4890
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
pariterre
left a comment
There was a problem hiding this comment.
@pariterre reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on mickaelbegon).
Codex analysisCI follow-up after commit 89c83a3:
No code change was required for the stale GitHub check metadata. |
Summary
Solution.print_cost()on a multiphase OCP whose phases have different state dimensionsSolutionDatabehavior against the regression reported in Error sol.print_cost() #712The faulty key-based concatenation described in #712 has already disappeared in the current implementation. This focused regression test documents and protects the corrected behavior.
Validation
pytest tests/shard5/test_global_torque_driven_ocp.py::test_phase_transition_uneven_variable_number_by_mapping -qThis change is