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

Add collision detection input param and test file #512

Open
wants to merge 7 commits into
base: indigo-devel
Choose a base branch
from

Conversation

yswhynot
Copy link

No description provided.

@k-okada
Copy link
Member

k-okada commented Jun 22, 2017

nice work, to keep backward compatibility, specially for real hardware, which do not have collision detection module at this moment (Pending at
#423) ,

@yswhynot
Copy link
Author

Hi @k-okada I've updated the files but I'm not quite sure about

add HRPSYS_PY_ARGS at -> https://github.com/start-jsk/rtmros_hironx/pull/512/files#diff-c61d113b474992382cc835f0eff0f44eR10 to turn on collision detection feature

what kind of args I need to put down?

@yswhynot
Copy link
Author

yswhynot commented Jul 4, 2017

@k-okada Oh it's been merged already, so I guess I can close this PR?

@k-okada
Copy link
Member

k-okada commented Jul 5, 2017

@yswhynot yes, you can close this PR, and I also create new PR based on your contribution at #515, this will run with collision detection enabled by roslaunch hironx_ros_bridge hironx_startup.launch

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