Skip to content

Support floating point numbers when setting and getting current limit#2

Open
jsphuebner wants to merge 1 commit into
leuzoe:mainfrom
jsphuebner:float_support_current_limit
Open

Support floating point numbers when setting and getting current limit#2
jsphuebner wants to merge 1 commit into
leuzoe:mainfrom
jsphuebner:float_support_current_limit

Conversation

@jsphuebner

Copy link
Copy Markdown

Der Webserver faultet, wenn man eine Dezimalstelle beim amx/amp angibt. Und das aktuelle Limit wird auch nur als int angezeigt (vielleicht in Anlehnung an go-e API?). Jedenfalls ist die PV-Überschuss-Regelung mit dieser Änderung genauer

@leuzoe

leuzoe commented Aug 1, 2022

Copy link
Copy Markdown
Owner

Danke für den Anpassungsvorschlag. In der Tat habe ich wegen der go-e API den Genauigkeitsverlust (bewusst) in Kauf genommen.
Ich werde den Vorschlag leider nicht weiter annehmen können, weil ich kein Gerät zum Testen mehr habe.
Das Modul gebe ich aber gerne in berufene Hände ab, wenn Interesse besteht.

@jsphuebner

Copy link
Copy Markdown
Author

Also einen Fork habe ich ja ohnehin schon angelegt

@maxgew

maxgew commented Aug 15, 2022

Copy link
Copy Markdown

Funkioniert die Regelung in 0,1A-Schritten auch i.V.m. OpenWB?

@jsphuebner

Copy link
Copy Markdown
Author

da bin ich überfragt. Jedenfalls wäre die Änderung abwärtskompatibel weil N € R :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants