Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logausgabe: Verbesserungsvorschlag #60

Open
Acgua opened this issue Jul 12, 2022 · 3 comments
Open

Logausgabe: Verbesserungsvorschlag #60

Acgua opened this issue Jul 12, 2022 · 3 comments

Comments

@Acgua
Copy link

Acgua commented Jul 12, 2022

Hi,
vielen Dank für das Script.

Ich habe hier ein paar Verbesserungen vorgeschlagen, im Code eingearbeitet:
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.

@Pittini
Copy link
Owner

Pittini commented Jul 15, 2022

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
Copy link
Author

Acgua commented Aug 9, 2022

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

@Pittini
Copy link
Owner

Pittini commented Aug 10, 2022

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants