-
Notifications
You must be signed in to change notification settings - Fork 4
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
Mosaic, WCS, skymatch #12
Comments
@nils-erik I see that the WCS keywords are set in the header, Can you please crosscheck? |
Hi Andrea,
They show up for me.
[image: Screen Shot 2020-02-18 at 2.56.52 PM.png]
…On Tue, Feb 18, 2020 at 12:53 PM Andrea Zonca ***@***.***> wrote:
@nils-erik <https://github.com/nils-erik> I see that the WCS keywords are
set in the header,
however they do not display with DS9:
[image: image]
<https://user-images.githubusercontent.com/383090/74777008-8412b780-524d-11ea-929c-10ac17f4f3f0.png>
Can you please crosscheck?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#12>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJPG6OLZC5LZVTL3644HTPDRDRDETANCNFSM4J3UYEPA>
.
|
can you post the image directly to Github? attachments to emails are not supported. |
I tested with a HST image and WCS worked fine, so I guess it is something wrong in the file. |
it is most probably related to the version of DS9, WCS works fine in DS9 7.5, but does not work on 8.1. |
JWST uses the drizzle algorithm to combine dithered images or mosaics, Drizzle parameters are defined in the CRDS, see the schema @ikashell is it fine if we use the same algorithm and adapt their pipeline to IRIS? |
I discovered the issue resulting in WCS not displaying in version 8 of ds9. In earlier versions, any invalid algorithm code for the CTYPE keyword in the header defaulted to a linear projection. In version 8 it appears that linear projections require a blank algorithm code (as was always true for spectral axes). |
@nils-erik |
@arunsurya77 do you mind to test one of the files on Galactica in:
can you load it into |
Hi Andrea, |
from @ikashell:
|
|
WCS with
|
@nils-erik I discovered that I need to create a "Generalized WCS" using the
Full names of the projections are available at https://docs.astropy.org/en/stable/_modules/astropy/modeling/projections.html Do you know if |
I fixed this just assuming the projection was TAN instead of LINEAR. |
here is the fixes I had to do to the input FITS to be able to use them with |
ok, resample is running to completion on the first image and gives InputOutput@arunsurya77 do you think this looks reasonable? |
From Nils:
The text was updated successfully, but these errors were encountered: