iobroker-nodemihome icon indicating copy to clipboard operation
iobroker-nodemihome copied to clipboard

Logausgabe: Verbesserungsvorschlag

Open Acgua opened this issue 2 years ago • 3 comments

Hi, vielen Dank für das Script.

Ich habe hier ein paar Verbesserungen vorgeschlagen, im Code eingearbeitet: https://github.com/Pittini/iobroker-nodemihome/compare/main...Acgua:iobroker-nodemihome:patch-1


RefreshGenericDpsTicker():

  • If mihome server is temporarily not reachable, an error was thrown every 10 secs. Therefore, added error handling and constant logLevelNetworkTimeout to set log level to desired level (info, debug, warn, error).

SetDevice():

  1. More user-friendly log if a setting changes, so like Setting Device Smartmi Fan 3: fan.on = true, and not Keyvalue=true key=0.
  2. Call device[i].setter[device[i].definition.common[key].name](i, keyvalue) directly and not within log(), as in my environment, this just logs an additional line with an empty object{} with no value to the user.

Acgua avatar Jul 12 '22 17:07 Acgua

Sieht ok für mich aus, denke Du hast das auch getestet?! Bitte noch die neue Version in die Readme aufnehmen und dann nen Pull Request machen, ich merge dann. Danke für Deine Engagement, ich hab leider aktuell für gar nix Zeit.

Pittini avatar Jul 15 '22 14:07 Pittini

Ja, auch über Tage produktiv getestet, läuft fehlerfrei. Leider hab ich aktuell auch keine Zeit, habe gerade andere Projekte am laufen. Mach ich, wie ich dazu komme, muss mich erst auch wieder in Github einarbeiten.

Eigentlich sollte das Script in einen Adapter umgewandelt werden. Ich habe mir noch ein Script geschrieben (nicht veröffentlicht), das die Geräte-Definitionen automatisch beibehält/aktualisiert.

image

Aber schöner wäre das direkt in einem Adapter. Mal sehen, ich lerne mich gerade ein in die Adapter-Entwicklung und komme auf dich zurück wenn ich soweit bin :)

Acgua avatar Aug 09 '22 20:08 Acgua

UI, für das Unterstützungsskript werden Dir etliche, mich eingeschlossen die Füße küssen, bitte unbedingt veröffentlichen. Ich rechne von Seiten des node-mihome Entwicklers nicht mehr mit ner Reaktion, von daher ist das super hilfreich. Adapter, ja wär toll, ich kanns leider nicht, bin froh dass ich das Ding überhaupt so weit ans laufen gekriegt hab, komm da an meine skill Grenzen was die node-mihome angeht. Solltest Du das angehen wollen, helf Dir gern soweit möglich.

Pittini avatar Aug 10 '22 11:08 Pittini