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/4983#issuecomment-791089119,https://api.github.com/repos/pydata/xarray/issues/4983,791089119,MDEyOklzc3VlQ29tbWVudDc5MTA4OTExOQ==,6628425,2021-03-05T02:01:23Z,2021-03-05T02:01:23Z,MEMBER,"For sure, thanks for getting started on a PR!  I like the more informative `AttributeError` you added; I agree the existing error for the `time` attribute is difficult to follow.  Let me know when it is ready for review.  

Regarding `da.time.dt.floor(""D"")`, we could also consider adding a `normalize` method (e.g. [in pandas](https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Series.dt.normalize.html)) which is shorthand for the same thing.  ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,819897789
https://github.com/pydata/xarray/issues/4983#issuecomment-790718449,https://api.github.com/repos/pydata/xarray/issues/4983,790718449,MDEyOklzc3VlQ29tbWVudDc5MDcxODQ0OQ==,43613877,2021-03-04T15:52:38Z,2021-03-04T15:53:22Z,CONTRIBUTOR,"I didn't thought of using `da.time.dt.floor(""D"")`. This is indeed great to know, but as there seems to be more folks who are expecting `da.time.dt.date` to work, so I'd still like to see this implemented.

The `time` attribute that is already implemented has the same issue that it does not exists in `cftime`:
```python
import numpy as np
import pandas as pd
import xarray as xr

attrs = {""units"": ""hours since 3000-01-01""}
ds = xr.Dataset({""time"": (""time"", [0, 1, 2, 3], attrs)})
xr.decode_cf(ds).time.dt.time
# AttributeError: 'CFTimeIndex' object has no attribute 'time'
```
I implemented the `date` attribute in PR #4994. The usage of `date` and `CFTimeIndex` raises an explicit AttributeError now and mentions the usage of `floor(""D"")`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,819897789
https://github.com/pydata/xarray/issues/4983#issuecomment-789275247,https://api.github.com/repos/pydata/xarray/issues/4983,789275247,MDEyOklzc3VlQ29tbWVudDc4OTI3NTI0Nw==,6628425,2021-03-02T22:38:50Z,2021-03-02T22:38:50Z,MEMBER,"Yes, I agree, this seems reasonable.  It's true that there is no object equivalent to `datetime.date` in cftime, but I do not think that is necessarily a requirement.  We can raise an error in that instance.

I'll admit I personally have not found a need for functionality like this -- I typically prefer to use something like `da.time.dt.floor(""D"")`, since it retains the `np.datetime64` or cftime type -- but if folks would find it useful I think it makes sense to add (this has come up [on StackOverflow](https://stackoverflow.com/questions/65182281/xarray-dt-date-dt-normalize-missing/) before too).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,819897789
https://github.com/pydata/xarray/issues/4983#issuecomment-789033264,https://api.github.com/repos/pydata/xarray/issues/4983,789033264,MDEyOklzc3VlQ29tbWVudDc4OTAzMzI2NA==,10194086,2021-03-02T16:26:31Z,2021-03-02T16:26:31Z,MEMBER,Sounds reasonable. Or maybe that's not supported in cftime? cc @spencerkclark ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,819897789