mikey0000 / Mammotion-HA

Home Assistant integration for Luba, Luba 2 and Yuka (Mammotion) lawn mowers
154 stars 22 forks source link

Yuka blade height wrong when docked #48

Closed TTLucian closed 1 month ago

TTLucian commented 3 months ago

The problem

Integration always shows 50mm blade height when the robot is docked. The set blade height is correct when the robot is mowing.

What Mammotion mower are you using?

Yuka

What version of the Mammotion intergration are you using?

0.0.4

What version of Home Assistant Core are you running?

core-2024.7.3

jLynx commented 3 months ago

We need to hide the blade height for Yuka as Yuka doesn't have a blade height sensor.

TTLucian commented 3 months ago

I think it does. It reports the correct value when it's mowing but not when it's docked.

romeoechovictor commented 3 months ago

Doesn't the Luba always reset it's blade height to 50mm when docking. So you see 50mm when docked and Xmm when mowing. Not sure about Yuka, but this sounds normal for Luba, ie not a bug.

TTLucian commented 3 months ago

I understand. Don't know about Luba. I own a Yuka. I've changed the blade height several times. It always reports the correct value when mowing but when docked, it reports 50 mm. I don't know if it's a bug in the firmware or the in the ha integration.

jLynx commented 3 months ago

The Yuka blade height is manually set where the Luba blade height is automatically set by the app. Because of this the Yuka doesn't have a blade height sensor. You can test this as you will not be able to see the blade height in the Mammotion app for the Yuka

romeoechovictor commented 3 months ago

When your Yuka docks does it reset it's blade height back to 50mm like it does on my Luba? Maybe you can check in the app when it's going back to the dock

jLynx commented 3 months ago

@romeoechovictor the Yuka does not have automatic blade hight adjustment. You have to change it manually yourself like an old lawnmower. As well as you cannot see the height in the app as it does not have a sensor

romeoechovictor commented 3 months ago

Ah ok. I understand now. Strange so that it reports a different value. I guess if docked then don't report any height to hass?

TTLucian commented 3 months ago

The app doesn't show the blade height setting. But the integration does and it is correct when the robot is mowing. If I set it to 70, the ha integration actually reports 70, but only when it's mowing. It reports 50 mm when docked. So I'm thinking that the robot actually does have a blade height sensor. I'll test this further and report back.

jLynx commented 3 months ago

@TTLucian hmm that's interesting, I would assume it's its internal default value changing. Can you test it by setting the height to 40 (or really something else) sending it out and seeing if it's still reporting the correct height?

TTLucian commented 3 months ago

I'll test it tomorrow and report back.

TTLucian commented 2 months ago

It would seem that it reports 50 when docked and 60 when mowing regardless of the actual setting. In my initial test I had the blade height set at 60 so I presumed that it reports correctly. It was just a coincidence. I don't know if it actually has a blade height sensor.

jLynx commented 2 months ago

Thabks for double checking. Thought that would be the case, so I'll continue wtih the original plan of removing the blade height for Yuka

dodavies commented 2 months ago

just ordered one says it does 20mm, I really hope it does ! so looking forward to adding to to HA

mikey0000 commented 2 months ago

Closing as the blade height entity is no longer displayed on Yuka devices.

TTLucian commented 2 months ago

Issue still present on v0.0.22. Did I miss something? Screenshot_2024-08-14-21-24-41-80_c3a231c25ed346e59462e84656a70e50

mikey0000 commented 2 months ago

I'll take a look

mikey0000 commented 2 months ago

We need to fix up the device identification logic. We'll fix it when we get to it :)