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

Remove rasterio converter #240

Merged
merged 6 commits into from
Mar 13, 2024
Merged

Remove rasterio converter #240

merged 6 commits into from
Mar 13, 2024

Conversation

huard
Copy link
Contributor

@huard huard commented Mar 12, 2024

Overview

  • Update xarray based raster converter.

This PR fixes #239

Changes:

  • Remove live flyingpigeon test.

Related Issue / Discussion

Additional Information

Links to other issues or sources.

@huard huard changed the title Remove rasterio converte Remove rasterio converter Mar 12, 2024
@huard huard requested a review from cehbrecht March 12, 2024 17:20
@cehbrecht
Copy link
Member

@huard test_raster_tif is failing. Do you also want to use pytest.importorskip("xarray") or maybe import the extra requirements?

@huard huard merged commit f7a1987 into master Mar 13, 2024
6 checks passed
@huard huard deleted the fix-239 branch March 13, 2024 17:39
tlvu added a commit to Ouranosinc/PAVICS-e2e-workflow-tests that referenced this pull request Mar 16, 2024
To pull this change bird-house/birdy#240.

To fix this error bird-house/birdy#239:
```
RavenPy-master/docs/notebooks/02_Extract_geographical_watershed_properties.ipynb::Cell 5
AttributeError: module 'xarray' has no attribute 'open_rasterio'
```
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 this pull request may close these issues.

Deprecated open_rasterio xarray
2 participants