-
-
Notifications
You must be signed in to change notification settings - Fork 698
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
Kostal Plenticore: add max charge rate, clarify naming and documentation #17740
base: master
Are you sure you want to change the base?
Conversation
@andig @deadrabbit87 bitte um Prüfung/Stellungnahme |
spaces
products: | ||
- brand: Kostal | ||
description: | ||
generic: Plenticore Hybrid (Gen2) | ||
generic: Plenticore Hybrid (nextgen) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What makes this next?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nextgen - next generation of template. we need the old one to keep older installations running. the new features (gridcharge/chargerate) makes it next - next on the version using modbus register 1042 which is not capable of the new features.
If we are able to track down the issues lead into keeping the old revision of the template we mind this and deprecate other versions I suggest.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anwender verstehen das nicht. Generationen von Templates? Huh?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Schwierig einen Konsens zu finden, die letzte Umbenennung ist inhaltlich falsch. Anwender verstehen vermutlich am besten "-gridcharge": aber erstens ist das nicht mehr für die zwei Funktionen gültig und zweitens hattest du zuletzt richtigerweise Bedenken, da der Name für PV unglücklich ist. Eine Auftrennung des Templates nur für die Batteriefunktionen ist nicht mehr umsetzbar und irgendwie auch unglücklich.
"Generationen von Templates".. hmm: ergänzende Funktionen brauchen einen Namen? Den ersten Namen wiederherzustellen hatte zwischen uns keinen Konsens.
Ich fänd den Namen mit Bezug auf die "erweiteren Funktionen" zutreffend. Hast du einen besseren Vorschlag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wenn "-ng" nicht zutreffend ist, wären "-plus", "-advanced" oder "-extended" zutreffender? Dazu würde dann aber auch gern nochmal Rücksprache halten. Gerne weitere Vorschläge..."-v2" geht schon wieder zu sehr in die Richtung der HW/SW Version des WR.
Am liebsten einigen wir uns auf "nextgen" lehnt sich an gen2 an, ohne eine HW-Generation zu benennen und bezieht sich auf die Funktionen, die das andere Template eben nicht hat.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich persönlich fände nach wie vor irgendwas mit -gridcharge
am Besten. Von mir aus auch -gridchargefeature
.
Was sprechendes wäre einfach sinnvoll, verstehe aber natürlich die Einwände von @andig.
Daher die Einigung auf -nextgen
.
-plus
finde ich wieder irreführend mit der Modellbezeichnung Plenticore Plus.
-extended
fände ich auch denkbar.
So oder so: Vielen Dank @iseeberg79 für deine Arbeit!
Die Änderung erfolgt abgestimmt mit der Umbenennung, die es zuletzt gegeben hat. Leider ist bisher nicht geklärt, warum einige HW+SW Versionen mit der Zurückschaltung in die interne Steuerung, nachdem die Batteriesperre oder Netzladung erfolgt ist, haben.
Da "gen2" zu Verwechselungen führt, "hw200" nicht zutreffend ist und "-gridcharge" nicht auf den Wechselrichter (PV) zutrifft, erscheint -ng als passender Kompromiss. Damit ist dann klar, das es die "nächste Generation" des Templates in Bezug auf Netzladung und Netzladungsleistung ist. Einen Zusammenhang mit der HW-Generation oder HW-Version soll die Benennung nicht haben, das sollte in der Dokumentation geschärft worden sein und weiter geschärft werden.