Skip to content

Add tutorial about dual arm controller#499

Open
chuoru wants to merge 29 commits intojrl-umi3218:masterfrom
HACHIX-CORPORATION:master
Open

Add tutorial about dual arm controller#499
chuoru wants to merge 29 commits intojrl-umi3218:masterfrom
HACHIX-CORPORATION:master

Conversation

@chuoru
Copy link

@chuoru chuoru commented Jan 21, 2026

This modification add tutorial for dual arm system using UR5e as the main robot and Kinova as the second robot.
Requested by: @fkanehiro

@fkanehiro
Copy link
Contributor

@ThomasDuvinage @phamthekhang632
Please review this

@chuoru
Copy link
Author

chuoru commented Jan 21, 2026

@ThomasDuvinage @phamthekhang632
In the last commit 0ffc04a, there is modification outside of my scope due to pre-commit clang format, I wonder why it is not fixed from previous commit from main branch.

@ThomasDuvinage
Copy link
Contributor

@chuoru Thanks for the documentation !
Indeed pre-commit haven't been applied to the latests commits.

@ThomasDuvinage ThomasDuvinage self-requested a review January 21, 2026 07:51
@chuoru
Copy link
Author

chuoru commented Jan 21, 2026

@ThomasDuvinage @phamthekhang632
Also, this repository https://github.com/HACHIX-CORPORATION/mc_rtc_tutorials provides environment setup with docker which is easier to run the controller. Please check.

@chuoru
Copy link
Author

chuoru commented Feb 19, 2026

@ThomasDuvinage @phamthekhang632
How about this pull request? Do you have any request for modifcation?

@chuoru
Copy link
Author

chuoru commented Feb 24, 2026

@ThomasDuvinage @phamthekhang632
I have re-applied pre-comit for this pull request, as I confirmed the remains pull request is this one and this one: isri-aist/dingo#1.
Please consider them.

@ThomasDuvinage
Copy link
Contributor

@chuoru thanks
We have a meeting with @phamthekhang632 to close this PR

@chuoru chuoru closed this Feb 27, 2026
@chuoru chuoru reopened this Feb 27, 2026
@chuoru
Copy link
Author

chuoru commented Mar 2, 2026

@ThomasDuvinage @phamthekhang632
Is any progress on the review? We need to close the project on our side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants