[Gkong5020]

Camera Research

Summary

• Added a README to summarize the main camera system design document
• Included older, initial research documents
• Contains detailed system design prototype with initial testing and camera recommendations

---

Controller research

Summary

• Compared two 3D controllers that could potentially work with the robot
• Weighed the pros and cons of each controller and decided on a main recommendation and fallback
• Mapped controls to each controller

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 9ee67dfd-ae7e-4382-a3a7-4db454381214

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[yassinsolim]

Good research, Gianna: the camera design, the Orbbec Gemini 2 recommendation, the controller comparison and the input mapping are all useful. A few points before it can merge:

1. docs/camera-research/README.md line 30 says the latency should "remain consistently >30-50ms". It should be below 30 to 50 ms.
2. Line 11 links to doctor_camera_system_design.md on your branch, which breaks after the merge. Please use a relative link.
3. Spelling: Electrial, seperate, attatched, exaxtly, ergonimic and qual-core.
4. Please add the measured delay. Run the doctor-view feed from #36 with a USB webcam for five minutes, at 1080p and 30 fps if the camera supports it, and report the average, minimum and maximum delay and the jitter, with a stopwatch cross-check. Note which numbers depend on the webcam rather than the Gemini 2.

Then re-request a review.

[yassinsolim]

Thanks, Gianna. Points 1 to 3 are fixed: the latency target, the relative link and the spelling. One new typo: doctor_camera_system_design.md has a double space in "Directly attached to arm".

Point 4 is still open. Add the measured delay from the doctor-view feed, which is on main now (camera:=true). Then re-request a review.