Skip to content
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

evcc soc module #1474

Merged
merged 9 commits into from
Mar 27, 2024
Merged

evcc soc module #1474

merged 9 commits into from
Mar 27, 2024

Conversation

MartinRinas
Copy link
Contributor

No description provided.

@MartinRinas
Copy link
Contributor Author

UI: openWB/openwb-ui-settings#426

@MartinRinas
Copy link
Contributor Author

@LKuemmel @benderl habt ihr diesen Fehler in den pytests bereits auf dem Schirm? Am Logging hat dieser PR nichts verändert.

@LKuemmel
Copy link
Contributor

Das hat nichts mit dem Logging zu tun. Der Test prüft, ob eine Exception beim Erstellen der Liste, zB für die Fahrzeuge, geworfen wird, damit am Ende kein Modul in den Auswahllisten fehlt.
Vermutlich kann GitHub gprc nicht importieren. Wenn er das EVCC-Modul für die Fahrzeug-Liste importieren will, kommt eine Exception. Die Pakete in der requirements.txt kennt GitHub. Manchmal muss man sie aber trotzdem noch unter packages/modules/conftest.py hinzufügen.

@MartinRinas
Copy link
Contributor Author

Ah I see, danke. Dann schau' ich mir das gleich nochmal an.

@MartinRinas
Copy link
Contributor Author

finde ich die exception selbst irgendwo? Hier sehe ich zwar dass es nicht klappt, kann aber nicht erkennen woran genau der Prozess scheitert. Sonst muss ich wohl mal das grpc Modul runterschmeißen und den Test lokal laufen lassen um den Fehler zu finden. In VSCode läuft bei mir aktuell alles durch, vermutlich weil das Modul ja vorhanden ist.
https://github.com/openWB/core/actions/runs/8265764211/job/22612181943?pr=1474#step:7:2872

@MartinRinas
Copy link
Contributor Author

@LKuemmel , so Fehler gefunden und behoben. Wäre nun fertig fürs Review & merge.

@LKuemmel LKuemmel added this to the 2.1.4 Step 1 milestone Mar 22, 2024
@LKuemmel LKuemmel merged commit 7308c4b into openWB:master Mar 27, 2024
1 check passed
@MartinRinas MartinRinas deleted the vehicle_evcc branch June 17, 2024 12:38
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.

None yet

2 participants