adrigzr / badconga

Deprecated plugin for Home Assistant for Cecotec Conga Vacuums.
27 stars 4 forks source link

5090 partly compatible #5

Open fabriba opened 3 years ago

fabriba commented 3 years ago

it returns but it seems to show most other information correctly, and also the "locate" action works.

happy to help troubleshooting.

adrigzr commented 3 years ago

Could you send me the output of the command SMSG_DEVICE_STATUS on several statuses?

Thanks!

Chris-mazo26 commented 3 years ago

Hola podrias agregar las fuciones de fregado y progrmas para las congas 5000....por favor....espero que no sea mucha molestia.....cualquier cosa me avisa

guat37 commented 3 years ago

Successfully tested on a 5090. All features are ok. Great Job.

conga

zzarne commented 3 years ago

This works quite well but the update including the charging base includes a bogus robot position. Maybe the schema doesn't match completely for the 5090. It can be seen in above screenshot where Robot x is way below map precision or on the map where the robot jumps to map origin from time to time. This can be fixed by ignoring the robot coordinates in handle_map_update if abs(schema.robotPoseInfo.poseX) + abs(schema.robotPoseInfo.poseY) + abs(schema.robotPoseInfo.posePhi)) is below, say, 6e-39. I can set up a pull request if this is the way to handle this but I'd prefer to decode all data correctly.