-
Notifications
You must be signed in to change notification settings - Fork 1
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
add Milankovitch cycles #34
Conversation
Codecov Report
@@ Coverage Diff @@
## main #34 +/- ##
===========================================
+ Coverage 98.71% 100.00% +1.28%
===========================================
Files 3 3
Lines 78 87 +9
===========================================
+ Hits 77 87 +10
+ Misses 1 0 -1
Continue to review full report at Codecov.
|
bors try |
tryBuild failed: |
bors try |
tryBuild succeeded: |
c10ef96
to
a264ae3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great to me! Thanks so much Clare. Some comments in the file. Also, the documentation landing page still is quite Earth-centric, while you made the rest more planet-agnostic. You might want to do the same on the landing page.
…ealing with dates to make Insolation work with Float32 values. Add tests and documentation for the new Milankovitch cycles. Specifically test that long-term variations in vernal equinox and perihelion date are correct. Change mean anomaly to be calculated relative to epoch not to vernal equinox. Remove Manifest files. Update epoch from float to string->DateTime.
bors try |
bors r+ |
tryBuild succeeded: |
Build succeeded: |
Closes #27 and closes #29
Docs preview here: https://clima.github.io/Insolation.jl/previews/PR34