-
Notifications
You must be signed in to change notification settings - Fork 85
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
BUG: Remove xarray crs
attribute in rio.write_crs (issue #488)
#490
BUG: Remove xarray crs
attribute in rio.write_crs (issue #488)
#490
Conversation
Thanks @GCBallesteros 👍 I see that you added it to |
Also, mind adding a simple test around here? It just needs to have a Dataframe with the |
A number of tests are failing now due to the removal of the crs attribute. Should I fix them by making use of the |
For For |
On `test_reproject__grid_mapping` the deletion via `_del_attr` of the crs has also been removed as it shouldn't be there.
Setting `mda.coords["spatial_ref"]` to 0 will make `mda.rio.crs` return None which then results on an error as we would be asking for `CRS.from_user_input(None)`.
The problem fixed by commit |
I pushed a fix for the scenario when the CRS isn't found by rasterio/GDAL, but can be determined from the attributes by rioxarray. |
crs
attribute in rio.write_crs (issue #488)
Thanks @GCBallesteros 👍 |
xarray.Dataset.reproject
doesn't modify the crs attribute #488docs/history.rst
for all changes anddocs/rioxarray.rst
for new API