FHEM Forum

FHEM - Hausautomations-Systeme => 1Wire => Thema gestartet von: epsrw1 am 16 Mai 2014, 12:57:03

Titel: OWTHERM readings
Beitrag von: epsrw1 am 16 Mai 2014, 12:57:03
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
Titel: Antw:OWTHERM readings
Beitrag von: ntruchsess am 19 Mai 2014, 13:12:57
schick mir bitte einen Pull-request mit Deinen Änderungen gegen meinen fhem-mirror auf Github (https://github.com/ntruchsess/fhem-mirror)
Titel: Antw:OWTHERM readings
Beitrag von: epsrw1 am 22 Mai 2014, 18:46:51
gäbe es evtl auch eine andere lösung? ich habe ehrlich gesagt keine ahnung wie der git-kram funktioniert.....
danke und gruß, florian
Titel: Antw:OWTHERM readings
Beitrag von: ntruchsess am 22 Mai 2014, 21:19:54
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
Titel: Antw:OWTHERM readings
Beitrag von: epsrw1 am 26 Mai 2014, 19:23:35
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 :)
Titel: Antw:OWTHERM readings
Beitrag von: Prof. Dr. Peter Henning am 28 Mai 2014, 09:21:52
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
Titel: Antw:OWTHERM readings
Beitrag von: ntruchsess am 28 Mai 2014, 10:11:08
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
Titel: Antw:OWTHERM readings
Beitrag von: epsrw1 am 28 Mai 2014, 18:57:29
ich schließe mich sinnvollen namen an, schlage allerdings vor die gleichen wie im set zu verwenden: tempHigh tempLow
gruß florian
Titel: Antw:OWTHERM readings
Beitrag von: Prof. Dr. Peter Henning am 29 Mai 2014, 12:08:53
Einverstanden mit der Namensgebung

LG

pah
Titel: Antw:OWTHERM readings
Beitrag von: epsrw1 am 29 Mai 2014, 17:22:09
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