Chlorinator improvements

[matthew-larner]

Great work on this integration cryptk!

It would be really helpful to make these improvements when you get time:

1. Add sensor values for chlorinator:

• Cell Temp
• Cell Current
• Cell Voltage
• Relay Polarity
• Status: On/off/chlorinating

2. Add slider for ORP set point

Here's a link to diagnostics: https://www.dropbox.com/scl/fi/1bnvancem5bu6f03u2g41/config_entry-omnilogic_local-01JJHF78M8Q59J80Z10SJNDB34.json?rlkey=8xipermdr9sdx3jejxh029u4n&dl=0

Thanks,
Matt

[cryptk]

I think I have all the info required to add this, just need to find the time to actually write the code for it.

[matthew-larner]

I think I have all the info required to add this, just need to find the time to actually write the code for it.

Great thanks. No pressure, but this would really help with a pool issue I'm troubleshooting. Also this issue if you get time: #150

[ronkmd]

+1 on this. Great Integration so far. Thanks!

[cryptk]

I have added the parts needed to the underlying library to decode the status/alert/error values that come from the Chlorinator. I have some more work I want to get done on the library, but I should be able to have this working "soon-ish"

[cryptk]

So, I have everything I need to expose the status/alert/error data, but I need a PCAP of someone viewing the chlorinator diagnostics/relay polarity/etc in order to get the rest of this.

[plasticfist]

looking forward to this as well!

[cryptk]

So the newly redesigned library has the ability to properly parse out the status, alert and error message fields. That means that I can now add enough data to the integration that you could determine the relay polarity and whether the chlorinator is on/off/paused/generating/etc. It also is able to set both pH and ORP targets for Chlorinator and CSAD.

You can see all of the possible status/alert/error flags here

This is high on my list of things to add to the integration now that I am working towards a 1.0.0 release

Here is what it can parse out of the diagnostics in this issue:

============================================================
Instance of Chlorinator:
============================================================
  alert_messages           : []
  avg_salt_level           : 3300
  bow_id                   : 1
  cell_type                : 3
  dispenser_type           : ChlorinatorDispenserType.SALT
  enabled                  : True
  error_messages           : []
  has_alert                : False
  has_error                : False
  instant_salt_level       : 0
  is_authenticated         : False
  is_generating            : False
  is_on                    : True
  is_paused                : True
  is_ready                 : True
  name                     : Chlorinator
  omni_type                : Chlorinator
  operating_mode           : 2
  operating_state          : 0
  orp_timeout              : 86400
  salt_level_status        : OK
  sc_mode                  : 0
  status                   : ['SYSTEM_PAUSED', 'K2_ACTIVE']
  superchlor_timeout       : 2
  system_id                : 6
  timed_percent            : 50
  timed_percent_telemetry  : 100

[cryptk]

Version 1.2.0b1 is out as a pre-release that adds a LOT of Chlorinator/CSAD functionality! Please feel free to try it out!

[plasticfist]

Gave it a try.

OmniLogic 5.1.85
Hayward TurboCell S3 model TCELLS340
Updated to the latest release 1.2.0, I hope to see the history for these
Cell Temp
Cell Current
Cell Voltage
Relay Polarity
Status: On/off/chlorinating

Issues

• some stats are present but not regularly updating, 4 hours ago last update of "Chlorinator Generating". The history for it looks like it has never generated. Do I need to do anything to clean slate the HACS module after/before install and restart?
• do not see Current, Voltage, Polarity...

I can see chlorine generator amps turning on/off and polarity reversing in the Hayward iPhone app.

These are stats I see in the integration card (no voltage, or current readings we see in the hayward iPhone app)
Chlorinator Average Salt Level 4,317 ppm
Chlorinator Error Present OK
Chlorinator Generating Off
Chlorinator Instant Salt Level 3,995 ppm
Chlorinator Super-Chlorinating Off
Filter Pump Power 126 W
FlowSensor Status Unavailable
Heater Heater Equipment Status Normal
Heater Status Unavailable
Heater Status Unavailable
Pool Flow On
WaterSensor 84.0 °F
Chlorinator Alert Present

Notes

• was using one the beta's and could see what appeared to be an accurate history on the salt cell (on/off correct % of the time), can't be sure which version that was...
• there is an outdoor WAP 6ft from the controller. It should never have local network issues between it and the HA server.

example

<?xml version="1.0" ?>
<STATUS version="1.12">
  <Backyard systemId="0" statusVersion="12" airTemp="104" state="1" ConfigChksum="2663292" mspVersion="R0502000"/>
  <BodyOfWater systemId="1" waterTemp="84" flow="1"/>
  <Filter systemId="2" filterState="1" filterSpeed="50" valvePosition="1" whyFilterIsOn="14" fpOverride="0" reportedFilterSpeed="50" power="119" lastSpeed="18"/>
  <VirtualHeater systemId="3" Current-Set-Point="86" enable="0" SolarSetPoint="87" Mode="0" SilentMode="0" whyHeaterIsOn="1"/>
  <Heater systemId="23" heaterState="0" temp="65535" enable="1" priority="0" maintainFor="24"/>
  <Chlorinator systemId="5" status="132" instantSaltLevel="4063" avgSaltLevel="4326" chlrAlert="0" chlrError="0" scMode="0" operatingState="2" Timed-Percent="42" operatingMode="1" enable="1"/>
  <ValveActuator systemId="7" valveActuatorState="0" whyOn="0"/>
  <ColorLogic-Light systemId="8" lightState="0" currentShow="2" speed="4" brightness="4" specialEffect="0"/>
  <Group systemId="58" groupState="0"/>
</STATUS>

Note: I know salt readings are off... I do manual tests. I've tried to move the offset in controller pad, I think I know how but it doesn't seem to work, the pad touch screen has issues?

[cryptk]

Gave it a try.

Awesome! Thanks!

* some stats are present but not regularly updating, 4 hours ago last update of "Chlorinator Generating".  The history for it looks like it has never generated.  Do I need to do anything to clean slate the HACS module after/before install and restart?

Yep, there is a bug where the re-worked binary sensors aren't updating their states. I'll have it fixed in the next version

* do not see Current, Voltage, Polarity...

Current and Voltage aren't in there yet, I think I need to do a separate diagnostics API call to get that data, and I haven't figured that API call out yet.

For polarity, you should have binary sensors in the diagnostics section (they will be disabled by default) for "K1 Active" and "K2 Active", I those should show you the data you are after for polarity.

* was using one the beta's and could see what appeared to be an accurate history on the salt cell (on/off correct % of the time), can't be sure which version that was...

Will be fixed in the next version, I found the bug

* there is an outdoor WAP 6ft from the controller.  It should never have local network issues between it and the HA server.

You would be surprised, the wifi on these units seems to be quite flaky, but some of the more recent versions should be much better with the retry and recovery logic

Note: I know salt readings are off... I do manual tests. I've tried to move the offset in controller pad, I think I know how but it doesn't seem to work, the pad touch screen has issues?

No idea, I don't have a Chlorinator or a CSAD and I almost never use the touchscreen on the Omni itself.

[cryptk]

@plasticfist I just released 1.3.0 that should resolve the binary sensors not updating