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

Update language for ND filters #22

Open
rzellem opened this issue Sep 12, 2024 · 3 comments
Open

Update language for ND filters #22

rzellem opened this issue Sep 12, 2024 · 3 comments
Assignees
Labels
documentation Improvements or additions to documentation update needed Good for newcomers

Comments

@rzellem
Copy link
Collaborator

rzellem commented Sep 12, 2024

from @sepito :

The only recommendation is to add some description for the neutral density filters (IPAC website has for example "Attenuation factor of Neutral Density filter used for target acquisition (ND 2.25) mounted in FPAM" etc.

@rzellem rzellem added documentation Improvements or additions to documentation update needed Good for newcomers labels Sep 12, 2024
@rzellem rzellem self-assigned this Sep 12, 2024
@rzellem
Copy link
Collaborator Author

rzellem commented Nov 8, 2024

Sounds good. I can add that language. As a heads up, the FSAM is not used for target acquisition (I believe that I'm using that for absolute flux calibrations...but I cannot confirm right now as I cannot get on to the JPL VPN). @sepito - would you mind removing the text about the FSAM ND4.75 filter being used for acquisition from the IPAC website, please?

@sepito
Copy link
Collaborator

sepito commented Nov 8, 2024

So the parameter to be removed is CGI_Bandpass_ND_4pt75_FSAM? I can do that but I think we'd prefer to make all these parameter updates at once as it is a somewhat involved process...so can this wait until I have confirmation on the questions that I submitted to you and Tom on GitHub? I'm curious also if Jeff Kruk is OK with our description language for those prism spectral continuum sensitivity parameters. Thanks!

@rzellem
Copy link
Collaborator Author

rzellem commented Nov 8, 2024

Happy for you to wait - I need to also verify/update the CGI parameters as well. I'll be doing that over the next few days and get back to you.

I haven't heard back from Jeff yet about our labeling question.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation update needed Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants