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

Offset available power by dc-generated battery charge #2735

Merged
merged 5 commits into from
Mar 17, 2022
Merged

Offset available power by dc-generated battery charge #2735

merged 5 commits into from
Mar 17, 2022

Conversation

andig
Copy link
Member

@andig andig commented Feb 28, 2022

Fix #2734

@andig andig added the bug Something isn't working label Feb 28, 2022
premultiply
premultiply approved these changes Feb 28, 2022
View reviewed changes
Copy link
Member

@premultiply premultiply left a comment

Choose a reason for hiding this comment

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

Uuuuh, der ist gut!
Wer denkt denn an sowas?

@andig
Copy link
Member Author

andig commented Feb 28, 2022 •
edited
Loading

Wer denkt denn an sowas?

Not me...

@andig
Copy link
Member Author

andig commented Mar 3, 2022

@premultiply nach Test müssten wir eher die Batterie komplett ignorieren um den Netzbezug weg zu bekommen (wird noch getestet):

if grid > 0 && battery < 0 {
	battery = 0
}

Das knallt aber sicher bei kleinen Grid-Werten im Rahmen der Trägheit der Batteriesteuerung. Hast Du eine Idee?

@premultiply
Copy link
Member

Ich glaub das da muss ich erst simulieren. Ich bekomme gerade Knoten ins Hirn...

@andig andig mentioned this pull request Mar 9, 2022
Closed
@andig
Copy link
Member Author

andig commented Mar 13, 2022

Ping @premultiply finden wir hier eine Lösung ohne weitere Konfigurationsparameter? Der PR funktioniert, würde aber m.E. bei langsamer Netzregelung des Batterie-WRs hässliche Nebeneffekte haben. Schwellwert? 50W?

1 similar comment
@andig

This comment was marked as duplicate.

Sign in to view
@premultiply
Copy link
Member

Wollen wir das mal nach Release testen wie sich das in der Realität verhält?

@andig
Copy link
Member Author

andig commented Mar 15, 2022

Gerne. Ich fürchte nur so wies jetzt ist würde ichs nicht testen wollen. Dann mit eingebautem Limit. Das kann aktuell nur schief gehen. Ich mach nen Vorschlag…

@andig
Copy link
Member Author

andig commented Mar 17, 2022

@ttjaden @premultiply ich hab 0a92bd8 hinzugefügt damit dieser Case nicht immer anschlägt sondern dem Batterie-WR etwas Luft zum regeln gibt.

Sollen wir das mal einchecken und ausprobieren?

@andig andig merged commit 38defb6 into master Mar 17, 2022
@andig andig deleted the fix/hybrid branch March 17, 2022 15:38
@andig andig mentioned this pull request Mar 26, 2022
Closed
@andig andig mentioned this pull request Apr 28, 2022
Closed
dontbyte pushed a commit to dontbyte/evcc that referenced this pull request Aug 2, 2022
@andig @dontbyte
Offset available power by dc-generated battery charge (evcc-io#2735)
b49cd32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@premultiply premultiply premultiply approved these changes

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

Successfully merging this pull request may close these issues.

Hybrid inverters with DC>AC power can result in unwanted grid supply
3 participants
@andig @premultiply @ttjaden