cryptk / haomnilogic-local

A Home Assistant integration for Hayward OmniLogic/OmniHub pool controllers using the local UDP api
Apache License 2.0
19 stars 4 forks source link

ORP Sensor #116

Open umchemist opened 1 month ago

umchemist commented 1 month ago

Hi!

This is a great add-on, far better than the Hayward app. For example, I can get the air temp reading!

I've have a pH and ORP set up, along with an ozone set up. Welcome the opportunity to contribute and get the pH/ORP values into this.

Please reach out.

daytonturner commented 1 month ago

I'm about to take a stab at adding these as well, will update here if I make any progress - also interested in getting detail beyond what the "official" cloud integration does which is simply show you the current pH and current ORP reading - I'd like to know when the chlorinator is actually chlorinating, what the target mV is, if the heater is heating, and ideally diagnostic data from each of the components which the app can access (including errors/warning notifications)

umchemist commented 1 month ago

Awesome, please let me know how I can help ! Some thoughts:

On Wed, Jul 24, 2024 at 1:41 PM daytonturner @.***> wrote:

I'm about to take a stab at adding these as well, will update here if I make any progress - also interested in getting detail beyond what the "official" cloud integration does which is simply show you the current pH and current ORP reading - I'd like to know when the chlorinator is actually chlorinating, what the target mV is, if the heater is heating, and ideally diagnostic data from each of the components which the app can access (including errors/warning notifications)

— Reply to this email directly, view it on GitHub https://github.com/cryptk/haomnilogic-local/issues/116#issuecomment-2248676835, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2UNKEUXRDA5W2NT3REIDTZN7YN5AVCNFSM6AAAAABLF6RLOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGY3TMOBTGU . You are receiving this because you authored the thread.Message ID: @.***>

krucka commented 1 month ago

I recently had my 1yr old SWG go out and now that I have my warranty replacement, was looking for a way to have better trending on the diagnostic reporting from the chlorinator. I have a TurboCell TCellS340W and am willing to help test if necessary. In the Android Omnilogic app, the same diagnostic window that shows instant and average salt (which already shows up in home assistant) shows: -Cell Temp -Cell Current -Cell Voltage -Relay Polarity From the local controller, there are other values as well such as board temp, that would also be nice to add.

cryptk commented 1 month ago

There should be a PH sensor already and ORP should be shown as an attribute on that PH sensor. I already plan to split the ORP data out to it's own sensor though as part of #106

If you don't have a pH sensor, can you please attach the diagnostic data from the addon to this issue so that I can look into it when I have time?

umchemist commented 1 month ago

Sounds great, unfortunately Im not seeing a pH sensor in the 7.5 build. Attached is the diagnostic data.

On Tue, Jul 30, 2024 at 11:48 AM cryptk @.***> wrote:

There should be a PH sensor already and ORP should be shown as an attribute on that PH sensor. I already plan to split the ORP data out to it's own sensor though as part of #106 https://github.com/cryptk/haomnilogic-local/issues/106

If you don't have a pH sensor, can you please attach the diagnostic data from the addon to this issue so that I can look into it when I have time?

— Reply to this email directly, view it on GitHub https://github.com/cryptk/haomnilogic-local/issues/116#issuecomment-2258786337, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2UNKF5E4O6AIHBCFZNNLDZO67WJAVCNFSM6AAAAABLF6RLOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJYG44DMMZTG4 . You are receiving this because you authored the thread.Message ID: @.***>