-
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
Reference image can't be found if no Direct in same filter as Grism for NIRISS (possibly other instruments too) #190
Comments
I think I'd prefer that the user have the responsibility for assigning the direct image reference for these cases. You could do this either by simply making a symbolic link of the file with the expected filename to some other one that actually exists. Or, alternatively you can explicitly specify the direct image reference with the grizli/grizli/pipeline/auto_script.py Lines 3296 to 3324 in 7450014
|
I agree that the user should specify what the fallback is. However, I thought that was the role Grism Reference Filters dictionary. If that's not the case, perhaps we can patch in a more appropriate error warning statement? |
I've opened #210 to at least add a message to the user if the corresponding direct image can't be found. I'm curious as to why the Grism Reference Filters dictionary isn't used for deciding filters with JWST in the same way as it is with HST. Can you provide any insight into this? |
I am working on a NIRISS program with F200W direct and GR150R grism in F200W, F150W, and F115W.
During
grism_prep
, the code will fail if there are no Direct observations in the same filter as the Grism observations:This is due to the following:
https://github.com/gbrammer/grizli/blob/3b3f09da271800163443c0d09091a35a1aa6d81f/grizli/pipeline/auto_script.py#L3071C1-L3084C40
which checks if there is an observation matching both the grism and pupil information. For my program, this will be true for F200W, F150W and F115W.
https://github.com/gbrammer/grizli/blob/3b3f09da271800163443c0d09091a35a1aa6d81f/grizli/pipeline/auto_script.py#L3071C1-L3084C40
The result is that the grism looks for a detection image in each of my grism bands, which causes an error:
https://github.com/gbrammer/grizli/blob/3b3f09da271800163443c0d09091a35a1aa6d81f/grizli/pipeline/auto_script.py#L3148C36-L3148C36
The preferred behavior would be the following:
This is especially relevant for pure parallel data where the grism/direct filter combos can be a grab bag.
I don't believe this affects NIRCam after taking a cursory look, but it may have a similar affect.
The text was updated successfully, but these errors were encountered: