Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Time synchronization in screenlogicpy/HA #32

Open
American1337 opened this issue Jan 27, 2023 · 6 comments · Fixed by #64 or #65
Open

Time synchronization in screenlogicpy/HA #32

American1337 opened this issue Jan 27, 2023 · 6 comments · Fixed by #64 or #65
Assignees
Labels
enhancement New feature or request home assistant Changes required in Home Assistant

Comments

@American1337
Copy link

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 dieselrabbit self-assigned this Jan 28, 2023
@dieselrabbit
Copy link
Owner

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

@dieselrabbit
Copy link
Owner

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

@American1337
Copy link
Author

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
Copy link
Author

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

@dieselrabbit dieselrabbit added the enhancement New feature or request label Feb 16, 2023
@dieselrabbit dieselrabbit added in progress Currently being worked on on hold Work is paused and removed in progress Currently being worked on labels Mar 6, 2023
@dieselrabbit dieselrabbit added in progress Currently being worked on and removed on hold Work is paused labels Mar 18, 2023
@dieselrabbit dieselrabbit added on hold Work is paused and removed in progress Currently being worked on labels Sep 4, 2023
@dieselrabbit dieselrabbit added in progress Currently being worked on home assistant Changes required in Home Assistant and removed on hold Work is paused labels Nov 14, 2023
@dieselrabbit dieselrabbit linked a pull request Nov 14, 2023 that will close this issue
@dieselrabbit dieselrabbit linked a pull request Dec 1, 2023 that will close this issue
@dieselrabbit dieselrabbit removed the in progress Currently being worked on label Dec 1, 2023
@robert-alfaro
Copy link

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
Copy link
Owner

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request home assistant Changes required in Home Assistant
Projects
None yet
3 participants