-
Notifications
You must be signed in to change notification settings - Fork 493
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
Foxy prep fixes #226
Foxy prep fixes #226
Conversation
@@ -84,6 +84,8 @@ class PR2ArmIKSolver : public KDL::ChainIkSolverPos | |||
|
|||
~PR2ArmIKSolver() override{}; | |||
|
|||
void updateInternalDataStructures() override; |
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.
For some reason this function was deleted in (https://github.com/ros-planning/moveit2/pull/60/files#diff-8404d089489122c76cd63a908826c88eL90) even though it overrides a virtual function. Not sure why this didn't complain with eloquent.
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.
Removed this since ros-eloquent-orocos-kdl
is at 3.2.1 which doesn't include this. @clalancette are there plans for making another eloquent release?
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
Could you please keep an eye on and add relevant changes to the moveit repo as well? |
Codecov Report
@@ Coverage Diff @@
## master #226 +/- ##
==========================================
- Coverage 47.43% 47.35% -0.08%
==========================================
Files 143 143
Lines 13347 13347
==========================================
- Hits 6331 6321 -10
- Misses 7016 7026 +10
Continue to review full report at Codecov.
|
General fixes that are required for Foxy to build. Since these are very generic, I think they should be merged into master.