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

Kostal Plenticore: add max charge rate, clarify naming and documentation #17740

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

iseeberg79
Copy link
Contributor

  1. hinzufügen der Funktion zur Anpassung der Leistung bei Netzladung, genauer 0-100% relativ möglicher Ladeleistung
  2. erneute Umbenennung zur besseren Erkennung der Funktion und Vermeidung der Verwechselung mit der Hardware-Generation des WR
  3. Anpassung der Dokumentation, um die Unterscheidung für Anwender einfacher zu gestalten inkl. Hinweis auf die Verwendung aktueller Firmware der Geräte, die bei der Netzladung beteiltigt sind (Wechselrichter und Batterie)

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.

@iseeberg79 iseeberg79 marked this pull request as draft December 14, 2024 20:46
@iseeberg79
Copy link
Contributor Author

@andig @deadrabbit87 bitte um Prüfung/Stellungnahme

@iseeberg79 iseeberg79 marked this pull request as ready for review December 14, 2024 21:03
products:
- brand: Kostal
description:
generic: Plenticore Hybrid (Gen2)
generic: Plenticore Hybrid (nextgen)
Copy link
Member

Choose a reason for hiding this comment

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

What makes this next?

Copy link
Contributor Author

@iseeberg79 iseeberg79 Dec 14, 2024

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.

Copy link
Member

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?

Copy link
Contributor Author

@iseeberg79 iseeberg79 Dec 14, 2024

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?

Copy link
Contributor Author

@iseeberg79 iseeberg79 Dec 14, 2024

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.

Copy link
Contributor

@deadrabbit87 deadrabbit87 Dec 15, 2024

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!

@andig andig added the devices Specific device support label Dec 14, 2024
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