[WAVE 2][RTR] Fix linear bounds rendering in plots - #1065
Conversation
a45e9b7 to
c7b9355
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1065 +/- ##
==========================================
+ Coverage 77.26% 77.86% +0.60%
==========================================
Files 196 196
Lines 21506 21506
==========================================
+ Hits 16616 16746 +130
+ Misses 4890 4760 -130
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:
|
Codex analysisThis PR is part of Wave 2 and is rebased onto current Review isolation: it changes only Validation completed by Codex:
The final commit ( @EveCharbie @pariterre, all local and online checks are green. This PR is ready for review as an independent Wave 2 change. |
Summary
InterpolationType.LINEARbounds with regular lines instead of post-stepsRoot cause
Bounds were always sent through
Axes.step(..., where="post"). This changes linear bounds into piecewise-constant values between shooting nodes, so they do not match integrated state trajectories such as collocation plots.Validation
MPLCONFIGDIR=/private/tmp/bioptim-mpl-973 python -m pytest tests/shard1/test_plot.py -q(6 passed)git diff --checkCloses #973
This change is