-
Notifications
You must be signed in to change notification settings - Fork 9
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
Implement lunar model #9
Comments
@crockosi @sbailey I am working on this now (finally!) My plan is to:
Any feedback is welcome. |
On the last point above, I implemented the V-band extinction coefficient calculation as:
where
Using the KPNO zenith extinction table, I calculate kV = 0.1515, compared with the value 0.162 that is hardcoded in |
What an awesome plot! |
This plot shows the dark sky (green), scattered moon (blue) and extinction coef (red) for a 25% moon at a zenith angle of 70 deg (same as the plot above): I am planning to add the green and blue surface brightness values in the simulation, which assumes that our KPNO "dark sky" spectrum has no scattered moonlight. However, I put the moon below the horizon in the default config, so that only the green curve is used (i.e., no change from the current simulations). |
This is even more awesome! I spot-checked it using the old IDL version of Thanks, David. I'll run a bunch of my other tests once this is merged. --Connie On Mon, Feb 22, 2016 at 5:32 PM, dkirkby [email protected] wrote:
|
Thanks for adding this. I defer to Connie for validating the results, but she sounds happy. Could you add a code snippet showing how to use this for simulating a series of spectra under different lunar conditions? In particular, if I create an Atmosphere giving it a Moon object, can I then directly call moon.update() to alter how the moon appears to the Atmosphere? Or do I create a new Atmosphere with that updated moon before simulating more spectra? Please add a docstring for Atmosphere.set_condition(). I think that gets a name from the constructor surface_brightness_dict, but the chain wasn't obvious without reading code. I think these do provide flexible development/testing alternatives to the Moon model, so I think it is worth keeping these two methods for similar functionality. |
@sbailey I made the changes you suggested. In particular, there is now an example of changing moon parameters on the fly in the docs: http://specsim.readthedocs.org/en/ks_moon/config.html#atmosphere I also added moon parameters to the
|
When one has both moon and clouds, the sky brightness can increase by quite a bit more. |
For comparison, the moon-only specsim predictions for those conditions are 18.8 mag/arcsec2 in z-band with the moon below the horizon (compared with ~19.0 in your graph, if you used the same units), but the typical increase at 50 deg from a 94% full moon is only ~0.8 mag/arcsec2. In the limit of single scattering, only the cloud along the boresight should matter. If that's a reasonable approximation, then perhaps thin cloud could be included in the KS1991 model as an additive correction to the extinction. |
Not sure if this should be a new or separate issue, but @djschlegel has posted some relevant data on the sky brightness as a function of moon angle and moon phase from the SDSS that specsim's model could be compared against. The relevant thread was sent to desi-bgs@ and is archived here: This issue is also relevant to the bright-time simulations open issue desihub/desisim#72 |
Thanks for flagging this John. I will take a look and then create a new issue. |
The IDL quicksim already has an implementation of Krisciunas & Schaefer (1991) in desimodel/pro/lunarmodel.pro. This issue is to implement the same functionality here as an intermediate (and more urgent) step towards #2.
The text was updated successfully, but these errors were encountered: