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-237315106,https://api.github.com/repos/pydata/xarray/issues/844,237315106,MDEyOklzc3VlQ29tbWVudDIzNzMxNTEwNg==,1217238,2016-08-03T17:53:38Z,2016-08-03T17:53:38Z,MEMBER,"OK, in it goes. Thanks @ocefpaf !
","{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 1, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-237157875,https://api.github.com/repos/pydata/xarray/issues/844,237157875,MDEyOklzc3VlQ29tbWVudDIzNzE1Nzg3NQ==,1217238,2016-08-03T06:56:03Z,2016-08-03T06:56:03Z,MEMBER,"Anyone else have an opinion on the name? I'd like to merge this shortly before releasing v0.8.0 tomorrow.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-237005109,https://api.github.com/repos/pydata/xarray/issues/844,237005109,MDEyOklzc3VlQ29tbWVudDIzNzAwNTEwOQ==,1217238,2016-08-02T18:53:06Z,2016-08-02T18:53:06Z,MEMBER,"Assuming we do add the `filter` method, maybe `filter_by_attrs`?
It's nice to use names with the same prefix because they appear next to each other in auto-complete.
","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-236991775,https://api.github.com/repos/pydata/xarray/issues/844,236991775,MDEyOklzc3VlQ29tbWVudDIzNjk5MTc3NQ==,1217238,2016-08-02T18:10:54Z,2016-08-02T18:10:54Z,MEMBER,"I'm OK with the specialized `get_variables_by_attributes`. It doesn't need to be mutually exclusive with the more generic option. I do think the name is a mouthful, though :).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-236986387,https://api.github.com/repos/pydata/xarray/issues/844,236986387,MDEyOklzc3VlQ29tbWVudDIzNjk4NjM4Nw==,1217238,2016-08-02T17:53:45Z,2016-08-02T17:53:45Z,MEMBER,"@ocefpaf Good point. It probably is indeed better to write be able to write something generic like `ds.filter(lambda x: x.attrs['standard_name'] == 'convective_precipitation_flux')` rather the specialized to attributes `ds.get_variables_by_attributes(standard_name='convective_precipitation_flux')`.
Another option would be simply renaming this to `filter_attrs`, which is more succinct.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-226240288,https://api.github.com/repos/pydata/xarray/issues/844,226240288,MDEyOklzc3VlQ29tbWVudDIyNjI0MDI4OA==,1217238,2016-06-15T16:20:11Z,2016-06-15T16:20:11Z,MEMBER,"Agreed -- this is almost ready. Please add to the API docs (`api.rst`) and do the docstring fixes.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-219094287,https://api.github.com/repos/pydata/xarray/issues/844,219094287,MDEyOklzc3VlQ29tbWVudDIxOTA5NDI4Nw==,1217238,2016-05-13T16:34:21Z,2016-05-13T16:34:21Z,MEMBER,"@ocefpaf We can squash on merge now in the web interface, so that makes things easier.
The appveyor build failure looks unrelated to this change -- something about conda dependencies.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-217170260,https://api.github.com/repos/pydata/xarray/issues/844,217170260,MDEyOklzc3VlQ29tbWVudDIxNzE3MDI2MA==,1217238,2016-05-05T14:33:28Z,2016-05-05T14:33:28Z,MEMBER,"Well, if you guys are mostly excited about using this for coordinate
variables, another consistent choice would be to return a list of matching
DataArrays. But if we want to return a Dataset, we should only do data
variables, because it's weird to lose all the describing coordinates when
you match, e.g., standard_name=""air_temperature"".
On Thu, May 5, 2016 at 7:05 AM, Kyle Wilcox notifications@github.com
wrote:
> @ocefpaf https://github.com/ocefpaf I agree that xarray model
> consistent should take precedence. I also use this method to (mostly) pull
> out coordinate variables and can continue to use netCDF4.Dataset to do that.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> https://github.com/pydata/xarray/pull/844#issuecomment-217163460
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324
https://github.com/pydata/xarray/pull/844#issuecomment-217043510,https://api.github.com/repos/pydata/xarray/issues/844,217043510,MDEyOklzc3VlQ29tbWVudDIxNzA0MzUxMA==,1217238,2016-05-05T00:05:11Z,2016-05-05T00:05:11Z,MEMBER,"An important design question: should this filter both data variables and coordinates or only data variables? My thought is that it's only worth filtering data variables -- filtering out unmatched coordinates is not very useful.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,153126324