spacetelescope / roman_datamodels

Datamodel support for the roman calibration pipeline
https://roman-datamodels.readthedocs.io
Other
7 stars 21 forks source link

RCAL-804: Split cal_step into L2 & L3 #334

Closed PaulHuwe closed 6 months ago

PaulHuwe commented 6 months ago

Resolves RCAL-804

Closes #

This PR splits the cal_step datamodels into L2 and L3 versions.

Checklist

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 97.33%. Comparing base (087a60d) to head (b54fe35). Report is 10 commits behind head on main.

Additional details and impacted files ```diff @@ 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 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

PaulHuwe commented 6 months ago

NOTE: these changes require updates to romancal regression test files as shown in the Jenkins URL.

nden commented 6 months ago

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.