Skip to content

Commit eec887b

Browse files
author
Paul Mitiguy
committed
Added a reminder TODO to change new functions in mobilizer.h to pure virtual when all subclasses have implemented the virtual function in mobilizer.h.
1 parent bb8a870 commit eec887b

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

multibody/tree/mobilizer.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -619,6 +619,8 @@ class Mobilizer : public MultibodyElement<T> {
619619
// @param[in] vdot 1ˢᵗ time derivatives of generalized velocities (v̇).
620620
// @param[out] qddot 2ⁿᵈ time derivatives of the generalized positions (q̈).
621621
// Note: Generalized positions and velocities are stored in `context`.
622+
// TODO(Mitiguy) change this function to a pure virtual function when it has
623+
// been overridden in all subclasses.
622624
virtual void MapVelocityDotToQDDot(const systems::Context<T>& context,
623625
const Eigen::Ref<const VectorX<T>>& vdot,
624626
EigenPtr<VectorX<T>> qddot) const;
@@ -627,6 +629,8 @@ class Mobilizer : public MultibodyElement<T> {
627629
// @param[in] qddot 2ⁿᵈ time derivatives of the generalized positions (q̈).
628630
// @param[out] vdot 1ˢᵗ time derivatives of generalized velocities (v̇).
629631
// Note: Generalized positions and velocities are stored in `context`.
632+
// TODO(Mitiguy) change this function to a pure virtual function when it has
633+
// been overridden in all subclasses.
630634
virtual void MapQDDotToVelocityDot(const systems::Context<T>& context,
631635
const Eigen::Ref<const VectorX<T>>& qddot,
632636
EigenPtr<VectorX<T>> v) const;

0 commit comments

Comments
 (0)