Proposer: Wim Meeussen
Present at review:
- List reviewers
Question / concerns / comments
Enter your thoughts on the API and any questions / concerns you have here. Please sign your name. Anything you want to address in the API review should be marked down here before the start of the meeting.
What is the reason for namespacing everything as pr2_mechanism? Should it be pr2_mechanism_model instead?
It seems like the main purpose of pr2_mechanism::Chain is to generate a KDL::Chain. Since KDL::Tree works well with the URDF, do we need the chain class?
- Robot has initXml. Should it be initialized from URDF instead?
- namespace should match package name
- Chain stays
- Remove gripper transmission and nonlinear transmission
Package status change mark change manifest)
Action items that need to be taken.
Major issues that need to be resolved