-
Notifications
You must be signed in to change notification settings - Fork 5
Update telesurgery from VDR feedback #188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
/build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the telesurgery workflow based on VDR feedback, reorganizing scripts and enriching documentation with demos and setup instructions.
- Renames and updates references to
surgeon/camera.pyassurgeon/viewer.pyand adjusts related command examples - Adds a new
gamepad.mdwith detailed MIRA gamepad control mappings - Expands the main README with demo GIFs, architecture diagram, real/ simulation environment setup, and updated script invocation
Reviewed Changes
Copilot reviewed 2 out of 9 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| workflows/telesurgery/docs/gamepad.md | New guide for gamepad controls, including mode, arm, and camera maps |
| workflows/telesurgery/README.md | Enhanced overview, demos, environment instructions, and script names |
Comments suppressed due to low confidence (1)
workflows/telesurgery/README.md:14
- [nitpick] Invalid HTML align value 'middle'; consider using
<div align="center">or native markdown for centering images.
<p align="middle">
Co-authored-by: Copilot <[email protected]> Signed-off-by: Mingxin Zheng <[email protected]>
Co-authored-by: Copilot <[email protected]> Signed-off-by: Mingxin Zheng <[email protected]>
|
/build |
Fixes #ISSUE_NUMBER
Description
workflows/telesurgery/docs, any concerns?