You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
In #732 ergab sich, dass getTotalPower() sehr häufig, von verschiedenen Programmteilen, aufgerufen wird und die zum TotalPower-Wert addierten Werte der drei Phasen nicht unbedingt aus dem selben Abfragezyklus stammen, was zu falschen (Zwischen-)Werten führte.
Describe the solution you'd like
Einige SDM-Powermeter, wie z. B. das SDM630, bieten zusätzlich zu den Leistungswerten der einzelnen Phasen auch schon einen Summenwert (SDM_TOTAL_SYSTEM_POWER) an. (Vgl. Tabelle in SDM.h.)
Dieser Wert müsste dann zusätzlich in readPowerMeter() per sdm.readVal() ausgelesen werden(?)
Falls für ein Modell nicht verfügbar, könnte dann ein fallback auf die aktuelle Lösung mit Addition der Phasenwerte erfolgen.
Hallo @cerise21 ich habe deinen wunsch in #1651 umgesetzt. Eine Test-Version ist dort in den Beschreibung verlinkt. Wäre super wenn du testen könntest ob weiterhin alles funktioniert, Danke.
Is your feature request related to a problem? Please describe.
In #732 ergab sich, dass getTotalPower() sehr häufig, von verschiedenen Programmteilen, aufgerufen wird und die zum TotalPower-Wert addierten Werte der drei Phasen nicht unbedingt aus dem selben Abfragezyklus stammen, was zu falschen (Zwischen-)Werten führte.
Describe the solution you'd like
Einige SDM-Powermeter, wie z. B. das SDM630, bieten zusätzlich zu den Leistungswerten der einzelnen Phasen auch schon einen Summenwert (SDM_TOTAL_SYSTEM_POWER) an. (Vgl. Tabelle in SDM.h.)
Dieser Wert müsste dann zusätzlich in readPowerMeter() per sdm.readVal() ausgelesen werden(?)
Falls für ein Modell nicht verfügbar, könnte dann ein fallback auf die aktuelle Lösung mit Addition der Phasenwerte erfolgen.
Describe alternatives you've considered
No response
Additional context
Ggf. mit der Bearbeitung von #763 kompatibel?
The text was updated successfully, but these errors were encountered: