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.

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

Add assisted grasp information to simulator state saving and restoring #278

Merged
merged 4 commits into from
Oct 26, 2023

Conversation

misoshiruseijin
Copy link
Contributor

  • Add AG state to simulator save and restore AG joint when restoring
  • Resolved datatype mismatch when restoring Tiago from JSON

@cgokmen cgokmen requested a review from cremebrule September 25, 2023 18:32
omnigibson/robots/manipulation_robot.py Outdated Show resolved Hide resolved
omnigibson/robots/manipulation_robot.py Outdated Show resolved Hide resolved
omnigibson/robots/tiago.py Outdated Show resolved Hide resolved
@misoshiruseijin
Copy link
Contributor Author

I have made the suggested changes and pushed. Thank you!

Copy link
Member

@ChengshuLi ChengshuLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cremebrule Would you mind taking a quick look at this PR as well? LGTM.

omnigibson/robots/tiago.py Outdated Show resolved Hide resolved
@ChengshuLi
Copy link
Member

@cremebrule Could you take brief look at this PR and merge it if you think it looks good? I already reviewed it.

@cgokmen
Copy link
Contributor

cgokmen commented Oct 26, 2023

I'm going to merge this now to avoid conflicts with the other PR which also does stuff with this.

@cgokmen cgokmen merged commit c5676c4 into og-develop Oct 26, 2023
2 of 3 checks passed
@cgokmen cgokmen deleted the save-restore-ag branch October 26, 2023 23:27
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.

3 participants