-
Notifications
You must be signed in to change notification settings - Fork 188
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
[WIP] Reverse k index so that k=1 is at the bottom. #107
Conversation
Also used ifelse function evaluations instead of if-else statements.
This is because at the surface and bottom it will attempt to access an array out of bounds.
Codecov Report
@@ Coverage Diff @@
## master #107 +/- ##
==========================================
+ Coverage 52.71% 53.15% +0.44%
==========================================
Files 19 19
Lines 645 649 +4
==========================================
+ Hits 340 345 +5
+ Misses 305 304 -1
Continue to review full report at Codecov.
|
We should add documentation on the numerical implementation of boundary conditions in this PR. |
Closing this PR as it's super stale. Can use the proposed changes as a reference when we revisit this issue. |
This PR reverses the vertical index k so that k=1 corresponds to the bottom of the domain. More details can be found in #90.
WIP because I'm not 100% sure I'm calculating differences and averages at k=1 and k=Nz correctly now. Will have to ask @jm-c to make sure.
Resolves #90