Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename of LazilyOuterIndexedArray to LazilyIndexedArray breaks rioxarray #5111

Closed
alexamici opened this issue Apr 5, 2021 · 0 comments · Fixed by #5112
Closed

Rename of LazilyOuterIndexedArray to LazilyIndexedArray breaks rioxarray #5111

alexamici opened this issue Apr 5, 2021 · 0 comments · Fixed by #5112

Comments

@alexamici
Copy link
Collaborator

What happened:

With current xarray master the rioxarray.open_rasterio crashes.

What you expected to happen:

With the released version 0.17.0 of xarray the open works as expected.

Minimal Complete Verifiable Example:

>>> import rioxarray
>>> rioxarray.open_rasterio("myfile.tif")
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-7-fedc16472305> in <module>
----> 1 da = rioxarray.open_rasterio("../../../FLAIR-PROC/rome-lockdown-2020/georeferenced/20200309+0130+417_VV-20200109-20200115-EPSG:32632-5.human-activity.tif")
      2 da

/usr/local/Caskroom/miniconda/base/envs/BOP/lib/python3.8/site-packages/rioxarray/_io.py in open_rasterio(filename, parse_coordinates, chunks, cache, lock, masked, mask_and_scale, variable, group, default_name, **open_kwargs)
    820 
    821     da_name = attrs.pop("NETCDF_VARNAME", default_name)
--> 822     data = indexing.LazilyOuterIndexedArray(
    823         RasterioArrayWrapper(
    824             manager,

AttributeError: module 'xarray.core.indexing' has no attribute 'LazilyOuterIndexedArray'

Anything else we need to know?:

LazilyOuterIndexedArray was recently renamed to LazilyIndexedArray by @aurghs in #4810 after a suggestion from @shoyer #4810 (comment)

Adding an alias would make the transition easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant