search

dieselrabbit / screenlogicpy

Interface for Pentair Screenlogic connected pool controllers over IP in Python
GNU General Public License v3.0
17 stars 5 forks source link

Time synchronization in screenlogicpy/HA #32

Open American1337 opened 1 year ago

American1337 commented 1 year ago

Thanks for all your help so far. A feature that would be helpful is to add the time sync feature that is available in the pentair application. While I want to place most of my automations in HA, a few automations I will leave in pentair such as the cleaning schedules. The feature circuit enablement and lighting I will place in HA.

My pentair system was running the pool cleaner a few hours off of expected schedule until I clicked sync time. I'd like to mostly avoid using the pentair app, use HA instead, and in that case time sync would never occur. It would be awesome if HA integration could do the trigger time sync periodically -- especially when there is power outage (i.e.: HA gets restarted).

Also, I'm wondering if it's possible to see the time/date on the pentair system? perhaps add that as a sensor or put in the logs.

Time sync feature in the application:

image

Could add option as part of integration settings:

image

dieselrabbit commented 1 year ago

Should be doable. I was able to identify 8110 to get the date & time and 8112 to set it.

dieselrabbit commented 1 year ago

What is the use case you are thinking of for seeing the controller time, either as a sensor or in the logs?

American1337 commented 1 year ago

Seeing the time was just for piece of mind that it is set correctly. If it prints out in log when debug logging is on that would be nice.

American1337 commented 1 year ago

To be clear, main thing I'm interested in ability to set/sync time with HA.

robert-alfaro commented 5 months ago

I assume this still needs hookup at the HA layer to be active? My controller's time drifts over time and I don't believe any syncing is taking place. How can I confirm / how can I help if needed?

dieselrabbit commented 5 months ago

I assume this still needs hookup at the HA layer to be active?

That is correct. Only the backend implementation in this lib is done.

The HA implementation is currently 3rd on my list behind Super Chlorination and IntelliChem chemistry LSI numbers.