Repository navigation
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Durch die Anpassung wird leider auch ein Standby-Verbrauch des WR als Ertrag (negativ) ausgegeben.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Das ist die exakte Umsetzung aus Software 1.9 - das war also lange Zeit so. Das Vorzeichen ist anscheinend leider nicht für alle VZLogger einheitlich.
Die alternative wäre ein Auswahlfeld für das Vorzeichen (Standardbelegung nicht invertiert), damit es abwärtskompatibel zu bisher eingerichteten Anlagen bleibt.
Standby-Verbrauch ist immer leidiges Thema da es nicht einheitlich gelöst ist. Viele Hersteller zeigen den gar nicht an (wird also genullt).
Bei VZLogger bin ich mir nicht sicher. Zeigt der denn den Standby Verbrauch, also ist das eine reale Gefahr? Mir sind auch aus 1.9 keine Problemmeldungen zu dem Modul bekannt.
Riskofrei ist das Auswahlfeld - bequemer für Kunden und falls nicht erforderlich die automatische Invertierung.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich kann das nicht beurteilen. Mir war nur aufgefallen, dass die Anpassung bei eventuellem Standbyverbrauch ein Problem hat. Wenn ich mir die unterstützte Hardware des VZ ansehe, wäre eine einstellbare Option sicher sinnvoll. Hauptsächlich werden allgemeine Zähler unterstützt, also erfolgt die Erfassung vor einem WR. Da wäre durchaus Standby enthalten.
Zusätzlich können einige WR direkt ausgelesen werden, was ggf. keine Anpassung des Vorzeichens erfordert.
Ich wäre daher für eine Option in der Komponente.