codecov-commenter
commented
9 months ago Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.
Comparison is base (
33090a8) 83.97% compared to head (7833389) 84.57%.
| Files | Patch % | Lines |
|---|---|---|
| mt_metadata/transfer_functions/io/zfiles/zmm.py | 88.23% | 6 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #193 +/- ## ========================================== + Coverage 83.97% 84.57% +0.59% ========================================== Files 269 269 Lines 20156 20253 +97 ========================================== + Hits 16926 17128 +202 + Misses 3230 3125 -105 ``` | [Flag](https://app.codecov.io/gh/kujaku11/mt_metadata/pull/193/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JP) | Coverage Δ | | |---|---|---| | [tests](https://app.codecov.io/gh/kujaku11/mt_metadata/pull/193/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JP) | `84.57% <95.12%> (+0.59%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JP#carryforward-flags-in-the-pull-request-comment) to find out more.:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
To speed up initialization of an
xarray.Datasetall the coordinates should have the same length. Need to change this inTF._initialize_transfer_functionsuch that eachxarray.DataArrayhas the same coordinates as they will all be filled in when merged. This removes redundancy and coordinate alignment slowdowns.