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/1271#issuecomment-280838479,https://api.github.com/repos/pydata/xarray/issues/1271,280838479,MDEyOklzc3VlQ29tbWVudDI4MDgzODQ3OQ==,6815844,2017-02-18T11:03:42Z,2017-02-18T11:03:42Z,MEMBER,"> I would strong prefer to avoid making the xarray data model more complex by adding another type of metadata (e.g., in addition to dims, coords, attrs and name on DataArray). Whitelisting specific attrs as ones that should be preserved in application code via set_options (or encouraging users to subclass DataArray in a ""safe"" way) is preferable in that regard.

Understood.
I close this issue.
I guess discussions for how this option would be implemented could be continued in #988.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280582281,https://api.github.com/repos/pydata/xarray/issues/1271,280582281,MDEyOklzc3VlQ29tbWVudDI4MDU4MjI4MQ==,10050469,2017-02-17T08:06:02Z,2017-02-17T08:06:02Z,MEMBER,"> I wonder if it might make sense to represent crs as a (scalar) coordinate, which would already be propagated in all unambiguous cases by xarray ops.

This looks like a brilliant idea, thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280574680,https://api.github.com/repos/pydata/xarray/issues/1271,280574680,MDEyOklzc3VlQ29tbWVudDI4MDU3NDY4MA==,1217238,2017-02-17T07:25:20Z,2017-02-17T07:28:29Z,MEMBER,"> Coordinate reference systems (crs) are an example of attribute which is always valid after selection, arithmetic, or even reduction since they apply to the coordinates of the DataArray.

I wonder if it might make sense to represent `crs` as a (scalar) coordinate, which would already be propagated in all unambiguous cases by xarray ops.

The main reason why this is annoying is that `ds.crs` would return a 0-dimensional DataArray, so you would need to write `ds.crs.item()` to access it directly (e.g., if you want to call a method on it).  Though you could actually already use the accessor interface to fix this (see [this gist for details](https://gist.github.com/shoyer/26986d85cae05f5b5211eb3f3ebc0851)).

> Could the setting not be module-wide, but be set on DataArray instances themselves? That setting could then be inherited when new ones are created from arithmetic operations.

I would strong prefer to avoid making the xarray data model more complex by adding another type of metadata (e.g., in addition to `dims`, `coords`, `attrs` and `name` on DataArray). Whitelisting specific `attrs` as ones that should be preserved in application code via `set_options` (or encouraging users to subclass DataArray in a ""safe"" way) is preferable in that regard.

Either way, this issue is related to the fuller ""hook"" system for customized attribute handling that I imagined over in #988.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280378409,https://api.github.com/repos/pydata/xarray/issues/1271,280378409,MDEyOklzc3VlQ29tbWVudDI4MDM3ODQwOQ==,221526,2017-02-16T16:19:21Z,2017-02-16T16:19:21Z,CONTRIBUTOR,"Could the setting not be module-wide, but be set on `DataArray` instances themselves? That setting could then be inherited when new ones are created from arithmetic operations.

I'd like to have the option of specifying specific attributes that should be kept, or maybe dropped. It will be *really* annoying to need to keep copying the `grid_mapping` attribute.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280259483,https://api.github.com/repos/pydata/xarray/issues/1271,280259483,MDEyOklzc3VlQ29tbWVudDI4MDI1OTQ4Mw==,10050469,2017-02-16T08:00:19Z,2017-02-16T08:00:19Z,MEMBER,"> divide attrs into two kinds, units and metadata

Coordinate reference systems (crs) are an example of attribute which is always valid after selection, arithmetic, or even reduction since they apply to the coordinates of the DataArray. A global option to keep certain attrs (like ``crs``) would be very useful to downstream libraries like salem. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280252632,https://api.github.com/repos/pydata/xarray/issues/1271,280252632,MDEyOklzc3VlQ29tbWVudDI4MDI1MjYzMg==,6815844,2017-02-16T07:24:14Z,2017-02-16T07:24:14Z,MEMBER,"Thank you for the information.
I agree that we should not strongly rely on attrs. Unit may change in arithmetic.

My sense is closest to the option 3 in #131.
Some attrs should be tracked and other should be dropped.
In the present stage, the most possible option is to add `keep_attrs` in `xr.set_options`?
Or any destructive change will be an option?
Such as to divide attrs into two kinds, `units` and `metadata`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322
https://github.com/pydata/xarray/issues/1271#issuecomment-280208293,https://api.github.com/repos/pydata/xarray/issues/1271,280208293,MDEyOklzc3VlQ29tbWVudDI4MDIwODI5Mw==,1217238,2017-02-16T02:11:22Z,2017-02-16T02:12:25Z,MEMBER,"Historically, we dropped attributes in arithmetic because attributes are often used for units and we didn't want to do computation that results in the wrong units. Blinding propagating metadata can lead to it ending up in the wrong place.

Also, if you are combining multiple DataArray objects with different `attrs`, there are a number of options for combining them, and it wasn't obvious which is the right strategy.

But in practice, this is something that a lot of people want (better to have stale metadata than no metadata at all), so maybe I made the wrong choice here. At the very least, I would be comfortable with [an option](http://xarray.pydata.org/en/stable/generated/xarray.set_options.html) to set `keep_attrs=True` for every operation and/or to specify a merge strategy for combining `attrs` in arithmetic. (I guess I've [changed my opinion](https://github.com/pydata/xarray/issues/131#issuecomment-43359850) from when rejected this proposal from @jhamman several years ago.)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,207962322