desihub / desisurvey

Code for desi survey planning and implementation
BSD 3-Clause "New" or "Revised" License
2 stars 7 forks source link

new bright time exposure factor #126

Open changhoonhahn opened 3 years ago

changhoonhahn commented 3 years ago

I've updated the bright time exposure factor calculation based on SV1 observations. The new exposure factor etc.bright_exposure_factor is calculated using a regression sky brightness model fit to SV1, CMX, and BOSS sky brightness observations. The exposure factor accounts for read noise and assumes a new reference BGS sky (nominal dark sky scaled to r=21.07 + moon/solar spectrum scaled to r=20.07). Below are specific changes to the modules

py/desisurvey/ephem.py

py/desisurvey/etc.py

py/desisurvey/scheduler.py

py/desisurvey/util.py

schlafly commented 3 years ago

Thanks Chang. Some comments:

changhoonhahn commented 3 years ago
schlafly commented 3 years ago

Great, yeah, I imagine the dark / gray time is the easier part of the model. Let's get that in this PR if it sounds feasible?

I agree that the read noise comes in only there. My comment was more to say that that same bright_exposure_factor could be used outside of bright time without doing damage.

And then it would be great to see the results from a surveysim run with these changes; i.e., when the dark / gray / bright surveys get finished.

changhoonhahn commented 3 years ago

@schlafly Could you check over this pull request again?

Below are the latest changes: