PointCloudLibrary / PointCloudLibrary/pcl
[common] Rename and deprecate run method in the random generators
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 11.1k
- Forks
- 4.7k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 6
Description
Deprecate and rename the pcl::common::UniformGenerator< T >::run method for pcl::common::UniformGenerator< T >::operator()
Is your feature request related to a problem? Please describe.
I was reviewing #3867 and the syntax looked moderately ugly.
There might be a case to be made to actually deprecate them both generators in favor of what the std library is providing.
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
Locate pcl::common::UniformGenerator::run in the common library and review the related syntax in #3867. Confirm how both generator APIs should be handled and what compatibility behavior is expected. Done means the requested naming and deprecation behavior are defined and covered by the relevant tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100