motors API: passing something other than speed percentage

Today in on_for_rotations(), etc the user passes the speed_pct because this is how the EV3-G block works. I started updating the MINDCUBER demo to use the new API but I end up doing the following all over the place (two lines, one to cald the speed_pct and a second to call the API)

        speed_pct = self.turntable.dps_to_speed(MindCuber.rotate_speed)
        self.turntable.on_to_position(speed_pct, temp_pos)

which works just fine but doesn’t look super clean (side note, on_to_position() is new). What if the on_to_XYZ() methods accepted either speed_pct or (speed_type, speed_value) tuple such as the following

        self.turntable.on_to_position(('dps', MindCuber.rotate_speed), temp_pos)

Valid types there could be ‘dps’, ‘dpm’, ‘rps’, or ‘rpm’.

[dwalton76]

Another option would be something like

class SpeedRPS(int):
    pass

class SpeedRPM(int):
    pass

class SpeedDPS(int):
    pass

class SpeedDPM(int):
    pass

and then the user would call via

    motor.on_for_rotations(SpeedRPS(400), 3)

That does look a little less hackish doesn’t it

[dwalton76]

yeah but I bet most programs only use one speed unit so hopefully won’t be bad…new code posted