lieber OWTHERM maintainer,
wäre es möglich die werte owg_th und owg_tl ebenfalls in die readings zu schreiben?
habe es manuell bei mir im modul eingefügt, das ist natürlich keine gute lösung da beim nächsten update wieder futsch....
merci beaucoup :)
florian
schick mir bitte einen Pull-request mit Deinen Änderungen gegen meinen fhem-mirror auf Github (https://github.com/ntruchsess/fhem-mirror)
gäbe es evtl auch eine andere lösung? ich habe ehrlich gesagt keine ahnung wie der git-kram funktioniert.....
danke und gruß, florian
Du kannst das geänderte Modul auch hier anhängen, dann schaue ich mir das an. Sollte halt die aktuelle Version aus dem SVN+deine Änderungen sein und nicht auf einem älteren Stand basieren
Gruß,
Norbert
hallo nochmal, soeben update durchgeführt und meine änderung erneut eingefügt. die zeilen 387 bis 392 sehen nun bei mir so aus:
#-- put into READINGS
readingsBeginUpdate($hash);
readingsBulkUpdate($hash,"owg_th",$vhigh);
readingsBulkUpdate($hash,"owg_tl",$vlow);
readingsBulkUpdate($hash,"temperature",$vval);
#-- STATE
21_OWTHERM.rar (http://scellius.de/admin/tmp/21_OWTHERM.rar)
merci :)
Diese Änderung passt nicht in die Systematik. "owg_tl" und "owg_th" sind keine klar erkennbaren Bezeichnungen - stattdessen sollte dort stehen AlarmHigh und AlarmLow, oder Ähnliches.
LG
pah
Da hast Du natürlich recht, owg_tl und th sind alles andere als sprechend. Als Namen haben wir ja schon tempHigh und tempLow. Jeweils sowohl als set-befehl, als auch als Attribute. Wäre sinnvoll diese Namen einfach für die beiden Readings zu übernehmen.
Gruß,
Norbert
ich schließe mich sinnvollen namen an, schlage allerdings vor die gleichen wie im set zu verwenden: tempHigh tempLow
gruß florian
Einverstanden mit der Namensgebung
LG
pah
aktueller stand bedeutet also:
zeilen 387 bis 392:
#-- put into READINGS
readingsBeginUpdate($hash);
readingsBulkUpdate($hash,"tempHigh",$vhigh);
readingsBulkUpdate($hash,"tempLow",$vlow);
readingsBulkUpdate($hash,"temperature",$vval);
#-- STATE