rand256 / valetudo

Valetudo RE - experimental vacuum software, cloud free
Apache License 2.0
669 stars 74 forks source link

Gen1: Return to dock in paused state not possible #280

Closed pidator closed 4 years ago

pidator commented 4 years ago

If my robot has reached a target point or if I pause a zoned cleaning, I can't send the robot back to the dock with the button on home tab.

Here's the output of /api/current_status in these situations:

paused zoned cleaning: {"msg_ver":8,"state":3,"battery":100,"clean_time":0,"clean_area":240000,"error_code":0,"map_present":1,"in_cleaning":0,"fan_power":60,"dnd_enabled":1,"human_state":"Idle","human_error":"No error","model":"rockrobo.vacuum.v1"}

robot at target point: {"msg_ver":8,"state":3,"battery":100,"clean_time":0,"clean_area":240000,"error_code":0,"map_present":1,"in_cleaning":0,"fan_power":60,"dnd_enabled":1,"human_state":"Idle","human_error":"No error","model":"rockrobo.vacuum.v1"}

Vacuum Model: Gen1 Valetudo Version: 0.9.7 (4007)

rand256 commented 4 years ago

Err.. Do you mean it just doesn't react on pressing that button?

Btw I guess paused cleaning should have state=10, so most likely the data from current_status should be different. It doesn't matter much in this exact case, though.

pidator commented 4 years ago

Err.. Do you mean it just doesn't react on pressing that button?

yes, no reaction at all after pressing the return to dock button.

Btw I guess paused cleaning should have state=10, so most likely the data from current_status should be different.

I think I've stoppped/paused the zoned cleaning on the way to the zone. So the "cleaning" hasen't started yet.

rand256 commented 4 years ago

Could you please try this build?

pidator commented 4 years ago

Could you please try this build?

It's working! Great, thanks!

rand256 commented 4 years ago

Glad to hear. I think I'll release this in a next few days as a final 0.9.8 if no more issues appears.

pidator commented 4 years ago

I will send you an PR for updated german translations in the next days too, perhaps this could still be integrated in 0.9.8. I'll close this issue here.