-
Notifications
You must be signed in to change notification settings - Fork 12
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
Search only returns results for NIRCam #50
Comments
Thanks @jotaylor. I think I see what's going on here. Bad logic for managing parameter values between defaults, command line inputs, and config file inputs. It looks like the instrument name is set to niriss from the config file, but is being set back to the default value of nircam later. I'll work on straightening it all out. |
This fixed the issue, thanks Bryan! |
I noticed it's now printing many lines that look like: |
Interesting. I didn't see that in any of my tests. I'll try with your query above on Monday and see if I can find a way to suppress those messages. |
I have cloned and installed the latest version of the repo, but when I supply a cfg file (attached) that specifies
instrument: niriss
, the code tries to retrieve nircam data. The output I get is below. I even tried a different cfg file that I know has worked in the past.jwst_query.txt
The text was updated successfully, but these errors were encountered: