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

Replacing concaveman #308

Closed
schochastics opened this issue Oct 10, 2023 · 2 comments
Closed

Replacing concaveman #308

schochastics opened this issue Oct 10, 2023 · 2 comments

Comments

@schochastics
Copy link

concaveman is just a suggested package, but it is quite heavy in terms of dependencies if it needs to be installed.
There is potentially more lightweight option RcppConcaveman which wraps a C++ implementation of concaveman (unfortunatelly not on CRAN, but it seems quite easy to wrap the cpp code). It also comes with the added benefit of being more efficient.

@thomasp85
Copy link
Owner

Oh, nice - we should def explore this

@thomasp85
Copy link
Owner

hmmm... It appears that C++ port is severely bug-riddled as it misses points that ends up outside the hull, and makes up new points along the hull that was never there to begin with + it doesn't create the same result on multiple runs

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

No branches or pull requests

2 participants