forked from snaptec/openWB
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Merger #20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If we have enough power for 3 phases but no car is charging phases are cyclically toggled all the time cause "ueberschuss" is enough but used current oldll is on minimum
Anpassung der Berechnung für Leistung und Energie da hier die Batterie vom WR mit eingerechnet wird entsprechend herausgerechnet werden muss
…ir überhaupt eine Berechnung anstellen müssen die möglicherewise am Schluss eine PV Generatorleistung anzeigt obwohl keine da ist. Wenn DC Seitig nix kommt muss die Generator Leistung in Richtung Netz null sein. Batterie Charge rausgenommen da dies auf jeden fall nicht stimmt (Mal schauen wie sich die PV Leistung definieren lässt wenn geladen wird)
…y nicht hybrid.
…angepasst, folgen der gleichen Logik wie die kw
# Conflicts: # packages/modules/sma_modbus_tcp/inverter_modbus_tcp.py
Neu werden beim ReRead und dem Start alle gelesenen mqtt Parameter protokolliert und validiert. Dazu wird die Datei samrtcon.log erstellt, welches nur die Config parameter enthält. Der Startup wurde leicht geändert, um dem null (alle config Parameter weg) Problem auf die Spur zu kommen. Aus jetziger Sicht läuft zu viel parallel auf dem mqtt Broker so dass nicht innent 5 Sekunden alle smarthome parameter gelesen werden können.
Co-authored-by: LKuemmel <[email protected]>
Fix to handle inverter response where yields.xml has <yields/>, i. e. no yield values available. This happens with Kostal Piko 3.6 MP. This fix addresses issue snaptec#2268
Eigenes Logging mit standard python logging ersetzt
Update kostal_steca.py
add solaredge
add package for byd
snaptec#2245 Auswahl für einen SMA Hybrid Wechselrichter hinzugefügt
remove not used version files
log überarbeitet
Prevent shell command injection
…-no-charging Avoid toggling u1p3p on no charging (but enough "ueberschuss")
global log, log_config entfernt
logging/validierung von mqtt parameter
Fix duplicated else causing misleading exception
fix float values
minor bug fixes; python-style equals NoneType
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.