You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think that this is consequence of the refactor done by @alexamici when he has removed _normalize_path: #4701
We decided to demand the path interpretation to the backends.
The reason to delegate all path normalisation to backends is that not all strings are paths, for example 'NETCDF:"~/sst.nc":tos' is a valid source string for GDAL / rasterio.
I would suggest to add path normalisation to the backends that needs them, in order to avoid any special case in the common code path.
@aurghs I suggest you add os.path.expanduser in the netCDF backend.
What happened:
open_dataset
doesn't replace~
any moreWhat you expected to happen:
should replace
~
for backward compatibilityMinimal Complete Verifiable Example:
Anything else we need to know?:
works on 0.17.0, fails on master
The text was updated successfully, but these errors were encountered: