-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RCAL-804: Split cal_step into L2 & L3 #334
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #334 +/- ##
==========================================
- Coverage 97.56% 97.33% -0.23%
==========================================
Files 30 31 +1
Lines 2788 2965 +177
==========================================
+ Hits 2720 2886 +166
- Misses 68 79 +11 ☔ View full report in Codecov by Sentry. |
for more information, see https://pre-commit.ci
NOTE: these changes require updates to romancal regression test files as shown in the Jenkins URL. |
…oman_datamodels into RCAL-804_SplitCalStep
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The latest changes in RAD are related to this PR. Since merging this will point roman_datamodels back to rad/main I'm going to go ahead and merge it so I can get romancal installable again. |
Resolves RCAL-804
Closes #
This PR splits the cal_step datamodels into L2 and L3 versions.
Checklist
CHANGES.rst
under the corresponding subsection