dimensionalOS / dimensionalOS/dimos
Standardize tf frame prefixing
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 4.5k
- Forks
- 808
- Avg merge
- 3d 5h
- Merged PRs (30d)
- 233
Description
RealSense already prefixes its ten frame names, but it does it by hand off its own camera_name config (camera/realsense/camera.py:138-176), so the pattern isn't standard and modules like PointLio get no prefixing at all (pointlio/module.py:185 broadcasts sensor_frame_id raw). All of a module's frames should live in one frame dictionary so every key can be prefixed reliably in one place.
Synced from DIM-1470 by jeff
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 with camera/realsense/camera.py:138-176 to understand the existing hand-written prefixing, then inspect pointlio/module.py:185 where sensor_frame_id is broadcast raw. Trace how each module stores and broadcasts frame names. Done means module frames share one frame dictionary and every key is prefixed consistently in one place.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- robotics
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100