Merge pull request #2530 from jcarpent/topic/coal

Fix var naming in Python
stack-of-tasks · Jan 9, 2025 · 0f1d998 · 0f1d998
2 parents 69f8b2e + ed1461c
commit 0f1d998
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,12 +8,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Fixed
 - Fix mjcf Euler angle parsing: use xyz as a default value for eulerseq compiler option ([#2526](https://github.com/stack-of-tasks/pinocchio/pull/2526))
+- Fix variable naming in Python ([#2530](https://github.com/stack-of-tasks/pinocchio/pull/2530))
 - Fix aba explicit template instantiation ([#2541](https://github.com/stack-of-tasks/pinocchio/pull/2541))
 - Add parsing meshes with vertices for MJCF format ([#2537](https://github.com/stack-of-tasks/pinocchio/pull/2537))
 - CMake: fix RPATH on macos ([#2546](https://github.com/stack-of-tasks/pinocchio/pull/2546))
 - Fix aba explicit template instantiation ([#2541](https://github.com/stack-of-tasks/pinocchio/pull/2541))
 - Fix mjcf parsing of keyframe qpos with newlines ([#2535](https://github.com/stack-of-tasks/pinocchio/pull/2535))
 
+
 ## [3.3.1] - 2024-12-13
 
 ### Added

diff --git a/include/pinocchio/bindings/python/collision/broadphase-manager.hpp b/include/pinocchio/bindings/python/collision/broadphase-manager.hpp
@@ -1,5 +1,5 @@
 //
-// Copyright (c) 2022 INRIA
+// Copyright (c) 2022-2024 INRIA
 //
 
 #ifndef __pinocchio_python_collision_broadphase_manager_hpp__
@@ -62,10 +62,10 @@ namespace pinocchio
       static void expose()
       {
         std::string derived_name = boost::typeindex::type_id<Derived>().pretty_name();
-        boost::algorithm::replace_all(derived_name, "hpp::fcl::", "");
+        boost::algorithm::replace_all(derived_name, "coal::", "");
         const std::string class_name = "BroadPhaseManager_" + derived_name;
 
-        const std::string class_doc = "Broad phase manager associated to hpp::fcl::" + derived_name;
+        const std::string class_doc = "Broad phase manager associated to coal::" + derived_name;
         bp::class_<Self> registered_class(class_name.c_str(), class_doc.c_str(), bp::no_init);
         registered_class.def(BroadPhaseManagerPythonVisitor());
 

diff --git a/include/pinocchio/bindings/python/collision/tree-broadphase-manager.hpp b/include/pinocchio/bindings/python/collision/tree-broadphase-manager.hpp
@@ -1,5 +1,5 @@
 //
-// Copyright (c) 2022 INRIA
+// Copyright (c) 2022-2024 INRIA
 //
 
 #ifndef __pinocchio_python_collision_tree_broadphase_manager_hpp__
@@ -49,11 +49,11 @@ namespace pinocchio
       static void expose()
       {
         std::string derived_name = boost::typeindex::type_id<Derived>().pretty_name();
-        boost::algorithm::replace_all(derived_name, "hpp::fcl::", "");
+        boost::algorithm::replace_all(derived_name, "coal::", "");
         const std::string class_name = "TreeBroadPhaseManager_" + derived_name;
 
         const std::string class_doc =
-          "Tree-based broad phase manager associated to hpp::fcl::" + derived_name;
+          "Tree-based broad phase manager associated to coal::" + derived_name;
         bp::class_<Self> registered_class(class_name.c_str(), class_doc.c_str(), bp::no_init);
         registered_class.def(TreeBroadPhaseManagerPythonVisitor());