Closed emmaai closed 1 year ago
@emmaai I can’t review this PR (outside my knowledge of the code) but I did want to say thanks for getting on top of this groupby issue and I’m looking forward to the update!
Base: 75.27% // Head: 75.57% // Increases project coverage by +0.30%
:tada:
Coverage data is based on head (
d694fb4
) compared to base (8f564b2
). Patch coverage: 97.29% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Changes
xarray
version constraintI don't have a sensible way to test the docker image against the current code since it's supposed to be a regression test and nothing should be broken. Maybe I shouldn't have updated the docker image in this pr. I'll leave the docker image test failed until next pr specially on it.