Skip to content

Conversation

@MartinRinas
Copy link
Contributor

@MartinRinas MartinRinas commented Feb 21, 2025

Liest die Lade/Entladeleistung aus den Modbusregistern aus anstelle diese durch Batteriespannung und -strom zu berechnen.

Fügt aktive Speichersteuerung für SMA Hybrid-WR hinzu.
Fügt Logging für gelesene Werte für alle SMA Batterietypen hinzu.

Ersetzt #2163

@MartinRinas

This comment was marked as resolved.

@rleidner
Copy link
Collaborator

@LKuemmel , @benderl hier werde ich Hilfe brauchen. Woher dieser Fehler in den Tests kommt ist mir leider nicht klar.

So einen pytest-Fehler hatte ich kürzlich auch.
Du kannst pytest auch lokal laufen lassen, z.B. "python -m pytest --log-cli-level=DEBUG packages/modules/configuration_test.py" aus dem Hauptverzeichnis.
Leider werden dann aber die logs nirgends geschrieben.
Ich habe dann in packages/modules/configuration.py temporär bei einigen exceptions zusätzlich zum log.exception noch ein print mit dem Fehler dazugebaut - dann konnte ich das Problem sehen, bei mir ein nicht ganz korrektes import...

@MartinRinas

This comment was marked as resolved.

@rleidner
Copy link
Collaborator

Die gleichen Importe gibts in anderen Modulen ebenfalls, z.b. smarthome/lambda_/watt.py oder auch im common/modbus.py
Vermutlich wurden die akzeptiert, als pytest noch nicht im github workflow aktiv war...
Ich hatte dasselbe Problem.
Ich habe das Modul dann ohne from komplett importiert und im Code das entsprechende Prefix davorgesetzt.

@LKuemmel
Copy link
Contributor

GitHub kennt die Pakete nicht. Du kannst ein Mock für die Klasse hinzufügen, so wie hier:

module.BinaryPayloadDecoder = Mock()

@LKuemmel LKuemmel merged commit 7aba53c into openWB:master Apr 7, 2025
1 check passed
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