-
Notifications
You must be signed in to change notification settings - Fork 101
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
Additional visualization #31
Conversation
please add better PR title in the future |
src/moveit_visual_tools.cpp
Outdated
if (!collision_obj.primitives[0].dimensions[shape_msgs::SolidPrimitive::BOX_Z]) | ||
collision_obj.primitives[0].dimensions[shape_msgs::SolidPrimitive::BOX_Z] = rviz_visual_tools::SMALL_SCALE; | ||
|
||
// ROS_INFO_STREAM_NAMED(name_,"CollisionObject: \n " << collision_obj); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove
@@ -347,6 +347,12 @@ class MoveItVisualTools : public rviz_visual_tools::RvizVisualTools | |||
const std::string& name, | |||
const rviz_visual_tools::colors& color = rviz_visual_tools::GREEN); | |||
|
|||
bool publishCollisionCuboid(const Eigen::Affine3d& pose, double x, double y, double z, const std::string& name, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add function documentation
* \param pose - position of the centroid of the cube | ||
* \param x - width | ||
* \param y - depth | ||
* \param z - height |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the standard in this file is to call these variables width, depth, height - not x,y,z which imply position
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
1a05bba
to
d16e3cf
Compare
* adding a visualization for publishing a box with width, height, and depth * adding additional publishCollisionCuboid method overloads * changing x,y,z to width, depth, height
cherry-picked to M |
@davetcoleman @mlautman Cherry-picking this to Melodic broke both this repo's melodic-branch as well as MoveIt's Travis (which is pulling in moveit_visual_tools via rosinstall file): Please fix. |
@rhaschke Working on it |
Just did a post-mortem with @mlautman and the takeaway is when doing direct cherry-picks between distros, check for anything TF related and if that is mentioned, do a full PR |
I also ran clang formatting