-
Notifications
You must be signed in to change notification settings - Fork 83
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
dtype for GDAL CInt16, rasterio complex_int16 #353
Conversation
@snowman2 would it be OK to put a small (100x100 Cint16) test here |
@scottyhq thanks for the fix 👍. Either way works for me. If you do add the raster, please compress it. |
@scottyhq thanks for you patience with this. It is looking good. I added a couple of comments for things that would be good to consider. |
@snowman2 I added more tests to cover dealing with masking and nodata. Given the single Tests pass for me locally, except for |
@scottyhq this looks great, thanks 👍 |
@scottyhq any issues if I squash and merge? |
docs/history.rst
for all changes anddocs/rioxarray.rst
for new APIWill follow up with a simple test and docs...