-
Notifications
You must be signed in to change notification settings - Fork 88
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
Create a common config for collision checking #448
Create a common config for collision checking #448
Conversation
@Levi-Armstrong I'd like to get your thoughts on this and the comments in #427. |
@Levi-Armstrong Any idea offhand what is going on with descartes? |
Oh, I bet I have to cast some things to FloatType |
@Levi-Armstrong if this passes it is ready for review. I have not fully converted the planners to use it, but I think that might come after #443 |
56f7a88
to
789b5c4
Compare
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.
A few minor things, looks good.
.../include/tesseract_motion_planners/descartes/impl/profile/descartes_default_plan_profile.hpp
Outdated
Show resolved
Hide resolved
.../include/tesseract_motion_planners/descartes/impl/profile/descartes_default_plan_profile.hpp
Show resolved
Hide resolved
.../include/tesseract_motion_planners/descartes/impl/profile/descartes_default_plan_profile.hpp
Show resolved
Hide resolved
789b5c4
to
fcf981b
Compare
I just deleted Verbosity from tesseract_common and squashed everything |
fcf981b
to
fd08265
Compare
Added the todos for the XML |
@Levi-Armstrong Ok with me merging? |
Sure |
Will Address #427