stack-of-tasks / pinocchio

A fast and flexible implementation of Rigid Body Dynamics algorithms and their analytical derivatives
http://stack-of-tasks.github.io/pinocchio/
BSD 2-Clause "Simplified" License
1.82k stars 383 forks source link

Homogenize internal namespace #2252

Open jorisv opened 4 months ago

jorisv commented 4 months ago

Pinocchio use different internal namespace for non public API. This should be homogenize to improve code readability.

Internal namespace are the following:

There is not a clear convention for the use of public namespace. Parsers are set in a specific namespace. Mathematical functions overload are in math namespace. boost serialization functions are in serialization namespace. It's less clear why a namespace exists for other functions and it's not releated to an include directory.