desihub / specsim

Quick simulations of spectrograph response
2 stars 9 forks source link

Implement a model of twilight sky brightness #84

Open dkirkby opened 6 years ago

dkirkby commented 6 years ago

We now have a functional twilight model that can be used together with the scattered moon model for more realistic bright-time simulations. The two main questions I would like to follow up on eventually are:

However, I think the general features of this model are a big improvement over nothing so should be merged soon unless anyone spots obvious problems.

This PR also removes the (now obsolete) sky-conditions config parameter, which was originally to support using the gray/bright spectra in desimodel but is now redundant with the more flexible moon and twilight models.

dkirkby commented 6 years ago

Update on the two issues mentioned earlier:

Three new plots to document the new model (which are now in the docs, along with instructions to reproduce them):

desi_moon_twilight_atmosphere

desi_twilight_polar

twilight_spectrum

weaverba137 commented 6 years ago

You'll want to bring in the latest changes from master to get tests to pass.

moustakas commented 6 years ago

@dkirkby It would be very helpful to have this updated sky model implemented to carry out some bright-time redshift success tests, even if there's still work to be done. Can you provide any updates on when we might be able to merge this? Thanks!

moustakas commented 5 years ago

Any chance of getting these changes into the 18.12 software release?

weaverba137 commented 4 years ago

@dkirkby, this PR is now about 3 years old. Should we continue to keep this open?