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
planner_cspace: fix pose duplication on path interpolation #648
planner_cspace: fix pose duplication on path interpolation #648
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Codecov Report
@@ Coverage Diff @@
## master #648 +/- ##
=======================================
Coverage 90.51% 90.51%
=======================================
Files 61 61
Lines 4556 4556
=======================================
Hits 4124 4124
Misses 432 432
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
LGTM. Minor comments added.
map_info.origin.position.y = -200.5; | ||
|
||
CyclicVecFloat<3, 2> m_prev; | ||
for (auto it = out.begin(); it != out.end(); ++it) |
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.
const auto
is better.
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.
++it
requires non-const.
BTW, rewrote as range-based for.
*/ | ||
|
||
#include <list> | ||
#include <iostream> |
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.
<iostream>
is not used.
[226] PASSED on melodicAll tests passed
|
[226] PASSED on noeticAll tests passed
|
Output path might contain duplicated poses due to interpolation logic and quantization error.