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

Homewizard: add cache #9158

Merged
merged 4 commits into from
Jul 30, 2023
Merged

Conversation

thierolm
Copy link
Contributor

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.

@andig andig added the enhancement New feature or request label Jul 29, 2023
@thierolm
Copy link
Contributor Author

@Spor36 kannst du morgen dann das neue nightly nochmal testen?
Eine Frage noch zu deinem Log. Welches interval hast du in der evcc.yaml eingestellt? Bei Nutzung der Steckdosen mit "kleiner" Rechnerkapazität, sind 15s immer eine ganz gute Wahl.

@Spor36
Copy link

Spor36 commented Jul 29, 2023

der evcc.yaml einge

Mache ich. interval: 30s # control cycle interval, dieser?

@thierolm
Copy link
Contributor Author

der evcc.yaml einge

Mache ich. interval: 30s # control cycle interval, dieser?

OK, passt :-)

@Spor36
Copy link

Spor36 commented Jul 30, 2023

EVCC_Nightly-3.csv

meter/homewizard/connection.go Outdated Show resolved Hide resolved
meter/homewizard/connection.go Outdated Show resolved Hide resolved
meter/homewizard/connection.go Outdated Show resolved Hide resolved
meter/homewizard/connection.go Outdated Show resolved Hide resolved
@thierolm
Copy link
Contributor Author

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 😀

@andig
Copy link
Member

andig commented Jul 30, 2023

Gerne!

@andig andig changed the title Homewizard: Enable Cache + Correct Enabled Endpoint Homewizard: add cache Jul 30, 2023
@andig
Copy link
Member

andig commented Jul 30, 2023

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?

@thierolm
Copy link
Contributor Author

thierolm commented Jul 30, 2023

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.

@andig andig merged commit 0c9dbe0 into evcc-io:master Jul 30, 2023
@thierolm
Copy link
Contributor Author

@Spor36 hab hier kurz im Biergarten in dein Log geschaut.
Die Timeouts sind weg. Funktioniert es denn, so wie du willst.

Eine letzte Bitte:
Schicke doch bitte morgen mit dem neuen Nightly noch einmal die Outputs von folgenden Testaufrufen (da sind dann auch die Imrovements von @andig drin):

a) evcc -l trace meter
b) evcc -l trace charger

Möchte nur final sicher gehen, dass die Funktionen so arbeiten wie gewünscht ...

@Spor36
Copy link

Spor36 commented Jul 31, 2023

EVCC_Nightly-6.csv

Ich bin mir nicht sicher, wie man a und b durchführt

@thierolm Vielen Dank und schöne Feiertage

@andig Danke auch für alles

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants