iobroker-nodemihome
iobroker-nodemihome copied to clipboard
Logausgabe: Verbesserungsvorschlag
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():
- More user-friendly log if a setting changes, so like
Setting Device Smartmi Fan 3: fan.on = true
, and notKeyvalue=true key=0
. - Call
device[i].setter[device[i].definition.common[key].name](i, keyvalue)
directly and not withinlog()
, as in my environment, this just logs an additional line with an empty object{}
with no value to the user.
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.
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.
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 :)
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.