Autor Thema: [patch] 98_dewpoint.pm  (Gelesen 4685 mal)

Offline till24

  • New Member
  • *
  • Beiträge: 28
Antw:[patch] 98_dewpoint.pm
« Antwort #30 am: 05 Dezember 2017, 19:45:30 »
bei mir kommt das:

2017.12.05 19:42:09 1: /usr/share/perl5/Math/Round.pm

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #31 am: 05 Dezember 2017, 21:06:52 »
bei mir kommt das:

2017.12.05 19:42:09 1: /usr/share/perl5/Math/Round.pm
Das heißt ja dann wohl:

Wenn man ein Modul benutzt, das Math::Round importiert, torpediert das andere Module inklusive fhem.pl.


Gesendet von meinem Nexus 5X mit Tapatalk


Offline mahowi

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1076
Antw:[patch] 98_dewpoint.pm
« Antwort #32 am: 05 Dezember 2017, 23:03:24 »
Bei mir ist es auch
/usr/share/perl5/Math/Round.pm

Math::Round wird von mehreren Modulen genutzt:
pi@raspberrypi:/opt/fhem/FHEM $ grep Math\:\:Round *
00_SIGNALduino.pm:#use Math::Round qw();
14_SD_WS09.pm:    #use Math::Round qw/nearest/;
31_MilightDevice.pm:#use Math::Round ();
31_MilightDevice.pm:  #return Math::Round::round($number);
31_MilightDevice.pm:  <p>Requires perl module Math::Round</p>
45_Plugwise.pm:#use Math::Round;
71_XiaomiSmartHome_Device.pm:use Math::Round qw/nearest/;
71_XiaomiSmartHome.pm:eval "use Math::Round";
98_MediaList.pm:use Math::Round qw/round/;

Meiner Meinung nach ist es etwas unglücklich, interne Funktionen genauso wie Perl-eigene zu benennen.
CUBe (MAX): HT, FK | CUBe (SlowRF): ESA2000WZ
JeeLink: LaCrosse | nanoCUL433: Smartwares SHS-51001-EU, EM1000GZ
ZME_UZB1: GreenWave PowerNode, Popp Thermostat | SIGNALDuino: HE877, X10 MS14A, Revolt NC-5462,  IT Steckdosen + PIR
tado° | Milight | HUE, Lightify | SmarterCoffee

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #33 am: 06 Dezember 2017, 09:36:44 »
Bei den meisten ist es kommentiert,

Über bleiben
pi@raspberrypi:/opt/fhem/FHEM $ grep Math\:\:Round *
71_XiaomiSmartHome_Device.pm:use Math::Round qw/nearest/;
71_XiaomiSmartHome.pm:eval "use Math::Round";
98_MediaList.pm:use Math::Round qw/round/;

Die 71_Xiaomi* sind nicht Teil des "offiziellen" Release. Bleibt eigentlich nur 98_MediaList.pm .

@till24: Benutzt Du eins von diesen Modulen ?

Offline till24

  • New Member
  • *
  • Beiträge: 28
Antw:[patch] 98_dewpoint.pm
« Antwort #34 am: 06 Dezember 2017, 11:06:07 »
Ja, ich benutze die Xiaomi Module..

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #35 am: 06 Dezember 2017, 12:53:18 »
Ich habe in den entsprechenden Foren einen Hinweis hinterlassen.

Gesendet von meinem Nexus 5X mit Tapatalk

Gefällt mir Gefällt mir x 2 Liste anzeigen

Offline mahowi

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1076
Antw:[patch] 98_dewpoint.pm
« Antwort #36 am: 06 Dezember 2017, 17:11:30 »
Ich sehe es nach wie vor als problematisch, das intern eine Funktion eines offiziellen Perl-Moduls überschrieben wird. Da nirgendwo daraufhin gewiesen wird, kann es hier auch zu unvorhersehbaren Problemen mit eigenen Routinen in z.B. 99_myUtils.pm kommen.

Vielleicht kann Rudi was dazu sagen, er ist Maintainer der 99_Utils.pm.

Edit: Ich habe mal einen neuen Thread dazu aufgemacht: [99_Utils.pm] round() in Konflikt mit Math::Round
« Letzte Änderung: 06 Dezember 2017, 17:29:44 von mahowi »
CUBe (MAX): HT, FK | CUBe (SlowRF): ESA2000WZ
JeeLink: LaCrosse | nanoCUL433: Smartwares SHS-51001-EU, EM1000GZ
ZME_UZB1: GreenWave PowerNode, Popp Thermostat | SIGNALDuino: HE877, X10 MS14A, Revolt NC-5462,  IT Steckdosen + PIR
tado° | Milight | HUE, Lightify | SmarterCoffee

Offline igami

  • Developer
  • Hero Member
  • ****
  • Beiträge: 2670
  • RTFM
    • commandref
