-
Notifications
You must be signed in to change notification settings - Fork 64
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
feature: get absorption- and float-voltage from MPPTs #1140
base: development
Are you sure you want to change the base?
Conversation
Hallo @schlimmchen Momentan wird mit der "absorption voltage" und der "float voltage" noch nichts gemacht. |
Das war mir auch schon aufgefallen, aber ich dachte das sei bei mir weggegangen nach Neuverdrahtung. Gerade nochmal die Konsole kurz mitlaufen lassen und bei mir scheinen die HEX-Anforderungen brav und zuverlässig beantwortet zu werden. Ich hatte dazu das Absenden der HEX-Requests synchronisiert mit den asynchronen TEXT Nachrichten, also verschicken nachdem eine valide TEXT Nachricht dekodiert wurde. In deinem Setup scheint das nicht auszureichen.
Die Idee hatte ich auch schon, hatte mir aber nicht gefallen, weil dann das Alter der Daten entsprechend anwächst. Das hast du wohl auch schon gesehen, denn du schreibst
Die richtige Lösung wäre einen asynchronen Task laufen zu lassen, pro MPPT Controller, der nichts anderes tut als eine Nachricht zu schicken, die Antwort zu dekodieren (oder Timeout), und dann die nächste Nachricht schicken, für alle Datenpunkte, die man einsammeln will. Das alles in eine Loop mit 1s Sleep zwischen den Runden.
Hm? Das war doch schon vorher da...
Da gibt es ein anders Issue/Diskussion dazu, das hab ich schon mitbekommen. Das Thema ist spannend, ich kenne das Problem und es ist lästig und ich hab noch keine Idee, wie man das lösen könnte. Ob man dafür die Float und Absorption Voltage kennen muss, wage ich allerdings zu bezweifeln, denn der Controller teilt seinen Mode ja bereits mit? |
Der Bug ist mir erst aufgefallen als ich die Abfrage der "absorption voltage" und der "float voltage" einfach angehängt habe. und dann kam ab und zu die "float voltage" auch nach 10 sec noch nicht an. Aber kein problem mit der "absorption voltage".
Ja, das wäre die richtige Lösung. Ich habe auch schon daran gedacht, war mir aber nicht sicher ob meine C++ Fähigkeiten schon reichen und deshalb eine einfache Lösung gewählt. Ein Problem haben wir momentan nicht weil ja
Ich regle über die Spannung und nicht über die Leistung/Strom. Und dann verwende ich gleich die im MPPT hinterlegen Spannungen. Ich denke das hat Vorteile. Fasse ich alles noch in Text ... Noch etwas Geduld bitte. |
Hallo @schlimmchen , 21:59:00.759 > [VE.Direct MPPT 7/17] Sending Hex Command: :7F6ED006B, Free FIFO-Buffer: 118 Mir ist beim Testen noch was aufgefallen. Ich konnte plötzlich kein Update über die Weboberfläche machen aber über die serielle Schnittstelle ging es ohne Probleme. Er hat erst wieder funktioniert nachdem ich die Größe der Datei etwas reduzieren konnte. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know you made this for #1167, but this could and should be its own feature, where we add the float and absorbtion values to the live view, MQTT and HASS. If you are not motivated for that, I can pick it up at some point in the future.
all changes have been implemented. If the test on my system is positive tomorrow then I will update the code. |
Ok, works on my system since yesterday |
91cc2fc
to
8ff94e7
Compare
I will do the missing implementations next. Absorption voltage: 28.6V On the "Device Info" block or on the "Output(Battery) block? |
Well, I think the "Output (Battery)" card is better suited for two reasons: (1) It has currently less rows than the "Device Info" card, and (2) these voltages have a relation to the battery voltage itself. So, I would say that card is ideal. |
PR includes the following: