Skip to content

RAD-248: Add wfi_parallel to visit section#800

Merged
PaulHuwe merged 4 commits into
spacetelescope:mainfrom
PaulHuwe:RAD-248_WFIParallel
Jan 21, 2026
Merged

RAD-248: Add wfi_parallel to visit section#800
PaulHuwe merged 4 commits into
spacetelescope:mainfrom
PaulHuwe:RAD-248_WFIParallel

Conversation

@PaulHuwe
Copy link
Copy Markdown
Collaborator

@PaulHuwe PaulHuwe commented Jan 15, 2026

Resolves RAD-248

Closes #748

This PR adds the wfi_parallel keyword block to the observation schema.

Tasks

  • Update or add relevant rad tests.
  • Update relevant docstrings and / or docs/ page.
  • Does this PR change any schema files?
    • Schema changes were discussed at RAD Review Board meeting.
  • Does this PR change any API used downstream? (If not, label with no-changelog-entry-needed.)
News fragment change types:
  • changes/<PR#>.feature.rst: new feature
  • changes/<PR#>.bugfix.rst: fixes an issue
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.removal.rst: deprecation or removal of public API
  • changes/<PR#>.misc.rst: infrastructure or miscellaneous change

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.36%. Comparing base (e3247f3) to head (90ced94).
⚠️ Report is 76 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #800   +/-   ##
=======================================
  Coverage   86.36%   86.36%           
=======================================
  Files          15       15           
  Lines        1137     1137           
=======================================
  Hits          982      982           
  Misses        155      155           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Collaborator

@jbrookens jbrookens left a comment

Choose a reason for hiding this comment

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

No changes for archive.

@schlafly
Copy link
Copy Markdown
Collaborator

For the romancal migration of wfi_parallel, you'll need this:

visit_file_sequence:
title: Visit File Sequence
description: |
The visit file sequence identifies the sequence within
the visit file group. A value of 1 indicates a prime instrument
exposure, and a value greater than 1 indicates a parallel
instrument exposure. The allowed range of visit file sequence
numbers of 1 to 5 inclusive.

From those docs, you can figure out if the observation is in a parallel mode on the basis of that 's' digit; >1 means parallel.

@PaulHuwe PaulHuwe marked this pull request as ready for review January 16, 2026 01:47
@PaulHuwe PaulHuwe requested review from a team and WilliamJamieson as code owners January 16, 2026 01:47
@PaulHuwe PaulHuwe merged commit a0900d7 into spacetelescope:main Jan 21, 2026
28 of 29 checks passed
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.

CLONE - Add wfi_parallel to visit section for B20

3 participants