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

[baxter_moveit_config] add OMPL + CHOMP pipeline launch #79

Open
wants to merge 1 commit into
base: kinetic-devel
Choose a base branch
from

Conversation

knorth55
Copy link
Contributor

@knorth55 knorth55 commented Jul 6, 2019

@knorth55 knorth55 changed the title [baxter_moveit_config] add OMPL + CHOMP pre-processor pipeline launch [baxter_moveit_config] add OMPL + CHOMP pipeline launch Jul 6, 2019
Copy link
Contributor

@rhaschke rhaschke left a comment

Choose a reason for hiding this comment

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

I'm not sure we want to merge all kinds of configuration variations for specific users.
These configs are basic examples only and you are responsible to handle your very own one yourself.
Other opinions?

@knorth55
Copy link
Contributor Author

knorth55 commented Jul 6, 2019

I can understand your point.
But I think it is good for users to have many example launches or sample launches, and try them easily.
As I mentioned in #78 (comment),
I think it is user-frindly to switch many moveit options by roslaunch args and try options from command line.
(However, I believe there would be some more user-frindly interface for samples than command line.)
This repo is for baxter moveit config, so it might be not good to add samples.
What do you think?

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