Antw:[patch] 98_dewpoint.pm
« Antwort #37 am: 14 Dezember 2017, 16:42:38 »
Bei mir wird für dewpoint zwar ein Event erzeugt, jedoch werden die Werte in der Detailansicht nicht aktualisiert und DbLog bekommt davon auch nichts mit.
Das gleiche Problem gab es schon beim powerMap Modul. Die Lösung ist hier: Events aus X_Notify() heraus erzeugen
Pi3 mit fhem.cfg + DbLog/logProxy
Komm vorbei zum FHEM Treffen im Kreis Gütersloh! Das nächste Mal im August 2019.

MAINTAINER: archetype, Heating_Control, LuftdatenInfo, monitoring, msgDialog, Nmap, powerMap, RandomTimer, WeekdayTimer
ToDo: adb, FluxLED

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #38 am: 14 Dezember 2017, 18:36:38 »
Baue ich demnächst ein.

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #39 am: 14 Dezember 2017, 18:46:09 »
Hhm, das Modul lag ja lange Jahre ziemlich brach.
Die Funktion deviceEvents aus dem Zitat wird hier noch gar nicht verwendet (wollte ich demnächst mal einbauen). Von daher passt das Beispiel aus dem Link (glaube ich zumindest) nicht so ganz. Muss ich mir noch mal genau ansehen.

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #40 am: 15 Dezember 2017, 16:33:03 »
Bei mir wird für dewpoint zwar ein Event erzeugt, jedoch werden die Werte in der Detailansicht nicht aktualisiert und DbLog bekommt davon auch nichts mit.
Das gleiche Problem gab es schon beim powerMap Modul. Die Lösung ist hier: Events aus X_Notify() heraus erzeugen
Ich habe das Verfahren aus dem Link jetzt mal sinngemäß übernommen.
Events gibt es jetzt nur noch, wenn das Reading dewpoint durch die Filter event-on-(update|change)-reading durchgeht.

Updates im Webinterface sind bei mir seit einigen Wochen generell sehr problematisch und funktionieren nur sporadisch oder zeitlich um Minuten verzögert. Ich vermute, dass das Webinterface hier auch beteiligt ist.

DbLog benutze ich nicht, bitte teste die anhängende Version doch mal.

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #41 am: 27 Dezember 2017, 14:31:04 »
Habe die Version soeben eingecheckt. Bei mir geht auch Dblog.

Offline Mr. Flash

  • Jr. Member
  • **
  • Beiträge: 52
Antw:[patch] 98_dewpoint.pm
« Antwort #42 am: 30 Dezember 2017, 14:18:08 »
Hallo Holger,

vielen Dank für die neue Version von 98_dewpoint.pm

Leider scheint sich da ein kleiner Fehler eingeschlichen zu haben, bei mir steht im Logfile:
2017.12.30 11:04:47 3: dewpoint_notify: alarm OFF
2017.12.30 11:07:21 3: dewpoint_notify: alarm ON
Undefined subroutine &main::Logr called at ./FHEM/98_dewpoint.pm line 386.

In Zeile 386 von "98_dewpoint.pm" steht dann anstelle von "Log3" ein "Logr", diese Funktion wird natürlich nicht gefunden.
Könntest Du das bitte korrigieren und ein Update bereitstellen? Vielen Dank.

Grüße,
Nik
RPi 2B/3B/ZW; Jessie lite; FHEM 5.9; S.USV basic; BME280; TSL25911.
HM: CFG-LAN (0.965),CFG-USB-2,CC-RT-DN,Dis-(EP-)WM55,ES-PMSw1-Pl,LC-Dim1T-FM,LC-Sw1-FM,LC-Sw2-FM,MOD-RPI-PCB,OU-CFM-Pl,RC-Dis-H-x-EU,SCI-3-FM,Sec-MDIR-2,-RHS,-SD,-SC-2,-SCo,-Sir-WM,-TiS,-WDS-2,TC-IT-WM-W-EU,WDS10-TH-O,WDS30-OT2-SM

Offline hotbso

  • Developer
  • Full Member
  • ****
  • Beiträge: 109
Antw:[patch] 98_dewpoint.pm
« Antwort #43 am: 30 Dezember 2017, 17:30:14 »
Hallo Nik,

peinlich 8-( .

Habe das korrigiert und eingecheckt.

Gruss
Holger

Offline Mr. Flash

  • Jr. Member
  • **
  • Beiträge: 52
Antw:[patch] 98_dewpoint.pm
« Antwort #44 am: 31 Dezember 2017, 17:37:56 »
Danke!
RPi 2B/3B/ZW; Jessie lite; FHEM 5.9; S.USV basic; BME280; TSL25911.
HM: CFG-LAN (0.965),CFG-USB-2,CC-RT-DN,Dis-(EP-)WM55,ES-PMSw1-Pl,LC-Dim1T-FM,LC-Sw1-FM,LC-Sw2-FM,MOD-RPI-PCB,OU-CFM-Pl,RC-Dis-H-x-EU,SCI-3-FM,Sec-MDIR-2,-RHS,-SD,-SC-2,-SCo,-Sir-WM,-TiS,-WDS-2,TC-IT-WM-W-EU,WDS10-TH-O,WDS30-OT2-SM

 

decade-submarginal