issue_comments

Table actions

5 rows where issue = 238731491 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: reactions, created_at (date), updated_at (date)

user 3

author_association 3

issue 1

  • Rasterio - Attribute Error · 5 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
311658625 https://github.com/pydata/xarray/issues/1466#issuecomment-311658625 https://api.github.com/repos/pydata/xarray/issues/1466 MDEyOklzc3VlQ29tbWVudDMxMTY1ODYyNQ== fmaussion 10050469 2017-06-28T13:23:54Z 2017-06-28T13:23:54Z MEMBER

I guess we would put it in extra_requires ?

Mh, rasterio should remain an optional dependency. We can however add a version check in the backend and raise an ImportError if the rasterio version is too old.

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Rasterio - Attribute Error 238731491
311656405 https://github.com/pydata/xarray/issues/1466#issuecomment-311656405 https://api.github.com/repos/pydata/xarray/issues/1466 MDEyOklzc3VlQ29tbWVudDMxMTY1NjQwNQ== ceholden 3585769 2017-06-28T13:15:35Z 2017-06-28T13:15:35Z CONTRIBUTOR

@fmaussion sorry for the delay. Looks like the switch to an actual CRS class happened just before rasterio=0.36 (commit), which was the last 0.x release before the alphas for 1.0 started up. Previously this to_string capability was a function inside of rasterio.crs, and now it's just on the rasterio.crs.CRS class.

Quite a bit of reorganization happened between 0.36 and 1.0a, so I'd agree sticking to rasterio>1.0a would be best. For specifying the requirement, I guess we would put it in extra_requires? Dask does a nice job of this for its various components (their setup.py).

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Rasterio - Attribute Error 238731491
311280592 https://github.com/pydata/xarray/issues/1466#issuecomment-311280592 https://api.github.com/repos/pydata/xarray/issues/1466 MDEyOklzc3VlQ29tbWVudDMxMTI4MDU5Mg== fmaussion 10050469 2017-06-27T07:47:56Z 2017-06-27T07:47:56Z MEMBER

@ceholden do you know when the to_crs() method was implemented? Should we make rasterio>1.0a a required dependency?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Rasterio - Attribute Error 238731491
311250589 https://github.com/pydata/xarray/issues/1466#issuecomment-311250589 https://api.github.com/repos/pydata/xarray/issues/1466 MDEyOklzc3VlQ29tbWVudDMxMTI1MDU4OQ== slharris 12929592 2017-06-27T04:35:14Z 2017-06-27T04:35:14Z NONE

That solved the issue - I should have checked that first!

thank you

 
{
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 1,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Rasterio - Attribute Error 238731491
311249992 https://github.com/pydata/xarray/issues/1466#issuecomment-311249992 https://api.github.com/repos/pydata/xarray/issues/1466 MDEyOklzc3VlQ29tbWVudDMxMTI0OTk5Mg== ceholden 3585769 2017-06-27T04:30:34Z 2017-06-27T04:30:34Z CONTRIBUTOR

Could you include the version of rasterio that you have installed? It's an optional dependency for xarray, and so a little tricky to enforce a minimum version number for, but my first guess would be to check the version:

bash python -c "import rasterio; print(rasterio.__version__)"

At one point in rasterio's development, the CRS information was stored as an simple dict instead of a rasterio.crs.CRS, which has a to_string method. Rasterio has been developing pretty rapidly, and it looks like it had been using a simple dict as soon as March, 2016 (version 0.33). The tests for xarray are running against rasterio=1.0a9 specifically (see docs/environment.yml), but any of the recent 1.0 alpha releases of rasterio will likely work. If your copy of rasterio is older than 1.0a[something], it might be as simple as upgrading the package.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Rasterio - Attribute Error 238731491

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options: 

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);