-
-
Notifications
You must be signed in to change notification settings - Fork 720
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
Homewizard: add cache #9158
Homewizard: add cache #9158
Conversation
@Spor36 kannst du morgen dann das neue nightly nochmal testen? |
Mache ich. interval: 30s # control cycle interval, dieser? |
OK, passt :-) |
Bin gerade auf der Landesgartenschau in &Fulda und ab morgen in Spanien in Urlaub... @andig wenn du die nötigen Anpassungen noch machen kannst, wäre prima weil ich bin dann mal für 3 Wochen weg 😀 |
Gerne! |
Was noch auffällt: den Product Type prüfen wir nur im Charger. Brauchts den Check? Und falls ja- gehört der nicht stattdessen in die Connection wo auch die Logik implementiert ist? |
Soweit ich die API Doku verstanden habe und auch im Log von @Spor36 gesehen habe, kann man auch das Homewizard Smart Meter Interface mit der API auslesen. Der ProduktCheck im charger prüft, ob es sich um eine schaltbare Device handelt. Man müsste in der Connection dann in Enable und Enabled prüfen. |
@Spor36 hab hier kurz im Biergarten in dein Log geschaut. Eine letzte Bitte: a) Möchte nur final sicher gehen, dass die Funktionen so arbeiten wie gewünscht ... |
Based on the first nightly logs I realised a lot of device timeouts. Therefore I added cache mechanism (1s). I also had to correct the api endpoint (data->state) in enabled function.