openWB Forum

rleidner hat geschrieben: ↑Di Okt 25, 2022 11:04 am Ich habe für den VWID SOC Modul PR 595 erzeugt: https://github.com/openWB/core/pull/595

Ich hab' gesehen dass Du auch deine Freude mit flake8 hast, Leerzeichen hier, Tab vs. Space dort etc. Womöglihc ist es Zeit dass wir mal ein Wiki zur Entwicklungsumgebung starten, die Kombination VSCode, GitHub plugin, vscode remote um direkt auf dem Raspbi arbeiten zu können, und flake8 wäre mal mein Ausgangspunkt.

@lenak ich starte das gern mit meinem gefährlichen Halbwissen, wäre sicher gut wenn Du deine Sicht der Dinge ergänzen kannst?
Ich hab' eben schon angefangen eine Seite zu schreiben, hab' aber keine Berechtigungen um das Wiki zu editieren. Ist das so gewollt, sollen wir hier auch den Weg über PRs & co gehen (Geht das auch beim Wiki?) oder passen die Berechtigungen nur noch nicht?

@rleidner Bis auf die Flake8-Meldungen sieht der PR gut aus.

mrinas hat geschrieben: ↑Mi Okt 26, 2022 6:28 am

rleidner hat geschrieben: ↑Di Okt 25, 2022 11:04 am Ich habe für den VWID SOC Modul PR 595 erzeugt: https://github.com/openWB/core/pull/595

Ich hab' gesehen dass Du auch deine Freude mit flake8 hast, Leerzeichen hier, Tab vs. Space dort etc. Womöglihc ist es Zeit dass wir mal ein Wiki zur Entwicklungsumgebung starten, die Kombination VSCode, GitHub plugin, vscode remote um direkt auf dem Raspbi arbeiten zu können, und flake8 wäre mal mein Ausgangspunkt.

@lenak ich starte das gern mit meinem gefährlichen Halbwissen, wäre sicher gut wenn Du deine Sicht der Dinge ergänzen kannst?
Ich hab' eben schon angefangen eine Seite zu schreiben, hab' aber keine Berechtigungen um das Wiki zu editieren. Ist das so gewollt, sollen wir hier auch den Weg über PRs & co gehen (Geht das auch beim Wiki?) oder passen die Berechtigungen nur noch nicht?

Nativ bietet GitHub keine PRs fürs Wiki an. Allerdings habe ich einen Workflow eingerichtet, der die Markdown-Dateien aus dem Repo mit dem Wiki synchronisiert. Wenn PR #597 gemergt ist, kannst Du mit den Beiträgen loslegen. https://github.com/openWB/core/wiki/Wik ... _erstellen Dort ist beschrieben, wie Du einen Beitrag erstellst.

LenaK hat geschrieben: ↑Mi Okt 26, 2022 1:50 pm @rleidner Bis auf die Flake8-Meldungen sieht der PR gut aus.

Vielen Dank, Lena.
Die flake8-Meldungen habe ich in den von mir erstellten scripts gefixed.

Lediglich in der libvwid.py, die aus einem anderen github Repository übernommen wird, sind noch etliche flake8 Meldungen übrig.

Hier stellt sich mir die Frage, wie das in Zukunft gemacht werden soll.

Wenn sich die originale libvwid.py ändert, muss man diese Änderungen wiederholen.
Das benötigt natürlich immer etwas Zeit und Aufwand.

Ich habe jetzt mal begonnen, die notwendigen Korrekturen in der aktuellen Version per sed-script einzubauen und werde die korrigierte Version incl. dieses Korrektur-Scripts noch einchecken.
Wenn sich die libvwid.py zukünftig ändert, wird das script vieles, aber vermutlich nicht alles lösen können.
Ich hoffe, das ist OK.

Edit: Ich habe jetzt auch libvwid.py "flake8-clean" eingechecked.
dazu gibt es ein neues script prepare_libvwid.sh mit folgenden Funtionen:
- download https://github.com/skagmo/ha_vwid/blob/ ... libvwid.py
- Reparieren der bekannten flake8-issues
- check des Resultats
- wenn nicht OK: Ausgabe der flake-8 Ergebnisse
- wenn OK: Frage ob libvwid.py durch die neue Version ersetzt werden soll
- Wenn ja (Y) wird dies gemacht.

In Github kommen nach dem commit immer noch Fehler, aber damit kann ich nichts anfangen: mit diesen Details: Lass mich wissen, falls ich noch was tun kann.

LenaK hat geschrieben: ↑Mi Okt 26, 2022 1:53 pm

