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/2351#issuecomment-411211864,https://api.github.com/repos/pydata/xarray/issues/2351,411211864,MDEyOklzc3VlQ29tbWVudDQxMTIxMTg2NA==,296686,2018-08-07T21:40:34Z,2018-08-07T21:40:34Z,CONTRIBUTOR,"Strangely, the Travis-CI build seems to have failed with some errors unrelated to this PR. The two errors are on:
- Python 3.6 with the development version of dask, giving an error of `AttributeError: module 'dask.array' has no attribute 'ghost'`
- Python 2.7 with cdat, iris and pynio which gives various errors of arrays not being equal for `test_to_and_from_cdms2_sgrid` and `test_to_and_from_cdms2_ugrid`
For what it's worth, all of the tests related to this PR passed.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,348462356
https://github.com/pydata/xarray/issues/2348#issuecomment-410864167,https://api.github.com/repos/pydata/xarray/issues/2348,410864167,MDEyOklzc3VlQ29tbWVudDQxMDg2NDE2Nw==,296686,2018-08-06T21:50:47Z,2018-08-06T21:50:47Z,CONTRIBUTOR,"Ok great, I'll make a PR to remove that last chunk - thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,348081353
https://github.com/pydata/xarray/pull/990#issuecomment-244801125,https://api.github.com/repos/pydata/xarray/issues/990,244801125,MDEyOklzc3VlQ29tbWVudDI0NDgwMTEyNQ==,296686,2016-09-05T19:14:38Z,2016-09-05T19:14:38Z,CONTRIBUTOR,"I realised that the reason that Travis and AppVeyor weren't running properly was because there were conflicts between this PR and master. I _think_ I've managed to rebase and push properly - but let me know if I've done anything wrong.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/990#issuecomment-244539361,https://api.github.com/repos/pydata/xarray/issues/990,244539361,MDEyOklzc3VlQ29tbWVudDI0NDUzOTM2MQ==,296686,2016-09-03T10:36:03Z,2016-09-03T10:36:03Z,CONTRIBUTOR,"Ah ok - that works (not entirely sure why, but it works!)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/990#issuecomment-244510329,https://api.github.com/repos/pydata/xarray/issues/990,244510329,MDEyOklzc3VlQ29tbWVudDI0NDUxMDMyOQ==,296686,2016-09-02T23:35:24Z,2016-09-02T23:35:24Z,CONTRIBUTOR,"Oops yes, sorry!
I'm having a bit of an issue with changing the strings to constants (which, by the way, I completely agree is how it should be done). I've defined the constants in `backends/api.py` and am then trying to import them in `core/dataarray.py`.
I am doing the import as:
`from ..backends import DATAARRAY_NAME, DATAARRAY_VARIABLE`
But this gives me the following error:
```
../../anaconda3/lib/python3.5/site-packages/py/_path/local.py:650: in pyimport
__import__(modname)
xarray/__init__.py:3: in
from .core.extensions import (register_dataarray_accessor,
xarray/core/extensions.py:3: in
from .dataarray import DataArray
xarray/core/dataarray.py:9: in
from ..backends import DATAARRAY_NAME, DATAARRAY_VARIABLE
xarray/backends/__init__.py:8: in
from .netCDF4_ import NetCDF4DataStore
xarray/backends/netCDF4_.py:6: in
from .. import Variable
E ImportError: cannot import name 'Variable'
```
It seems like my relative import is somehow mucking up another relative import inside the netCDF4 stuff. Do you have any ideas how I can solve this? Am I just doing my imports wrong?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/990#issuecomment-244395018,https://api.github.com/repos/pydata/xarray/issues/990,244395018,MDEyOklzc3VlQ29tbWVudDI0NDM5NTAxOA==,296686,2016-09-02T14:44:24Z,2016-09-02T14:44:24Z,CONTRIBUTOR,"Great, thanks. I think I've done these changes (frustratingly I can't see a way to use `*args`/`*kwargs` after using the full argument list in the function definition, so I've had to write it out. Let me know if there is a better way, but I suspect there isn't).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/issues/988#issuecomment-243952679,https://api.github.com/repos/pydata/xarray/issues/988,243952679,MDEyOklzc3VlQ29tbWVudDI0Mzk1MjY3OQ==,296686,2016-09-01T01:41:51Z,2016-09-01T01:41:51Z,CONTRIBUTOR,"I also prefer an approach that _doesn't_ use context managers: I agree with @darothen's comments about the issues with their use.
Regardless of the exact implementation, I am strongly in favour of this functionality being added to xarray - I can already think of a number of very useful ways I could use it!
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173612265
https://github.com/pydata/xarray/pull/990#issuecomment-243951504,https://api.github.com/repos/pydata/xarray/issues/990,243951504,MDEyOklzc3VlQ29tbWVudDI0Mzk1MTUwNA==,296686,2016-09-01T01:33:25Z,2016-09-01T01:33:25Z,CONTRIBUTOR,"Thanks - I've got that working with the `close()` method, and I did it 'properly' using the `BaseDataObject` class.
The only failure on here now is the same as #991 - a conda error creating one of the environments.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/991#issuecomment-243946499,https://api.github.com/repos/pydata/xarray/issues/991,243946499,MDEyOklzc3VlQ29tbWVudDI0Mzk0NjQ5OQ==,296686,2016-09-01T00:58:54Z,2016-09-01T00:58:54Z,CONTRIBUTOR,"The Travis builds seem to have all passed except for `py27-cdat+pynio` which has failed with a conda error that seems to be unrelated to these changes (`Error: HTTPError: 403 Client Error: Forbidden for url`)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/990#issuecomment-243946174,https://api.github.com/repos/pydata/xarray/issues/990,243946174,MDEyOklzc3VlQ29tbWVudDI0Mzk0NjE3NA==,296686,2016-09-01T00:56:50Z,2016-09-01T00:56:50Z,CONTRIBUTOR,"Also, it seems like there might be an issue with not closing the Dataset - but only on Windows (see the AppVeyor build failure at https://ci.appveyor.com/project/jhamman/xarray-injyf/build/1.0.268/job/kr8ss684ijxg55be).
Any ideas how this could be sorted without having to implement `.close()` for `DataArray` (or, alternatively, how to go about implementing that, given that I don't think a DataArray contains a reference to the Dataset that it may have come from)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/991#issuecomment-243933961,https://api.github.com/repos/pydata/xarray/issues/991,243933961,MDEyOklzc3VlQ29tbWVudDI0MzkzMzk2MQ==,296686,2016-08-31T23:32:56Z,2016-08-31T23:32:56Z,CONTRIBUTOR,"I think I've dealt with all of these.
Refactoring the tests has made them a bit cleaner and safer due to not using mutation - however I still have quite a few repeated lines of code. I've also combined them all into one test with lots of asserts - as otherwise I couldn't keep the helper functions within the test without repeating them loads of times.
Does anything need adding to the docs for this? I'm assuming not as it's just making the code do more checking, it's not adding a feature as such?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/990#issuecomment-243929210,https://api.github.com/repos/pydata/xarray/issues/990,243929210,MDEyOklzc3VlQ29tbWVudDI0MzkyOTIxMA==,296686,2016-08-31T23:05:12Z,2016-08-31T23:05:12Z,CONTRIBUTOR,"Thanks for the useful comments.
I've added some bits to the Sphinx docs in the places that I think they should go - but let me know if I need to add any more.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/990#issuecomment-243863092,https://api.github.com/repos/pydata/xarray/issues/990,243863092,MDEyOklzc3VlQ29tbWVudDI0Mzg2MzA5Mg==,296686,2016-08-31T18:50:41Z,2016-08-31T18:50:41Z,CONTRIBUTOR,"I think I've dealt with the comments/issues now, but let me know if more needs doing.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/pull/991#issuecomment-243001377,https://api.github.com/repos/pydata/xarray/issues/991,243001377,MDEyOklzc3VlQ29tbWVudDI0MzAwMTM3Nw==,296686,2016-08-28T21:43:49Z,2016-08-28T21:43:49Z,CONTRIBUTOR,"Thanks, I'll deal with these modifications.
Also, I'm seeing errors from TravisCI saying `ValueError: cannot read or write netCDF files without netCDF4-python or scipy installed` - what do I need to do to make sure these tests work on Travis? (They pass fine on my local machine)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/990#issuecomment-243000159,https://api.github.com/repos/pydata/xarray/issues/990,243000159,MDEyOklzc3VlQ29tbWVudDI0MzAwMDE1OQ==,296686,2016-08-28T21:20:44Z,2016-08-28T21:20:44Z,CONTRIBUTOR,"Thanks @shoyer - I wasn't actually aware that DataArrays can have a name _before_ they're put into a Dataset (that's one of the things I love about contributing to OSS: you always learn things about the software that you never knew before!). I'll deal with the design issues ASAP.
How would you suggest testing this? I wasn't sure where to put the tests, and what exactly to test? Presumably something about the equivalence between this and saving after manually converting to a dataset, but I wasn't really sure how best to go about it.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173632183
https://github.com/pydata/xarray/issues/913#issuecomment-234096888,https://api.github.com/repos/pydata/xarray/issues/913,234096888,MDEyOklzc3VlQ29tbWVudDIzNDA5Njg4OA==,296686,2016-07-20T21:59:14Z,2016-07-20T21:59:14Z,CONTRIBUTOR,"I'm not really sure how to work out which one it is a bug in - do you have any advice? Or should I just report it on the dask issue tracker too and see what they think?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,166642852
https://github.com/pydata/xarray/issues/911#issuecomment-233874116,https://api.github.com/repos/pydata/xarray/issues/911,233874116,MDEyOklzc3VlQ29tbWVudDIzMzg3NDExNg==,296686,2016-07-20T08:00:08Z,2016-07-20T08:00:08Z,CONTRIBUTOR,"Thanks. I might raise an issue on the netCDF4-Python project to see if we can improve the error message.
Would you be happy for me to submit a PR updating the xarray docs to note that you can only export attributes with certain values to a netCDF file? (and possibly, longer-term, we could provide an error inside xarray if you try to save out a file with objects as attributes)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,166511736
https://github.com/pydata/xarray/pull/637#issuecomment-232153175,https://api.github.com/repos/pydata/xarray/issues/637,232153175,MDEyOklzc3VlQ29tbWVudDIzMjE1MzE3NQ==,296686,2016-07-12T19:28:10Z,2016-07-12T19:28:10Z,CONTRIBUTOR,"Coming here from #897, it seems that we just need to decide to go with matplotlib's version, and therefore switch round the tuple given as `figsize` at https://github.com/pydata/xarray/pull/637/commits/400b1ef1d142c3c0dec1ff3c9f3cdfffa2229c10#diff-4ee153eb90c5207a5ed2e2b997bf6ce9R212, and then update the docs to go with it.
Does that make sense?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,112928260