Miguel Jimenez

Thanks @MaceKuailv for catching this up! you are correct and this should be an easy fix! Will work on this over the weekend

I was looking into solving this problem. But couldn't reproduce the error.... I ran the exact code above (after loading the ECCO dataset on sciserver, using Ali's tutorial notebook) but...

We decided to include an `Error about using data in which time is not a dimension (e.g. time average)`. So that the best practice is to always have a dataset...

@MaceKuailv FYI I just checked the `oceanspy_test` environment that I used for testing in my local computer and matplotlib indeed is version `3.5.1` (also checked and xgcm is version 0.6...

> We just looked into this me and @MaceKuailv , and the environment used for ci testing in my PR (which passed all tests) installed matplotib version 3.3.5. @MaceKuailv will...

@MaceKuailv don't forget to sync your fork with the new changes I added to oceanspy, so that your PR does not revert them...

Good to know about the zarr development route. I can't wait to see the new behavior, although I understand the timeline... Here is a better description of the problem with...

Yes, that does it. That is ```python In []: from collections.abc import MutableMapping In []: isinstance(mapper, MutableMapping) Out []: True ``` Thanks @martindurant !

With this sorted (and working), should I submit a PR to set ``Intake-xarray`` to use ``open_dataset`` when opening zarr stores (i have a working fork)?

Awesome, I can take a look in the next couple of days and see what I can do. It shouldn't be much trouble, I hope. Thanks a lot @weiji14!