Balboa-GS510SZ icon indicating copy to clipboard operation
Balboa-GS510SZ copied to clipboard

Gs514sz with vl702s topside hasn’t got full functionality

Open awallace24335 opened this issue 2 years ago • 16 comments

Hello and thank you so much for this project.

I have set everything up using the HA example and 1/2 of the functions seem to work fine and the rest don’t.

The functions that are not functioning are: Mode button Temp up Temp down Lights button + lights on data

Heater status, pump 1, pump 2, display data and temp all work fine.

My topside is slightly different being the Vl702s so wondering if there could be a slight variation in the coding.

is there anything I can try to do, to get this fully functioning.

these are the results using the serial reader:

Water temperature): 34.50 Set temperature: 39.50 LCD Display: 34.5C Display button: 0 Standard mode: 1 Heater: 1 Pump 1: 0 Pump 2: 0 Light: 0

Many thanks

awallace24335 avatar Jan 26 '24 20:01 awallace24335

IMG_2643

awallace24335 avatar Jan 26 '24 20:01 awallace24335

Since that display is different than I developed the code for you need to go back to the basis. The only way I figured out was to study the communication behaviour between the control unit and the display using oscilloscope to determine and mimic each button push.

MagnusPer avatar Jan 27 '24 21:01 MagnusPer

If you using the https://github.com/MagnusPer/Balboa-GS510SZ/tree/main/examples/Balboa_GS_Read example and try increasing the temp by enter “1” in serial console, does anything change on display?

MagnusPer avatar Jan 27 '24 21:01 MagnusPer

Thank you for your comment, after doing some investigating most of the functions that a written through the serial console work as expected. the only problem is when I press "7" it activates the lights and pressing "4" has no response. How would I change these functions around?

I now just need to plug the board back in with the Home Assistant firmware on to see if that works as expected.

Thank you for your help.

edit: using the mqtt firmware I can remotely send temp up / temp down commands but there seems to be a problem with the temp commands / set temp when using Home Assistant.

awallace24335 avatar Jan 28 '24 09:01 awallace24335

Are you using the MQTT example or this one examples/Balboa_GS_HomeAssistant/Balboa_GS_HomeAssistant.ino @awallace24335 ?

netmindz avatar Jan 28 '24 13:01 netmindz

Try adding a space in the name here so the command mapping matches the name of this button. I don't own this tub so wasn't able to fully test myself

https://github.com/MagnusPer/Balboa-GS510SZ/blob/46f41fbbb927061c465f90602a6bfaaeec2079b3/examples/Balboa_GS_HomeAssistant/Balboa_GS_HomeAssistant.ino#L252

netmindz avatar Jan 28 '24 13:01 netmindz

Are you using the MQTT example or this one examples/Balboa_GS_HomeAssistant/Balboa_GS_HomeAssistant.ino @awallace24335 ?

I was trying to use the Home assistant example but I could get the temp commands working. I then started from the start and used the read example which was working other than lights / pump3 was mixed up.

I have just tried the Mqtt example and I can get everything working other that the lights / pump3 being mixed up, so i thing there must be a problem in the home assistant example

awallace24335 avatar Jan 28 '24 16:01 awallace24335

Yeah there is a fault in the home assistant example, the line I listed should be "Temp Up” not "TempUp" and similar for down

netmindz avatar Jan 28 '24 16:01 netmindz

Try adding a space in the name here so the command mapping matches the name of this button. I don't own this tub so wasn't able to fully test myself

https://github.com/MagnusPer/Balboa-GS510SZ/blob/46f41fbbb927061c465f90602a6bfaaeec2079b3/examples/Balboa_GS_HomeAssistant/Balboa_GS_HomeAssistant.ino#L252

adding this space on both up and down fixed this issue of the Temp Up and Temp Down commands. The only thing that I am missing is the Climate option (temp) I don't think its receiving all the values.

image

image

Thank you

awallace24335 avatar Jan 28 '24 16:01 awallace24335

Yeah there is a fault in the home assistant example, the line I listed should be "Temp Up” not "TempUp" and similar for down

Edit: I have partially sorted it but feel it’s not the correct way as the update target temp isn’t working just yet.

image

https://pastebin.com/xWgw1sWf

awallace24335 avatar Jan 28 '24 20:01 awallace24335

The current and target match between the climate and sensors in that image, so I'm guessing you mean the spa itself hasn't changed?

netmindz avatar Jan 29 '24 11:01 netmindz

The current and target match between the climate and sensors in that image, so I'm guessing you mean the spa itself hasn't changed?

Correct, the only reason the target temp works is because i added "hvac.setTargetTemperature(Balboa.setTemperature);" which i now believe is wrong. to fix the water temp i changed "hvac.setCurrentTargetTemperature(Balboa.waterTemperature);" to "hvac.setCurrentTemperature(Balboa.waterTemperature);"

when i press pump 3 it activates the lights, do you know if this can be switched around?

thank you for your help

awallace24335 avatar Jan 29 '24 13:01 awallace24335

Clearly different outputs have been used on your controller to the author of the library

Just swap this line

else if (s_payload == "Pump3") {

With the line about light

netmindz avatar Jan 30 '24 21:01 netmindz

Clearly different outputs have been used on your controller to the author of the library

Just swap this line

else if (s_payload == "Pump3") {

With the line about light

No problem i will do that. Thank you

I don't think the status is tracked of Pump 3 to verify if its on or not. Is it possible to add it here or will it not work?

image

awallace24335 avatar Jan 31 '24 10:01 awallace24335

I've created a PR with some untested code that might help you along https://github.com/MagnusPer/Balboa-GS510SZ/pull/19

netmindz avatar Apr 18 '24 21:04 netmindz

Hi, I have the same controller, and it's doing exactly the same thing to me as yours. Would it be possible for you to send me your code? I'd be very grateful. Thank you.

surro5 avatar Apr 24 '25 18:04 surro5