home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 812453263

This data as json

html_url issue_url id node_id user created_at updated_at author_association body reactions performed_via_github_app issue
https://github.com/pydata/xarray/issues/5098#issuecomment-812453263 https://api.github.com/repos/pydata/xarray/issues/5098 812453263 MDEyOklzc3VlQ29tbWVudDgxMjQ1MzI2Mw== 226037 2021-04-02T09:33:24Z 2021-04-02T09:33:43Z MEMBER

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.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  847014702
Powered by Datasette · Queries took 0.916ms · About: xarray-datasette