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.

Sign up for GitHub

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

Jump to bottom

Fronius Gen24: remove ChaGriSet and InOutWRte_RvrtTms #18386

Merged
merged 5 commits into from
Jan 26, 2025
Merged

Fronius Gen24: remove ChaGriSet and InOutWRte_RvrtTms #18386

merged 5 commits into from
Jan 26, 2025
+0 −28

Conversation

TomF79
Copy link
Contributor

@TomF79 TomF79 commented Jan 23, 2025 •
edited by premultiply
Loading

Remove ChaGriSet and InOutWRte_RvrtTms

Fix #18277
Fix #17583

@TomF79
Fronius Gen24: remove ChaGriSet and InOutWRte_
8aa372d 
Remove ChaGriSet
Remove InOutWRte
Add InWrte
Add MinRsvPct
@andig andig added the devices Specific device support label Jan 23, 2025
@andig andig mentioned this pull request Jan 23, 2025
Closed
@TomF79
Copy link
Contributor Author

TomF79 commented Jan 24, 2025

@andig

fixed format because test failed.

@Wisley7
Copy link

Wisley7 commented Jan 24, 2025

Bei der Steuerung meines Akkus über den Fronius Symo-Hybrid setze ich einfach den OutWrte auf folgende Werte:
Normal: 100
Hold: 0
Aus Netz laden : -100

StorCtl_Mod steht bei mir immer auf 3

Sonst wurde und wird nix verändert.
Ich denke das wird beim Gen24 auch funktionieren

@TomF79
Copy link
Contributor Author

TomF79 commented Jan 24, 2025

Bei der Steuerung meines Akkus über den Fronius Symo-Hybrid setze ich einfach den OutWrte auf folgende Werte: Normal: 100 Hold: 0 Aus Netz laden : -100

StorCtl_Mod steht bei mir immer auf 3

Sonst wurde und wird nix verändert. Ich denke das wird beim Gen24 auch funktionieren

siehe #18277 (reply in thread)

@Wisley7
Copy link

Wisley7 commented Jan 24, 2025

Bei der Steuerung meines Akkus über den Fronius Symo-Hybrid setze ich einfach den OutWrte auf folgende Werte: Normal: 100 Hold: 0 Aus Netz laden : -100
StorCtl_Mod steht bei mir immer auf 3
Sonst wurde und wird nix verändert. Ich denke das wird beim Gen24 auch funktionieren

siehe #18277 (reply in thread)

Da steht nix zu meinem Beitrag…

@TomF79
Copy link
Contributor Author

TomF79 commented Jan 24, 2025

Ich denke das wird beim Gen24 auch funktionieren

Da steht nix zu meinem Beitrag…

Der Link ist zur Info welche Register beim Gen24 geschrieben werden müssen.

@andig
Copy link
Member

andig commented Jan 25, 2025

Siehe auch #18364 (comment)

Ping @premultiply

@Wisley7
Copy link

Wisley7 commented Jan 25, 2025

Ich denke das wird beim Gen24 auch funktionieren

Da steht nix zu meinem Beitrag…

Der Link ist zur Info welche Register beim Gen24 geschrieben werden müssen.

Stimmt, allerdings könnte man das Modbus für fronius einheitlich gestalten, und nur OutWrte von 100 über 0 auf -100 setzen, denn das funktioniert bei allen Fronius Wechselrichtern gleich. Das was zur Zeit beim Gen24 gesetzt wird, ist der minimale SOC , welcher für den Notbetrieb vorgehalten werden soll.
bei OutWrte auf -100 wird einfach ein Nachladen erzwungen.

@TomF79
Copy link
Contributor Author

TomF79 commented Jan 25, 2025

Ich denke das wird beim Gen24 auch funktionieren

Da steht nix zu meinem Beitrag…

Der Link ist zur Info welche Register beim Gen24 geschrieben werden müssen.

Stimmt, allerdings könnte man das Modbus für fronius einheitlich gestalten, und nur OutWrte von 100 über 0 auf -100 setzen, denn das funktioniert bei allen Fronius Wechselrichtern gleich. Das was zur Zeit beim Gen24 gesetzt wird, ist der minimale SOC , welcher für den Notbetrieb vorgehalten werden soll. bei OutWrte auf -100 wird einfach ein Nachladen erzwungen.

Es geht aber nicht nur ums Laden sondern um die 3 Fälle "normal", "hold", "charge" und für diese gibt es Standardwerte siehe #18277 (reply in thread)

