Localize and MoveArm have the wrong baseclass #553
Labels
good first issue
Good for newcomers
improvement
Improvement to existing functionality
stale
This issue or pull request already exists
Describe the bug
@dataclass class MotionStep(Step): """ Base class for all steps which should move the robot, but not return a result. """
@dataclass class Localize(MotionStep): """ Step which causes the robot to localize """
@dataclass class MoveArm(MotionStep): """ Step which causes the robot to move its arm """
The Localize and MoveArm classes should not have MotionStep as they do not move the robot. The description for MotionStep could also be more precise in the description so it is clear that MotionStep should only include step that move the base of the robot
This would make it easier to use the base classes in the different robots too.
The text was updated successfully, but these errors were encountered: