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

Sungrow-Hybrid: fix charge #12215

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Sungrow-Hybrid: fix charge #12215

wants to merge 1 commit into from

Conversation

premultiply
Copy link
Member

No description provided.

@premultiply premultiply changed the title fix charge Sungrow-Hybrid: Fix charge Feb 12, 2024
@premultiply premultiply added the devices Specific device support label Feb 12, 2024
@naltatis
Copy link
Member

naltatis commented Feb 12, 2024

Scheint ein Fehler im Template zu sein:

[main  ] FATAL 2024/02/12 15:49:59 cannot create meter 'sungrow_battery': cannot create meter type 'template': cannot create meter type 'custom': battery mode: 1 error(s) decoding:

* '' has invalid keys: set

@premultiply
Copy link
Member Author

@andig Da braucht das Modbus-Plugin erst noch eine passende Set-Erweiterung, oder?

@andig
Copy link
Member

andig commented Feb 12, 2024

Was solls denn tun?

@premultiply
Copy link
Member Author

Wert aus Register A lesen (, skalieren) und dann in Register B schreiben.

So wie mit const-Plugin als Festwert auch.
Nur halt dynamisch via Modbus gelesen.

@andig
Copy link
Member

andig commented Feb 12, 2024

Mhhm. Keine Ahnung ob das mit dem Go Plugin gehen sollte (@GrimmiMeloni ?). Sonst bräuchte es wohl noch etwas Neues (copy). Langsam wirds unübersichtlich :O

@premultiply
Copy link
Member Author

Wie funktioniert das denn bei const?

@andig
Copy link
Member

andig commented Feb 12, 2024

Genauso. Aber in https://github.com/evcc-io/evcc/pull/12215/files#diff-0b0f0388fa0ea5962b68de7565e1131be1912c96e6d198ad57f6b53d77775b17R175 ist es ein modbus Plugin- das hat kein set.

Anders gesagt: const ist eigentlich eine Mischung aus copy und const. Die müssten wir jetzt- wenns orthogonal bleiben soll- aufsplitten und auch gleich alle Templates mit anpassen :O. Dann gäbe es ein eigenständiges copy plugin das hier zu verwenden wäre. Spannend wird dann nochmal, den passenden Typ zu finden bzw. zu matchen, ob input und output zueinander passen. Aber das haben wir ja schonmal irgendwie beim go Plugin gelöst. Müsste dann hier die gleiche Lösung werden.

Long story short: es braucht einen weiteren PR, der const umbaut und aufteilt.

@GrimmiMeloni
Copy link
Sponsor Collaborator

Sorry, in der Ecke habe ich mich noch gar nicht umgeschaut.
Ganz neutral gefragt: was spricht denn gegen den Vorschlag von Premultiply? Der Beschreibung nach doch recht generisch einsetzbar?

@andig
Copy link
Member

andig commented Feb 14, 2024

Welches Problem wird denn hier überhaupt gelöst? Es ist kein Ticket verlinkt.

@andig
Copy link
Member

andig commented Feb 14, 2024

Und was hat das mit #12203 zu tun?

@premultiply
Copy link
Member Author

Die Zwangsladung funktioniert derzeit nicht.
Dies soll dieser PR beheben.

@andig andig changed the title Sungrow-Hybrid: Fix charge Sungrow-Hybrid: fix charge Feb 22, 2024
@github-actions github-actions bot added the stale Outdated and ready to close label Mar 14, 2024
@github-actions github-actions bot closed this Mar 19, 2024
@premultiply premultiply reopened this Mar 19, 2024
@premultiply premultiply added infrastructure Basic functionality backlog Things to do later and removed stale Outdated and ready to close labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Things to do later devices Specific device support infrastructure Basic functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants