add reset_encoding to dataset/dataarray/variable#7689
Conversation
keewis
left a comment
There was a problem hiding this comment.
I was about to suggest naming it clear_encoding, but dict.clear works in-place, which I don't think we want to follow.
However, given #5336 (comment) we should probably discuss the general strategy in the meeting tomorrow before merging.
|
I think we can merge regardless of #5082. This is pretty useful right now |
|
In before a user asks for it, should we allow deleting only certain keys like |
Not opposed to this but do you think we'll really be asked for this? What sort of API do you envision? |
|
I agree that it may not be necessary. |
|
I see how we could do that on the variable but its a bit messy on the Dataset/DataArray. I suggest we wait until this is a requested feature. |
|
Sounds good to me! Thanks @jhamman |
The failure is a result of an xarray bug that can occur after subsetting data that was itself loaded from netcdf. See pydata/xarray#6352 for the issue and pydata/xarray#7689 for the fix used to create the workaround.
whats-new.rstapi.rst