Skip to content

Astro cal header cleanup #712#750

Merged
semaphoreP merged 3 commits intodevelopfrom
Astro-cal-header-cleanup-#712
Feb 16, 2026
Merged

Astro cal header cleanup #712#750
semaphoreP merged 3 commits intodevelopfrom
Astro-cal-header-cleanup-#712

Conversation

@aneeshb97
Copy link
Contributor

@aneeshb97 aneeshb97 commented Feb 14, 2026

Describe your changes

Type of change

Please delete options that are not relevant (and this line).

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Reference any relevant issues (don't forget the #)

Checklist before requesting a review

  • I have verified that all unit tests pass in a clean environment and added new unit tests, as appropriate
  • I have checked that I am merging into the right branch
  • I have checked the output of the latest Github Actions run associated with this PR and confirmed running pytest did not produce any warnings
  • I have checked if my code modifies an existing step function or modifies the data format docs. If it does, I've added my PR to the list maintained here.

Header cleanup done separately for pri_hdr and ext_hdr as SATSPOTS/IFSHOWFSC keywords present in both need to deleted from pri_hdr and invalidated in ext_hdr
Copy link
Contributor

@semaphoreP semaphoreP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@semaphoreP semaphoreP merged commit 07e4067 into develop Feb 16, 2026
1 check passed
@semaphoreP semaphoreP deleted the Astro-cal-header-cleanup-#712 branch February 16, 2026 20:39
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.

2 participants