Skip to content

Plot improvements: line_colors, is_subplot colorbar sizing, inversion panels#242

Merged
Jammy2211 merged 1 commit intomainfrom
feature/plot-improvements-4
Mar 26, 2026
Merged

Plot improvements: line_colors, is_subplot colorbar sizing, inversion panels#242
Jammy2211 merged 1 commit intomainfrom
feature/plot-improvements-4

Conversation

@Jammy2211
Copy link
Copy Markdown
Collaborator

Summary

  • plot_array: add line_colors parameter to draw tangential/radial critical curves and caustics in distinct colours (black/white)
  • plot_inversion_reconstruction: add line_colors and is_subplot parameters; thread is_subplot to _apply_colorbar for correct subplot tick-label sizing; apply box_aspect from zoomed extent to fix rectangular mesh colorbar size
  • _plot_rectangular / _plot_delaunay: accept is_subplot and pass to _apply_colorbar
  • plot_mapper: add line_colors parameter and pass through to plot_inversion_reconstruction

Test plan

  • Run a rectangular pixelization fit and verify the source reconstruction colorbar is correctly sized in subplots
  • Verify critical curves (black=tangential, white=radial) appear on image-plane panels
  • Verify caustics appear on source-plane inversion reconstruction panels

🤖 Generated with Claude Code

…nversion panels

- plot_array: add line_colors parameter to draw tangential/radial curves in distinct colours
- plot_inversion_reconstruction: add line_colors and is_subplot parameters; thread is_subplot to _apply_colorbar for correct subplot tick label sizing; apply box_aspect from zoomed extent
- _plot_rectangular / _plot_delaunay: accept is_subplot and pass to _apply_colorbar
- plot_mapper: add line_colors parameter and pass through to plot_inversion_reconstruction

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Jammy2211 Jammy2211 merged commit e432f30 into main Mar 26, 2026
0 of 8 checks passed
@Jammy2211 Jammy2211 deleted the feature/plot-improvements-4 branch March 26, 2026 20:23
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