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.

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

Merged
merged 14 commits into from
Mar 24, 2025
Merged

Add Buderus/Bosch/Junkers #19977

merged 14 commits into from
Mar 24, 2025

Conversation

Tihmann
Copy link
Contributor

@Tihmann Tihmann commented Mar 21, 2025

Fix #19660

@andig andig added the devices Specific device support label Mar 21, 2025
Tihmann and others added 4 commits March 21, 2025 17:15
Co-authored-by: andig <cpuidle@gmail.com>
Co-authored-by: andig <cpuidle@gmail.com>
@andig
Copy link
Member

andig commented Mar 22, 2025

@Tihmann du hast noch nciht alle Korrekturen angenommen/ umgesetzt, daher auch der Fehler.

@andig andig added the heating Heating label Mar 22, 2025
@andig
Copy link
Member

andig commented Mar 22, 2025

Took me a bit to figure out why the tests don't alter: tempalte MUST be in templates/definition/charger...

@andig
Copy link
Member

andig commented Mar 22, 2025

@Tihmann ...und du musst das Template natürlich testen...

@andig andig marked this pull request as draft March 22, 2025 15:04
@andig andig removed the devices Specific device support label Mar 22, 2025
@andig andig changed the title buderus.yaml zur Einbindung von Buderus/Bosch/Junkers Wärmepumpen Add Buderus/Bosch/Junkers Mar 22, 2025
@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 22, 2025

Took me a bit to figure out why the tests don't alter: tempalte MUST be in templates/definition/charger...

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 ?

@andig
Copy link
Member

andig commented Mar 22, 2025

Verschieben ist erledigt und getestet werden muss es trotzdem.

@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 22, 2025

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

@andig
Copy link
Member

andig commented Mar 22, 2025

Für den lokalen Test ist egal wo die Datei liegt.

@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 23, 2025

Der erste Entwurf klappt soweit okay. Es fehlen noch zwei Punkte:

korrekte Umsetzung von getmode
korrekte Umsetzung der Zieltemperatur

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.

@andig
Copy link
Member

andig commented Mar 23, 2025

limittemp, siehe andere Templates, z.B. kermi

@andig andig marked this pull request as ready for review March 23, 2025 21:28
@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 23, 2025

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:
source: go
script: |
res := 1
switch {
case SG1 == "1" && SG4 == "0": res = 3
case SG1 == "0" && SG4 == "0": res = 1
case SG1 == "0" && SG4 == "1": res = 2
case SG1 == "1" && SG4 == "1": res = 2
}
res
in:
- name: SG1
type: str
config:
source: http
uri: http://{{ .host }}/api/boiler/hpin1opt
jq: .value ^.{0,1}
- name: SG4
type: str
config:
source: http
uri: http://{{ .host }}/api/boiler/hpin4opt
jq: .value ^.{0,1}

@MarkusMit
Copy link

@Tihmann, probier mal einfach '.value[0:1]'

@andig
Copy link
Member

andig commented Mar 24, 2025

…oder mal mit jqplay ausprobieren!

@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 24, 2025

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 ?

Screenshot_20250324-094000

@andig
Copy link
Member

andig commented Mar 24, 2025

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?

@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 24, 2025

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...

@andig
Copy link
Member

andig commented Mar 24, 2025

Du kannst nur auf weniger limitieren, nicht auf mehr.

Wenn Du Zieltemp auf 30°C ziehst wird evcc dann abschalten. Bei 60°C schaltet die WP bei 43° ab.

@Tihmann
Copy link
Contributor Author

Tihmann commented Mar 24, 2025

Du kannst nur auf weniger limitieren, nicht auf 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.

@andig andig merged commit fccd7ba into evcc-io:master Mar 24, 2025
6 checks passed
@Tihmann Tihmann deleted the patch-2 branch March 24, 2025 13:33
@Tihmann Tihmann restored the patch-2 branch March 25, 2025 09:04
Tihmann added a commit to Tihmann/evcc that referenced this pull request Mar 25, 2025
Tihmann added a commit to Tihmann/evcc that referenced this pull request Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
heating Heating
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wärmepumpen - Unterstützung von Buderus/Bosch/Junkers Wärmepumpen mit ems-esp
3 participants