humanoid-path-planner / hpp-doc

Documentation for project Humanoid Path Planner
https://humanoid-path-planner.github.io/hpp-doc
ISC License
29 stars 23 forks source link

Dependency graph #25

Closed nim65s closed 6 years ago

nim65s commented 6 years ago

Hi,

I'm currently trying to generate a graph of all the dependencies between gepetto's packages, and I am using the graph generated by hpp-doc to do some checks.

I found a difference: hpp-manipulation depends on hpp-pinocchio in http://homepages.laas.fr/gsaurel/graph.svg and not in the graph generated by hpp-doc.

So either,

What do you think ?

jmirabel commented 6 years ago

Pushing this to its extreme, the only dependency left would be hpp-core...

I would prefer to keep the explicit dependence to hpp-pinocchio, for clarity. However, it isn't a strong preference.

florent-lamiraux commented 6 years ago

The directive ADD_DOC_DEPENDENCY is necessary to build html links to hpp-pinocchio classes in hpp-manipulation documentation. It does not however add hpp-pinocchio in the "Requires" filed of hpp-manipulation.pc. That is why the dependency does not appear in the graph generated by hpp-doc.

I think that this behaviour is desirable.

nim65s commented 6 years ago

understood, thanks ! :)