Skip to content

Merge upstream into this repo#8

Merged
egordon merged 2 commits intopersonalrobotics:masterfrom
AndrejOrsula:master
Aug 15, 2023
Merged

Merge upstream into this repo#8
egordon merged 2 commits intopersonalrobotics:masterfrom
AndrejOrsula:master

Conversation

@egordon
Copy link

@egordon egordon commented Aug 15, 2023

@amalnanavati Lmk if there are any issues with the new argument, I don't see why there would be.

AndrejOrsula and others added 2 commits June 6, 2023 23:40
Signed-off-by: Andrej Orsula <orsula.andrej@gmail.com>
Although `target_link` is a parameter to `move_to_pose`, it isn't used if `self.__execute_via_moveit` is False. This commit addresses that.
@egordon egordon requested a review from amalnanavati August 15, 2023 16:58
Copy link

@amalnanavati amalnanavati left a comment

Choose a reason for hiding this comment

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

Oh yeah there won't be an issue, I was the one who added the new argument haha. Thanks for updating!

@egordon egordon merged commit 24465de into personalrobotics:master Aug 15, 2023
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