During the review of https://github.com/personalrobotics/aikido/pull/588, we realized that (for some reason) these methods take StateSpace::State* instead of MetaSkeletonStateSpace::State*. Since we're working with ConcreteRobot we must be working with MetaSkeletonStateSpace::State* (unless the user is doing something very stupid).
We should fix this at some point, potentially during @aditya-vk's refactor of the Robot classes.
During the review of https://github.com/personalrobotics/aikido/pull/588, we realized that (for some reason) these methods take
StateSpace::State*instead ofMetaSkeletonStateSpace::State*. Since we're working withConcreteRobotwe must be working withMetaSkeletonStateSpace::State*(unless the user is doing something very stupid).We should fix this at some point, potentially during @aditya-vk's refactor of the Robot classes.