mrinas hat geschrieben: ↑Mi Okt 26, 2022 6:28 am

rleidner hat geschrieben: ↑Di Okt 25, 2022 11:04 am Ich habe für den VWID SOC Modul PR 595 erzeugt: https://github.com/openWB/core/pull/595

Ich hab' gesehen dass Du auch deine Freude mit flake8 hast, Leerzeichen hier, Tab vs. Space dort etc. Womöglihc ist es Zeit dass wir mal ein Wiki zur Entwicklungsumgebung starten, die Kombination VSCode, GitHub plugin, vscode remote um direkt auf dem Raspbi arbeiten zu können, und flake8 wäre mal mein Ausgangspunkt.

@lenak ich starte das gern mit meinem gefährlichen Halbwissen, wäre sicher gut wenn Du deine Sicht der Dinge ergänzen kannst?
Ich hab' eben schon angefangen eine Seite zu schreiben, hab' aber keine Berechtigungen um das Wiki zu editieren. Ist das so gewollt, sollen wir hier auch den Weg über PRs & co gehen (Geht das auch beim Wiki?) oder passen die Berechtigungen nur noch nicht?

Nativ bietet GitHub keine PRs fürs Wiki an. Allerdings habe ich einen Workflow eingerichtet, der die Markdown-Dateien aus dem Repo mit dem Wiki synchronisiert. Wenn PR #597 gemergt ist, kannst Du mit den Beiträgen loslegen. https://github.com/openWB/core/wiki/Wik ... _erstellen Dort ist beschrieben, wie Du einen Beitrag erstellst.

ich hab' mal einen PR für eine neue Wiki-Seite eingestellt. Denke man muss mal sehen wie praktikabel das ist, sehe hier zwei konkurrierende Ansprüche. Einerseits soll das Wiki einfach von möglichst vielen befüllt, ergänzt und angepasst werden, was für direkte Schreibrechte aufs Wiki analog zum snaptec repo sprechen würde - andererseits ermöglicht der Weg über PRs ein zweites Paar Augen, aber eben auch höhere Latenz, eine deutlich höhere Hürde (mal eben schnell ist nicht) und nicht zuletzt höheren Aufwand für euch die die Rechte fürs merge haben.

Der Test für workflow PR 595 in github läuft jetzt fehlerfrei.
Neben der Behebung der flake8 Fehler müssen bisher nicht vorhandene python module noch "gemocked" werden, also dafür Platzhalter definiert werden:
In packages/modules/conftest.py musste für den vwid-SOC-Adapter folgendes neu eintragen werden:

rleidner hat geschrieben: ↑Mi Nov 02, 2022 9:16 am Neben der Behebung der flake8 Fehler müssen bisher nicht vorhandene python module noch "gemocked" werden

Das kann natürlich in bestimmten Fällen die beste Lösung sein. Die "normale" Lösung wäre allerdings diese Module zu installieren. Dafür das Modul zum Installationskommando für die Pakete in den GitHub-Actions hinzufügen.

rleidner hat geschrieben: ↑Mi Nov 02, 2022 9:16 am Der Test für workflow PR 595 in github läuft jetzt fehlerfrei.
Neben der Behebung der flake8 Fehler müssen bisher nicht vorhandene python module noch "gemocked" werden, also dafür Platzhalter definiert werden:
In packages/modules/conftest.py musste für den vwid-SOC-Adapter folgendes neu eintragen werden:

Code: Alles auswählen

sys.modules['aiohttp'] = type(sys)('aiohttp')
sys.modules['lxml'] = type(sys)('lxml')
sys.modules['lxml.html'] = type(sys)('lxml.html')

Danke fürs MQTT SoC Modul, ich bau' mir gerade noch eine Spielwiese parallel zur produktiv genutzten 2.0er openWB auf, da passt das sehr gut. Welche Topics muss ich denn befüllen um den SoC dort 'reinzubekommen?

PR für die Beschreibung im UI ist in Arbeit.

LenaK hat geschrieben: ↑Mi Nov 02, 2022 2:50 pm

Code: Alles auswählen

openWB/set/vehicle/<id>/get/soc
openWB/set/vehicle/<id>/get/range (optional)

PR für die Beschreibung im UI ist in Arbeit.

danke! damit landet der SoC immerhin schon mal im vehicle topic, aber noch nicht im chargepoint/*/get/connected_vehicle

Aber womöglich hab' ich mir inzwischen schon andere Dinge kaputt gemacht und muss noch mal sauber neu starten, mag das nicht ausschließen.