@TomF79
Copy link
Contributor Author

TomF79 commented Jan 25, 2025

Siehe auch #18364 (comment)

Ping @premultiply

Zur Vereinheitlichung beider Files sicher sinnvoll.

@Wisley7
Copy link

Wisley7 commented Jan 25, 2025

Ich denke das wird beim Gen24 auch funktionieren

Da steht nix zu meinem Beitrag…

Der Link ist zur Info welche Register beim Gen24 geschrieben werden müssen.

Stimmt, allerdings könnte man das Modbus für fronius einheitlich gestalten, und nur OutWrte von 100 über 0 auf -100 setzen, denn das funktioniert bei allen Fronius Wechselrichtern gleich. Das was zur Zeit beim Gen24 gesetzt wird, ist der minimale SOC , welcher für den Notbetrieb vorgehalten werden soll. bei OutWrte auf -100 wird einfach ein Nachladen erzwungen.

Es geht aber nicht nur ums Laden sondern um die 3 Fälle "normal", "hold", "charge" und für diese gibt es Standardwerte siehe #18277 (reply in thread)

Genau, und das habe ich im Beitrag davor geschrieben. Das es völlig ausreicht OutWrte zu verändern. Für normal(100) für hold(0) und für charge (-100)
Außer „ StorCtl_Mod „ einmalig auf 3 setzen sind keine weiteren Werte erforderlich.

@TomF79
Copy link
Contributor Author

TomF79 commented Jan 25, 2025

Ich denke das wird beim Gen24 auch funktionieren

Da steht nix zu meinem Beitrag…

Der Link ist zur Info welche Register beim Gen24 geschrieben werden müssen.

Stimmt, allerdings könnte man das Modbus für fronius einheitlich gestalten, und nur OutWrte von 100 über 0 auf -100 setzen, denn das funktioniert bei allen Fronius Wechselrichtern gleich. Das was zur Zeit beim Gen24 gesetzt wird, ist der minimale SOC , welcher für den Notbetrieb vorgehalten werden soll. bei OutWrte auf -100 wird einfach ein Nachladen erzwungen.

Es geht aber nicht nur ums Laden sondern um die 3 Fälle "normal", "hold", "charge" und für diese gibt es Standardwerte siehe #18277 (reply in thread)

Genau, und das habe ich im Beitrag davor geschrieben. Das es völlig ausreicht OutWrte zu verändern. Für normal(100) für hold(0) und für charge (-100) Außer „ StorCtl_Mod „ einmalig auf 3 setzen sind keine weiteren Werte erforderlich.

Für den Fall "normal" wäre es empfehlenswert alle Werte wieder auf Fronius Standard zu setzen, so wie es jetzt im File geändert wurde, um zu vermeiden dass sich andere Anwendungen, von evcc Benutzern, in die Quere kommen.

Für den Fall "charge" sollte ChaGridSet sicherheitshalber drin bleiben, falls Benutzer dies nicht standardmäßig aktiviert haben.

Für den Fall "hold" reicht StorCtl_Mod und OutWrte.

"ChaGridSet" musste raus, da genau das der Fall war, das Laderegeln in der Weboberfäche nicht mehr funktionierten.
"InOutWRte_RvrtTms" musste raus, da laut Fronius zeitlich mit den Befehlen etwas nicht stimmt und deshalb ein Spannungsfehler auftrat.

@Wisley7
Copy link

Wisley7 commented Jan 25, 2025

Und auch das würde ohne Probleme funktionieren.
StorCtl_Mod kann auf 3 bleiben. Dieser Register besagt bei 3 nur, dass OutWrte und InWrte berücksichtigt werden. Da OutWrte bei „normal“ wieder auf 100 gesetzt wird, funktioniert der WR wie er soll, ohne Einschränkungen!
und somit könnte man mit Veränderung eines Registers alle 3 Modi schalten.

@premultiply premultiply changed the title Fronius Gen24: remove ChaGriSet and InOutWRte_ Fronius Gen24: remove ChaGriSet and InOutWRte_RvrtTms Jan 26, 2025
@premultiply premultiply merged commit 075393f into evcc-io:master Jan 26, 2025
6 checks passed
@TomF79 TomF79 deleted the patch-2 branch January 27, 2025 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@premultiply premultiply

Labels
devices Specific device support
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fronius Modbus Batteriesteuerung fehlerhaft bei Netzladen der Hausbatterie
4 participants
@TomF79 @Wisley7 @andig @premultiply