Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add SO-ARM100 #105

Merged
merged 4 commits into from
Nov 23, 2024
Merged

Add SO-ARM100 #105

merged 4 commits into from
Nov 23, 2024

Conversation

chernyadev
Copy link
Contributor

@chernyadev chernyadev commented Oct 28, 2024 •
edited
Loading

Adding 5-DOF Standard Open Arm 100 v1.3, closes #104.

image

@chernyadev chernyadev mentioned this pull request Oct 28, 2024
Closed
@kevinzakka kevinzakka self-requested a review October 28, 2024 17:36
@kevinzakka
Copy link
Collaborator

Thanks @chernyadev, excellent PR!

@yuvaltassa This is ready to be merged.

@kevinzakka
Copy link
Collaborator

Hi @chernyadev, things got delayed due to CoRL. We'll get to this next week at the latest, thanks again for your contribution!

@copybara-service copybara-service bot merged commit 4ae825c into google-deepmind:main Nov 23, 2024
2 of 3 checks passed
@saran-t
Copy link
Member

saran-t commented Nov 24, 2024

Sorry, @kevinzakka just pointed out to me that I've misconfigured Copybara and commit 4ae825c contained no new files.

Will fix this ASAP.

copybara-service bot pushed a commit that referenced this pull request Nov 25, 2024
@saran-t @copybara-github
Actually commit the trs_so_arm100 directory.
a88bc45 
The previous commit 4ae825c which was supposed to have merged #105 was misconfigured internally, causing the new files from the PR to not be committed to the repo.

PiperOrigin-RevId: 699899488
Change-Id: I228162c2e0c0986cf00e81093b9006b54fa73822
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saran-t saran-t saran-t approved these changes

@kevinzakka kevinzakka kevinzakka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add SO-ARM 100
3 participants
@chernyadev @kevinzakka @saran-t