-
Notifications
You must be signed in to change notification settings - Fork 174
Allow skymatch/skysub for one image #5423
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5423 +/- ##
==========================================
- Coverage 52.37% 52.37% -0.01%
==========================================
Files 414 414
Lines 37707 37710 +3
Branches 5838 5839 +1
==========================================
Hits 19750 19750
- Misses 16697 16700 +3
Partials 1260 1260
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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 reasonable, but just a couple questions and comments.
CHANGES.rst
Outdated
@@ -414,6 +414,9 @@ pipeline | |||
|
|||
- Enable NIRSpec lamp processing in calspec2 pipeline. [#5267] | |||
|
|||
- Update ``Image3Pipeline`` to allow sky subtraction when input contains |
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.
It looks like this got added down in the section for one of the previous builds. It should be in the section at the top, for release 0.17.2. Just add a section for "pipeline" between "outlier_detection" and "ramp_fitting" and move this entry up there.
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.
Fixed this
This PR allows
skymatch
step to run even for a single input image. This essentially will become simple sky background subtraction without matching. Also see helpdesk issue INC0159018.