Merge tag 'v3.6.0'

Release of version 3.6.0.

Author: nim65s

CMakeLists.txt

@@ -25,6 +25,7 @@ SET(DOXYGEN_USE_MATHJAX YES)
 # JRL-cmakemodule setup
 INCLUDE(cmake/base.cmake)
 INCLUDE(cmake/boost.cmake)
+INCLUDE(cmake/lapack.cmake)
 INCLUDE(cmake/python.cmake)
 INCLUDE(cmake/sphinx.cmake)
 
@@ -36,18 +37,18 @@ PROJECT(${PROJECT_NAME} ${PROJECT_ARGS})
 ADD_PROJECT_DEPENDENCY(sot-tools REQUIRED PKG_CONFIG_REQUIRES sot-tools)
 ADD_PROJECT_DEPENDENCY(example-robot-data)
 
-SET(BOOST_COMPONENTS filesystem system unit_test_framework)
+IF(BUILD_TESTING)
+  FIND_PACKAGE(Boost REQUIRED COMPONENTS unit_test_framework)
+ENDIF(BUILD_TESTING)
 
 IF(BUILD_PYTHON_INTERFACE)
   FINDPYTHON()
+  SEARCH_FOR_BOOST_PYTHON(REQUIRED)
   STRING(REGEX REPLACE "-" "_" PYTHON_DIR ${CUSTOM_HEADER_DIR})
