Skip to content

DOC: interpolate: add a note against using triangulation based intrpolators on a regular grid #17906

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

Merged
merged 1 commit into from
Feb 18, 2023

Conversation

L-Sandler
Copy link
Contributor

Reference issue

Attempting to close #17378

What does this implement/fix?

Increases clarity for 4 multivariate interpolation functions.

  • scipy.interpolate.griddata
  • scipy.interpolate.LinearNDInterpolator
  • scipy.interpolate.NearestNDInterpolator
  • scipy.interpolate.CloughTocher2DInterpolator

Additional information

Some tests failed on my local environment, but I was instructed to send a PR by this message.

@L-Sandler L-Sandler requested a review from ev-br as a code owner February 1, 2023 16:05
@j-bowhay j-bowhay added scipy.interpolate Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org labels Feb 1, 2023
@j-bowhay
Copy link
Member

j-bowhay commented Feb 1, 2023

Lint failures here are unrelated and due to recent infrastructure changes

@ev-br
Copy link
Member

ev-br commented Feb 1, 2023

Would also be nice to add RegularGridInterpolator and interpn to See Also sections.

@ev-br ev-br changed the title DOC: See ticket #17378 for background. DOC: interpolate: add a note against using triangulation based intrpolators on a regular grid Feb 1, 2023
@ev-br
Copy link
Member

ev-br commented Feb 18, 2023

My last comment on adding RGI and interpn to see also sections can be addressed in a follow-up; this PR is a clear improvement, so let's merge it as is. Thank you @L-Sandler and congrats with what I believe is your first scipy PR. Keep them coming!

@ev-br ev-br merged commit 5b07737 into scipy:main Feb 18, 2023
@ev-br ev-br added this to the 1.11.0 milestone Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org scipy.interpolate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

griddata linear / LinearNDInterpolator unexpected behavior
3 participants