Skip to content
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

Add "nrcalong"/"nrcblong" to detector_list #922

Open
eteq opened this issue Oct 28, 2024 · 0 comments
Open

Add "nrcalong"/"nrcblong" to detector_list #922

eteq opened this issue Oct 28, 2024 · 0 comments

Comments

@eteq
Copy link
Contributor

eteq commented Oct 28, 2024

I was about to create an issue asking to add nrcalong/nrcblong to the valid detectors... but on a lark I tried nircam.detector = 'nrcalong' and was surprised it worked! The reason I was surprised is that nircam.detector_list() yields:

['NRCA1',
 'NRCA2',
 'NRCA3',
 'NRCA4',
 'NRCA5',
 'NRCB1',
 'NRCB2',
 'NRCB3',
 'NRCB4',
 'NRCB5']

And it took me quite a while to realize 5 = long. Given the JWST data products use the *long naming scheme, I suggest either 1) addint nrcalong/nrcblongto the list, or just change the "real" name fromnrc?5tonrc?long`.

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

No branches or pull requests

1 participant