-
Notifications
You must be signed in to change notification settings - Fork 937
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
Fix jog arm integrationtest CI / array out-of-bounds #1466
Conversation
and re-enable jog-arm integrationtest
drop hard-coded plain c array and directly add Eigen vectors without loop and start from non-singular robot state Fixes moveit#1435
@AndyZe please review. |
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.
Thanks! I verified that jogging still works and the tests still pass locally, too. I thought this issue would be more difficult to fix than this.
delta_x[3] = vector_toward_singularity[3] / scale; | ||
delta_x[4] = vector_toward_singularity[4] / scale; | ||
delta_x[5] = vector_toward_singularity[5] / scale; | ||
delta_x = vector_toward_singularity / scale; |
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.
That's a nice simplification
moveit_experimental/jog_arm/test/integration/test_jog_arm_integration.py
Outdated
Show resolved
Hide resolved
41a6d29
to
6861aec
Compare
It was still hard enough to track down. |
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.
Thanks for fixing the test.
* rename .test launch file and re-enable jog-arm integration test * fix array out-of-bounds access drop hard-coded plain c array and directly add Eigen vectors without loop * start from non-singular robot state
* rename .test launch file and re-enable jog-arm integration test * fix array out-of-bounds access drop hard-coded plain c array and directly add Eigen vectors without loop * start from non-singular robot state
Description
Drop hard-coded plain c array and directly add Eigen vectors without loop
and start from non-singular robot state. This fixes the travis CI integration test.
Checklist