new bright time exposure factor #126
Open
changhoonhahn wants to merge 17 commits intodesihub:mainfrom
Open
Conversation
will be calculated based on observing conditions
exposure factor calculation for available tiles.
Contributor
|
Thanks Chang. Some comments:
|
Author
|
Contributor
|
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. |
… times; etc.sky_exposure_factor modified to take nominal exposure time as an input
…t_weather in surveysim.nightops
accurately accounts for read noise. I've also added some documentation on sky_level
Author
|
@schlafly Could you check over this pull request again? Below are the latest changes:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I've updated the bright time exposure factor calculation based on SV1 observations. The new exposure factor
etc.bright_exposure_factoris 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 modulespy/desisurvey/ephem.pypy/desisurvey/etc.pybright_exposure_factorfunction calculates the bright time exposure factor based on airmass, moon illumination, moon separation, moon altitude, sun separation, and sun altitude. The exposure factor is how much the bright time exposure has to be scaled to match the SNR of the reference BGS sky (see notebook for new ref. sky). The calculation includes read noise contributions.bright_Isky5000_notwilight_regressionfunction calculates the sky surface brightness without twilight contributions at 5000A using a regression model. This sky model was fit using SV1, CMX, and BOSS observationsbright_Isky5000_twilight_regressionfunction calculates the twilight sky contribution at 5000A using a regression model. The twilight contribution was fit using BOSS twilight observations with sun alt > -18.py/desisurvey/scheduler.pyScheduler.init_night: now initializes moon Alt, Az, sun RA, Dec, Alt, Az in addition moon RA and Dec.Scheduler.next_tile: for BRIGHT programetc.bright_exposure_factorin included in the exposure factor calculation.py/desisurvey/util.pyimport astropy._erfa.coresince it wasn't used and caused import errors.