Skip to content

Bump SOC schemas to 2.0.0#876

Merged
PaulHuwe merged 10 commits into
spacetelescope:mainfrom
braingram:v2_with_ssc
May 11, 2026
Merged

Bump SOC schemas to 2.0.0#876
PaulHuwe merged 10 commits into
spacetelescope:mainfrom
braingram:v2_with_ssc

Conversation

@braingram

@braingram braingram commented May 11, 2026

Copy link
Copy Markdown
Collaborator

Closes #859

This has:

  • no archive changes
  • no functional changes to any schemas

Bump all SOC schema versions to 2.0.0 to provide a clear marker for pre-comissioning (<2) schemas.

Since every SSC schema references the rad metaschema (and a few also reference other SSC schemas) this also updates those schemas to point to the new 2.0.0 versions (which are functionally identical with only a version number difference).

Regression tests all pass: https://github.com/spacetelescope/RegressionTests/actions/runs/25557470637

This will conflict with:

as both of those bump the manifest to 1.10.0 (instead of 2.0.0). I suggest that we merge this one first then update those PRs (which will also be easier to review since they will contain many fewer changes).

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

codecov Bot commented May 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.03%. Comparing base (5e16410) to head (10a1d09).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #876      +/-   ##
==========================================
- Coverage   86.56%   86.03%   -0.54%     
==========================================
  Files          15       15              
  Lines        1154     1131      -23     
==========================================
- Hits          999      973      -26     
- Misses        155      158       +3     

☔ 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.

@braingram braingram marked this pull request as ready for review May 11, 2026 12:31
@mairanteodoro

Copy link
Copy Markdown
Contributor

@braingram I'll wait until this PR is merged before merging #870.

@schlafly schlafly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good to me, thanks Brett!

@schlafly

Copy link
Copy Markdown
Collaborator

As this has no archive changes, I'm fine with this going in now, but I'll let Paul give the final all clear / merge.

@PaulHuwe

Copy link
Copy Markdown
Collaborator

@ejoliet Can you please review this PR? SSC files are modified, but they changes are only linked versions of:
rad_schema, wfi_detector, wfi_optical_element - which are identical to what presently exists, only with updated version numbers.

@PaulHuwe PaulHuwe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great. After SSC approves and #868 is merged, this should go in. Then we can make new, simple versions of #870 and #875.

@ejoliet ejoliet self-requested a review May 11, 2026 22:51

@ejoliet ejoliet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should be fine for SSC. THanks.

@PaulHuwe PaulHuwe merged commit d07e999 into spacetelescope:main May 11, 2026
21 of 22 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.

Bump major version of schemas for each phase of flight

5 participants