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!

Copy link
Member

Choose a reason for hiding this comment

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

Mich interessiert weniger der Name- das ist technishces Detail- sondern v.a. die Beschreibung. Mit nexgen kann ein Anwender nichts anfangen. Ebensowenig Grid Charge- warum sollte er das nicht auswählen?

Gibts denn keine Aussage von Kostal dazu???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nein, keine Aussagen von Kostal - leider!! Die Bezeichnung soll dem Anwender helfen, das gelingt hier in keinem Fall gut aktuell?

Ich gehe davon aus, wenn ein Anwender Netzladen möchte, spricht ihn die Bezeichnung "gridcharge" direkt an. Warum er das nicht wählen sollte, steht exkl. im Kleingedruckten der Doku. Könnte man das geschickt als Hilfe bei der Auswahl des UI Templates eigentlich auch anzeigen?

Das dies technisch dann nicht eindeutig ist, wäre mE zu vernachlässigen.

Es geht einzig darum, dem Anwender die richtige Entscheidung leichter zu machen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gibts denn keine Aussage von Kostal dazu???

Aktuell gibt es noch insbesondere bei SW 01.xx und HW0100/HW201 "Herausforderungen" - meine quick and dirty Zusammenfassung gerade mag noch Lücken haben. Mal geht eine G1 HW Version/mal nicht - noch keine klaren Zusammenhänge erkennbar. Wir geben uns weiter Mühe, während auf Rückmeldung gewartet wird (mehrere Nutzer!)

Copy link
Contributor Author

@iseeberg79 iseeberg79 Dec 16, 2024

Choose a reason for hiding this comment

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

Die letzen drei Posts mal Revue passieren lassen, könnte ich mir einen anderen Weg auch vorstellen. Um dem Anwender die Auswahl der Templates zu erleichtern könnten wir:

  1. die Bezeichnung des alten Template überarbeiten (Fallback/Safemode/failsafe/old/stable) als Ergänzung am sonst unveränderten Template; am neuen Template verschwindet die Ergänzung: führt den Anwender zur neuen Version,

  2. die Bezeichnung des alte Template unverändert; die neue einfach nur "New" - vielleicht mit einem hohen Sternchen, um damit Aufmerksamkeit auf die Doku zu lenken: "testing/beta" oder "experimentell" fallen für mich raus, da die Funktion stabil für neue Wechselrichter ist

Nur, um nochmal einen anderen Blickwinkel einzunehmen. Die technischen Bezeichnungen sind zweitrangig, sehe ich auch so.

@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