search

borglab / GTDynamics

Full kinodynamics constraints for arbitrary robot configurations with factor graphs.
BSD 2-Clause "Simplified" License
39 stars 10 forks source link

Rename `PoseFactor` to `JointFactor` #283

Closed varunagrawal closed 2 years ago

varunagrawal commented 2 years ago

PoseFactor is not a very descriptive name for what the factor is doing. A better name would be JointFactor or JointConstraintFactor. I prefer JointFactor since it is easy to remember and encapsulates what the factor is all about.

Another design decision to consider with @dellaert.

varunagrawal commented 2 years ago

PoseFactor is consistent with TwistFactor naming scheme, so closing.