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/618#issuecomment-147512768,https://api.github.com/repos/pydata/xarray/issues/618,147512768,MDEyOklzc3VlQ29tbWVudDE0NzUxMjc2OA==,1217238,2015-10-12T20:35:46Z,2015-10-12T20:35:46Z,MEMBER,"OK, why don't we merge this, then I'll take a look at the remaining failing test? I think I can fix that by adjusting `.travis.yml`.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147512127,https://api.github.com/repos/pydata/xarray/issues/618,147512127,MDEyOklzc3VlQ29tbWVudDE0NzUxMjEyNw==,2443309,2015-10-12T20:32:42Z,2015-10-12T20:32:42Z,MEMBER,"Okay - that worked. I think we're good with numpy 1.10 here. There is still one failing test related to pandas/numpy. This may be more of a travis thing.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147490424,https://api.github.com/repos/pydata/xarray/issues/618,147490424,MDEyOklzc3VlQ29tbWVudDE0NzQ5MDQyNA==,2443309,2015-10-12T18:53:47Z,2015-10-12T18:53:47Z,MEMBER,"ugh indeed. I'll figure out how to coerce the datetime64 object into something that works for now.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147281012,https://api.github.com/repos/pydata/xarray/issues/618,147281012,MDEyOklzc3VlQ29tbWVudDE0NzI4MTAxMg==,1217238,2015-10-12T03:10:33Z,2015-10-12T03:10:33Z,MEMBER,"In any case, I was wrong about what had changed with NumPy 1.10. This appears to be inadvertent... (sigh).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147274729,https://api.github.com/repos/pydata/xarray/issues/618,147274729,MDEyOklzc3VlQ29tbWVudDE0NzI3NDcyOQ==,1217238,2015-10-12T02:28:26Z,2015-10-12T02:28:26Z,MEMBER,"Hmm. Something seem to have broken/changed with NumPy 1.10:
```
In [1]: import numpy as np
In [2]: np.datetime64('NaT') == np.datetime64('NaT')
Out[2]: True
In [3]: np.datetime64('NaT') == np.datetime64('NaT', 'us')
/Users/shoyer/miniconda/envs/tmp-np1.10/bin/ipython:1: DeprecationWarning: elementwise == comparison failed; this will raise an error in the future.
#!/bin/bash /Users/shoyer/miniconda/envs/tmp-np1.10/bin/python.app
Out[3]: False
```
I'm not quite sure what's going on there, so I'll file an issue in the NumPy repo.
In the meantime, we may want to update our test to coerce to `datetime64[ns]` first.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147142725,https://api.github.com/repos/pydata/xarray/issues/618,147142725,MDEyOklzc3VlQ29tbWVudDE0NzE0MjcyNQ==,2443309,2015-10-11T01:08:01Z,2015-10-11T01:08:01Z,MEMBER,"I'm actually going to paste the traceback from one of the test results (using `py.test`) to show how nonsensical this is:
```
=========================================================================================================== FAILURES ============================================================================================================
_____________________________________________________________________________________________ TestVariable.test_index_0d_not_a_time _____________________________________________________________________________________________
self =
def test_index_0d_not_a_time(self):
d = np.datetime64('NaT')
x = self.cls(['x'], [d])
# Wasn't able to figure out why this was failing.
# AssertionError: numpy.datetime64('NaT') != numpy.datetime64('NaT')
> self.assertIndexedLikeNDArray(x, d, None)
xray/test/test_variable.py:115:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
xray/test/test_variable.py:61: in assertIndexedLikeNDArray
self.assertEqual(variable.values[0], expected_value0)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
self = , a1 = numpy.datetime64('NaT'), a2 = numpy.datetime64('NaT')
def assertEqual(self, a1, a2):
> assert a1 == a2 or (a1 != a1 and a2 != a2)
E AssertionError: assert (numpy.datetime64('NaT') == numpy.datetime64('NaT') or (numpy.datetime64('NaT') != numpy.datetime64('NaT')))
xray/test/__init__.py:154: AssertionError
```
I don't get how neither of these are true:
```
(numpy.datetime64('NaT') == numpy.datetime64('NaT') or\
(numpy.datetime64('NaT') != numpy.datetime64('NaT')))
```
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147142509,https://api.github.com/repos/pydata/xarray/issues/618,147142509,MDEyOklzc3VlQ29tbWVudDE0NzE0MjUwOQ==,2443309,2015-10-11T00:59:04Z,2015-10-11T00:59:04Z,MEMBER,"> Previously, NaT == NaT was true. Now it's false, like how NaN works.
@shoyer - I don't think this is true.
``` Python
In [9]: import numpy as np
In [10]: t = np.datetime64('NaT')
In [11]: t == t
Out[11]: True
In [12]: n = np.nan
In [13]: n == n
Out[13]: False
In [14]: np.__version__
Out[14]: '1.10.0'
```
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147130696,https://api.github.com/repos/pydata/xarray/issues/618,147130696,MDEyOklzc3VlQ29tbWVudDE0NzEzMDY5Ng==,2443309,2015-10-10T22:30:22Z,2015-10-10T22:30:22Z,MEMBER,"I'll figure out something to do with the NaT comparison.
I think we're okay on Travis. We just have one failing test.
As for the verbosity of the numpy warnings. It is now just showing up in a lot more places (https://travis-ci.org/xray/xray/jobs/84683346#L665-L702). It seems like matplotlib and numpy itself haven't dealt with the warnings on their end yet.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147129634,https://api.github.com/repos/pydata/xarray/issues/618,147129634,MDEyOklzc3VlQ29tbWVudDE0NzEyOTYzNA==,1217238,2015-10-10T22:07:32Z,2015-10-10T22:07:32Z,MEMBER,"The ""numpy equal will not check object identity in the future"" warning is really annoying. The problem is that we put `NaN` in object arrays to indicate missing values (like pandas). Equals on object arrays currently applies the elementwise check `x is y or x == y`, which of course is different from `x == y` for NaN.
Of course, we have our own check for NaN which means this warning isn't relevant for us. If the warnings have gotten more verbose, I'll take another look. If you're interested in trying out some fixes, this is where they originate from: https://github.com/xray/xray/blob/a2fa27004f6cac476a3dbec2fa0774716f05f198/xray/core/ops.py#L183
I'll futz with the Travis-CI settings and see if I can get it installing happily again.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123
https://github.com/pydata/xarray/pull/618#issuecomment-147128970,https://api.github.com/repos/pydata/xarray/issues/618,147128970,MDEyOklzc3VlQ29tbWVudDE0NzEyODk3MA==,1217238,2015-10-10T22:01:26Z,2015-10-10T22:01:26Z,MEMBER,"NumPy fixed comparison operations with NaT in 1.10. Previously, `NaT == NaT` was true. Now it's false, like how NaN works.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110806123