Conversation
There was a problem hiding this comment.
Wenn der Code mit einem Speicher mit älterer Firmware aufgerufen wird, werden zwei ungültige Modbus Abfragen gesendet. Würde es nicht Sinn machen, nach Z. 43 ein Flag zu setzen, dass eine Firmware größer oder kleiner 95.09 erkannt wurde? Dadurch kann der zweite ungültige Aufruf in Zeile 55 vermieden werden.
Noch besser wäre es natürlich, wenn man die Version der FW über Modbus auslesen könnte. Gibt es dafür ein dokumentiertes Register?
enhance usage of firmware_check
fix type hint
benderl
left a comment
benderl
left a comment
There was a problem hiding this comment.
Ich habe noch kleinere Anpassungen gemacht. So ist deutlicher, wo firmware_check gesetzt wird.
Jetzt gibt es zwei unterschiedliche Eigenschaften, die beide die Version abgrenzen: firmware_check und firmware in der Konfiguration. Das ist sehr unübersichtlich und sollte bei Gelegenheit mal gerade gezogen werden.
2 participants
Mit der neuen Firmware 95.09 sind wieder Änderungen an den Modbus Registern erfolgt und Register 13021 hat wieder kein Vorzeichen, dafür gibt es ein neues Register 5213