Autor Thema: dewpoint: Nach Update deutlich CPU-Lastiger  (Gelesen 491 mal)

Offline heikoh81

  • Sr. Member
  • ****
  • Beiträge: 525
dewpoint: Nach Update deutlich CPU-Lastiger
« am: 11 März 2018, 18:36:20 »
Hallo zusammen,

ich schreibe auf Bitte von rudolf, da ich ein Performance-Problem nach einem Update von dewpoint festgestellt habe.
(siehe auch unsere Diskussion: https://forum.fhem.de/index.php/topic,83537.30.html).

Ich hatte in meinem fhem bislang folgende dewpoint-Definition:
define dewpointToAllDeviceReadings dewpoint dewpoint .* temperature humidity dewpoint
attr dewpointToAllDeviceReadings room Temperaturen

Mit Rev. 6757 gab es keine Performance-Probleme, d.h. fhem war weiterhin sehr flink.
Nach Update auf Rev. 15551 dagegen wurde mein gesamtes fhem bei ca. jedem 2-3 Klick komplett lahmgelegt. Die Perl-Auslastung ging auf 100% hoch.

Ich möchte an dieser Stelle nicht über die Sinnhaftigkeit meiner Regex-Definition diskutieren.
Dass diese alle Devices umfasst, hatte rudolf bereits festgestellt.
Worum es mir geht ist, dass mit Rev. 6757 die Performance trotz dieser Definition noch sehr gut war, mit Rev. aber nicht mehr.
D.h. im Code von dewpoint wurde irgendetwas geändert, was zu einer deutlich ineffizienteren Abarbeitung der dewpoint-Berechnung führt. Dies könnte auch bei anderen Benutzern mit dewpoint-Definitionen zu Problemen führen, weswegen ich dies hier poste.

Da ich selbst dewpoint nicht weiterverfolgt habe in meiner Haussteuerung, waren für mich die Performance-Probleme durch Löschen der Definition beseitigt. Gerne kann ich aber zukünftige Code-Änderungen in meiner recht umfangreichen Installation (3000+ entities) testen.

Viele Grüße,
Heiko

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 14270
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #1 am: 11 März 2018, 18:41:13 »
Man kann sich ja die Unterschiede zwischen den Revisionen im SVN anschauen.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier

Offline heikoh81

  • Sr. Member
  • ****
  • Beiträge: 525
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #2 am: 12 März 2018, 21:09:24 »
Ja, mit diesem Post soll ich ja den Entwickler von dewpoint erreichen.
Ich bin kein Programmierer, somit kann ich den Fehler nicht finden.

Wenn es das falsche Forum ist, bitte verschieben.
In den Development-Foren konnte ich keine neuen Beiträge erstellen...

Offline CoolTux

  • Developer
  • Hero Member
  • ****
  • Beiträge: 14270
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #3 am: 12 März 2018, 21:23:17 »
Hier liest Du nach im welchen Forum dewpoint besprochen wird
https://svn.fhem.de/trac/browser/trunk/fhem/MAINTAINER.txt

Danach verschiebst Du selbst Deinen Beitrag. Ganz unten ganz links THEMA VERSCHIEBEN
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.me/MOldenburg
Mein GitHub: https://github.com/LeonGaultier

Offline heikoh81

  • Sr. Member
  • ****
  • Beiträge: 525
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #4 am: 12 März 2018, 21:32:25 »
Thema gemäß maintainer.txt ins Forum "Automatisierung" verschoben.

Offline heikoh81

  • Sr. Member
  • ****
  • Beiträge: 525
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #5 am: 15 April 2018, 12:56:36 »
Der Maintainer von dewpoint hat sich bislang noch nicht hier geäußert.
Sollte dies laut maintainer.txt nicht nach spätestens 3 Wochen geschehen?

Ich als Anwender kann nicht den Code aus dem SVN vergleichen, oder gar im Code erkennen, was jetzt die höhere CPU-Last erzeugt.

The third column specifies, where/how the maintainer should be contacted. If there is no reaction from the mainainer within 3 weeks, then rudolfkoenig (forum.fhem.de/FHEM Forum) should be contacted, in order to assign a new maintainer.

Offline blofield

  • New Member
  • *
  • Beiträge: 32
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #6 am: 16 April 2018, 15:00:02 »
Dann lass' das Modul weg und implementiere via 99_myUtils, wie hier beschrieben:
https://wiki.fhem.de/wiki/Dewpoint#Variante_.C3.BCber_99_myUtils

klappt super, performant und seit langem stabil ;)

Gruß
blofield

Offline Beta-User

  • Hero Member
  • *****
  • Beiträge: 2902
Antw:dewpoint: Nach Update deutlich CPU-Lastiger
« Antwort #7 am: 16 April 2018, 15:19:12 »
Der Maintainer von dewpoint hat sich bislang noch nicht hier geäußert.
Sollte dies laut maintainer.txt nicht nach spätestens 3 Wochen geschehen?
An sich ja, aber wie ich neulich lernen durfte ist es so, dass das Verschieben der Beiträge nicht immer optimal ist.
In der Regel ist es das richtige Vorgehen, aber nicht zwangsläufig bei echten PR's, die den Maintainer erreichen sollen:

Es werden beim Verschieben nämlich keine Benachrichtigungen versandt. Wer also z.B. nur Benachrichtigungen aus einem bestimmten Forumsbereich erhält, bekommt auch keine Benachrichtigung. Ein Maintainer, der ggf. diese Vorgehensweise wählt, bekommt daher uU. einen entsprechenden PR gar nicht mit (u.A. Rudi selbst hält es so).

Bitte daher für diesen Fall einen Doppelpost anlegen (kannst ja hierhin verlinken und - ebenfalls ausnahmsweise - den anderen Thread gleich wieder zu machen (oder umgekeht: diesen hier zu machen)).

Gruß, Beta-User
Server: HP-T5740 mit Debian stretch (i386) | ConfigDB | VCCU mit einiger HM-Hardware | MySensors seriell (2.2.0, RS485+nRF24, div. konkrete Hardware, u.a. einige DS18B20) | Milight@ESP-GW | SIGNALduino | MapleCUN

 

decade-submarginal