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 flange frame #506

Conversation

gavanderhoorn
Copy link
Member

@gavanderhoorn gavanderhoorn commented Jun 11, 2020

As per subject.

This makes ur_description adhere to the (WIP) REP Coordinate Frames for Serial Industrial Manipulators and aligns it with other ROS-Industrial robot support packages (such as those for ABB and Fanuc).

Users will have to use flange as the EEF attachment point, instead of ee_link. tool0 will always be coincident with an unconfigured (or "all zeros") toolframe, and must not be changed.

This is for #448.

@gavanderhoorn
Copy link
Member Author

@fmauch: could I ask you to check the base->tool0 transform corresponds to what the TP shows?

I've had to redo/include some more rotations and want to make sure it's all correct.

@fmauch
Copy link
Contributor

fmauch commented Jun 15, 2020

@fmauch: could I ask you to check the base->tool0 transform corresponds to what the TP shows?

I've had to redo/include some more rotations and want to make sure it's all correct.

Yep, seems to work properly.

Also remove the tiny collision object, as it's no longer needed now that MoveIt has been fixed.
This is the correct EEF attachment point: REP-103 aligned and no unwanted rotations (as tool0).
Now that flange is its parent.
Aligned with other ROS-Industrial robot support packages.
@gavanderhoorn gavanderhoorn merged commit 61fcb59 into ros-industrial:melodic-devel-staging Jun 15, 2020
@gavanderhoorn gavanderhoorn deleted the add_flange_frame branch June 15, 2020 13:24
@gavanderhoorn
Copy link
Member Author

Thanks @fmauch for the review 👍

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.

2 participants