Visitar URL original
Use existing np.arrays when calling `_qhull.delaunay` by iccir · Pull Request #32417 · matplotlib/matplotlib · GitHub
Skip to content

Use existing np.arrays when calling _qhull.delaunay - #32417

Merged
scottshambaugh merged 1 commit into
matplotlib:mainfrom
iccir:delaunay-arrays
Oct 1, 2026
Merged

scottshambaugh merged 1 commit into
matplotlib:mainfrom
iccir:delaunay-arrays

Conversation

@iccir

@iccir iccir commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

PR summary

Triangulation.__init__ converts array-like x and y arguments into numpy arrays and stores as self.x and self.y. However, it passes the original arrays to _qhull.delaunay, which converts again as part of pybind11 bridging.

Instead, pass the numpy arrays.

AI Disclosure

None used for this specific fix. Found while working on #32411 where I did use AI.

PR quality check

  • Use an expressive title, e.g. "Fix title font property precedence"
  • [/] New and changed code is tested
  • [/] Plotting related features are demonstrated in an example
  • [/] New features and API changes have release notes
  • [/] Documentation complies with general and docstring guidelines

@scottshambaugh
scottshambaugh merged commit 266c9a3 into matplotlib:main Oct 1, 2026
45 of 46 checks passed
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.

3 participants