8000 MOS background sources are assigned lower left source position · Issue #8459 · spacetelescope/jwst · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

MOS background sources are assigned lower left source position #8459

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

Closed
stscijgbot-jp opened this issue May 6, 2024 · 3 comments · Fixed by #8461
Closed

MOS background sources are assigned lower left source position #8459

stscijgbot-jp opened this issue May 6, 2024 · 3 comments · Fixed by #8461

Comments

@stscijgbot-jp
Copy link
Collaborator

Issue JP-3612 was created on JIRA by Melanie Clarke:

In assign_wcs for NIRSpec MOS observations, for background-only slits, any input x and y position for the source is ignored ('estimated_source_in_shutter_x', 'estimated_source_in_shutter_y' in the MSA file).  The 'source_xpos' and 'source_ypos' values for the slit are set to 0.0 (line 665-666, jwst/assign_wcs/nirspec.py).  However, these values are later corrected by -0.5, to account for the difference between the MSA coordinate origin and the model coordinate origin (line 724-725, jwst/assign_wcs/nirspec.py).  This puts the background source location at -0.5, -0.5, which is the lower left corner of the slit.

I think it was probably intended to put the source position for the background at the center of the slit, not the lower left corner.  Line 665-666 should be corrected to set the background slit source position to 0.5, 0.5.  This will impact the point source pathloss correction, computed and stored by default for every slit, but should not otherwise impact processing for background slits, since they are always treated as uniform sources.

@stscijgbot-jp
Copy link
Collaborator Author

Comment by Howard Bushouse on JIRA:

Fixed by #8461

@stscijgbot-jp
Copy link
Collaborator Author

Comment by David Law on JIRA:

Christian Hayes Can you please take a look at this for the NIRSpec team and close it out?

@stscijgbot-jp
Copy link
Collaborator Author

Comment by Christian Hayes on JIRA:

The change looks good to me, closing this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant
0