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/2773#issuecomment-841508030,https://api.github.com/repos/pydata/xarray/issues/2773,841508030,MDEyOklzc3VlQ29tbWVudDg0MTUwODAzMA==,14808389,2021-05-14T21:16:04Z,2021-05-14T21:16:04Z,MEMBER,"it might be better to not special-case the `""units""` attribute, so this should be fixed by `_repr_inline_`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,411365882
https://github.com/pydata/xarray/issues/2773#issuecomment-670015358,https://api.github.com/repos/pydata/xarray/issues/2773,670015358,MDEyOklzc3VlQ29tbWVudDY3MDAxNTM1OA==,14808389,2020-08-06T15:54:38Z,2020-08-06T16:36:28Z,MEMBER,"#4248 pushes the formatting to duck arrays, so unit-aware arrays like `pint` can provide a function to properly format itself. If we still want to format `units` attributes, we'd need to reopen this.
Edit: reopened it until we have a decision about formatting the `units` attributes","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,411365882
https://github.com/pydata/xarray/issues/2773#issuecomment-662110153,https://api.github.com/repos/pydata/xarray/issues/2773,662110153,MDEyOklzc3VlQ29tbWVudDY2MjExMDE1Mw==,14808389,2020-07-21T21:14:46Z,2020-07-21T21:15:31Z,MEMBER,"Instead of trying to come up with our own formatting, how about supporting a `_repr_short_(self, length)` method on the duck array (with a fall back to the current behavior)? That way duck arrays have to explicitly define the format (or have a compatibility package like `pint-xarray` provide it for them) if they want something different from their normal repr and we don't have to add duck array specific code.
This won't help with displaying the `units` attributes (which we don't really need once we have support for pint arrays in indexes).","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,411365882
https://github.com/pydata/xarray/issues/2773#issuecomment-565518795,https://api.github.com/repos/pydata/xarray/issues/2773,565518795,MDEyOklzc3VlQ29tbWVudDU2NTUxODc5NQ==,14808389,2019-12-13T16:57:36Z,2019-12-13T16:57:36Z,MEMBER,"it was argued in pint that the unit is part of the data, so we should keep it as close to the data as possible. How about
```
Dimensions: (time: 3, x: 988, y: 822)
Coordinates:
* x (x) [m] float64 ...
* y (y) [m] float64 ...
* time (time) [s] datetime64[ns] ...
Data variables:
rainfall (time, y, x) [mm] float32 ...
max_temp (time, y, x) [deg C] float32 ...
```
or
```
Dimensions: (time: 3, x: 988, y: 822)
Coordinates:
* x (x) float64 [m] ...
* y (y) float64 [m] ...
* time (time) datetime64[ns] [s] ...
Data variables:
rainfall (time, y, x) float32 [mm] ...
max_temp (time, y, x) float32 [deg C] ...
```
The issue with the second example is that it is easy to confuse with numpy's dtype, though. Maybe we should use parentheses instead?
re special casing: I think would be fine for attributes since we already special case them for plotting, but I don't know about duck arrays. Even if we want to special case them, there are many unit libraries with different interfaces so we would either need to special case all of them or require a specific interface (or a function to retrieve the necessary data?).
Also, we should keep in mind is that using more horizontal space for the units results in less space for data. And we should not forget about https://github.com/dask/dask/issues/5329#issue-485927396, where a different kind of format was proposed, at least for the values of a `DataArray`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,411365882