PointCloudLibrary / PointCloudLibrary/pcl
TextureMapping (textureMeshwithMultipleCameras) - Massive increase in performance upon adding openmp here
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 11.1k
- Forks
- 4.7k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 6
Description
I direct you to this section of the code:
We require this library for our system but a large amount of compute time is spent on each iteration of this particular loop. You can modify your code as such to receive almost a three times performance increase.
https://gist.github.com/soulslicer/a883485ee03b25a2e988f2a8d85d3903
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
Start at surface/include/pcl/surface/impl/texture_mapping.hpp around line 848, in the textureMeshwithMultipleCameras loop, and compare the current implementation with the linked gist. Benchmark that loop before and after the proposed OpenMP change; done means the performance improvement is confirmed without changing texture-mapping behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- computer-vision, performance
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100