Skip to content

improve messages #3342

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.

Sign up for GitHub

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

Draft
LKuemmel wants to merge 3 commits into
base: master
Choose a base branch
from
+18 −31
Draft

improve messages #3342

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f5bcda0
improve messages
LKuemmel May 4, 2026
d161012
adopt test
benderl May 6, 2026
f33e77c
fix wording
benderl May 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 2 additions & 11 deletions packages/helpermodules/utils/error_handling.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,7 @@

log = logging.getLogger(__name__)

CP_ERROR = ("Anhaltender Fehler beim Auslesen des Ladepunkts. Soll-Stromstärke, Lade- und Stecker-Status wird "
"zurückgesetzt.")

INTERNAL_ERROR_HINT = ("Liebe Kunden, das Log ist zur Auswertung durch Support-Mitarbeiter der openWB GmbH gedacht. "
"Meldungen, die hier erscheinen können wie Fehlermeldungen aussehen, sind aber oft ganz normal. "
"Beispielsweise führen wir Abfragen der internen Hardware mehrere tausend mal pro Stunde aus "
"(wir gehen bis ans Limit der seriellen Kommunikation um eine möglichst feine Auflösung zu "
"erreichen), eine Abfrage-Fehlerquote von 1-2% ist dabei normal. Wirklich relevante "
"Fehlermeldungen erscheinen in der grafischen Nutzeroberfläche an prominenter Stelle. Bitte "
"belastet unseren Support nicht mit Fragen nach euch unbekannten Log-Meldungen.")
CP_ERROR = "Soll-Stromstärke, Lade- und Stecker-Status wird zurückgesetzt."


class ErrorTimerContext:
Expand All @@ -42,7 +33,7 @@ def __exit__(self, exception_type, exception, exception_traceback) -> bool:
# keine Werte mehr gepublished.
return False
else:
log.error(f"{exception}\n{INTERNAL_ERROR_HINT}")
log.error(exception)
return True
return True

Expand Down
32 changes: 14 additions & 18 deletions packages/modules/common/hardware_check.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,19 +15,15 @@
MAX_ATTEMPTS = 3
RETRY_DELAY_SECONDS = 0.3

OPEN_TICKET = (" Bitte nehme bei anhaltenden Problemen über die Support-Funktion in den Einstellungen Kontakt mit " +
"uns auf.")
RS485_ADAPTER_BROKEN = ("Auslesen von Zähler UND Evse nicht möglich. Vermutlich ist {} defekt oder zwei "
"Busteilnehmer haben die gleiche Modbus-ID. Bitte die Zähler-ID prüfen.")
USB_ADAPTER_BROKEN = RS485_ADAPTER_BROKEN.format('der USB-Adapter')
LAN_ADAPTER_BROKEN = (f"{RS485_ADAPTER_BROKEN.format('der LAN-Konverter abgestürzt,')} "
"Bitte den openWB series2 satellit stromlos machen.")
METER_PROBLEM = "Der Zähler konnte nicht ausgelesen werden. Vermutlich ist der Zähler falsch konfiguriert oder defekt."
METER_BROKEN_VOLTAGES = "Die Spannungen des Zählers konnten nicht korrekt ausgelesen werden: {}V Der Zähler ist defekt."
METER_NO_SERIAL_NUMBER = ("Die Seriennummer des Zählers für das Ladelog kann nicht ausgelesen werden. Wenn Du die "
"Seriennummer für Abrechnungszwecke benötigst, wende Dich bitte an unseren Support. Die "
RS485_ADAPTER_BROKEN = "Erneutes Auslesen des {}"
USB_ADAPTER_BROKEN = RS485_ADAPTER_BROKEN.format('USB-Adapters')
LAN_ADAPTER_BROKEN = RS485_ADAPTER_BROKEN.format('LAN-Konverters')
METER_PROBLEM = "Erneutes Auslesen des Zählers"
METER_BROKEN_VOLTAGES = "Erneutes Auslesen der Spannungen am Zähler"
METER_NO_SERIAL_NUMBER = ("Die Seriennummer des Zählers für das Ladeprotokoll kann nicht ausgelesen werden. Wenn Du "
"die Seriennummer für Abrechnungszwecke benötigst, wende Dich bitte an unseren Support. Die "
"Funktionalität wird dadurch nicht beeinträchtigt!")
EVSE_BROKEN = "Auslesen der EVSE nicht möglich. Vermutlich ist die EVSE defekt oder hat eine unbekannte Modbus-ID. "
EVSE_BROKEN = "Erneutes Auslesen der EVSE"


def check_meter_values(counter_state: CounterState, fault_state: Optional[FaultState] = None) -> None:
Expand Down Expand Up @@ -110,18 +106,18 @@ def request_and_check_hardware(self: ClientHandlerProtocol,
if meter_check_passed is False:
if evse_check_passed is False:
if isinstance(self.client, ModbusTcpClient_):
raise Exception(LAN_ADAPTER_BROKEN + OPEN_TICKET)
raise Exception(LAN_ADAPTER_BROKEN)
else:
raise Exception(USB_ADAPTER_BROKEN + OPEN_TICKET)
raise Exception(USB_ADAPTER_BROKEN)
else:
raise Exception(meter_error_msg + OPEN_TICKET)
raise Exception(meter_error_msg)
elif evse_check_passed and meter_check_passed and meter_error_msg is not None:
fault_state.warning(meter_error_msg + OPEN_TICKET)
fault_state.warning(meter_error_msg)
if evse_check_passed is False:
if meter_error_msg is not None:
raise Exception(EVSE_BROKEN + " " + meter_error_msg + OPEN_TICKET)
raise Exception(EVSE_BROKEN + " " + meter_error_msg)
else:
raise Exception(EVSE_BROKEN + OPEN_TICKET)
raise Exception(EVSE_BROKEN)
return evse_state, counter_state

def check_meter(self: ClientHandlerProtocol) -> Tuple[bool, Optional[str], CounterState]:
Expand Down
4 changes: 2 additions & 2 deletions packages/modules/common/hardware_check_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from modules.common.component_state import CounterState, EvseState
from modules.common.evse import Evse
from modules.common.hardware_check import (
EVSE_BROKEN, LAN_ADAPTER_BROKEN, METER_BROKEN_VOLTAGES, METER_PROBLEM, OPEN_TICKET, USB_ADAPTER_BROKEN,
EVSE_BROKEN, LAN_ADAPTER_BROKEN, METER_BROKEN_VOLTAGES, METER_PROBLEM, USB_ADAPTER_BROKEN,
SeriesHardwareCheckMixin, _check_meter_values)
from modules.common.modbus import NO_CONNECTION, ModbusSerialClient_, ModbusTcpClient_
from modules.conftest import SAMPLE_IP, SAMPLE_PORT
Expand All @@ -21,7 +21,7 @@
[pytest.param(Exception("Modbus"), None, [230]*3, None, False, ModbusSerialClient_, EVSE_BROKEN,
id="EVSE defekt"),
pytest.param(Exception("Modbus"), None, [230, 0, 230], None, False, ModbusSerialClient_,
EVSE_BROKEN + " " + METER_BROKEN_VOLTAGES.format([230, 0, 230]) + OPEN_TICKET,
EVSE_BROKEN + " " + METER_BROKEN_VOLTAGES.format([230, 0, 230]),
id="EVSE defekt und Zähler eine Phase defekt"),
pytest.param(None, Exception("Modbus"), None, None, None,
ModbusSerialClient_, METER_PROBLEM, id="Zähler falsch konfiguriert"),
Expand Down
Loading