Skip to content

Report the earliest and latest times in SpectrogramSequence.__str__ - #331

Open
nabobalis wants to merge 2 commits into
sunpy:mainfrom
nabobalis:fix-sequence-time-range
Open

nabobalis wants to merge 2 commits into
sunpy:mainfrom
nabobalis:fix-sequence-time-range

Conversation

@nabobalis

Copy link
Copy Markdown
Member

PR Description

The time range used the first time of the first cube and the last time of the last cube, so a sequence whose time runs backwards along an axis (for example an IRIS v34 raster) printed its range with the start after the end.

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.

K

The time range used the first time of the first cube and the last time of the
last cube, so a sequence whose time runs backwards along an axis (for example
an IRIS v34 raster) printed its range with the start after the end.
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