search

ubi-agni / tactile_toolbox

software to handle tactile sensors in ROS
9 stars 5 forks source link

Includes restructure #30

Closed guihomework closed 2 years ago

guihomework commented 2 years ago

This PR changes the header file location to easily permits installing the files and allow external plugins to re-use classes or plugin displays of rviz_tactile_plugins

guihomework commented 2 years ago

Seems I broke the runtime now... 

[ERROR] [1661794645.104098490]: PluginlibFactory: The plugin for class 'rviz_tactile_plugins/Tactile State Display' failed to load. 
 Error: Failed to load library catkin_ws_noetic/devel/lib//librviz_tactile_plugins.so. 
   Make sure that you are calling the PLUGINLIB_EXPORT_CLASS macro in the library code, and that names are consistent between this macro and your XML.
 Error string: Could not load library (Poco exception = catkin_ws_noetic/devel/lib//librviz_tactile_plugins.so: undefined symbol: _ZNK4rviz7tactile17TactileVisualBase10metaObjectEv)

somebody solved this type of issues when adding the header files in the build https://answers.ros.org/question/267548/rviz-throws-undefined-symbol-exception-when-including-from-include-folder/?answer=268163

will try this now

guihomework commented 2 years ago

the solution I copied from the forum works, additionally to the missed export. if you don't like it, please suggest a cleaner way

rhaschke commented 2 years ago

Thanks!