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

FIX: use "krogh" as interpolator method-string instead of "krog" #8187

Merged
merged 3 commits into from
Sep 16, 2023

Conversation

kmuehlbauer
Copy link
Contributor

While working on #8146 this came up. Aligning with upstream naming convention.

  • Closes #xxxx
  • Tests added
  • User visible changes (including notable bug fixes) are documented in whats-new.rst
  • New functions/methods are listed in api.rst

Co-authored-by: Deepak Cherian <[email protected]>
@dcherian dcherian added the plan to merge Final call for comments label Sep 16, 2023
@dcherian dcherian merged commit 45bce44 into pydata:main Sep 16, 2023
max-sixty pushed a commit to max-sixty/xarray that referenced this pull request Sep 17, 2023
…ata#8187)

* FIX: use "krogh" as interpolator method-string instead of "krog"

* Update xarray/tests/test_missing.py

Co-authored-by: Illviljan <[email protected]>

* Update xarray/core/missing.py

Co-authored-by: Deepak Cherian <[email protected]>

---------

Co-authored-by: Illviljan <[email protected]>
Co-authored-by: Deepak Cherian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants