PointCloudLibrary / PointCloudLibrary/pcl
[surface] Update poisson library/Use it as external library
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 11.1k
- Forks
- 4.7k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 6
Description
Currently we have a version of the poisson library within the code. As the directory is named poission4 I think the code could be based on Poisson V4 - but the current version is 13.74 (see here). Currently they are working on a CMake integration (see https://github.com/mkazhdan/PoissonRecon/pull/231), so maybe we could integrate this as external dependency, so we do not need to maintain this code anymore (e.g. it contains some compiler warnings).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the existing poission4 directory and compare it with PoissonRecon version 13.74. Review the referenced PoissonRecon CMake integration pull request (#231) first. Done means PCL uses PoissonRecon as an external dependency instead of maintaining the current internal code, with the existing compiler warnings addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, computer-vision
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100