diff --git a/math/pinocchio/Makefile b/math/pinocchio/Makefile index 5dd0c8207..00c0070b5 100644 --- a/math/pinocchio/Makefile +++ b/math/pinocchio/Makefile @@ -4,8 +4,7 @@ ORG= stack-of-tasks NAME= pinocchio -VERSION= 3.0.0 -PKGREVISION= 1 +VERSION= 3.2.0 CATEGORIES= math COMMENT= Efficient rigid body dynamics diff --git a/math/pinocchio/PLIST b/math/pinocchio/PLIST index 95fbd7ed9..69ff530b1 100644 --- a/math/pinocchio/PLIST +++ b/math/pinocchio/PLIST @@ -272,8 +272,6 @@ include/pinocchio/math/taylor-expansion.hpp include/pinocchio/math/tensor.hpp include/pinocchio/math/triangular-matrix.hpp include/pinocchio/math/tridiagonal-matrix.hpp -include/pinocchio/multibody/constraint-base.hpp -include/pinocchio/multibody/constraint-generic.hpp include/pinocchio/multibody/data.hpp include/pinocchio/multibody/data.hxx include/pinocchio/multibody/data.txx @@ -348,6 +346,7 @@ include/pinocchio/multibody/visitor/fusion.hpp include/pinocchio/multibody/visitor/joint-binary-visitor.hpp include/pinocchio/multibody/visitor/joint-unary-visitor.hpp include/pinocchio/parsers/config.hpp +include/pinocchio/parsers/meshloader-fwd.hpp include/pinocchio/parsers/mjcf.hpp include/pinocchio/parsers/mjcf/geometry.hxx include/pinocchio/parsers/mjcf/mjcf-graph.hpp @@ -372,7 +371,6 @@ include/pinocchio/serialization/frame.hpp include/pinocchio/serialization/fwd.hpp include/pinocchio/serialization/geometry.hpp include/pinocchio/serialization/inertia.hpp -include/pinocchio/serialization/joints-constraint.hpp include/pinocchio/serialization/joints-data.hpp include/pinocchio/serialization/joints-model.hpp include/pinocchio/serialization/joints-motion-subspace.hpp diff --git a/math/pinocchio/distinfo b/math/pinocchio/distinfo index 28de0fc9a..d361170a4 100644 --- a/math/pinocchio/distinfo +++ b/math/pinocchio/distinfo @@ -1,8 +1,4 @@ -SHA1 (pinocchio-3.0.0.tar.gz) = 72192a2ea9914c74413b6ef92f066ced802cd62f -RMD160 (pinocchio-3.0.0.tar.gz) = 53ab1059dd86c34ae9a2a71af26ef24b2119d186 -Size (pinocchio-3.0.0.tar.gz) = 100744571 bytes -SHA1 (patch-2264) = eb7cba2191020c3bb6026369fb236239e22e86db -SHA1 (patch-2284) = b179e8b3ef082515af5b4ee48f3b9d9267e509a3 -SHA1 (patch-an) = 99018d12bd2a563b05685fc25fae0dad02303164 +SHA1 (pinocchio-3.2.0.tar.gz) = 01d46b7a1b64feb6874b8b173bb96b8283e100a8 +RMD160 (pinocchio-3.2.0.tar.gz) = 12e75cd4238297d0414daebb61f6c523441e2fec +Size (pinocchio-3.2.0.tar.gz) = 101261496 bytes SHA1 (patch-ao) = 82388cd763283a6d70581fdabcd1ae1035046ab8 -SHA1 (patch-ap) = e68ebc96a023535f8bde33eae5eb027638ec3a1c diff --git a/math/pinocchio/patches/patch-2264 b/math/pinocchio/patches/patch-2264 deleted file mode 100644 index 649eee38a..000000000 --- a/math/pinocchio/patches/patch-2264 +++ /dev/null @@ -1,86 +0,0 @@ -From 511ab722a9a81ad231dde8ee602913455a4c697c Mon Sep 17 00:00:00 2001 -From: Guilhem Saurel -Date: Thu, 30 May 2024 12:28:06 +0200 -Subject: [PATCH] unittest: add 2 valgrind suppressions - -To fix packaging on ArchLinux, when valgrind is installed. - -They are the same as the 2 previous ones, except: -- realloc instead of alloc -- some extra `fun:UnknownInlinedFun` ---- - ...ings_build_geom_from_urdf_memorycheck.supp | 64 +++++++++++++++++++ - 1 file changed, 64 insertions(+) - -diff --git a/unittest/python/bindings_build_geom_from_urdf_memorycheck.supp b/unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -index e36b8329e..b34069998 100644 ---- unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -+++ unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -@@ -273,3 +273,67 @@ - fun:UnknownInlinedFun - fun:_imp_create_dynamic - } -+{ -+ -+ Memcheck:Leak -+ match-leak-kinds: definite -+ fun:realloc -+ fun:d_growable_string_resize -+ fun:d_growable_string_append_buffer -+ fun:d_growable_string_callback_adapter -+ fun:d_print_flush -+ fun:d_print_callback -+ fun:d_demangle_callback.constprop.0 -+ fun:d_demangle -+ fun:__cxa_demangle -+ fun:_ZN5boost6python6detail12gcc_demangleEPKc -+ fun:name -+ fun:visit > -+ fun:visit >, boost::python::class_ > -+ fun:visit > -+ fun:def > -+ fun:_ZN9pinocchio6python15exposeCollisionEv -+ fun:_Z36init_module_pinocchio_pywrap_defaultv -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python21handle_exception_implENS_9function0IvEE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python6detail11init_moduleER11PyModuleDefPFvvE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_imp_create_dynamic -+ fun:cfunction_vectorcall_FASTCALL -+ fun:UnknownInlinedFun -+ fun:_PyEval_EvalFrameDefault.cold -+} -+{ -+ -+ Memcheck:Leak -+ match-leak-kinds: definite -+ fun:realloc -+ fun:d_growable_string_resize -+ fun:d_growable_string_append_buffer -+ fun:d_growable_string_callback_adapter -+ fun:d_print_flush -+ fun:d_print_callback -+ fun:d_demangle_callback.constprop.0 -+ fun:d_demangle -+ fun:__cxa_demangle -+ fun:_ZN5boost6python6detail12gcc_demangleEPKc -+ fun:name -+ fun:_ZNK9pinocchio6python28GeometryFunctorPythonVisitorINS_15ComputeDistanceEE5visitIN5boost6python6class_IS2_NS6_6detail13not_specifiedES9_S9_EEEEvRT_.isra.0 -+ fun:visit >, boost::python::class_ > -+ fun:visit > -+ fun:def > -+ fun:_ZN9pinocchio6python15exposeCollisionEv -+ fun:_Z36init_module_pinocchio_pywrap_defaultv -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python21handle_exception_implENS_9function0IvEE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python6detail11init_moduleER11PyModuleDefPFvvE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_imp_create_dynamic -+ fun:cfunction_vectorcall_FASTCALL -+} diff --git a/math/pinocchio/patches/patch-2284 b/math/pinocchio/patches/patch-2284 deleted file mode 100644 index e1ca57fa3..000000000 --- a/math/pinocchio/patches/patch-2284 +++ /dev/null @@ -1,149 +0,0 @@ -From 7c7a74680fb73590af0506e77656edc56f7697e8 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:47:40 +0000 -Subject: [PATCH 1/6] Do not specify size of Identity for constant size - matrices. - ---- - include/pinocchio/multibody/joint/joint-helical.hpp | 2 +- - include/pinocchio/multibody/joint/joint-revolute.hpp | 2 +- - 2 files changed, 2 insertions(+), 2 deletions(-) - -diff --git a/include/pinocchio/multibody/joint/joint-helical.hpp b/include/pinocchio/multibody/joint/joint-helical.hpp -index 82e62da72..842c8ecfd 100644 ---- include/pinocchio/multibody/joint/joint-helical.hpp -+++ include/pinocchio/multibody/joint/joint-helical.hpp -@@ -201,7 +201,7 @@ namespace pinocchio - } - AngularType rotation() const - { -- AngularType m(AngularType::Identity(3)); -+ AngularType m(AngularType::Identity()); - _setRotation(m); - return m; - } -diff --git a/include/pinocchio/multibody/joint/joint-revolute.hpp b/include/pinocchio/multibody/joint/joint-revolute.hpp -index ae3876032..1a77dd170 100644 ---- include/pinocchio/multibody/joint/joint-revolute.hpp -+++ include/pinocchio/multibody/joint/joint-revolute.hpp -@@ -186,7 +186,7 @@ namespace pinocchio - } - AngularType rotation() const - { -- AngularType m(AngularType::Identity(3)); -+ AngularType m(AngularType::Identity()); - _setRotation(m); - return m; - } - -From 3d1b11a4bbc5e8be35601a4178482f10d91a48aa Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:49:13 +0000 -Subject: [PATCH 2/6] Add a test on access to pose in joint data. - ---- - unittest/all-joints.cpp | 20 ++++++++++++++++++++ - 1 file changed, 20 insertions(+) - -diff --git a/unittest/all-joints.cpp b/unittest/all-joints.cpp -index 44e8e802b..072188ed9 100644 ---- unittest/all-joints.cpp -+++ unittest/all-joints.cpp -@@ -193,6 +193,18 @@ struct TestJointModelIsEqual : TestJointModel - } - }; - -+struct TestJointModelTransform : TestJointModel -+{ -+ template -+ static void test(const JointModelBase & jmodel) -+ { -+ typedef typename JointModel::JointDataDerived JointData; -+ JointData jdata = jmodel.createData(); -+ Eigen::Matrix v = jdata.M_accessor().translation(); -+ Eigen::Matrix R = jdata.M_accessor().rotation(); -+ } -+}; -+ - BOOST_AUTO_TEST_CASE(isEqual) - { - typedef JointCollectionDefault::JointModelVariant JointModelVariant; -@@ -211,6 +223,14 @@ BOOST_AUTO_TEST_CASE(isEqual) - BOOST_CHECK(jmodel_any != jmodelx); - } - -+BOOST_AUTO_TEST_CASE(transform) -+{ -+ typedef JointCollectionDefault::JointModelVariant JointModelVariant; -+ boost::mpl::for_each(TestJointModelTransform()); -+ -+ TestJointModelTransform()(JointModel()); -+} -+ - struct TestJointModelCast : TestJointModel - { - template - -From 52b3bf170f3f102bbe1ee80de9163a5031e71ed3 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:49:56 +0000 -Subject: [PATCH 3/6] Return Vector3 instead of Vector3::ConstandReturnType - - The compiler otherwise fails to evaluate method - TransformHelicalTpl::translation. ---- - include/pinocchio/multibody/joint/joint-helical.hpp | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/include/pinocchio/multibody/joint/joint-helical.hpp b/include/pinocchio/multibody/joint/joint-helical.hpp -index 842c8ecfd..092357979 100644 ---- include/pinocchio/multibody/joint/joint-helical.hpp -+++ include/pinocchio/multibody/joint/joint-helical.hpp -@@ -79,7 +79,7 @@ namespace pinocchio - typedef Matrix3 AngularType; - typedef Matrix3 AngularRef; - typedef Matrix3 ConstAngularRef; -- typedef typename Vector3::ConstantReturnType LinearType; -+ typedef Vector3 LinearType; - typedef typename Vector3::ConstantReturnType LinearRef; - typedef const typename Vector3::ConstantReturnType ConstLinearRef; - typedef typename traits::ActionMatrixType ActionMatrixType; - -From 1389b6a540e6e8b4cbd725e07f634ec289a90f32 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Tue, 11 Jun 2024 21:56:29 +0200 -Subject: [PATCH 4/6] [algorithm] resolve ambiguity in JointModel type. - - When using a different JointCollection, JointModel is not of the right type - and causes a compilation error. ---- - include/pinocchio/algorithm/model.hxx | 8 ++++---- - 1 file changed, 4 insertions(+), 4 deletions(-) - -diff --git a/include/pinocchio/algorithm/model.hxx b/include/pinocchio/algorithm/model.hxx -index 9c1ec8700..e123fcef8 100644 ---- include/pinocchio/algorithm/model.hxx -+++ include/pinocchio/algorithm/model.hxx -@@ -380,8 +380,8 @@ namespace pinocchio - for (JointIndex joint_idA = 1; joint_idA < modelA.joints.size(); ++joint_idA) - { - const JointIndex joint_id = model.getJointId(modelA.names[joint_idA]); -- const JointModel & joint_model = model.joints[joint_id]; -- const JointModel & joint_modelA = modelA.joints[joint_idA]; -+ const typename Model::JointModel & joint_model = model.joints[joint_id]; -+ const typename Model::JointModel & joint_modelA = modelA.joints[joint_idA]; - - joint_model.jointConfigSelector(config_vector) = - joint_modelA.jointConfigSelector(config_vectorA); -@@ -409,8 +409,8 @@ namespace pinocchio - for (JointIndex joint_idB = 1; joint_idB < modelB.joints.size(); ++joint_idB) - { - const JointIndex joint_id = model.getJointId(modelB.names[joint_idB]); -- const JointModel & joint_model = model.joints[joint_id]; -- const JointModel & joint_modelB = modelB.joints[joint_idB]; -+ const typename Model::JointModel & joint_model = model.joints[joint_id]; -+ const typename Model::JointModel & joint_modelB = modelB.joints[joint_idB]; - - joint_model.jointConfigSelector(config_vector) = - joint_modelB.jointConfigSelector(config_vectorB); - diff --git a/math/pinocchio/patches/patch-an b/math/pinocchio/patches/patch-an deleted file mode 100644 index f106ab2c8..000000000 --- a/math/pinocchio/patches/patch-an +++ /dev/null @@ -1,20 +0,0 @@ ---- CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ CMakeLists.txt 2024-05-28 08:48:46.596303442 +0200 -@@ -297,17 +297,8 @@ - # Check whether hpp-fcl python bindings are available. - set(BUILD_WITH_HPP_FCL_PYTHON_BINDINGS FALSE) - if(BUILD_PYTHON_INTERFACE) -- execute_process( -- COMMAND ${PYTHON_EXECUTABLE} -c "import hppfcl" -- RESULT_VARIABLE _hpp_fcl_python_bindings_not_found -- OUTPUT_QUIET ERROR_QUIET) -- if(_hpp_fcl_python_bindings_not_found EQUAL 0) - set(BUILD_WITH_HPP_FCL_PYTHON_BINDINGS TRUE) - message(STATUS "Found hpp-fcl Python bindings.") -- else() -- message(STATUS "hpp-fcl Python bindings NOT found.") -- endif() -- unset(_hpp_fcl_python_bindings_not_found) - endif(BUILD_PYTHON_INTERFACE) - endif(BUILD_WITH_HPP_FCL_SUPPORT) - diff --git a/math/pinocchio/patches/patch-ap b/math/pinocchio/patches/patch-ap deleted file mode 100644 index d2403a436..000000000 --- a/math/pinocchio/patches/patch-ap +++ /dev/null @@ -1,16 +0,0 @@ -boost-1.85 removed get_child with temporary second argument - ---- src/parsers/mjcf/mjcf-graph.cpp~ 2024-05-27 21:47:00.000000000 +0200 -+++ src/parsers/mjcf/mjcf-graph.cpp 2024-08-06 14:31:31.848800244 +0200 -@@ -37,8 +37,9 @@ - std::string path = el + "."; - if (parent.get_child_optional(path)) - { -- ptree attr_parent = parent.get_child(path, ptree()); -- ptree attr_current = current.get_child(path, ptree()); -+ const ptree dflt; -+ ptree attr_parent = parent.get_child(path, dflt); -+ ptree attr_current = current.get_child(path, dflt); - // To only copy non existing attribute in current, we copy all current - // attribute (replacing) into a parent copy then we replace current with the new - // ptree diff --git a/math/py-pinocchio/Makefile b/math/py-pinocchio/Makefile index 3c035fadb..bba03ba8d 100644 --- a/math/py-pinocchio/Makefile +++ b/math/py-pinocchio/Makefile @@ -4,13 +4,13 @@ ORG= stack-of-tasks NAME= pinocchio -VERSION= 3.0.0 -PKGREVISION= 1 +VERSION= 3.2.0 CATEGORIES= math COMMENT= Efficient rigid body dynamics (python bindings) HOMEPAGE= http://${ORG}.github.io/${NAME}/ +#CMAKE_ARGS+= -DBUILD_WITH_CASADI_SUPPORT=ON CMAKE_ARGS+= -DBUILD_WITH_COLLISION_SUPPORT=ON CMAKE_ARGS+= -DBUILD_BENCHMARK=OFF CMAKE_ARGS+= -DBUILD_TESTING=OFF @@ -46,6 +46,7 @@ include ../../math/py-eigenpy/depend.mk include ../../path/hpp-fcl/depend.mk include ../../path/py-hpp-fcl/depend.mk include ../../mapping/octomap/depend.mk +#include ../../optimization/py-casadi/depend.mk include ../../pkgtools/pkg-config/depend.mk include ../../mk/sysdep/boost-headers.mk include ../../mk/sysdep/boost-libs.mk diff --git a/math/py-pinocchio/distinfo b/math/py-pinocchio/distinfo index 6bc98ffcc..7288e873c 100644 --- a/math/py-pinocchio/distinfo +++ b/math/py-pinocchio/distinfo @@ -1,10 +1,7 @@ -SHA1 (pinocchio-3.0.0.tar.gz) = 72192a2ea9914c74413b6ef92f066ced802cd62f -RMD160 (pinocchio-3.0.0.tar.gz) = 53ab1059dd86c34ae9a2a71af26ef24b2119d186 -Size (pinocchio-3.0.0.tar.gz) = 100744571 bytes -SHA1 (patch-2264) = eb7cba2191020c3bb6026369fb236239e22e86db -SHA1 (patch-2284) = b179e8b3ef082515af5b4ee48f3b9d9267e509a3 -SHA1 (patch-af) = bf6c1d2fe04942c52d783be76b2a3d5386189953 -SHA1 (patch-ag) = 929490a70c419afe239844b87f556e3789060c63 +SHA1 (pinocchio-3.2.0.tar.gz) = 01d46b7a1b64feb6874b8b173bb96b8283e100a8 +RMD160 (pinocchio-3.2.0.tar.gz) = 12e75cd4238297d0414daebb61f6c523441e2fec +Size (pinocchio-3.2.0.tar.gz) = 101261496 bytes +SHA1 (patch-ag) = 35b62ca6bd5676f51a5bef1d4a98c74113ca6a03 SHA1 (patch-ah) = 365652d212fc8b2e82c7fee33818d08541491893 -SHA1 (patch-ak) = 59ebee7b94ff79fae81702a21be3763e9502b8ba +SHA1 (patch-ak) = 73bff9b8278abb37c761375b4120b9f93edf9226 SHA1 (patch-al) = 35d232f6e5f416848aa5f7444c882bb3ddddea04 diff --git a/math/py-pinocchio/patches/patch-2264 b/math/py-pinocchio/patches/patch-2264 deleted file mode 100644 index 649eee38a..000000000 --- a/math/py-pinocchio/patches/patch-2264 +++ /dev/null @@ -1,86 +0,0 @@ -From 511ab722a9a81ad231dde8ee602913455a4c697c Mon Sep 17 00:00:00 2001 -From: Guilhem Saurel -Date: Thu, 30 May 2024 12:28:06 +0200 -Subject: [PATCH] unittest: add 2 valgrind suppressions - -To fix packaging on ArchLinux, when valgrind is installed. - -They are the same as the 2 previous ones, except: -- realloc instead of alloc -- some extra `fun:UnknownInlinedFun` ---- - ...ings_build_geom_from_urdf_memorycheck.supp | 64 +++++++++++++++++++ - 1 file changed, 64 insertions(+) - -diff --git a/unittest/python/bindings_build_geom_from_urdf_memorycheck.supp b/unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -index e36b8329e..b34069998 100644 ---- unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -+++ unittest/python/bindings_build_geom_from_urdf_memorycheck.supp -@@ -273,3 +273,67 @@ - fun:UnknownInlinedFun - fun:_imp_create_dynamic - } -+{ -+ -+ Memcheck:Leak -+ match-leak-kinds: definite -+ fun:realloc -+ fun:d_growable_string_resize -+ fun:d_growable_string_append_buffer -+ fun:d_growable_string_callback_adapter -+ fun:d_print_flush -+ fun:d_print_callback -+ fun:d_demangle_callback.constprop.0 -+ fun:d_demangle -+ fun:__cxa_demangle -+ fun:_ZN5boost6python6detail12gcc_demangleEPKc -+ fun:name -+ fun:visit > -+ fun:visit >, boost::python::class_ > -+ fun:visit > -+ fun:def > -+ fun:_ZN9pinocchio6python15exposeCollisionEv -+ fun:_Z36init_module_pinocchio_pywrap_defaultv -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python21handle_exception_implENS_9function0IvEE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python6detail11init_moduleER11PyModuleDefPFvvE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_imp_create_dynamic -+ fun:cfunction_vectorcall_FASTCALL -+ fun:UnknownInlinedFun -+ fun:_PyEval_EvalFrameDefault.cold -+} -+{ -+ -+ Memcheck:Leak -+ match-leak-kinds: definite -+ fun:realloc -+ fun:d_growable_string_resize -+ fun:d_growable_string_append_buffer -+ fun:d_growable_string_callback_adapter -+ fun:d_print_flush -+ fun:d_print_callback -+ fun:d_demangle_callback.constprop.0 -+ fun:d_demangle -+ fun:__cxa_demangle -+ fun:_ZN5boost6python6detail12gcc_demangleEPKc -+ fun:name -+ fun:_ZNK9pinocchio6python28GeometryFunctorPythonVisitorINS_15ComputeDistanceEE5visitIN5boost6python6class_IS2_NS6_6detail13not_specifiedES9_S9_EEEEvRT_.isra.0 -+ fun:visit >, boost::python::class_ > -+ fun:visit > -+ fun:def > -+ fun:_ZN9pinocchio6python15exposeCollisionEv -+ fun:_Z36init_module_pinocchio_pywrap_defaultv -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python21handle_exception_implENS_9function0IvEE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_ZN5boost6python6detail11init_moduleER11PyModuleDefPFvvE -+ fun:UnknownInlinedFun -+ fun:UnknownInlinedFun -+ fun:_imp_create_dynamic -+ fun:cfunction_vectorcall_FASTCALL -+} diff --git a/math/py-pinocchio/patches/patch-2284 b/math/py-pinocchio/patches/patch-2284 deleted file mode 100644 index e1ca57fa3..000000000 --- a/math/py-pinocchio/patches/patch-2284 +++ /dev/null @@ -1,149 +0,0 @@ -From 7c7a74680fb73590af0506e77656edc56f7697e8 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:47:40 +0000 -Subject: [PATCH 1/6] Do not specify size of Identity for constant size - matrices. - ---- - include/pinocchio/multibody/joint/joint-helical.hpp | 2 +- - include/pinocchio/multibody/joint/joint-revolute.hpp | 2 +- - 2 files changed, 2 insertions(+), 2 deletions(-) - -diff --git a/include/pinocchio/multibody/joint/joint-helical.hpp b/include/pinocchio/multibody/joint/joint-helical.hpp -index 82e62da72..842c8ecfd 100644 ---- include/pinocchio/multibody/joint/joint-helical.hpp -+++ include/pinocchio/multibody/joint/joint-helical.hpp -@@ -201,7 +201,7 @@ namespace pinocchio - } - AngularType rotation() const - { -- AngularType m(AngularType::Identity(3)); -+ AngularType m(AngularType::Identity()); - _setRotation(m); - return m; - } -diff --git a/include/pinocchio/multibody/joint/joint-revolute.hpp b/include/pinocchio/multibody/joint/joint-revolute.hpp -index ae3876032..1a77dd170 100644 ---- include/pinocchio/multibody/joint/joint-revolute.hpp -+++ include/pinocchio/multibody/joint/joint-revolute.hpp -@@ -186,7 +186,7 @@ namespace pinocchio - } - AngularType rotation() const - { -- AngularType m(AngularType::Identity(3)); -+ AngularType m(AngularType::Identity()); - _setRotation(m); - return m; - } - -From 3d1b11a4bbc5e8be35601a4178482f10d91a48aa Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:49:13 +0000 -Subject: [PATCH 2/6] Add a test on access to pose in joint data. - ---- - unittest/all-joints.cpp | 20 ++++++++++++++++++++ - 1 file changed, 20 insertions(+) - -diff --git a/unittest/all-joints.cpp b/unittest/all-joints.cpp -index 44e8e802b..072188ed9 100644 ---- unittest/all-joints.cpp -+++ unittest/all-joints.cpp -@@ -193,6 +193,18 @@ struct TestJointModelIsEqual : TestJointModel - } - }; - -+struct TestJointModelTransform : TestJointModel -+{ -+ template -+ static void test(const JointModelBase & jmodel) -+ { -+ typedef typename JointModel::JointDataDerived JointData; -+ JointData jdata = jmodel.createData(); -+ Eigen::Matrix v = jdata.M_accessor().translation(); -+ Eigen::Matrix R = jdata.M_accessor().rotation(); -+ } -+}; -+ - BOOST_AUTO_TEST_CASE(isEqual) - { - typedef JointCollectionDefault::JointModelVariant JointModelVariant; -@@ -211,6 +223,14 @@ BOOST_AUTO_TEST_CASE(isEqual) - BOOST_CHECK(jmodel_any != jmodelx); - } - -+BOOST_AUTO_TEST_CASE(transform) -+{ -+ typedef JointCollectionDefault::JointModelVariant JointModelVariant; -+ boost::mpl::for_each(TestJointModelTransform()); -+ -+ TestJointModelTransform()(JointModel()); -+} -+ - struct TestJointModelCast : TestJointModel - { - template - -From 52b3bf170f3f102bbe1ee80de9163a5031e71ed3 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Thu, 6 Jun 2024 06:49:56 +0000 -Subject: [PATCH 3/6] Return Vector3 instead of Vector3::ConstandReturnType - - The compiler otherwise fails to evaluate method - TransformHelicalTpl::translation. ---- - include/pinocchio/multibody/joint/joint-helical.hpp | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/include/pinocchio/multibody/joint/joint-helical.hpp b/include/pinocchio/multibody/joint/joint-helical.hpp -index 842c8ecfd..092357979 100644 ---- include/pinocchio/multibody/joint/joint-helical.hpp -+++ include/pinocchio/multibody/joint/joint-helical.hpp -@@ -79,7 +79,7 @@ namespace pinocchio - typedef Matrix3 AngularType; - typedef Matrix3 AngularRef; - typedef Matrix3 ConstAngularRef; -- typedef typename Vector3::ConstantReturnType LinearType; -+ typedef Vector3 LinearType; - typedef typename Vector3::ConstantReturnType LinearRef; - typedef const typename Vector3::ConstantReturnType ConstLinearRef; - typedef typename traits::ActionMatrixType ActionMatrixType; - -From 1389b6a540e6e8b4cbd725e07f634ec289a90f32 Mon Sep 17 00:00:00 2001 -From: Florent Lamiraux -Date: Tue, 11 Jun 2024 21:56:29 +0200 -Subject: [PATCH 4/6] [algorithm] resolve ambiguity in JointModel type. - - When using a different JointCollection, JointModel is not of the right type - and causes a compilation error. ---- - include/pinocchio/algorithm/model.hxx | 8 ++++---- - 1 file changed, 4 insertions(+), 4 deletions(-) - -diff --git a/include/pinocchio/algorithm/model.hxx b/include/pinocchio/algorithm/model.hxx -index 9c1ec8700..e123fcef8 100644 ---- include/pinocchio/algorithm/model.hxx -+++ include/pinocchio/algorithm/model.hxx -@@ -380,8 +380,8 @@ namespace pinocchio - for (JointIndex joint_idA = 1; joint_idA < modelA.joints.size(); ++joint_idA) - { - const JointIndex joint_id = model.getJointId(modelA.names[joint_idA]); -- const JointModel & joint_model = model.joints[joint_id]; -- const JointModel & joint_modelA = modelA.joints[joint_idA]; -+ const typename Model::JointModel & joint_model = model.joints[joint_id]; -+ const typename Model::JointModel & joint_modelA = modelA.joints[joint_idA]; - - joint_model.jointConfigSelector(config_vector) = - joint_modelA.jointConfigSelector(config_vectorA); -@@ -409,8 +409,8 @@ namespace pinocchio - for (JointIndex joint_idB = 1; joint_idB < modelB.joints.size(); ++joint_idB) - { - const JointIndex joint_id = model.getJointId(modelB.names[joint_idB]); -- const JointModel & joint_model = model.joints[joint_id]; -- const JointModel & joint_modelB = modelB.joints[joint_idB]; -+ const typename Model::JointModel & joint_model = model.joints[joint_id]; -+ const typename Model::JointModel & joint_modelB = modelB.joints[joint_idB]; - - joint_model.jointConfigSelector(config_vector) = - joint_modelB.jointConfigSelector(config_vectorB); - diff --git a/math/py-pinocchio/patches/patch-af b/math/py-pinocchio/patches/patch-af deleted file mode 100644 index 7c155f250..000000000 --- a/math/py-pinocchio/patches/patch-af +++ /dev/null @@ -1,40 +0,0 @@ -* From 06c61fcc25077b83a02baf61601e60d8190bfe9c Mon Sep 17 00:00:00 2001 - From: Guilhem Saurel - Date: Wed, 3 Jul 2024 11:00:20 +0200 - Subject: [PATCH] CMake: modernize link to console_bridge - -* Don't install .pc file: it would conflict between installs for - multiple python versions. In case it is needed, something must be found to - handle name conflicts. - ---- bindings/python/CMakeLists.txt~ 2024-05-27 21:47:00.000000000 +0200 -+++ bindings/python/CMakeLists.txt 2024-07-03 19:09:30.772531780 +0200 -@@ -81,7 +81,12 @@ - if(BUILD_WITH_URDF_SUPPORT) - # Link directly against console_bridge since we bind some enums and call - # console_bridge::setLogLevel function. -- target_link_libraries(${PYTHON_LIB_NAME} PUBLIC console_bridge::console_bridge) -+ modernize_target_link_libraries( -+ ${PYTHON_LIB_NAME} -+ SCOPE PUBLIC -+ TARGETS console_bridge::console_bridge -+ LIBRARIES ${console_bridge_LIBRARIES} -+ INCLUDE_DIRS ${console_bridge_INCLUDE_DIRS}) - endif() - if(BUILD_WITH_HPP_FCL_PYTHON_BINDINGS) - target_compile_definitions(${PYTHON_LIB_NAME} -@@ -270,14 +275,6 @@ - set(_PKG_CONFIG_PYWRAP_CFLAGS "${_PKG_CONFIG_PYWRAP_CFLAGS} ${cflags}") - endforeach() - -- configure_file("${CMAKE_CURRENT_SOURCE_DIR}/pinocchiopy.pc.cmake" -- "${CMAKE_CURRENT_BINARY_DIR}/pinocchiopy.pc") -- -- install( -- FILES "${CMAKE_CURRENT_BINARY_DIR}/pinocchiopy.pc" -- DESTINATION ${CMAKE_INSTALL_LIBDIR}/pkgconfig -- PERMISSIONS OWNER_READ GROUP_READ WORLD_READ OWNER_WRITE) -- - if(DOXYGEN_FOUND AND DOXYGEN_VERSION VERSION_GREATER 1.8.17) - set(DOXYGEN_GENERATE_HTML YES) - set(DOXYGEN_GENERATE_LATEX NO) diff --git a/math/py-pinocchio/patches/patch-ag b/math/py-pinocchio/patches/patch-ag index 2ea0a4c23..6fe7a0617 100644 --- a/math/py-pinocchio/patches/patch-ag +++ b/math/py-pinocchio/patches/patch-ag @@ -1,29 +1,29 @@ py-pinocchio needs pinocchio ---- CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ CMakeLists.txt 2024-05-28 10:17:29.362489152 +0200 -@@ -183,6 +183,7 @@ +--- CMakeLists.txt.orig 2024-08-27 16:45:25.000000000 +0000 ++++ CMakeLists.txt 2024-08-29 13:19:21.747133530 +0000 +@@ -192,6 +192,7 @@ # ---------------------------------------------------- - set(CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake/find-external/CppAD/" ${CMAKE_MODULE_PATH}) + set(CMAKE_MODULE_PATH "${JRL_CMAKE_MODULES}/find-external/CppAD/" ${CMAKE_MODULE_PATH}) add_project_dependency(Eigen3 REQUIRED PKG_CONFIG_REQUIRES "eigen3 >= 3.0.5") +add_project_dependency(${PROJECT_NAME} REQUIRED) if(BUILD_WITH_URDF_SUPPORT) add_project_dependency(urdfdom_headers REQUIRED) ---- bindings/python/CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ bindings/python/CMakeLists.txt 2024-05-28 10:18:04.832481756 +0200 -@@ -76,7 +76,7 @@ +--- bindings/python/CMakeLists.txt.orig 2024-08-27 18:45:25.000000000 +0200 ++++ bindings/python/CMakeLists.txt 2024-08-29 15:19:55.570423979 +0200 +@@ -75,7 +75,7 @@ ${PYTHON_LIB_NAME} SCOPE PUBLIC TARGETS eigenpy::eigenpy) - target_link_libraries(${PYTHON_LIB_NAME} PUBLIC ${PROJECT_NAME}) -+ target_link_libraries(${PYTHON_LIB_NAME} PUBLIC ${PROJECT_NAME}::${PROJECT_NAME}) ++ target_link_libraries(${PYTHON_LIB_NAME} PUBLIC ${PROJECT_NAME}::${PROJECT_NAME}) if(BUILD_WITH_URDF_SUPPORT) # Link directly against console_bridge since we bind some enums and call ---- unittest/CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ unittest/CMakeLists.txt 2024-05-28 10:18:52.605797982 +0200 -@@ -70,26 +70,26 @@ +--- unittest/CMakeLists.txt.orig 2024-08-27 18:45:25.000000000 +0200 ++++ unittest/CMakeLists.txt 2024-08-29 15:21:21.067157905 +0200 +@@ -71,26 +71,26 @@ target_include_directories(${TEST_NAME} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}) if(NOT unit_test_HEADER_ONLY) @@ -56,11 +56,11 @@ py-pinocchio needs pinocchio endif() modernize_target_link_libraries( ---- examples/CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ examples/CMakeLists.txt 2024-05-28 10:19:23.895782651 +0200 +--- examples/CMakeLists.txt.orig 2024-08-27 18:45:25.000000000 +0200 ++++ examples/CMakeLists.txt 2024-08-29 15:22:20.099674281 +0200 @@ -14,22 +14,22 @@ get_filename_component(EXAMPLE_NAME ${EXAMPLE} NAME) - set(EXAMPLE_NAME "example-cpp-${EXAMPLE_NAME}") + set(EXAMPLE_NAME "${PROJECT_NAME}-example-cpp-${EXAMPLE_NAME}") add_unit_test(${EXAMPLE_NAME} "${EXAMPLE}.cpp") - target_link_libraries(${EXAMPLE_NAME} PUBLIC ${PROJECT_NAME}) + target_link_libraries(${EXAMPLE_NAME} PUBLIC ${PROJECT_NAME}::${PROJECT_NAME}) diff --git a/math/py-pinocchio/patches/patch-ak b/math/py-pinocchio/patches/patch-ak index d5e2b363a..d8911dcee 100644 --- a/math/py-pinocchio/patches/patch-ak +++ b/math/py-pinocchio/patches/patch-ak @@ -1,7 +1,7 @@ Don't install files already installed by pinocchio ---- CMakeLists.txt.orig 2024-05-27 21:47:00.000000000 +0200 -+++ CMakeLists.txt 2024-05-28 10:22:24.912306337 +0200 +--- CMakeLists.txt.orig 2024-08-27 18:45:25.000000000 +0200 ++++ CMakeLists.txt 2024-08-29 15:24:08.330632746 +0200 @@ -11,7 +11,6 @@ ) set(PROJECT_URL "http://github.com/stack-of-tasks/pinocchio") @@ -10,11 +10,10 @@ Don't install files already installed by pinocchio set(PROJECT_USE_KEYWORD_LINK_LIBRARIES TRUE) set(PROJECT_COMPATIBILITY_VERSION AnyNewerVersion) set(PINOCCHIO_PROJECT_SOURCE_DIR ${CMAKE_CURRENT_LIST_DIR}) -@@ -499,18 +498,3 @@ - endforeach() +@@ -507,19 +506,4 @@ pkg_config_append_cflags("${CFLAGS_DEPENDENCIES}") pkg_config_append_cflags("${CFLAGS_OPTIONS}") -- + -# Install catkin package.xml -install(FILES package.xml DESTINATION share/${PROJECT_NAME}) -# Allows Colcon to find non-Ament packages when using workspace underlays @@ -29,3 +28,5 @@ Don't install files already installed by pinocchio - "prepend-non-duplicate;PYTHONPATH;${PYTHON_SITELIB}") -install(FILES ${CMAKE_CURRENT_BINARY_DIR}/share/${PROJECT_NAME}/hook/python_path.dsv - DESTINATION share/${PROJECT_NAME}/hook) +- + setup_project_finalize()