krahabb / meross_lan

Home Assistant integration for Meross devices
MIT License
408 stars 45 forks source link

MFC100 #393

Open mschingel opened 6 months ago

mschingel commented 6 months ago

Device works as expected however the outlet 2 that is created is actually suppose to be a Fan. Outlet 1 is actually the switch when used will turn on and off both fan and lights. The Fan that has 3 speed settings. Can you please add the speed settings to the fan and updated the entity types.

krahabb commented 6 months ago

The new Moonlight release could already be able to manage that..else I'd need a diagnostic trace

mschingel commented 6 months ago

The new Moonlight release could already be able to manage that..else I'd need a diagnostic trace

I can provide a diagnostics trace if given the steps. I just updated to moonlight and will check

krahabb commented 6 months ago

https://github.com/krahabb/meross_lan/wiki/Diagnostics#download-diagnostics

Contrary to the docs it should now work even through a proxy and is the quickest path. Else you'd need to setup 'tracing' (help in the same wiki page)

mschingel commented 6 months ago

https://pastebin.com/4qiFKpXi Edit:

This has trace in the code.... https://pastebin.com/amDhTgmZ

This is the Download Diagnostics. on Moonlight.

To add, after the update. The device doesnt work the same. The outlet 1 turned into a fan but doesnt do anything, outlet 2 is still the fan as a switch with only on off. not the 3 fan speeds. Outlet 1 used to be a switch that turns on both the light and the fan at the sametime.

krahabb commented 6 months ago

Aww..well... I was hoping it was exactly as the map100 (fresh implementation though) but it also has a light so there's some mess in interpreting the device 'layout' I'll check the diagnostic in the coming days..thank you

mschingel commented 5 months ago

Hello, sorry for the delay. @krahabb

Several issues occur with the updated made. The fan doesn't turn off and the state is wrong or can be wrong un till updated.

I think its best to go back to how it was originally, i can always create helpers to change the device type in home assistant. I dont really need to adjust the fan speed, bc i can do that with homekit.

If u want to debug more i certainly can try.

krahabb commented 5 months ago

Hello @mschingel, This would need some detailed tests and inspection of behavior but I don't have the time at the moment. I'll be back onto this asap.

mschingel commented 4 months ago

@krahabb That works. In the meantime could you role back the changes made since it worked better the first way please?

mschingel commented 4 months ago

@krahabb by chance if you can rolls this back. I understand things come up. But it simply worked better the way it first was.

krahabb commented 4 months ago

Ah yeah..I'm very unable to release any quick fix at the moment...wouldn't it be possible for you to downgrade to the previous (better working) release?