netmindz / balboa_GL_ML_spa_control

Control protocol between GL2000 controller and ML series compatibile top panel
16 stars 8 forks source link

Send command v2 #27

Closed netmindz closed 1 year ago

netmindz commented 1 year ago

Sending commands, but now with v2 of the Android Home Assistant library so we can use select drop-down for mode, pump speed etc

netmindz commented 1 year ago

I am getting pretty close to merging this down into main.

Can't test pumps as I have the wrong FB values at the moment and onPumpSwitchStateChanged and onModeSwitchStateChanged can't handle when you need to go "down" off the bottom of the list of choices to loop back to the option you need.

Other than that, I think we are in a pretty good position.

Would be nice to remove the resend command hack, but that can be its own PR

Will need to update the install guide to add pin5, remove the level shifter and add a suitable disclaimer about trying your ESP

netmindz commented 1 year ago

onPumpSwitchStateChanged tested ok for single speed, do you have dual speed @tmjo ?

netmindz commented 1 year ago

I've tried to write onModeSwitchStateChanged, but is still buggy

tmjo commented 1 year ago

onPumpSwitchStateChanged tested ok for single speed, do you have dual speed @tmjo ?

Yes, mine is a dual speed. I'll try to do some more testing this week.