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/4601#issuecomment-735210680,https://api.github.com/repos/pydata/xarray/issues/4601,735210680,MDEyOklzc3VlQ29tbWVudDczNTIxMDY4MA==,10194086,2020-11-28T10:16:04Z,2020-11-28T10:16:04Z,MEMBER,"I can open a PR but is there a clean way to handle accessors? Do you prefer `ABC`s or `NotImplementedError` for the missing methods in `DataWithCoords`?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,748684119 https://github.com/pydata/xarray/issues/4601#issuecomment-732440475,https://api.github.com/repos/pydata/xarray/issues/4601,732440475,MDEyOklzc3VlQ29tbWVudDczMjQ0MDQ3NQ==,10194086,2020-11-23T21:35:47Z,2020-11-23T21:35:47Z,MEMBER,"> Do we want other libraries which do da.longitude to raise a mypy error? That may be a tradeoff with raising the true error on da.jsel Good point. Given that the accessors are also going via `__getattr__` I would not remove the typing. Due to the accessors it also needs to be `-> Any`. In conclusion `DataWithCoords` only makes sense as a mixin. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,748684119