Skip to content
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

Update saj-h2.yaml Documentation #16356

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

Conversation

UDicke
Copy link

@UDicke UDicke commented Sep 27, 2024

Die aktuelle Beschreibung der Parameter entspricht nicht der aktuellen Funktion der Parameter. Die Beschreibungen hier entsprechen der tatsächlichen Funktion.

Die aktuelle Beschreibung der Parameter entspricht nicht der aktuellen Funktion der Parameter. Die Beschreibungen hier entsprechen der tatsächlichen Funktion.
@andig
Copy link
Member

andig commented Sep 27, 2024

Mir ist nicht so richtig klar, warum wir das genau hier aber nicht bei allen anderen Templates für battery control machen sollten?

@andig andig added the devices Specific device support label Sep 27, 2024
@UDicke
Copy link
Author

UDicke commented Sep 27, 2024

Der Parameter "defaultmode" wird nur im saj-h2.yaml verwendet lt. Suche.
Diese Beschreibung hier von minsoc und maxsoc entspricht der Funktion im SAj-H2 bzw. Ampere.Strorage.Pro.
Bei einigen anderen Templates die diese Parameter nutzen, scheint die Beschreibung zu passen. Ich kann das aber nicht pauschal sagen, weil ich nicht weiss wofür die Parameter woanders genutzt werden.
Kann es sein das minsoc und maxsoc bei einigen Modellen auch für das "Laden aus dem Netz" genutzt werden? Auch die ursprüngliche Beschreibung von minsoc deutet darauf hin.
Was machen die Parameter z.B. in templates/definition/meter/sunspec-battery-control.yaml ?

@andig andig marked this pull request as draft September 27, 2024 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants