Skip to content

Conversation

@r3kste
Copy link
Collaborator

@r3kste r3kste commented Oct 29, 2025

Summary of changes

  1. Refactored slice_plot by separating data extraction logic from plotting code.
  2. Added new parameters for range and share_axes, and vectorized labels in grid_plot.
  3. Introduced MarkerData dataclass.

Plot Changes

Before After
oldplot newplot

The only change is with the horizontal and vertical spacing.

Backend Comparision

Plotly Matplotlib
oldplot slice_plot

PR Checklist

  • Compared images before and after making changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/optimagic/visualization/backends.py 99.00% <100.00%> (+0.21%) ⬆️
src/optimagic/visualization/convergence_plot.py 95.45% <ø> (ø)
src/optimagic/visualization/plotting_utilities.py 52.50% <100.00%> (-40.97%) ⬇️
src/optimagic/visualization/slice_plot.py 96.77% <100.00%> (+1.82%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@r3kste r3kste force-pushed the slice_plot_refactor branch from f6c8e5d to aa76f21 Compare October 30, 2025 19:19
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@r3kste r3kste force-pushed the slice_plot_refactor branch from aa76f21 to 9ec8b9a Compare October 30, 2025 19:56
@r3kste r3kste requested a review from timmens October 30, 2025 20:14
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