Ardumower / Sunray

Ardumower Sunray
74 stars 59 forks source link

Alfred serial robot driver: incorrectly swapping left and right motor encoder values? #151

Open greymfm opened 9 months ago

greymfm commented 9 months ago

This looks like a bug (alfred firmware sends left, right encoder values and serial robot drivers receives them as right, left encoder values):

https://github.com/Ardumower/Sunray/blob/6afe8502d9281efa0e8c05e36e277ab9feecad45/sunray/src/driver/SerialRobotDriver.cpp#L286

We need to verify and debug this (e.g. with some printf on console and turning motor only left via the app)...

EinEinfach commented 9 months ago

requestMotorPwm is also affected, comparing to Alfred mcu fw the sequence should be vice versa