Skip to content

Plot improvements: arcsec tick labels, circular import fix, test imports#241

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

Plot improvements: arcsec tick labels, circular import fix, test imports#241
Jammy2211 merged 1 commit intomainfrom
feature/plot-improvements-3

Conversation

@Jammy2211
Copy link
Copy Markdown
Collaborator

Summary

  • Restore " arcsec suffix on tick labels in _arcsec_labels (matches old with_appended_suffix behaviour: strips .0 when all labels end in .0, then appends ")
  • Fix circular import caused by adding dataset plot functions to autoarray/plot/__init__.py; tests now import subplot_imaging_dataset / subplot_interferometer_dataset / subplot_interferometer_dirty_images directly from their submodules
  • Stage modified test files for dataset and fit plotters

Test plan

  • python -m pytest test_autoarray/dataset/plot/ passes
  • python -m pytest test_autoarray/fit/plot/ passes
  • No circular import errors on import autoarray

🤖 Generated with Claude Code

… import, export dataset plot functions

- Restore `"` arcsec suffix on tick labels in `_arcsec_labels` (strips .0 suffix
  when all labels end in .0, matching old `with_appended_suffix` behaviour)
- Fix circular import: revert `autoarray/plot/__init__.py` additions that caused
  `ImportError` via `dataset/plot/imaging_plots` → `plot/utils` → `__init__` cycle
- Update dataset plot tests to import `subplot_imaging_dataset` /
  `subplot_interferometer_dataset` / `subplot_interferometer_dirty_images`
  directly from their submodules instead of via `aplt.*`

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