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/issues/1449#issuecomment-497130146,https://api.github.com/repos/pydata/xarray/issues/1449,497130146,MDEyOklzc3VlQ29tbWVudDQ5NzEzMDE0Ng==,2448579,2019-05-29T22:21:55Z,2019-05-29T22:21:55Z,MEMBER,still an issue...,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235224055
https://github.com/pydata/xarray/issues/1449#issuecomment-308105236,https://api.github.com/repos/pydata/xarray/issues/1449,308105236,MDEyOklzc3VlQ29tbWVudDMwODEwNTIzNg==,10194086,2017-06-13T12:50:38Z,2017-06-13T12:50:38Z,MEMBER,"Well, not specifying the time would conform to the CF conventions... (http://cfconventions.org/cf-conventions/v1.6.0/cf-conventions.html#time-coordinate), so this might be an issue in panoply.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235224055
https://github.com/pydata/xarray/issues/1449#issuecomment-307839267,https://api.github.com/repos/pydata/xarray/issues/1449,307839267,MDEyOklzc3VlQ29tbWVudDMwNzgzOTI2Nw==,1217238,2017-06-12T16:16:07Z,2017-06-12T16:16:07Z,MEMBER,"I didn't realize that some applications would have trouble with time-offsets without hours, but we can easily change this. The logic in controlled by [these](https://github.com/pydata/xarray/blob/aeb4108b1f39cde2bac765ca58c70b5011b188c5/xarray/conventions.py#L208) [lines](https://github.com/pydata/xarray/blob/aeb4108b1f39cde2bac765ca58c70b5011b188c5/xarray/conventions.py#L233) (called [from here](https://github.com/pydata/xarray/blob/aeb4108b1f39cde2bac765ca58c70b5011b188c5/xarray/conventions.py#L280-L283)), as you can see use different helper functions for formatting the time-offset string -- either `format_timestamp(x)` or `str(pd.Timestamp(x))`.

Either way, you certainly have identified as inconsistency. It doesn't make sense to ""clean-up"" manually specified time units to add the ""T"" but not include that in our auto-generated time units. We should probably switch our time units to include ""T"" because that's ISO-8601, and I'm no longer sure why we normalize time units at all rather than just writing them unmodified.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,235224055