Skip to content

Visualization cleanup#411

Merged
Jammy2211 merged 2 commits intomainfrom
feature/visualization_clean_up
Apr 2, 2026
Merged

Visualization cleanup#411
Jammy2211 merged 2 commits intomainfrom
feature/visualization_clean_up

Conversation

@Jammy2211
Copy link
Copy Markdown
Collaborator

Summary

  • Critical curves plotted once, interferometer/point plotter passdown
  • Plotter and visualizer improvements across imaging, interferometer, and point models
  • Fix JAX array leak from PointSolver

Test plan

  • Verify visualization output renders correctly

🤖 Generated with Claude Code

Jammy2211 and others added 2 commits April 2, 2026 10:43
…sdown

- Lift critical curve computation to visualizer level in interferometer and point
  so it is computed once and passed through the plotter call chain
- Add image_plane_lines/source_plane_lines parameters to interferometer subplot_fit,
  subplot_fit_dirty_images, subplot_fit_real_space and point subplot_fit
- Thread critical curve parameters through PlotterInterferometer.fit_interferometer,
  Plotter.tracer, PlotterPoint.fit_point and VisualizerPoint.visualize
- Sync fit_imaging_plots with interferometer equivalent for consistency

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Jammy2211 Jammy2211 merged commit 5beb6b9 into main Apr 2, 2026
8 checks passed
@Jammy2211 Jammy2211 deleted the feature/visualization_clean_up branch April 2, 2026 11:30
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