JSON SoC Modul

[MartinRinas]

No description provided.

[MartinRinas]

UI; openWB/openwb-ui-settings#598

[andlem74]

Hallo Martin,
es wäre aus meiner Sicht wünschenswert, die zwei JSON-Parsing-Schritte für SoC und Range auf einem einzigen JSON-Objekt aus derselben HTTP-Abfrage anzuwenden oder wenigstens anwenden zu können. Der Sinn eines JSON-Objekts besteht doch in den meisten Fällen darin, mehrere Eigenschaften in einem Objekt zu verpacken. Es ist m. E. nicht zielführend, dieselbe URL zweimal abzufragen, weil die Abfragehäufigkeit der Vendor-API so gering wie möglich gehalten werden muss, damit diese nicht wegen Überlastung eingeschränkt oder abgestellt wird.
Des weiteren wäre es gut, Konfigurationsmöglichkeiten für die Option "Berechnung während der Ladung" und das "Timeout" vorzusehen.
Grüße
Andreas

[MartinRinas]

Hi, klar schau' ich mir gern an.

• nur ein Abruf sofern SoC und Range von der identischen URL gelesen werden soll
• manuelle Berechnung während des Ladens ermöglichen
• timeout als optionale Konfigurationsoption

[andlem74]

Ich stelle meinen PSA Car Controller PR solange als Draft zurück, bis ich getestet habe, ob ich die Funktion mit Deinem Modul gleichermaßen abbilden kann. Falls ja, was sehr zu erwarten ist, lösche ich meinen PR. Wir können es dann ja einmal im Forum erwähnen, wie es mit Deinem JSON-Modul umsetzbar ist.

[MartinRinas]

@andlem74 das sollte soweit nun passen, teste das doch gern mal bei Dir.

Die HTTP URL wird nur einmal aufgerufen sofern range und soc auf die identische URL zeigen. Im Log taucht das doppelt auf, wenngleich tatsächlich nur ein HTTP Aufruf stattfindet. Konnte ich in meinen HTTP Server logs bestätigen. Doppelte Protokollierung ist wohl ein fall für ein anderen PR.

[LKuemmel]

Ich stimme @andlem74 zu, dass ein json-Objekt abgefragt und ausgewertet wird. die Angabe von separaten URLs macht es unnötig kompliziert. Kannst Du das Modul bitte so ändern, dass analog zum json-Modul für Geräte eine URL angegeben werden kann?

[MartinRinas]

@LKuemmel habe ich geändert, UI ist auch angepasst.