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 interpolation documentation #1346

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

fgallardo-mpie
Copy link
Contributor

@fgallardo-mpie fgallardo-mpie commented Jun 1, 2021

Overview

Change documentation of interpolation to correctly describe the method for getting points and the method for weighing those points.

Resolves Issue #1336

Details

new description is

    n_points : int, optional
        If given, specifies the number of the closest points used to form
        the interpolation basis. This will invalidate the radius argument
        in favor of an N closest points approach. This typically has poorer
        results.

@codecov
Copy link

codecov bot commented Jun 1, 2021

Codecov Report

Merging #1346 (7ab4bbb) into master (a4f148d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1346   +/-   ##
=======================================
  Coverage   90.01%   90.01%           
=======================================
  Files          56       56           
  Lines       12188    12188           
=======================================
  Hits        10971    10971           
  Misses       1217     1217           

@GuillaumeFavelier GuillaumeFavelier merged commit c892560 into pyvista:master Jun 1, 2021
@GuillaumeFavelier
Copy link
Contributor

Thank you @fgallardo-mpie !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants