GH5005 fix documentation on open_rasterio

[gabriel-abrahao]

• Closes Example documentation in open_rasterio #5005
• Addresses open_rasterio does not read coordinates from netCDF file properly with netCDF4>=1.4.2 #3185
• Fixed an example line in the open_rasterio docstring that extracted an affine transform in an old way that no longer works.

[keewis]

@gabriel-abrahao, do you want to try converting the code block to doctest? Otherwise this looks good to me and should be ready for merging.

[gabriel-abrahao]

@gabriel-abrahao, do you want to try converting the code block to doctest? Otherwise this looks good to me and should be ready for merging.

I tried, but building the documentation is failing on both my Linux computers just like in #4257 , so unfortunately I can't make it.

[keewis]

you could also use the CI to verify the changes (the doctests CI is pretty fast), but as I said we can leave that to a new PR.

Unless someone else has any comments I'll merge this tomorrow.

[pep8speaks]

Hello @gabriel-abrahao! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-03-15 13:46:59 UTC

[gabriel-abrahao]

you could also use the CI to verify the changes (the doctests CI is pretty fast), but as I said we can leave that to a new PR.

Unless someone else has any comments I'll merge this tomorrow.

Just did that, thanks! Is there a way to make the CI run just the check I want it to?

I'm not sure if every doctest has to import xarray as xr and import numpy as np, so I added those just in case and it's passing.

Also, right multiplying matrices will be deprecated in numpy, so now transform is on the left side.

[keewis]

thanks, this looks great.

Is there a way to make the CI run just the check I want it to?

for future reference, you can add a [skip-ci] tag to the first line of a commit message. This will run only the style and doctest checks and build the docs.

[keewis]

thanks for the update, @gabriel-abrahao. I noticed this is you first PR here. Welcome to xarray!