fix sma hm

[LKuemmel]

No description provided.

[benderl]

Ich kann nicht beurteilen, ob der Datentyp der Seriennummer immer eine Zahl ist. Eventuell gibt es auch Seriennummern mit Nullen am Anfang?
Ich würde bei solchen Daten, die nicht klar definiert sind und nicht von uns beeinflusst werden können, immer einen String als Datentyp nehmen. Dann bekommt man nicht irgendwann doch Probleme, falls SMA die Zahlen ausgehen und z.B. auf Hexadezimal umgestellt wird.

[LKuemmel]

@yankee42 Du hast mal diese Prüfung eingebaut:
log.error("Serial <%s> must bei an int or None, but is <%s>. Assuming None.", serial, type(serial))
Hast Du weitere Infos zur Seriennummer?

[benderl]

Wenn es beim Datentyp Int bleiben soll, macht es Sinn, eine Anzahl Stellen vorzugeben, um ein paar Falscheingaben abzufangen. Einfach diese Parameter für openwb-base-number-input ergänzen:
min="1000000"
max="9999999"
Wobei ich keine Ahnung habe, wieviele Stellen so eine Seriennummer haben kann.

[yankee42]

@yankee42 Du hast mal diese Prüfung eingebaut: log.error("Serial <%s> must bei an int or None, but is <%s>. Assuming None.", serial, type(serial)) Hast Du weitere Infos zur Seriennummer?

Oh, ein Rechtschreibfehler. Es muss natürlich "be an int" heißen.

Aber zu deiner Frage: Die Seriennummer wird im Speedwire-Protokoll binär als 4-Byte-Integer übertragen.

Ich kann nicht beurteilen, ob der Datentyp der Seriennummer immer eine Zahl ist. Eventuell gibt es auch Seriennummern mit Nullen am Anfang?

Es ist immer eine Zahl. Man kann auch Nullen vorranstellen, wenn man die Darstellung schöner findet, macht aber keinen Unterschied.

Dann bekommt man nicht irgendwann doch Probleme, falls SMA die Zahlen ausgehen und z.B. auf Hexadezimal umgestellt wird.

Da kann ich dich beruhigen. Wenn SMA das Ändern will, müssen sie das Speedwire Protokoll ändern und dann geht das ganze Modul nicht mehr.

Was mir zu dem PR einfällt: Bis jetzt ist es so, dass man none (oder irgendwas, was keine Zahl ist) eingeben kann, damit die Seriennummer gänzlich ignoriert wird. Geht irgend sowas nach dem PR noch?

[LKuemmel]

Ja, dann das Feld leer lassen.