Skip to content

Name expected world objects in crop component-count error#939

Open
nabobalis wants to merge 1 commit into
sunpy:mainfrom
nabobalis:crop-error-message
Open

Name expected world objects in crop component-count error#939
nabobalis wants to merge 1 commit into
sunpy:mainfrom
nabobalis:crop-error-message

Conversation

@nabobalis

Copy link
Copy Markdown
Member

PR Description

When a crop point has the wrong number of components, the error now lists the expected world objects and their order, so users migrating from a WCS with fewer world objects learn the fix from the error itself.

This was very frustrating to not know what I need to pass in.

AI Assistance Disclosure

AI tools were used for:

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding
  • No AI tools were used

Regardless of AI use, the human contributor remains fully responsible for correctness, design choices, licensing compatibility, and long-term maintainability.

When a crop point has the wrong number of components, the error now
lists the expected world objects and their order, so users migrating
from a WCS with fewer world objects learn the fix from the error itself.
@nabobalis nabobalis force-pushed the crop-error-message branch from 58452dd to 4559dd5 Compare June 11, 2026 21:41
@nabobalis nabobalis marked this pull request as ready for review June 11, 2026 21:41
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