Logausgabe: Verbesserungsvorschlag

Pittini / iobroker-nodemihome

Steuerung von bisher nicht unterstützten Xiaomi Geräten in Iobroker via node-mihome

MIT License

27 stars 15 forks source link

Logausgabe: Verbesserungsvorschlag #60

Open Acgua opened 2 years ago

Acgua commented 2 years ago

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 not Keyvalue=true key=0.
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.

Pittini commented 2 years ago

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.

Acgua commented 2 years ago

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 :)

Pittini commented 2 years ago

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 / iobroker-nodemihome

Logausgabe: Verbesserungsvorschlag #60

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.

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.