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

Allow the atmospheric transmission curves to vary by airmass #179

Closed
cwwalter opened this issue Oct 15, 2018 · 4 comments
Closed

Allow the atmospheric transmission curves to vary by airmass #179

cwwalter opened this issue Oct 15, 2018 · 4 comments
Assignees
Labels
Milestone

Comments

@cwwalter
Copy link
Member

imSim uses transmission curves from the project controlled throughputs repo.

The atmospheric curves are located here.

https://github.com/lsst/throughputs/tree/master/atmos

They are produced with the MODTRAN model set to typical expected conditions at Cerro Pachon including added aerosols.

Currently the bandpasses loaded by the sims code are fixed to be at airmass=1.2. However the throughputs directory does include files for air-masses between 1.0 and 2.5. @danielsf points out that
"There are methods in sims_photUtils to take a hardware-only bandpass (which also exists in throughputs) and multiply it by an arbitrary airmass transmission curve."

Creating code to leverage those routines would increase the realism of the atmospheric model and would be useful for the PC group etc.

@cwwalter
Copy link
Member Author

cwwalter commented Jun 26, 2023

@jmeyers314 Note this existing issue related to airmass varying throughput.

Should we replace this with your suggestion of a multiplicative function and is this something we can implement quickly for the release candidate?

@jmeyers314
Copy link
Member

Yeah, I suspect we can take the total throughput files from throughputs/baseline, divide out the atmos_12.dat file and multiply in an interpolation of the nearest atmos_XY.dat files from throughputs/atmos.

@cwwalter cwwalter removed the DC3 label Jun 26, 2023
@cwwalter cwwalter modified the milestones: imSim 2.1, imSim 2.0 Jun 26, 2023
@cwwalter cwwalter modified the milestones: imSim 2.0, imSim 2.1 Jul 20, 2023
@cwwalter
Copy link
Member Author

This is a sprint day project on Friday.

@jchiang87
Copy link
Collaborator

Addressed by #444

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

No branches or pull requests

3 participants