ros-controls / ros-controls/kinematics_interface
Comment on Heap Allocation for Realtime USe
Open
Nobody has claimed this yet.
persistent
question
- Dominant language
- C++
- Stars
- 50
- Forks
- 25
- Avg merge
- 5h 54m
- Merged PRs (30d)
- 11
Description
The following linked comment seems to label Eigen::Map as allocating memory on the heap. I understand Eigen::Map to provide a zero-copy view of the input memory, so there are no allocations on the heap. Is this an older comment, and should be removed? Or is my understanding about Eigen::Map incorrect?
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
Open kinematics_interface/src/kinematics_interface.cpp at line 31 and inspect the comment about Eigen::Map and heap allocation. Confirm the comment's accuracy against the surrounding usage, then update or remove it so the documentation matches the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100