Skip to content

roman/epsf-hook#1144

Open
alphasentaurii wants to merge 3 commits into
spacetelescope:mainfrom
alphasentaurii:CCD-1614-epsf-header-precondition
Open

roman/epsf-hook#1144
alphasentaurii wants to merge 3 commits into
spacetelescope:mainfrom
alphasentaurii:CCD-1614-epsf-header-precondition

Conversation

@alphasentaurii
Copy link
Copy Markdown
Collaborator

@alphasentaurii alphasentaurii commented Jul 1, 2025

Resolves CCD-1614

This PR adds a precondition header hook for Level 3 data model header time specifications not recognized by CRDS. The hook replaces MJD Mean Time keyword (looping over list of possible keys) with standard ROMAN.META.EXPOSURE.START_TIME used by Level 2 files.

Tasks

  • update or add relevant tests
  • update relevant docstrings and / or docs/ page
  • Does this PR change any API used downstream? (if not, label with no-changelog-entry-needed)
    • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
news fragment change types...
  • changes/<PR#>.hst.rst: HST reference files
  • changes/<PR#>.jwst.rst: JWST reference files
  • changes/<PR#>.roman.rst: Roman reference files
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.testing.rst: change to tests or test automation
  • changes/<PR#>.general.rst: infrastructure or miscellaneous change

@alphasentaurii
Copy link
Copy Markdown
Collaborator Author

DO NOT MERGE - this PR is for historical tracking purposes only (header keys will be preconditioned by roman cal instead)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant