Skip to content

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