Skip to content

Sonnenbatterie 5/6 support batterymode#20742

Closed
ngehrsitz wants to merge 3 commits into
evcc-io:masterfrom
ngehrsitz:sonnen-eco56-batterymode
Closed

Sonnenbatterie 5/6 support batterymode#20742
ngehrsitz wants to merge 3 commits into
evcc-io:masterfrom
ngehrsitz:sonnen-eco56-batterymode

Conversation

@ngehrsitz
Copy link
Copy Markdown
Contributor

This adds batterymode control support for the Sonnenbatterie eco 5/6 line.
There has to be a delay between the http request to set the battery into slave mode and setting the charging power. If both requests are sent without a delay the charging power will be set and returned by the api, but the battery won´t start charging, even on subsequent requests that set the charging power again.
This appears to be some sort of race condition with the timer that stops charging if no requests are received for three minutes.
Therefore I added a sleep plugin.

- name: capacity
advanced: true
- name: chargepower
default: 3000
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The maximum power is also available from the API under S65, so it would be better if there was a way to fetch that default and store it into a variable.
Is there a way to do this with templates?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Soll ich diesen PR dementsprechend umbauen? Bzw. gibt es zu dem memory leak schon eine diskussion irgendwo?

Comment thread templates/definition/meter/sonnenbatterie_eco56.yaml
@andig andig marked this pull request as draft April 22, 2025 11:47
@andig andig added the devices Specific device support label Apr 22, 2025
@github-actions github-actions Bot added the stale Outdated and ready to close label Apr 29, 2025
@github-actions github-actions Bot removed the stale Outdated and ready to close label Apr 30, 2025
@andig
Copy link
Copy Markdown
Member

andig commented May 1, 2025

No progress, closing.

@andig andig closed this May 1, 2025
@ngehrsitz
Copy link
Copy Markdown
Contributor Author

@andig Ich wollte mit diesem PR noch warten, bis #20451 gemerged wurde. Ich habe jetzt aber mal diesen PR aktualisiert.

@andig
Copy link
Copy Markdown
Member

andig commented May 2, 2025

Ich kann den PR blöderweise nicht wieder öffnen. Falls Du keinen neuen erzeugen willst- Github Ticket ist auf gemacht.

@ngehrsitz
Copy link
Copy Markdown
Contributor Author

Ich warte mal bis der andere PR durch ist, dann eröffne ich einen neuen.

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.

3 participants