Closed niksirbi closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 99.66%. Comparing base (
a97ae86
) to head (8dc0a4e
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
No data about Duplication
Description
What is this PR
Why is this PR needed? Closes #158 . Thanks @antortjim for reporting the bug.
What does this PR do?
We now depend on
xarray[accel,viz]
, instead of justxarray[accel]
, meaning we also requirematplotlib
andseaborn
.Accordingly,
matplotlib
is no longer listed as an additional dependency for docs and binder (no need, since it will now come bundled withmovement
).References
See #158 and xarray's installation instructions.
How has this PR been tested?
Example notebooks were run in a fresh environment.
Is this a breaking change?
No.
Does this PR require an update to the documentation?
No.
Checklist: