[henrikjacobsenfys]

Problem

With analysis.plot_data_and_model(plot_residuals=True) the x axes of the data panel and the residuals panel did not line up: the residuals axis started further right than the data axis.

slicerplot_with_residuals drew the residuals in a second Matplotlib figure placed in the widget's bottom bar. plopp runs tight_layout() on each figure independently, so each axis' left margin followed its own y tick labels and y label (10⁻¹ / [dimensionless] vs 0.4 / Residuals [dimensionless]). Both Analysis and Analysis1d go through this function.

Fix

Both panels are now two axes of one Matplotlib figure (sharex=True, height ratio 2:1), passed to plopp via ax= (SlicerPlot(..., ax=main_ax) and pp.linefigure(..., ax=res_ax)). plopp supports several plopp figures sharing one Matplotlib figure and links their home buttons.

Behaviour changes:

• The axes share x limits, scale and zoom/pan, so the main plot's logX button now applies to the residuals too.
• The residuals panel hides its own logX/logY buttons: logX is shared, and a log y axis does not suit residuals, which can be negative.
• figsize now sizes the whole figure. The default (6, 6) keeps the previous 6×4 + 6×2 proportions. Before, a user figsize only resized the main panel.

Measured with the ipympl backend: before, the axes' left edges were at 77.6 px and 87.6 px; after, both span 87.6 → 585.0 px with identical x limits.

Tests

In tests/unit/easydynamics/utils/test_plotting.py:

• A mocked test checking that both panels share one figure and its x axis, that figsize is passed through (default and custom), and that the residuals' log buttons are hidden.
• A regression test that renders for real under the ipympl backend (switched in for the test, as in the integration tests) with panels whose y tick labels differ in width, and asserts identical on-screen axis edges and x limits.

The new tests fail on the old code and pass with the fix. pixi run check passes.

Fixes #269

🤖 Generated with Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.18%. Comparing base (762b081) to head (2c1033a).
⚠️ Report is 6 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #270      +/-   ##
===========================================
+ Coverage    98.15%   98.18%   +0.03%     
===========================================
  Files           60       60              
  Lines         6066     6067       +1     
  Branches      1022     1022              
===========================================
+ Hits          5954     5957       +3     
+ Misses          58       57       -1     
+ Partials        54       53       -1     

Flag	Coverage Δ
unittests	98.18% <100.00%> (+0.03%)	⬆️

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

Files with missing lines	Coverage Δ
src/easydynamics/utils/plotting.py	94.11% <100.00%> (+6.23%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.