-  ADD_PROJECT_DEPENDENCY(dynamic-graph-python REQUIRED
+  ADD_PROJECT_DEPENDENCY(dynamic-graph-python 4.0.0 REQUIRED
     PKG_CONFIG_REQUIRES dynamic-graph-python)
-  SET(BOOST_COMPONENTS ${BOOST_COMPONENTS} python)
 ENDIF(BUILD_PYTHON_INTERFACE)
 
-SEARCH_FOR_BOOST()
-
 # Main Library
 SET(${PROJECT_NAME}_HEADERS
   include/${CUSTOM_HEADER_DIR}/dynamic-pinocchio.h
@@ -69,8 +70,7 @@ SET(${PROJECT_NAME}_SOURCES
 ADD_LIBRARY(${PROJECT_NAME} SHARED
   ${${PROJECT_NAME}_SOURCES} ${${PROJECT_NAME}_HEADERS})
 TARGET_INCLUDE_DIRECTORIES(${PROJECT_NAME} PUBLIC $<INSTALL_INTERFACE:include>)
-TARGET_LINK_LIBRARIES(${PROJECT_NAME} ${Boost_LIBRARIES}
-  sot-core::sot-core)
+TARGET_LINK_LIBRARIES(${PROJECT_NAME} sot-core::sot-core)
 
 IF(SUFFIX_SO_VERSION)
   SET_TARGET_PROPERTIES(${PROJECT_NAME} PROPERTIES SOVERSION ${PROJECT_VERSION})

README.md

@@ -2,7 +2,7 @@ sot-dynamic-pinocchio
 Encapsulate Pinocchio in SoT
 ===========
 
-[![Building Status](https://travis-ci.org/stack-of-tasks/sot-dynamic-pinocchio.svg?branch=master)](https://travis-ci.org/stack-of-tasks/sot-dynamic-pinocchio)
+[![License](https://img.shields.io/badge/License-BSD%202--Clause-green.svg)](https://opensource.org/licenses/BSD-2-Clause)
 [![Pipeline status](https://gitlab.laas.fr/stack-of-tasks/sot-dynamic-pinocchio/badges/master/pipeline.svg)](https://gitlab.laas.fr/stack-of-tasks/sot-dynamic-pinocchio/commits/master)
 [![Coverage report](https://gitlab.laas.fr/stack-of-tasks/sot-dynamic-pinocchio/badges/master/coverage.svg?job=doc-coverage)](http://projects.laas.fr/gepetto/doc/stack-of-tasks/sot-dynamic-pinocchio/master/coverage/)
 

cmake

@@ -5,6 +5,5 @@ IMAGE_PATH             = @CMAKE_SOURCE_DIR@/doc/pictures
 FILE_PATTERNS          = *.cc *.cpp *.h *.hh *.hxx
 
 TAGFILES               = \
-"@JRL_MAL_DOXYGENDOCDIR@/jrl-mal.doxytag = @JRL_MAL_DOXYGENDOCDIR@" \
 "@DYNAMIC_GRAPH_DOXYGENDOCDIR@/dynamic-graph.doxytag = @DYNAMIC_GRAPH_DOXYGENDOCDIR@" \
 "@SOT_CORE_DOXYGENDOCDIR@/sot-core.doxytag = @SOT_CORE_DOXYGENDOCDIR@"

doc/Doxyfile.extra.in

@@ -17,6 +17,7 @@
 /* STD */
 #include <string>
 #include <map>
+#include <memory>
 
 /* SOT */
 #include <pinocchio/fwd.hpp>
@@ -29,6 +30,7 @@
 #include <sot/core/matrix-geometry.hh>
 /* Matrix */
 #include <dynamic-graph/linear-algebra.h>
+#include <sot/dynamic-pinocchio/deprecated.hh>
 
 /* PINOCCHIO */
 #include <pinocchio/macros.hpp>
@@ -80,7 +82,7 @@ class SOTDYNAMIC_EXPORT DynamicPinocchio : public dg::Entity {
 
   /*  --- MODEL ATRIBUTES --- */
   pinocchio::Model* m_model;
-  pinocchio::Data* m_data;
+  std::unique_ptr<pinocchio::Data> m_data;
 
   /*  --- MODEL ATRIBUTES --- */
 
@@ -164,7 +166,15 @@ class SOTDYNAMIC_EXPORT DynamicPinocchio : public dg::Entity {
 
   void setModel(pinocchio::Model*);
 
-  void setData(pinocchio::Data*);
+  void createData();
+
+  /// \deprecated this function does nothing. This class has its own
+  /// pinocchio::Data object, which can be access with \ref getData.
+  void setData(pinocchio::Data*) SOT_DYNAMIC_PINOCCHIO_DEPRECATED;
+
+  pinocchio::Model* getModel() { return m_model; };
+
+  pinocchio::Data* getData() { return m_data.get(); };
 
   /* --- GETTERS --- */
 

include/sot/dynamic-pinocchio/dynamic-pinocchio.h

@@ -1,6 +1,6 @@
 <package format="2">
   <name>sot-dynamic-pinocchio</name>
-  <version>3.5.2</version>
+  <version>3.6.0</version>
   <description>
     Pinocchio bindings for dynamic-graph
   </description>

package.xml

@@ -18,7 +18,8 @@ SET(plugins
   )
 
 SET(integrator-force-rk4_plugins_deps integrator-force)
-SET(integrator-force-exact_plugins_deps integrator-force)
+SET(integrator-force-exact_plugins_deps integrator-force lapack)
+SET(mass-apparent_plugins_deps integrator-force)
 
 FOREACH(plugin ${plugins})
   GET_FILENAME_COMPONENT(LIBRARY_NAME ${plugin} NAME)
@@ -28,7 +29,8 @@ FOREACH(plugin ${plugins})
     SET_TARGET_PROPERTIES(${LIBRARY_NAME} PROPERTIES SOVERSION ${PROJECT_VERSION})
   ENDIF(SUFFIX_SO_VERSION)
 
-  TARGET_LINK_LIBRARIES(${LIBRARY_NAME} ${PROJECT_NAME} ${${LIBRARY_NAME}_deps})
+  TARGET_LINK_LIBRARIES(${LIBRARY_NAME}
+    PUBLIC ${PROJECT_NAME} ${${LIBRARY_NAME}_plugins_deps})
 
   IF(NOT INSTALL_PYTHON_INTERFACE_ONLY)
     INSTALL(TARGETS ${LIBRARY_NAME} EXPORT ${TARGETS_EXPORT_NAME}
@@ -37,10 +39,15 @@ FOREACH(plugin ${plugins})
 
   IF(BUILD_PYTHON_INTERFACE)
     STRING(REPLACE - _ PYTHON_LIBRARY_NAME ${LIBRARY_NAME})
-    DYNAMIC_GRAPH_PYTHON_MODULE("${PYTHON_DIR}/${PYTHON_LIBRARY_NAME}"
-      ${LIBRARY_NAME} ${PROJECT_NAME}-${PYTHON_LIBRARY_NAME}-wrap
-      1 "src/python-module-py.cpp")
-    TARGET_LINK_BOOST_PYTHON(${PROJECT_NAME}-${PYTHON_LIBRARY_NAME}-wrap)
+    if(EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/${plugin}-python-module-py.cc")
+      DYNAMIC_GRAPH_PYTHON_MODULE("${PYTHON_DIR}/${PYTHON_LIBRARY_NAME}"
+        ${LIBRARY_NAME} ${PROJECT_NAME}-${PYTHON_LIBRARY_NAME}-wrap
+        SOURCE_PYTHON_MODULE "${CMAKE_CURRENT_SOURCE_DIR}/${plugin}-python-module-py.cc")
+    elseif(EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/${plugin}-python.hh")
+      DYNAMIC_GRAPH_PYTHON_MODULE("${PYTHON_DIR}/${PYTHON_LIBRARY_NAME}"
+        ${LIBRARY_NAME} ${PROJECT_NAME}-${PYTHON_LIBRARY_NAME}-wrap
+        MODULE_HEADER "${CMAKE_CURRENT_SOURCE_DIR}/${plugin}-python.hh")
+    endif()
   ENDIF(BUILD_PYTHON_INTERFACE)
 ENDFOREACH(plugin)
 

src/CMakeLists.txt

@@ -0,0 +1,3 @@
+#include <sot/dynamic-pinocchio/angle-estimator.h>
+
+typedef boost::mpl::vector< dynamicgraph::sot::AngleEstimator > entities_t;

src/angle-estimator-python.hh

@@ -75,8 +75,7 @@ class GetJointNames : public Command {
     }
     const std::vector<std::string>& jointNames = robot.m_model->names;
     // Remove first joint names 'universe'
-    std::size_t n(jointNames.size());
-    assert(n >= 1);
+    assert(jointNames.size() >= 1);
     std::vector<Value> res;
     for (std::size_t i = 1; i < jointNames.size(); ++i) {
       res.push_back(Value(jointNames[i]));

src/dynamic-command.h

@@ -0,0 +1,26 @@
+#include <dynamic-graph/python/module.hh>
+
+#include <sot/dynamic-pinocchio/dynamic-pinocchio.h>
+
+namespace dg = dynamicgraph;
+namespace dgs = dynamicgraph::sot;
+
+typedef bp::return_value_policy<bp::reference_existing_object> reference_existing_object;
+
+BOOST_PYTHON_MODULE(wrap)
+{
+  bp::import("dynamic_graph");
+  bp::import("pinocchio");
+
+  dg::python::exposeEntity<dgs::DynamicPinocchio, bp::bases<dg::Entity>, dg::python::AddCommands>()
+    .add_property("model",
+        bp::make_function(&dgs::DynamicPinocchio::getModel, reference_existing_object()),
+        bp::make_function(&dgs::DynamicPinocchio::setModel))
+    .add_property("data",
+        bp::make_function(&dgs::DynamicPinocchio::getData, reference_existing_object()),
+        bp::make_function(&dgs::DynamicPinocchio::setData))
+    .def("setModel", &dgs::DynamicPinocchio::setModel)
+    .def("createData", &dgs::DynamicPinocchio::createData)
+    .def("setData", &dgs::DynamicPinocchio::setData)
+    ;
+}