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

default Solax Inverter Work Mode #16856

Merged
merged 16 commits into from
Nov 1, 2024
Merged

default Solax Inverter Work Mode #16856

merged 16 commits into from
Nov 1, 2024

Conversation

farcorben
Copy link
Contributor

fügt einen parameter hinzu der den default Workmode der Solax und Qcells Wechselrichter aus den favorisierten Modus nach "Fast Charge" oder "Charge plan" zurücksetzt.

Issue: #16830

@andig: Ist es ungefähr dies was du dir vorstellst?
Bekomme nur das Dropdown Menü nicht schön hin.. Würdet ihr da nochmal drauf schauen, bitte?

image
image

Test an meinem Solax X3 Hybrid G4 war erfolgreich
funktioniert

@andig andig added the devices Specific device support label Oct 25, 2024
util/templates/defaults.yaml Outdated Show resolved Hide resolved
@farcorben farcorben marked this pull request as ready for review October 26, 2024 18:03
@andig
Copy link
Member

andig commented Oct 31, 2024

Der PR ist super... ich scheue mich allerdings ein bisschen davor, künftig für jeden WR eine ganze Liste von Modi pflegen zu müssen. Ich würde daher den defaultmode lieber als einfachen INT ohne Auswahlliste umsetzen. Wäre das ok für Dich?

@farcorben
Copy link
Contributor Author

Hey @andig, du hast vollkommen recht. Solax hat allein die letzten drei Monate ganze drei neue Modis eingeführt. Der letzte ist noch nicht einmal dokumentiert in der aktuellen Modbus Dokumentation. Ich würde das nochmal auf einfachen INT umbauen und dir neu zum review geben wenn ich fertig bin. Danke für den Hinweis

@farcorben farcorben marked this pull request as draft October 31, 2024 12:48
templates/definition/meter/solax.yaml Outdated Show resolved Hide resolved
templates/definition/meter/solax.yaml Outdated Show resolved Hide resolved
@farcorben farcorben marked this pull request as ready for review October 31, 2024 18:50
@andig andig merged commit cd78552 into evcc-io:master Nov 1, 2024
6 checks passed
jonilala796 pushed a commit to jonilala796/evcc that referenced this pull request Jan 3, 2025
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.

2 participants