-
Notifications
You must be signed in to change notification settings - Fork 52
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
Propagate WCS alignment to gwcs
keywords
#81
Comments
This is implemented in jwst_utils.match_gwcs_to_sip, but isn't yet run in the pipeline. |
You can also use the https://tweakwcs.readthedocs.io/en/latest/index.html The |
Thanks @jdavies-st. I agree, but the reason to implement things this way in grizli with the SIP headers is so that I can make consistent WCS between HST and JWST instruments and use the same (old) low-level |
Cool. This might be something you can outsource to Or if you don't like the SIP fitting or tweaking currently in Btw, if you find that your SIP fitting works better (or is more reliable) than that in the JWST pipeline, then you may want to push those changes or fixes upstream to |
Agreed. I'll look into the |
The alignment scripts in
prep
now appear to work for the SIP headers generated for JWST exposures, but the alignment updates (shifts, rotation, scale) aren't propagated to the keywords that are read when running theassignWCS
pipeline step. These seem to beRA_REF
,DEC_REF
andROLL_REF / PA_V3
in theSCI
extension of the FITS files. It's not necessarily true that theREF
position is the same as the SIPCRPIX
pixel, so the shifts and rotation derived to align the SIP headers may not necessarily be the same forgwcs
, depending on the order of operations.The text was updated successfully, but these errors were encountered: