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/3291#issuecomment-824354016,https://api.github.com/repos/pydata/xarray/issues/3291,824354016,MDEyOklzc3VlQ29tbWVudDgyNDM1NDAxNg==,1217238,2021-04-21T20:59:18Z,2021-04-21T21:00:41Z,MEMBER,"> If `ds[""time""]` is a 1D indexed coordinate, is `ds[""time""].data ≡ ds.indexes[""time""].data`? If so, that would just be a `pd.DatetimeIndex` which is timezone-aware and then this problem is solved because we don't maintain a separate numpy array. Am I understanding this correctly?
No, unfortunate it is not possible to use a `pandas.Index` directly inside `Variable.data`, because pandas.Index is not compatible with the NumPy array API -- in particular it is stuck with 1D data. Instead, we will need to wrap the array in some adapter class to make it compatible. Ideally this wrapper would be a fully N-dimensional wrapper for `pandas.Series` objects, but for a first pass it would probably be fine to raise an error if indexing would create a higher dimensional array.
The bigger issue is that elsewhere in Xarray probably needs updates to avoid assuming that all dtype objects are `numpy.dtype` instances.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490618213
https://github.com/pydata/xarray/issues/3291#issuecomment-824146151,https://api.github.com/repos/pydata/xarray/issues/3291,824146151,MDEyOklzc3VlQ29tbWVudDgyNDE0NjE1MQ==,2448579,2021-04-21T15:18:57Z,2021-04-21T15:18:57Z,MEMBER,"I am confused on the following point after reading the [indexing refactor design notes on removing IndexVariable](https://github.com/pydata/xarray/blob/master/design_notes/flexible_indexes_notes.md#4-indexvariable).
If `ds[""time""]` is a 1D indexed coordinate, is `ds[""time""].data ≡ ds.indexes[""time""].data`? If so, that would just be a `pd.DatetimeIndex` which is timezone-aware and then this problem is solved because we don't maintain a separate numpy array. Am I understanding this correctly?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490618213
https://github.com/pydata/xarray/issues/3291#issuecomment-823833533,https://api.github.com/repos/pydata/xarray/issues/3291,823833533,MDEyOklzc3VlQ29tbWVudDgyMzgzMzUzMw==,3924836,2021-04-21T07:12:11Z,2021-04-21T07:13:45Z,MEMBER,"Just wanted to rekindle discussion here and ping @dcherian and @benbovy , the current workaround for pandas DatetimeIndex with timezone info (dtype='datetime64[ns, EST]') is to drop the timezone piece or use `to_index()` and operate in pandas, then reassign the time coordinate: See https://github.com/pydata/xarray/issues/1036 and https://github.com/pydata/xarray/issues/3163.
If I'm following https://github.com/pydata/xarray/blob/master/design_notes/flexible_indexes_notes.md this is another potential example of improved user-friendliness where we could have timezone-aware indexes and therefore call pandas methods like `pandas.core.indexes.datetimes.DatetimeIndex.tz_convert()` directly as a DataArray method?
This would definitely be great for remote sensing data that is usually stored with UTC timestamps, but often analysis requires converting to local time.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490618213
https://github.com/pydata/xarray/issues/3291#issuecomment-531522690,https://api.github.com/repos/pydata/xarray/issues/3291,531522690,MDEyOklzc3VlQ29tbWVudDUzMTUyMjY5MA==,1217238,2019-09-15T00:00:37Z,2019-09-15T00:00:37Z,MEMBER,"You should be getting a warning about this if you use the latest version of pandas. In the future, this behavior will change to return an object dtype array full of pandas Datetime objects. Unfortunately NumPy doesn't have a built-in datetime with time-zone stype, so this is about the best we can do.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,490618213