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

SolarEdge: fix battery control #10906

Merged
merged 11 commits into from
Nov 26, 2023
Merged

SolarEdge: fix battery control #10906

merged 11 commits into from
Nov 26, 2023

Conversation

andig
Copy link
Member

@andig andig commented Nov 24, 2023 •
edited
Loading

TODO

  • watchdog

Idea from #10899 (comment):

ich denke für den WDT an sowas wie ein neuer Plugin watchdog das einen timeout Wert bekommt. Geschrieben wird dann immer der letzte Wert alle timeout/2. Um das "nicht mehr" schreiben in den Griff zu bekommen könnten wir dem noch ein reset value verpassen. Wenn dieses geschrieben würde (oder wird, ist egal) dann wird die Goroutine für den WDT beendet. Wäre hier also der Fall, wenn minsoc geschrieben würde. Wie findest Du die Idee?

Refs Masterminds/sprig#388

@andig andig added the bug Something isn't working label Nov 24, 2023
@andig andig mentioned this pull request Nov 24, 2023
Merged
1 task
@premultiply
Copy link
Member

Aha, jetzt verstehe ich das besser. 👍🏼

Ich befürchte aber das reicht bei SE so noch nicht, da man nicht das Mode-Register ständig schreiben muss sondern das Timeout-Register selbst.
Jedenfalls verstehe ich die Doku derzeit so.

@andig
Copy link
Member Author

andig commented Nov 25, 2023

Was war das Timeout Register?

@premultiply
Copy link
Member

0xE00B

Siehe hier: #10767 (reply in thread)

@andig
Copy link
Member Author

andig commented Nov 25, 2023

Watchdog und wait sind drin zum testen:

evcc meter -b hold -w 2m

-> dann läuft der Watchdog zum testen 2m.

andig
andig commented Nov 25, 2023
View reviewed changes
cmd/flags.go Outdated Show resolved Hide resolved
andig and others added 2 commits November 25, 2023 21:47
@andig @premultiply
Update templates/definition/meter/solaredge-hybrid.yaml
de4f026 
Co-authored-by: premultiply <4681172+premultiply@users.noreply.github.com>
@andig
Use configured watchdog timeout
84ba6d1
@andig andig merged commit ea2fe72 into master Nov 26, 2023
6 checks passed
@andig andig deleted the fix/se-battery branch November 26, 2023 04:43
@premultiply
Copy link
Member

🏝️ Jetzt bin ich aber schon etwas gespannt.

@andig andig added this to the Battery Control milestone Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@premultiply premultiply premultiply left review comments

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
Battery Control
Development

Successfully merging this pull request may close these issues.
2 participants
@andig @premultiply