Luxtronik Warmwasser Zirkulation ZIP

[ch.eick]

Leider werden _alle_ Zeiten überschrieben.

Das war, wenn ich mich recht erinnere auch der Grund es nicht mit einzubauen.

Hast Du auch mal den Code Part, der mit rein soll?

[frober]

Ok, hier die Änderungen:

ab Zeile 312

# Warmwasser Zirkulation
   elsif( int(@_)==4 && ( $cmd eq 'hotWaterCircPumpRun' ) ) {
        Log3 $name, 3, "set $name $cmd $val";
        $hash->{LOCAL} = 1;
        if( $val eq "on" ) {
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation506", $val);
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation507", $val);
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation508", $val);
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation697", $val);
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation698", $val);
        }
        elsif( $val eq "off" ) {
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation507", $val);
         $resultStr = LUXTRONIK2_SetParameter ($hash, "zirkulation508", $val);         
        }
      else{
         $resultStr = "$name Error: Wrong parameter given - use on,off";
      }
        $hash->{LOCAL} = 0;
        return $resultStr;
   }

ab Zeile 1525

     #Zirkulation
  elsif ($parameterName eq "zirkulation506") {
      $setParameter = 506; #Umschalten auf Woche(Mo-So)
      $setValue = '0';
   }
   elsif ($parameterName eq "zirkulation507") {
      $setParameter = 507; #Einschaltzeit
      $setValue = $realValue eq "on" ? '60' : '0';
   }
   elsif ($parameterName eq "zirkulation508") {
      $setParameter = 508; #Ausschaltzeit
      $setValue = '0';
   }
   elsif ($parameterName eq "zirkulation697") {
      $setParameter = 697; #Laufzeit Ein
      $setValue = '1';
   }
   elsif ($parameterName eq "zirkulation698") {
      $setParameter = 698; #Laufzeit Aus
      $setValue = '0';
   }

Den Code habe ich gerade angepasst und kurz getestet.

Bezgl. des Attributs bräuchte ich Zeit, so etwas habe ich noch nicht gemacht. Diese Anpassung war wie bei westgate c&p.

P.S. ich habe nur die wesentlichen Zeilen bei mir eingebaut (zirkulatio507 + 508), der Rest verändert sich ja normalerweise nicht mehr.

Grüsse
Bernd

[frober]

Was mir noch aufgefallen ist, die Parameternamen erscheinen als Reading.
Nicht tragisch, aber unschön da nicht aussagekräftig und eigentlich nicht nötig.

[ch.eick]

Was mir noch aufgefallen ist, die Parameternamen erscheinen als Reading.
Nicht tragisch, aber unschön da nicht aussagekräftig und eigentlich nicht nötig.

Du kannst sie ja schön formatiert als stateFormat anzeigen, wenn es aber einmal läuft schaut man eh nicht mehr hin :-)

[frober]

Du kannst sie ja schön formatiert als stateFormat anzeigen, wenn es aber einmal läuft schaut man eh nicht mehr hin :-)

Ich brauche diese Readings nicht
War nur eine Anmerkung, falls es doch Mal eingecheckt wird.

[ch.eick]

Ich brauche diese Readings nicht
War nur eine Anmerkung, falls es doch Mal eingecheckt wird.

Aber wenn sie verwendet werden, dann sind sie halt auch sichtbar. Beim initialen device Anlegen werden sie nicht abgefragt und auch nicht angezeigt.
Also ist alles so wie es sein soll.

[westgate]

Hallo frober und ch.eick,

ich glaube ihr habt das Problem wirklich gelöst. Ich hatte halt nie etwas anderes als Woche(Mo-SO) genutzt. Deshalb bin ich gar nicht auf diese Idee gekommen.

@Tupol    Jetzt könnte man die Funktion wirklich einbauen!

Der Paramter "506 Einst_SuZIP_akt" wählt in der Tat aus welche Zeittabelle genutzt wird:
Woche(Mo-SO)     = 0
5+2(Mo-Fr,Sa-So) = 1
Tage(Mo,Di...)       = 2

Mann sollte als bei Einschalten 506 Einst_SuZIP_akt : 0 setzen.
Dann könnte man bei Ausschalten aber z.B. auf 506 Einst_SuZIP_akt : 2 wechseln so wären wieder die Tagesgenauen Einstellungen aktiv.

Ich werde mal die erste Seite so aktualisieren.

Gruß Westgate

[westgate]

Beitrag ist aktualisiert.
Bitte lest mal quer ob alles so stimmt.

[frober]

Dann könnte man bei Ausschalten aber z.B. auf 506 Einst_SuZIP_akt : 2 wechseln so wären wieder die Tagesgenauen Einstellungen aktiv.

Das war auch mein Gedanke/ Hoffnung. Die Daten werde jedoch komplett (zumindest war das bei mir so) überschrieben.
Ich habe die Zeitvariablen nicht gezählt, vermute aber dass die Variablen mehrfach für die 3 Zeittabellen verwendet werden.

[ch.eick]

Das war auch mein Gedanke/ Hoffnung. Die Daten werde jedoch komplett (zumindest war das bei mir so) überschrieben.
Ich habe die Zeitvariablen nicht gezählt, vermute aber dass die Variablen mehrfach für die 3 Zeittabellen verwendet werden.

Wie wäre den der default? Bei mir hatte ich da nie etwas eingetragen.

[frober]

Beitrag ist aktualisiert.
Bitte lest mal quer ob alles so stimmt.

Vom Code her passt es, danke für die Aktualisierung.
Bzgl. der Zeittabellen müsste ich nochmals testen...

Eine weitere Anmerkung: Ein FS20-Dummy ist nicht mehr nötig. Ein normaler Dummy mit useSetExtensions funktioniert auch.

[frober]

Wie wäre den der default? Bei mir hatte ich da nie etwas eingetragen.

Kann ich nicht mehr sagen, ich vermute das per Default Woche von 00:00 - 23:59 gesetzt ist. Als Dauerlauf, Taktung war, glaube ich 5:15.

Zum Testen unter Tage einfach eine Zeit eintragen und Pumpe über Fehm starten. Wenn die eingetragen Zeiten noch vorhanden sind währe es toll.
Ich teste Morgen auch nochmal...

[frober]

Ich habe es noch einmal getestet, die Zeiten bleiben doch erhalten.
Da muss beim ersten Mal etwas schief gegangen sein.

Die Variablen werden auch nicht doppelt benutzt, habe sie gezählt...

Grüße Bernd