Skip to content

Overhaul plot styling and extract fits_* output functions#308

Merged
Jammy2211 merged 2 commits intomainfrom
feature/plot-improvements
Mar 25, 2026
Merged

Overhaul plot styling and extract fits_* output functions#308
Jammy2211 merged 2 commits intomainfrom
feature/plot-improvements

Conversation

@Jammy2211
Copy link
Copy Markdown
Collaborator

Summary

  • _resolve_colormap delegates to _default_colormap() so the autoarray segmentdata colormap is used everywhere by default
  • subplot_galaxies: hide_unused_axes() fixes blank 6th panel (5 panels in a 2×3 grid)
  • save_galaxy_images_fits() extracted to galaxies_plots.py
  • save_adapt_images_fits() extracted to adapt_plots.py
  • plotter.py: fits_* blocks replaced with one-liner delegate calls; unused imports removed
  • Inversion subplot_of_mapper() replaces manual scatter/colorbar code in plotter.py

Test plan

  • Verify subplot_galaxies no longer shows a blank 6th panel
  • Check galaxy_images.fits and adapt_images.fits still written correctly
  • Confirm autoarray colormap applied to all galaxy/adapt plots

🤖 Generated with Claude Code

Jammy2211 and others added 2 commits March 25, 2026 12:05
- _resolve_colormap now delegates to _default_colormap() (autoarray)
  so the segmentdata colormap is used everywhere by default
- subplot_galaxies: hide_unused_axes() fixes blank 6th panel
- save_galaxy_images_fits() extracted to galaxies_plots.py
- save_adapt_images_fits() extracted to adapt_plots.py
- plotter.py: fits_* blocks replaced with one-liner delegate calls;
  unused imports (numpy, hdu_list_for_output_from) removed
- inversion subplot_of_mapper() replaces manual scatter/colorbar code

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