diogoalmeida / generic_control_toolbox

Generic code for robot experiments
MIT License
13 stars 7 forks source link

Build fails on ROS kinetic #19

Open RaduCorcodel opened 3 years ago

RaduCorcodel commented 3 years ago

Hi,

I tried building on Kinetic today and got this error messages:

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp: In member function ‘void generic_control_toolbox::KDLManager::updateSolvers()’:
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1134:25: error: ‘class KDL::ChainIkSolverVel’ has no member named ‘updateInternalDataStructures’
     ikvel_[item.first]->updateInternalDataStructures();
                         ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1135:25: error: ‘class KDL::ChainFkSolverPos_recursive’ has no member named ‘updateInternalDataStructures’
     fkpos_[item.first]->updateInternalDataStructures();
                         ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1136:30: error: ‘class KDL::ChainJntToJacSolver’ has no member named ‘updateInternalDataStructures’
     jac_solver_[item.first]->updateInternalDataStructures();
                              ^
/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1137:33: error: ‘class KDL::ChainDynParam’ has no member named ‘updateInternalDataStructures’
     dynamic_chain_[item.first]->updateInternalDataStructures();

On this system, there is this version of KDL installed.

ros-kinetic-orocos-kdl is already the newest version (1.3.2-1xenial-20201017-070916+0000).

On ROS Melodic it builds just fine. I tried updating all ROS packages but that didn't help. Any ideas on what else I could do?

Radu,

diogoalmeida commented 3 years ago

Might be that the kdl version from kinetic is just not compatible with the current version of the toolbox. The common way to address this would be to have a "kinetic" branch that hosts a version of the code that is compatible with that ros version.

I can look into it, but cannot do it in the near future. If you have a version that works, I can create a kinetic branch and you can make a PR into it.


From: RaduCorcodel notifications@github.com Sent: Wednesday, December 16, 2020 5:35:26 PM To: diogoalmeida/generic_control_toolbox generic_control_toolbox@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [diogoalmeida/generic_control_toolbox] Build fails on ROS kinetic (#19)

Hi,

I tried building on Kinetic today and got this error messages:

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp: In member function ‘void generic_control_toolbox::KDLManager::updateSolvers()’:

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1134:25: error: ‘class KDL::ChainIkSolverVel’ has no member named ‘updateInternalDataStructures’

 ikvel_[item.first]->updateInternalDataStructures();

                     ^

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1135:25: error: ‘class KDL::ChainFkSolverPos_recursive’ has no member named ‘updateInternalDataStructures’

 fkpos_[item.first]->updateInternalDataStructures();

                     ^

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1136:30: error: ‘class KDL::ChainJntToJacSolver’ has no member named ‘updateInternalDataStructures’

 jac_solver_[item.first]->updateInternalDataStructures();

                          ^

/homes/corcodel/grasping/src/generic_control_toolbox/src/kdl_manager.cpp:1137:33: error: ‘class KDL::ChainDynParam’ has no member named ‘updateInternalDataStructures’

 dynamic_chain_[item.first]->updateInternalDataStructures();

On this system, there is this version of KDL installed.

ros-kinetic-orocos-kdl is already the newest version (1.3.2-1xenial-20201017-070916+0000).

On ROS Melodic it builds just fine. I tried updating all ROS packages but that didn't help. Any ideas on what else I could do?

Radu,

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/diogoalmeida/generic_control_toolbox/issues/19, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AALOBTNPEOQDS4EWZ7RYYWTSVDVV5ANCNFSM4U6K25DQ.

RaduCorcodel commented 3 years ago

Sounds good. I will roll back to the newest commit that still runs under kinetic and hopefully that fixes it. Thanks so much, I think that would work just fine.