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 Readme.md #501

Merged
merged 3 commits into from Dec 4, 2020
Merged

Update Readme.md #501

merged 3 commits into from Dec 4, 2020

Conversation

tkempf
Copy link
Contributor

@tkempf tkempf commented Dec 4, 2020

explaining the different behaviour of Min+PV mode since 0.33

@andig
Copy link
Member

andig commented Dec 4, 2020

/cc @premultiply

@andig andig added the enhancement New feature or request label Dec 4, 2020
@andig
Copy link
Member

andig commented Dec 4, 2020

Wäre es nicht sinnvoll, die Kommentare zum SHM zusammen in einen Anschnitt zu legen? 95% der Anwender haben vmtl gar keinen und werden hier eher verwirrrt?

@tkempf
Copy link
Contributor Author

tkempf commented Dec 4, 2020

Grundsätzlich schon, aber in dem Fall ändert sich ja tatsächlich das Verhalten des Min+PV. Da die Standardeinstellung im Sunnyportal relativ viel PV voraussetzt, wird der Min+PV relativ häufig unterbrechen sobald ein SHM in defaulteinstellung mitspielt. Deshalb sollte m.E. bei der Beschreibung des Min+PV zumindest ein Verweis auf den Abschnitt mit SHM stehen, in dem dann die genauere Erklärung der Unterschiede steht.

@andig
Copy link
Member

andig commented Dec 4, 2020

Finde ich gut. Dann bei den PV Modi kurzen Hinweis auf SHM und restliche Erläuterung im HEMS/SHM Kapitel?

@tkempf
Copy link
Contributor Author

tkempf commented Dec 4, 2020

So, ich habs jetzt nochmal ein wenig in die Richtung angepasst.

@andig andig merged commit 9e99fe0 into evcc-io:master Dec 4, 2020
@andig
Copy link
Member

andig commented Dec 4, 2020

Top, danke!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants