home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 799076455

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/pull/4984#issuecomment-799076455 https://api.github.com/repos/pydata/xarray/issues/4984 799076455 MDEyOklzc3VlQ29tbWVudDc5OTA3NjQ1NQ== 5635139 2021-03-15T04:13:46Z 2021-03-15T04:13:46Z MEMBER

Great re the dimensions!

I reviewed the tests more fully, they look great.

It looks like we need a requires_numexpr decorator on the tests — would you be OK to add that?

Could we add a simple method to DataArray which converts to a Dataset, calls the functions, and converts back too? (there are lots of examples already of this, let me know any issues)

And we should add the methods to api.rst, and a whatsnew entry if possible.


Does anyone have any other thoughts? I think the API is very reasonable. I could imagine a more sophisticated API that could take a single query, rather than a dict of them by dimension — currently it's da.query(x="x>3", y="y>4"), but that would require more work and decisions, even if it were preferable.

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