Repository navigation
Universelles Modbus-Modul: Faktor zur Registerauslesung - #3741
Conversation
There was a problem hiding this comment.
Pull request overview
Erweitert das Universell-Modbus-Modul um einen konfigurierbaren Faktor pro Register, um gelesene Werte (z. B. Vorzeichen/Skalierung) direkt bei der Registerauslesung anzupassen.
Changes:
RegisterConfigumreg_factorerweitert (Default-Faktor).read_value()multipliziert gelesene Registerwerte mit dem konfigurierten Faktor.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/modules/devices/generic/modbus/helper.py | Skaliert gelesene Registerwerte über reg_factor. |
| packages/modules/devices/generic/modbus/config.py | Fügt reg_factor zur Register-Konfiguration hinzu. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| if value is not None: | ||
| value *= register_config.reg_factor | ||
| return value |
LKuemmel
left a comment
LKuemmel
left a comment
There was a problem hiding this comment.
Bitte die Github-Syntax zur Verknüpfung von Issue und PR verwenden: https://docs.github.com/de/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
| unit=unit, | ||
| ) | ||
|
|
||
| factor = register_config.reg_factor if register_config.reg_factor is not None else 1 |
There was a problem hiding this comment.
Hast Du nicht in Zeile 9 bereits ausgeschlossen, dass reg_factor None ist?
LKuemmel
changed the title
* Add Faktor-Input zur Registerauslesung * Add Copilot suggestions * Update type von reg_factor * fix
openWB/openwb-ui-settings#1033
Resolve: #3732