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/523#issuecomment-131187220,https://api.github.com/repos/pydata/xarray/issues/523,131187220,MDEyOklzc3VlQ29tbWVudDEzMTE4NzIyMA==,2443309,2015-08-14T17:22:02Z,2015-08-14T17:22:02Z,MEMBER,"I submitted an feature request over at numpy. I'm going to close this now. @rabernat - thanks for the PR and keep them coming. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,99847237 https://github.com/pydata/xarray/pull/523#issuecomment-131166492,https://api.github.com/repos/pydata/xarray/issues/523,131166492,MDEyOklzc3VlQ29tbWVudDEzMTE2NjQ5Mg==,2443309,2015-08-14T16:17:10Z,2015-08-14T16:17:10Z,MEMBER,"> Worth? Yes. Any hope to actually get it in there? No... I think I disagree. There is almost no chance anyone outside of the climate community is going to spend time on this but, if calendar support was added in a responsible way to numpy and pandas, I don't see why they wouldn't be interested. So it will need to come from the climate users community, which IMHO, is under represented in the dev community. If you don't want open the issue, I will. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,99847237 https://github.com/pydata/xarray/pull/523#issuecomment-129101660,https://api.github.com/repos/pydata/xarray/issues/523,129101660,MDEyOklzc3VlQ29tbWVudDEyOTEwMTY2MA==,2443309,2015-08-09T03:43:09Z,2015-08-09T03:43:09Z,MEMBER,"I have a few general comments, 1. Can you point me to where in the [CF Conventions](http://cfconventions.org/Data/cf-conventions/cf-conventions-1.6/build/cf-conventions.html#time-coordinate) or [UDUNITS](http://www.unidata.ucar.edu/software/udunits/#documentation) the valid time coordinate units defined? 2. We should think about whether or not this fix belongs in xray or netCDF4. I am of the opinion that if the CF Conventions do in fact support the units in question, we should apply this fix in the netCDF package. If they don't, I don't think we wan to support it here either. 3. @shoyer is out for a bit but he will almost certainly want to weigh in on this. If we end up going this route, you'll want to add some unit tests in [test_conventions.py](https://github.com/xray/xray/blob/master/xray/test/test_conventions.py#L234-L254). ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,99847237