gridap / gridap/MiniQhull.jl

Implement `delaunay!`, in-place version of `delaunay`

Open
#15 1 comment 0 reactions 0 assignees View on GitHub
help wanted
Dominant language
Julia
Stars
31
Forks
9
PR merge metrics
No merged PRs in 30d

Description

Now, `delaunay` allocates the output. It would be nice to have the corresponding in-place version `delaunay!`, where the user provides the output. A part of performance reasons, the in-place version would allow the user to select the container type (as long as it has the correct memory layout).

Once `delaunay!` is available, `delaunay` can be implemented by calling `delaunay!`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the existing `delaunay` implementation and its surrounding API to understand how output is allocated and represented. Add a `delaunay!` entry point that accepts a caller-provided container with the required memory layout, then make `delaunay` use it while preserving its current behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
julia
Domain
computer-graphics
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.