Fix measuredPhases out of sync due to delayed phase switching by charger #7162
Conversation
Fix measuredPhases out of sync due to delayed phase switching by charger
Prevent phase warning due to delayed phase switching by charger
Fix compile error
Refactor measuredPhases fix and correct compile errors
Refactor measuredPhases fix and correct compile errors
| if lp.guardGracePeriodElapsed() { | ||
| lp.log.WARN.Printf("ignoring inconsistent phases: %dp < %dp observed active", phases, measuredPhases) | ||
| } | ||
| lp.resetMeasuredPhases() |
There was a problem hiding this comment.
Ich gehe bei allem mit- nur warum sollten wir das tun? Wir messen doch bei jedem Durchlauf, die Werte sind also aktuell!
There was a problem hiding this comment.
Ich gehe bei allem mit- nur warum sollten wir das tun? Wir messen doch bei jedem Durchlauf, die Werte sind also aktuell!
Sind sie leider nicht - es wird nur während der Ladung gemessen. Das Problem tritt ja genau dann auf wenn das Laden geendet hat.
There was a problem hiding this comment.
Dann müsste da aber eigentlich ja auch sofort 0 rauskommen.
Warum tut es das nicht?
There was a problem hiding this comment.
Das tut es nicht weil die alte Messung vor dem Umschaltung noch vorhanden ist. Umschaltung ist träge und kann zurück kommen, bevor umgeschaltet wurde. Dann bleibt auch der alte Messwert stehen.
Mit dieser Erklärung passt das für mich. Was meinst Du?
There was a problem hiding this comment.
Ja, ok, jetzt verstehe ich es auch.
Solange dann nicht nochmal ein neuer "alter" Realmesswert kommt...
There was a problem hiding this comment.
dann würde ja im Fehlerfall wieder resettet.
|
Nightly in 20min |
Fix #7166