-
Notifications
You must be signed in to change notification settings - Fork 21
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This adds six methods to `caput.time.Observer` to calculate (solar/lunar) (transits/risings/settings). These functions are originally from `ch_util.ephemeris` where they originally worked only on the `chime` `Observer`. They've since been enhanced to use any `Observer`, so there's no reason now not to have them in the `Observer` object directly, now. CHANGE FROM CH_UTIL VERSION There is one major change to these functions from what was in `ch_util.ephemeris`: instead of a `diameter` of 0.6 degrees = 36 arcminutes, these methods now use a `diameter` of 100 arcminutes. This means these functions now compute astronomical sun/moon rise/set, which is conventionally taken to occur when the centre of the sun/moon is 50 arcminutes below the horizon. The 50 arcminute number comes from a solar/lunar radius of 16 arcminutes plus 34 arcminutes of atmospheric refraction at the horizon. This change fixes the 3-minute discrepancy Mateus noticed when these functions were first ported from PyEphem (which _does_ account for atmospheric refraction) to skyfield (which does _not_ account for the refraction), here: chime-experiment/ch_util_private@b505cf0
- Loading branch information
1 parent
0123612
commit 9f4bb1b
Showing
2 changed files
with
257 additions
and
3 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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