Skip to content

Adapt subframes tutorial#476

Merged
rhaschke merged 6 commits intomoveit:masterfrom
rhaschke:subframes-tutorial
Apr 27, 2020
Merged

Adapt subframes tutorial#476
rhaschke merged 6 commits intomoveit:masterfrom
rhaschke:subframes-tutorial

Conversation

@rhaschke
Copy link
Copy Markdown
Contributor

moveit/moveit#1757 fixes getFrameTransform() to return global transforms for subframes as well (before, it returned local frames w.r.t. the attach link).
However, this requires some adaptions to the target poses in the subframes tutorial.
I also added visualization of some target and eef frames.

Obviously, depends on moveit/moveit#1757

@rhaschke rhaschke requested a review from felixvd March 27, 2020 08:57
Copy link
Copy Markdown
Contributor

@felixvd felixvd left a comment

Choose a reason for hiding this comment

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

I couldn't test this today either, but if it runs it looks good to me mostly.

}
// END_SUB_TUTORIAL

void createArrowMarker(visualization_msgs::Marker& marker, const geometry_msgs::Pose& pose, const Eigen::Vector3d& dir,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Isn't there something in rviz_visual_tools for this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was hoping for this as well. But, unfortunately, rviz_visual_tools can only handle a single reference frame_id for markers.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It could be added to rvt if you would like

orientation.setRPY(0, (-20.0 / 180.0 * M_PI), 0);
fixed_pose.pose.orientation = tf2::toMsg(orientation);

// BEGIN_SUB_TUTORIAL quaternions1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like how clean the file is now, and there must be a better place for this quaternion stuff, but I'm sure that a refresher on how to handle rotations with TF will really help some newbies out here. Do you have any good materials to link? Worst case, a comment like this may help: "Note that you can split up your rotations into intermediate steps and multiply tf2 quaternions<http://docs.ros.org/api/tf2/html/classtf2_1_1Quaternion.html>_ to create your desired orientation more easily."

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am using http://wiki.ros.org/agni_tf_tools to interactively setup my rotations.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In my opinion that's definitely worth linking. It's not easy for newcomers to find these kinds of tools.

@rhaschke rhaschke merged commit b38517a into moveit:master Apr 27, 2020
@rhaschke rhaschke deleted the subframes-tutorial branch April 27, 2020 22:55
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