home / github / issue_comments

Menu
  • Search all tables
  • GraphQL API

issue_comments: 217132574

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/844#issuecomment-217132574 https://api.github.com/repos/pydata/xarray/issues/844 217132574 MDEyOklzc3VlQ29tbWVudDIxNzEzMjU3NA== 950575 2016-05-05T11:41:19Z 2016-05-05T11:41:19Z CONTRIBUTOR

should this filter both data variables and coordinates or only data variables?

I thought a little bit more about this and now I am on the fence. The pros of filtering only data variables are a nice and clean Dataset object, and overall consistency with the high level xarray model. The cons are that we cannot do the filtering on the coords (obviously), but most of the time that we need to do that we go to a lower level object like netCDF4.Dataset. However, it would be nice if both xarray and netCDF4 behaved the same way...

I am 51% with filtering both (and the current implementation does that) but I will leave the final decision to you.

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