Skip to content

Fix Sungrow iHM Counter values - #3350

Merged
benderl merged 2 commits into
openWB:masterfrom
seaspotter:fix_sungrow_ihm
May 7, 2026
Merged

benderl merged 2 commits into
openWB:masterfrom
seaspotter:fix_sungrow_ihm

Conversation

@seaspotter

Copy link
Copy Markdown
Collaborator

@benderl benderl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich war mal so frei und habe den Kommentar etwas erweitert.
Es sollte auf jeden Fall festgehalten werden, dass die Berechnung auch nur eine Näherung ist, bevor das nach und nach bei jedem Modul oder sogar in Kernfunktionen so implementiert wird. Sonst haben wir irgendwann auch wieder so einen Mischmasch wie bei den Stömen (Wirkleistung, Blindleistung, Scheinleistung).

@benderl benderl self-assigned this May 7, 2026
@benderl benderl added the bug Something isn't working label May 7, 2026
@benderl benderl added this to the 2.2.0-Patch.1 milestone May 7, 2026
@benderl
benderl merged commit 8a95d9f into openWB:master May 7, 2026
1 check passed
@seaspotter
seaspotter deleted the fix_sungrow_ihm branch May 7, 2026 09:37
ndrsnhs pushed a commit to ndrsnhs/core that referenced this pull request Aug 24, 2026
* Fix power sign and update voltage calculation

* explain voltage calculation

---------

Co-authored-by: Lutz Bender <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants