Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename reset_encoding to drop_encoding #8287

Merged
merged 11 commits into from
Oct 12, 2023

Conversation

max-sixty
Copy link
Collaborator

Closes #8259

xarray/core/dataarray.py Outdated Show resolved Hide resolved
max-sixty and others added 2 commits October 9, 2023 06:30
Co-authored-by: Illviljan <14371165+Illviljan@users.noreply.github.com>
Copy link
Collaborator

@headtr1ck headtr1ck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should add this to api

xarray/core/dataarray.py Outdated Show resolved Hide resolved
xarray/core/dataset.py Outdated Show resolved Hide resolved
xarray/core/variable.py Outdated Show resolved Hide resolved
max-sixty and others added 6 commits October 11, 2023 22:33
Co-authored-by: Michael Niklas  <mick.niklas@gmail.com>
Co-authored-by: Michael Niklas  <mick.niklas@gmail.com>
Co-authored-by: Michael Niklas  <mick.niklas@gmail.com>
@max-sixty max-sixty added the plan to merge Final call for comments label Oct 12, 2023
@max-sixty
Copy link
Collaborator Author

The failures are unrelated, instead re #8296, so planning to merge this unless there are any comments...

@max-sixty max-sixty merged commit d50a5e5 into pydata:main Oct 12, 2023
28 of 30 checks passed
@max-sixty max-sixty deleted the drop-reset-encoding branch October 12, 2023 17:11
dcherian added a commit to dcherian/xarray that referenced this pull request Oct 14, 2023
* upstream/main: (46 commits)
  xfail flaky test (pydata#8299)
  Most of mypy 1.6.0 passing (pydata#8296)
  Rename `reset_encoding` to `drop_encoding` (pydata#8287)
  Enable `.rolling_exp` to work on dask arrays (pydata#8284)
  Fix `GroupBy` import (pydata#8286)
  Ask bug reporters to confirm they're using a recent version of xarray (pydata#8283)
  Add pyright type checker (pydata#8279)
  Improved typing of align & broadcast (pydata#8234)
  Update ci-additional.yaml (pydata#8280)
  Fix time encoding regression (pydata#8272)
  Allow a function in `.sortby` method (pydata#8273)
  make more args kw only (except 'dim') (pydata#6403)
  Use duck array ops in more places (pydata#8267)
  Don't raise rename warning if it is a no operation (pydata#8266)
  Mandate kwargs on `to_zarr` (pydata#8257)
  copy  the `dtypes` module to the `namedarray` package. (pydata#8250)
  Add xarray-regrid to ecosystem.rst (pydata#8270)
  Use strict type hinting for namedarray (pydata#8241)
  Update type annotation for center argument of dataaray_plot methods (pydata#8261)
  [pre-commit.ci] pre-commit autoupdate (pydata#8262)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan to merge Final call for comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should .reset_encoding be .drop_encoding?
3 participants