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

SMA: fix battery control #17393

Merged
merged 2 commits into from
Nov 25, 2024
Merged

SMA: fix battery control #17393

merged 2 commits into from
Nov 25, 2024

Conversation

premultiply
Copy link
Member

@premultiply premultiply commented Nov 21, 2024

Fixes #17115
Fixes #17374
Fixes #17201

@premultiply premultiply added the devices Specific device support label Nov 21, 2024
@premultiply premultiply self-assigned this Nov 21, 2024
@andig
Copy link
Member

andig commented Nov 21, 2024

Sind die Templates jetzt identisch? Dann könnten wir sie auch kombinieren.

@premultiply
Copy link
Member Author

Ja, ich traue mich aber noch nicht. Es könnte ja sein, dass es doch noch mit gereifter Erkenntnis Unterscheidung braucht.
Daher würde ich erstmal eine Zeit lang so ins Rennen gehen. Zusammengelegt ist ja schnell...

@andig
Copy link
Member

andig commented Nov 22, 2024

@premultiply betrifft das auch #17201?

@premultiply
Copy link
Member Author

Vermutlich.
Wir müssen es probieren.

Mit meinem SBS3.7-10 funktionieren ALLE bisherigen und diese neue Variante problemlos.

@andig
Copy link
Member

andig commented Nov 22, 2024

Dann rein damit?

@andig
Copy link
Member

andig commented Nov 23, 2024

ping @premultiply ab ins nightly?

@premultiply premultiply marked this pull request as ready for review November 24, 2024 14:27
@andig andig merged commit f0aeb9b into master Nov 25, 2024
6 checks passed
@andig andig deleted the devices/sma-batctrl branch November 25, 2024 08:07
@czam70
Copy link

czam70 commented Nov 25, 2024

Ist das denn mit 0.131.7 schon implementiert worden? Oder kommt das mit der nächsten Verision?
Im Changelog hier https://github.com/evcc-io/evcc/releases/tag/0.131.7 habe ich es nicht gesehen.

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
3 participants