Skip to content

unify maxchargepower (BC) #20451

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

unify maxchargepower (BC) #20451

wants to merge 4 commits into from

Conversation

premultiply
Copy link
Member

  • unify charge power limit parameter name to maxchargepower for battery inverter templates. This is the limit or setpoint used for forced charge only.
  • deprecate other names
  • update parameter help
  • define base default 4200 W where no other defaults are set in template

Note:
To ensure static conformity with §14a EnWG (Germany) in typical use cases, you should set this parameter to 4200.

@andig
Copy link
Member

andig commented Apr 6, 2025

Das ist breaking, also BC.

@andig andig added the devices Specific device support label Apr 6, 2025
@premultiply premultiply changed the title unify maxchargepower unify maxchargepower (BC) Apr 6, 2025
@premultiply
Copy link
Member Author

...zumindest was die Auswertung eventuell vorhandener, andersnamiger Parameter angeht.

Es sollte aber ansonsten keine fatalen Konfigurationsfehler werfen und mit dem Standardwert starten.

@zachelnet
Copy link
Contributor

zachelnet commented Apr 6, 2025

Hi @premultiply
irgendwie passt maxchargepower beim sungrow-hybrid.yaml nicht:

@andig andig marked this pull request as draft April 8, 2025 18:07
@andig
Copy link
Member

andig commented Apr 8, 2025

@zachelnet die Lösung wäre einfach in dem Template das Default zu ändern. Dann fällt auch das if/else weg. Gerne PR.

@andig
Copy link
Member

andig commented Apr 8, 2025

Bzw: da muss einfach das if/else weg. @premultiply

@zachelnet
Copy link
Contributor

zachelnet commented Apr 8, 2025

@zachelnet die Lösung wäre einfach in dem Template das Default zu ändern. Dann fällt auch das if/else weg. Gerne PR.

hab euch mal ein PR erstellt, kannst du dir dies bitte anschauen ob's so passt. #20497

@hameno please look my PR because I change your part of "Reset max battery discharge power"

@premultiply
Copy link
Member Author

@zachelnet Wozu genau muss man die maxdischargepower per Hand konfigurieren können?
Warum nicht immer fest 10600 W oder immer rated power lesen?

@andig
Copy link
Member

andig commented Apr 8, 2025

Siehe PR vorher. Funktioniert nicht bei allen.

@zachelnet
Copy link
Contributor

zachelnet commented Apr 8, 2025

@zachelnet Wozu genau muss man die maxdischargepower per Hand konfigurieren können? Warum nicht immer fest 10600 W oder immer rated power lesen?

Dies kam von folgende PR #19882, da hat @hameno ein Fehler mit sein SH5.0RT.

Denk aber wir könnten BDC rated power als Value setzten

        # Reset max battery discharge power
        - source: go
          script: power
          in:
          - name: power
            type: int
            config:
              source: modbus
              {{- include "modbus" . | indent 12 }}
              register:
                address: 5627 # BDC rated power
                type: input
                decode: uint16
              scale: 10
          out:
          - name: power
            type: int
            config:
              source: modbus
              {{- include "modbus" . | indent 12 }}
              register:
                address: 33047 # Battery max discharge power
                type: writesingle
                decode: uint16

Ich probiere es bei mir mal aus -> Habs gerade bei mir ausprobiert, soweit klappt es bei mein SH8.0RT den wert auf 800 ( =8kw ) im Register zu setzten.

type: writesingle
decode: uint16
{{- else }}
{{- if lt .maxchargepower 1 }}
Copy link
Contributor

@zachelnet zachelnet Apr 9, 2025

Choose a reason for hiding this comment

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

Hi @premultiply
denk mit {{- if eq .maxchargepower "0" }} sollte es zu keinem Fehler kommen im test

@github-actions github-actions bot added the stale Outdated and ready to close label Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support stale Outdated and ready to close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants