PID-Modul: Vorschläge (gelöst)

[John]

Hi Udo
delta kann positiv u. negativ werden.

Hallo,
auch mit der neuen Version Logeinträge nur bei negativem Delta und keine Befehle an Stellantriebe.

Hans

Code: [Auswählen]

if($delta >= AttrVal($name,'pidDeltaTreshold','0')) {

John

[betateilchen]

ja, das ist aber im Moment nicht das Problem.

[Hans Franz]

lösche mal das Attribut pidActorTreshold

Tata,

Code: [Auswählen]

2013.10.07 23:50:46 3: aDiff:0 pAT:0  tDiff:4.38690185546875e-05 pAI:0
2013.10.07 23:50:46 3: PID heizung.04: set stellantrieb.04 valve 98

Das sieht doch schon besser aus.
Aber wieso ist $aDIFF immer 0?

[betateilchen]

Weil der Stellantrieb noch nie gestellt wurde. Eigentlich sollte das JETZT nicht mehr 0 sein.

[John]

Nochmal,
wenn der Istwert goesser als der Sollwert ist, dann geht der Regler in Pause, da Delta negativ wird.
Das ist nun wirklich ein Problem oder ?


Wie wärs mit

Code: [Auswählen]

if(abs($delta) >= AttrVal($name,'pidDeltaTreshold','0')) {

Dann sind wohl auch die anderen Probleme mit der Stellausgabe gelöst.
John

[Hans Franz]

Weil der Stellantrieb noch nie gestellt wurde. Eigentlich sollte das JETZT nicht mehr 0 sein.

Leider doch:

Code: [Auswählen]

2013.10.07 23:53:48 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:53:48 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:53:48 3: get stellantrieb.04 valve : 97
2013.10.07 23:53:48 3: PID heizung.04: set heizung.04 desired 21
2013.10.07 23:53:49 3: aDiff:0 pAT:0  tDiff:60.8145229816437 pAI:0
2013.10.07 23:53:49 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:53:49 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:54:49 3: aDiff:0 pAT:0  tDiff:60.3309800624847 pAI:0
2013.10.07 23:54:49 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:54:49 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:54:49 3: get stellantrieb.04 valve : 97
2013.10.07 23:54:49 3: PID heizung.04: set heizung.04 desired 21
2013.10.07 23:54:50 3: aDiff:0 pAT:0  tDiff:60.8117101192474 pAI:0
2013.10.07 23:54:50 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:54:50 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:55:50 3: aDiff:0 pAT:0  tDiff:60.3267669677734 pAI:0
2013.10.07 23:55:50 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:55:50 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:55:50 3: get stellantrieb.04 valve : 97
2013.10.07 23:55:50 3: PID heizung.04: set heizung.04 desired 21
2013.10.07 23:55:51 3: aDiff:0 pAT:0  tDiff:60.8101971149445 pAI:0
2013.10.07 23:55:51 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:55:51 3: FHT8V set stellantrieb.04 valve 98
2013.10.07 23:56:51 3: aDiff:0 pAT:0  tDiff:60.3250930309296 pAI:0
2013.10.07 23:56:51 3: PID heizung.04: set stellantrieb.04 valve 98
2013.10.07 23:56:51 3: FHT8V set stellantrieb.04 valve 98

[betateilchen]

Wenn schon, dann bitte so:

Code: [Auswählen]

if(abs($delta) >= abs(AttrVal($name,'pidDeltaTreshold','0'))) {

Aber um dieses Thema geht es gerade überhaupt nicht!

[betateilchen]

Leider doch:

Mach nochmal bitte ein "list <pidName>"

[Hans Franz]

Code: [Auswählen]

Internals:
   DEF        CUL_WS_5 stellantrieb.04
   NAME       heizung.04
   NR         360
   STATE      18.8 (delta 1.2)
   TYPE       PID
   Readings:
     2013-10-08 00:15:51   actuation       0
     2013-10-08 00:15:51   delta           0
     2013-10-08 00:16:45   desired         20
     2013-10-08 00:15:51   integrator      0
     2013-10-08 00:21:47   measured        18.8
     2013-10-07 22:57:21   p_d             16.4705882352941
     2013-10-07 22:57:21   p_i             3.13725490196076
     2013-10-07 22:57:21   p_p             96.8627450980392
     2013-10-08 00:21:46   sensorState     alive
     2013-10-08 00:21:47   state           paused
   Helper:
     actor      stellantrieb.04
     command    valve
     dFactor    5.88235294117647
     errorState
     iFactor    3.05882352941176
     pFactor    25.4901960784314
     pausedSince 1381184151.72795
     reading    temperature
     regexp     ([\d\.]*)
     satMax     100
     satMin     0
     sensor     CUL_WS_5
Attributes:
   alias      Heizungen_4
   room       Heizungen

[John]

setActor wird niemals aufgerufen solange {lastActVal} nicht definiert ist.

Code: [Auswählen]


my $aDiff = 0;
$aDiff = abs($a - $hash->{helper}{lastActVal}) if($hash->{helper}{lastActVal});
.....
if($aDiff >= $pAT && $tDiff >= $pAI) {
# send command to actor
setActor($hash,$a);
}



damit bleibt aDiff dauerhaft auf 0.

lastActVal wird aber nur in setActor aktualisiert.

[Hans Franz]

Code: [Auswählen]

if(abs($delta) >= AttrVal($name,'pidDeltaTreshold','0')) {
führt zu

Code: [Auswählen]

Use of uninitialized value $i in numeric gt (>) at ./FHEM/98_PID.pm line 305.
Use of uninitialized value $i in numeric lt (<) at ./FHEM/98_PID.pm line 306.
Use of uninitialized value $i in addition (+) at ./FHEM/98_PID.pm line 309.
Use of uninitialized value $a in int at ./FHEM/98_PID.pm line 310.
Use of uninitialized value $i in int at ./FHEM/98_PID.pm line 310.
Use of uninitialized value $i in sprintf at ./FHEM/98_PID.pm line 312.

also wieder rausgeschmissen.

dennoch jetzt:

Code: [Auswählen]

Use of uninitialized value $a in int at ./FHEM/98_PID.pm line 310.
Use of uninitialized value $a in sprintf at ./FHEM/98_PID.pm line 312.
Use of uninitialized value $a in int at ./FHEM/98_PID.pm line 316.

Ich blick' den Code leider überhaupt noch nicht:(.

(heizung.04 ist mein Schlafzimmer. Ich geh' erst mal lüften :).)

[betateilchen]

Moin. Ich hab grad keine Lust mehr.
Lasst mich doch einfach mal in Ruhe einen Fehler suchen, anstatt hier durch sinnlose Code-Basteleien, die scheitern müssen den Thread noch unübersichtlicher zu machen, als er sowieso schon ist.

setActor wird niemals aufgerufen solange {lastActVal} nicht definiert ist.

Diese Aussage ist so schlichtweg falsch.

[Hans Franz]

Moin,
lastActVal ist nun intialisiert, wieso auch immer.
Wenn desired grösser ist als measured wird das Ventil auch geöffnet. Alles OK.
Aber wenn desired kleiner als measured (negatives Delta) bleiben actuation und Delta in den Readings auf den alten Werten.
Das Ventil wird nicht geschlossen.

[betateilchen]

Aber wenn desired kleiner als measured (negatives Delta) bleiben actuation und Delta in den Readings auf den alten Werten.
Das Ventil wird nicht geschlossen.

Ja, ich weiß...

[fhainz]

Bis auf den negatives Delta Fehler funktioniert's jetzt bei mir. :)