-
Notifications
You must be signed in to change notification settings - Fork 15
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
MDA8 #1224
MDA8 #1224
Conversation
Notes from old PR: #1165
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main-dev #1224 +/- ##
============================================
+ Coverage 79.31% 79.33% +0.02%
============================================
Files 131 133 +2
Lines 20248 20284 +36
============================================
+ Hits 16059 16092 +33
- Misses 4189 4192 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
bc37418
to
817f23f
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.
Small changes. Otherwise looks good
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.
😺
Change Summary
Implements the possibility to calculate mda8 values for specific variables. This is done by creating an additional ColocatedData object after colocation.
Related issue number
closes #863
Checklist