-
-
Notifications
You must be signed in to change notification settings - Fork 802
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
Add Buderus/Bosch/Junkers #19977
Add Buderus/Bosch/Junkers #19977
Conversation
Co-authored-by: andig <cpuidle@gmail.com>
Co-authored-by: andig <cpuidle@gmail.com>
@Tihmann du hast noch nciht alle Korrekturen angenommen/ umgesetzt, daher auch der Fehler. |
Took me a bit to figure out why the tests don't alter: tempalte MUST be in templates/definition/charger... |
@Tihmann ...und du musst das Template natürlich testen... |
Also die Datei ins entsprechende Verzeichnis verschieben ? Ich meine auch fürs Testen nach der von Dir beschriebenen Methode, oder ist es für den Test egal ? |
Verschieben ist erledigt und getestet werden muss es trotzdem. |
Vielen Dank für's Verschieben in GitHub. Meine Frage war nur ob die Testdatei auch lokal entsprechend verschoben werden muss, oder ob es lokal keine Rolle spielt, wo die Datei liegt |
Für den lokalen Test ist egal wo die Datei liegt. |
Der erste Entwurf klappt soweit okay. Es fehlen noch zwei Punkte: korrekte Umsetzung von getmode Im Moment würde ich mich auf die Zieltemperatur konzentrieren, in der evcc Oberfläche steht bei Zieltemperatur noch 100°C und eine Änderung in der Oberfläche hat keine Auswirkung. Wie kann man den Parameter Zieltemperatur zuordnen ? Einen Wert in emsesp gibt es dazu. |
limittemp, siehe andere Templates, z.B. kermi |
Wollte noch getmode: anpassen, doch leider bekomme ich die .jq Abfrage nicht hin. Fehler: cannot create charger type 'template': cannot create charger type 'sgready': SG1: invalid jq query '.value ^.{0,1}': unexpected token "^" getmode: |
@Tihmann, probier mal einfach |
…oder mal mit jqplay ausprobieren! |
ich habe den Parameter limittemp jetzt getestet. Der Wert wird jetzt rechts neben "Verbunden" als Heizungslimit eingeblendet. Der Wert Zieltemperatur rechts unten ist davon nicht beinträchtigt (steht bei Start auf 100) und Änderungen an Zieltemperatur unten rechts haben keine Auswirkungen. @andig WIe ist denn der Parameter Zieltemperatur zu setzen ? |
Gar nicht. Das ist das harte Limit aus dem Gerät. Du kannst nur auf weniger limitieren, nicht auf mehr. Ansonsten sieht das ja jetzt gut aus- rein damit? |
Zieltemp. unten rechts ist ein Auswahlmenü, in dem die Zieltemperatur in 1 Grad Schritten ausgewählt werden kann. Ist Zieltemp. unten rechts gleich limittemp ? Im Moment haben Änderungen von Zieltemp. gar keine Auswirkungen, weder weniger noch mehr... |
Wenn Du Zieltemp auf 30°C ziehst wird evcc dann abschalten. Bei 60°C schaltet die WP bei 43° ab. |
Okay, das klappt. Logik verstanden. Gerne rein damit, dann können die Tests ausgeweitet werden, es gab im Thread ja einige Testwillige. |
This reverts commit fccd7ba.
This reverts commit fccd7ba.
Fix #19660