cryptk
ORP Sensor
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.
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)
Awesome, please let me know how I can help ! Some thoughts:
- The pH/ORP controls just a high voltage relay
- pH calibration is just a single point (odd, most are two or three points)
- Is there an ability to control the run time of the relay? That's not seen anywhere, but could be handy when using CO2 or a strong acid to pump and reduce pH.
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: @.***>
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.
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?
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: @.***>
There is not attachment to that comment. I don't know if attachments work with email replies, you might need to attach it in the GitHub UI itself
Right- sorry about that. File now attached. config_entry-omnilogic_local-01J384ZFW1R0K0GS7NBM7NRQ0C(1).json
@cryptk, I'm obviously not in a rush, and was checking back in on this and wondered if what I posted in your Python repo is any help?
https://github.com/cryptk/python-omnilogic-local/issues/54?notification_referrer_id=NT_kwDOARl-1LQxMTY3MTgwMjU3OToxODQ0ODA4NA#issuecomment-2264596409
https://github.com/cryptk/python-omnilogic-local/issues/54?notification_referrer_id=NT_kwDOARl-1LQxMTY3MTgwMjU3OToxODQ0ODA4NA#issuecomment-2264636279
@umchemist your issue is because your chlorinator is a CO2 type, which wasn't mentioned in the documentation that I have. I'll work on getting that supported.
@krucka this issue is caused by something different than that issue
@cryptk, i recently updated to the latest version, and now i'm getting this error related to the chlorinator. nothing on my pool equipment end has changed:
enabled debug logging but nothing shows up. screenshot above was all i could find
@krucka can you downgrade the integration to get back to a working state, pull diagnostic info from the integration and then open a new issue with that diag info?
Actually, your issue might be resolved by a change I just merged that I will be releasing shortly.
@cryptk, do you want me to wait first to see if your new release fixes it before downgrading?
@krucka try the new version (0.7.12) and if you still have issues, then downgrade, pull diagnostics and open a NEW issue with those diagnostics attached.
This issue was originally for getting the PH/ORP sensor values available in the integration. When doing a simulation run with the diagnostics provided, those are now available:
Going to close this issue as the other parts are out of scope for the original request (althought I do acknowledge that further Chlorinator/CSAD support work is needed, and is actively on the way)