-
Notifications
You must be signed in to change notification settings - Fork 25
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
JP-3519: Add noutputs and bunit to readnoise and superbias schemas #250
JP-3519: Add noutputs and bunit to readnoise and superbias schemas #250
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #250 +/- ##
=======================================
Coverage 64.84% 64.84%
=======================================
Files 103 103
Lines 5694 5694
=======================================
Hits 3692 3692
Misses 2002 2002 ☔ View full report in Codecov by Sentry. |
I think the errors should be fixed by #249 Would you rebase this PR and fix the changes conflict? Otherwise looks good to me! |
a8888dd
to
cf23293
Compare
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! (devdeps are known failures due to numpy 2.0)
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
Resolves JP-3519
Closes spacetelescope/jwst#8217
This PR addresses a request to add keyword schemas for noutputs and bunit to the readnoise and superbias schemas, due to CRDS selection criteria updates for these reference files.
Checklist
CHANGES.rst
(either inBug Fixes
orChanges to API
)