-
Notifications
You must be signed in to change notification settings - Fork 13
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
Intervall per Script setzen #348
Comments
Das ist eine coole Idee. Ich schaue mir das an - würde aber erst mal den PR bearbeiten. War ein wenig Pause mit dem Adapter da ich mit die Checks von Github nicht repariert bekommen habe, hatte aber gestern tolle Hilfe. |
Hi, bin jetzt soweit dass ich da ran gehen würde. Was brauchst Du denn? Eine Änderung der Abfragehäufigeit "an sich" oder eine Änderung der Frequenz alle wieviele "normale" Abfragen die Zell-Details abgefragt werden? |
Hi, frohe Festtage :-), für mich würde das Einstellen der Abfragehäufigeit "an sich" sehr gut passen (also ein Datenpunkt, mit dem dynamisch das Intervall gesetzt werden kann, welches bisher fest in der Instanz gesetzt wird). |
@Ralph1234567 Gerade habe ich eine Version hochgeladen in der Dein Wunsch erfüllt ist. - spätes Weihnachtsgeschenk. In der Konfiguration kann man jetzt anwählen, dass ein State verwendet werden soll, der State steht unter system (overridepoll). Kannst Du es bitte ausprobieren und ein Feedback geben? Danke! |
@christianh17 |
Hi,
wäre es möglich, das Intervall der Datenabfrage von "außen" änderbar zu machen (z B. ein Datenpunkt unter system.adapter.bydhvs, der per eigenem Script beschrieben werden kann)?
Hintergrund:
Ich logge die CellVoltages. An der Lade-/Entladegrenze ist dies nur sinnvoll möglich, wenn das Intervall klein genug ist, um die Änderungen zu erfassen. Das erzeugt allerdings auch dann eine entsprechende Datenmenge, wenn sich der Akku nicht an der Lade-/Entladegrenze befindet.
Idee deshalb: Bei entsprechend hoher bzw. niedriger Spannung Intervall kurz setzen, dazwischen Intervall lang setzen.
Viele Grüße
Ralph
The text was updated successfully, but these errors were encountered: