-
Notifications
You must be signed in to change notification settings - Fork 180
JP-3859: Style updates for flat_field #9464
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-3859: Style updates for flat_field #9464
Conversation
|
Regtests here: https://github.com/spacetelescope/RegressionTests/actions/runs/15031169793 Tests are passing. |
emolter
left a comment
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.
Looks great Melanie, thanks! I left a few small comments but TBH despite the +-850 code lines I was unable to find much, including nitpicks
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9464 +/- ##
==========================================
+ Coverage 77.80% 77.90% +0.10%
==========================================
Files 363 362 -1
Lines 36388 36334 -54
==========================================
- Hits 28311 28307 -4
+ Misses 8077 8027 -50 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Resolves JP-3859
Style and documentation fixes for the flat field step.
Tasks
Build 12.0(use the latest build if not sure)no-changelog-entry-needed)changes/:echo "changed something" > changes/<PR#>.<changetype>.rst(see changelog readme for instructions)docs/pageokify_regteststo update the truth files