Closed jerrylin96 closed 10 months ago
Merging #68 (f81ced9) into main (75729a4) will decrease coverage by
1.76%
. The diff coverage is1.53%
.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@ Coverage Diff @@
## main #68 +/- ##
========================================
- Coverage 8.66% 6.90% -1.76%
========================================
Files 1 1
Lines 612 796 +184
========================================
+ Hits 53 55 +2
- Misses 559 741 +182
Files | Coverage Δ | |
---|---|---|
climsim_utils/data_utils.py | 6.90% <1.53%> (-1.76%) |
:arrow_down: |
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!
@jerrylin96 thanks for tagging me! Just checking, are the Files Changed here accurate? Seems like possibly there are some things in there that are not part of this PR but were committed accidentally? Either that, or perhaps I don't understand the scope of this PR 😄
@jerrylin96 thanks for tagging me! Just checking, are the Files Changed here accurate? Seems like possibly there are some things in there that are not part of this PR but were committed accidentally? Either that, or perhaps I don't understand the scope of this PR 😄
Sorry just saw this... For some reason I don't get notifications on GitHub! Do you think you could tag me on slack next time? Really sorry about that
Mostly changed data_utils.py to make it more flexible with respect to variables and saving.