cycl0ne / TibberLox

Creative Commons Zero v1.0 Universal
2 stars 1 forks source link

Commited my version in a more 'pythonic' manner. I did the modification in parallel to yours and wanted to share my results. #1

Closed JacknJo closed 1 year ago

JacknJo commented 1 year ago

Hi and many thanks for your efforts in the Lox-Tibber script.

I used your script (in the loxforum version) and used it as a baseline for the script here in the PR. Feel free to have a look at it and take parts if you think it might be of use.

Features i've added:

cycl0ne commented 1 year ago

Hmm, also erst wollte ich es blind mergen. Dann hab ich mir mal den Source angeschaut und ich muss ne Nacht darüber schlafen. Ich persönlich finde das schon wieder "zuviel". Da die Idee dahinter war: ich ändere den Konfig Part per Editor, schmeiße es auf die NAS und dann geh ich in die GUI und lass ich es über den Cron Job laufen. Was man nun machen muss -> Ins Nas kopieren, per SSH draufgehen und dann ausführen und dann Konfig eingeben und dann wieder raus um dann wieder im Cron das einzusetzen. -> setzt mehr Expertise vom User vorraus. Dieser Konfig Part ist halt nur eine einmalige Sache.

JacknJo commented 1 year ago

Verstehe ich, aber ist eben auch Geschmackssache und hängt stark vom jeweiligen Use-Case ab. Für den von dir beschriebenen Use-Case würde ich vorschlagen, dass zusätzlich ein default-config File mit eingecheckt wird, das der User falls er möchte direkt im Editor ändern kann und direkt mit dem tibberlox.py-script auf die NAS oder ähnliches schieben kann -> keine Config-prompt mehr.