8000 Allow skymatch/skysub for one image by mcara · Pull Request #5423 · spacetelescope/jwst · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 3 commits into from
Oct 23, 2020

Conversation

mcara
Copy link
Member
@mcara mcara commented Oct 22, 2020

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.

@codecov
Copy link
codecov bot commented Oct 22, 2020

Codecov Report

Merging #5423 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            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              
Flag Coverage Δ
#unit 52.37% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
jwst/pipeline/calwebb_image3.py 32.55% <0.00%> (-2.45%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ef0894d...4dcb90a. Read the comment docs.

@hbushouse hbushouse added this to the Build 7.7 milestone Oct 22, 2020
8000
Copy link
Collaborator
@hbushouse hbushouse left a 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
Copy link
Collaborator

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.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed this

@hbushouse hbushouse merged commit 656b7ab into spacetelescope:master Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0