PointCloudLibrary / PointCloudLibrary/pcl
Deprecate TicToc in favor of StopWatch
Open
Nobody has claimed this yet.
good first issue
kind: todo
module: common
- Dominant language
- C++
- Stars
- 11.1k
- Forks
- 4.7k
- Avg merge
- 4d 10h
- Merged PRs (30d)
- 6
Description
Who knows... But we have a different set of classes in common/time.h. Perhaps this TicToc can be deprecated in favor of StopWatch...
Originally posted by @taketwo in https://github.com/PointCloudLibrary/pcl/pull/2913#issuecomment-472336272
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 by reading common/time.h and comparing the TicToc and StopWatch classes. Determine where TicToc is used and what deprecation scope is appropriate. Done means the intended replacement and any affected usage are consistently addressed, but the issue does not name tests or a more specific acceptance check.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100