-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update documentation #81
Conversation
Codecov ReportBase: 81.90% // Head: 81.90% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #81 +/- ##
=======================================
Coverage 81.90% 81.90%
=======================================
Files 3 3
Lines 525 525
=======================================
Hits 430 430
Misses 95 95 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
modified: src/SurfaceFluxes.md
18f6072
to
df88893
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 seems to be directly copied from the design doc? I'm ok with merging it in. We may want to further clean up the design doc in the future, and can update the docs here accordingly.
Yeah for now they are a just ported summary of the equations in our code. The next step is to add more specific docs related to the |
bors r+ |
Build succeeded: |
Purpose
Update documentation (linked issue #24 )
Content