Every-Flavor-Robotics / motorgo-mini-driver

Driver software for the MotorGo Mini board
MIT License
2 stars 1 forks source link

Voltage control mode is confusing #86

Open SwapnilPande opened 8 months ago

SwapnilPande commented 8 months ago

Setting a voltage command in voltage control is actually clsoer to setting a current command when the motor phase resistance, inductance, and KV is provided. Similarly, voltage_limit is just as confusing because it's simply constraining the "current" command in voltage-based torque control. This can lead to the unintended conseqence of the command voltage to the motor actually being higher than the voltage limit, after being transformed by simple FOC based on KV and phase resistance.

Naming conventions should be defined to make this clearer.