Neues Modul 82_UmHorizon.pm - Unitymedia Horizon Box

Begonnen von LarsMie, 24 April 2017, 00:18:15

Vorheriges Thema - Nächstes Thema

MaMi7880

Hallo,

ich schließe mich dem Vorredner an, hätte auch Interesse an dem Modul und bin für ausführliche Tests bereit. Getestet habe ich das aktuelle Modul noch nicht, bin soeben erst über den Beitrag gestolpert.

Gruß
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

MaMi7880

#16
Hallo,

ich habe mir erlaubt (ohne vorherige Erlaubnis des Modulautors) etwas in dem Modul rum zuschrauben.

Grund war eigentlich die Integration in mein Stimmen-gesteuertes System. Wie es so ist kam eins zu dem anderen, dabei kam dann folgendes raus:


# 20171026 version to 0.0.2
# + Input youth-protection-pin (send OK-OK-PIN-OK),
#   set attr UM_pin <dddd> to activate
# + sort Keys into groups like Remotecontroll (NUM-Block/DPAD-Block/etc.)
# + integrade automatic Socket-(dis)connect (default is auto),
#   turn off with attr <name> UM_auto false


Vielleicht ist es ja im Sinne des Autors ansonsten Entschuldige ich mich hiermit...
Das wäre mein erstes mal, dass ich sowas mache also bitte um Verständnis für den Programmier-Stil, normalerweise programmiere ich andere Sachen  :P

Beste Grüße
Manuel

Edit: Änderung geht erst mal an LarsMie
Edit: Patch angehangen
Edit: komplette Datei angehangen
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

CoolTux

Erstelle bitte ein Patch File und informiere dem Modulauthor per Mail.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

CoolTux

Gerade gesehen das der Author das letzte Mal im Mai online war. Ist also schon ne Weile her.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

MaMi7880

Zitat von: CoolTux am 26 Oktober 2017, 18:22:15
Erstelle bitte ein Patch File und informiere dem Modulauthor per Mail.

Danke für die Antwort...
Wie gesagt, ist das erste mal... Ich werde natürlich den Regeln folgen und hole dies nach ... Habe einfach angefangen und drauf los getippt ohne weiter auf die SVN-Regeln zu achten... mia culpa ...
Ich nehme die Datei raus und wende mich erst mal an den Modulautor...
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

CoolTux

Datei kannst drin lassen, oder besser noch. Erstelle nur einen patch den du dann hier dran hängst.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

MaMi7880

Den Patch kann ich doch nur erstellen wenn das Modul um SVN vorhanden ist, oder? Ich finde es nur nicht.
Oder reicht ein einfaches delta?
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

CoolTux

Du nimmst das Ausgangsmodul ist ja glaube hier im ersten Post angehangen und Dein jetziges Modul und erstellst aus beiden ein Patch. Einfach mal googeln da findest bestimmt was.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

MaMi7880

#23
Super, danke für die Info...

Erledigt, Patch oben abgehangen... Hoffe es ist richtig so...
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

Brause

Hallo MaMi

könntest Du doch bitte nochmal die komplette pm-Datei anhängen oder mir zukommen lassen.
Wenn ich versuche das Patch einzuspielen ändert sich bei mir nur der Versionskommentar.


patching file 82_UmHorizon.pm
Hunk #2 FAILED at 54.
Hunk #3 FAILED at 71.
Hunk #4 FAILED at 140.
Hunk #5 FAILED at 157.
Hunk #6 FAILED at 184.
Hunk #7 FAILED at 197.
Hunk #8 FAILED at 278.
Hunk #9 FAILED at 368.
Hunk #10 FAILED at 377.
Hunk #11 FAILED at 409.
Hunk #12 FAILED at 423.
Hunk #13 FAILED at 449.
Hunk #14 FAILED at 467.
13 out of 14 hunks FAILED -- saving rejects to file 82_UmHorizon.pm.rej


Danke und Gruss aus der Schweiz
Brause

MaMi7880

Zitat von: Brause am 28 Oktober 2017, 17:29:59
Hallo MaMi

könntest Du doch bitte nochmal die komplette pm-Datei anhängen oder mir zukommen lassen.
Wenn ich versuche das Patch einzuspielen ändert sich bei mir nur der Versionskommentar.


patching file 82_UmHorizon.pm
Hunk #2 FAILED at 54.
Hunk #3 FAILED at 71.
Hunk #4 FAILED at 140.
Hunk #5 FAILED at 157.
Hunk #6 FAILED at 184.
Hunk #7 FAILED at 197.
Hunk #8 FAILED at 278.
Hunk #9 FAILED at 368.
Hunk #10 FAILED at 377.
Hunk #11 FAILED at 409.
Hunk #12 FAILED at 423.
Hunk #13 FAILED at 449.
Hunk #14 FAILED at 467.
13 out of 14 hunks FAILED -- saving rejects to file 82_UmHorizon.pm.rej


Danke und Gruss aus der Schweiz
Brause

Hallo Brause,

Ich habe die komplette Moduldatei an meinen oberen Beitrag noch zusätzlich angehangen. Wie schon gesagt, die Änderungen liefen ohne Zustimmung des Modulautors. Leider hat sich LarsMie noch nicht gemeldet. Würde mich auch gerne als Developer anbieten wenn sonst kein Interesse mehr besteht...

Das mit der Patch-Datei tut mir Leid. War das erste mal, ich lerne noch.

Beste Grüsse
MANUEL
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

TottiToad

Hi,

würde mich sehr freuen wenn das Modul weiter entwickelt wird !

CoolTux wird bestimmt was dazu sagen können ob @MaMi7880 das Modul weiterentwickeln kann, falls sich LarsMie nicht mehr meldet.


Ich wäre jedenfalls sehr dankbar die HorizonBox vernünftig einbinden zu können.
Teste gerne ;)

Grüße & Danke
Totti

CoolTux

Wen der ursprüngliche Modulauthor angeschrieben wurde, gib ihm 2 Wochen Zeit zum antworten. Passiert nichts kannst du einen Fork machen und am besten in einem neuen Thread das neue Modul/Fork anbieten.
Achte im neuen Modul darauf einen Verweis im Rahmen der GPL2 auf das Ursprungsmodul und den Author ein zu tragen.



Grüße
Leon
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

MaMi7880

ja, ist klar, ich will ja keinem so schnell etwas weg nehmen.
Ich warte.. ;-)

Beste Grüße
Manuel
RaspberryPi 3 mit nanoCUL & JeeLink | FHEM 6.0 mit IT, HM  - Sensoren/Aktoren & Lacrosse Temp./Humi. | Shelly | Amazon Echo | Logitech Harmony Hub | Philips HUE | andFHEM

LarsMie

#29
Hallo zusammen, ich habe das ganze erst jetzt mitbekommen, da ich längere Zeit nicht in der Heimat war und mich auf meinen neuen Job konzentriert habe.

Das Projekt hatte ich aber noch im Hinterkopf.

Allen, die am Modul (Mit-)Arbeiten möchten, können dies sehr gerne tun. Da habe ich überhaupt kein Problem mit! (mein Bananapi ist ständig abgestürzt und irgendwann hatte ich keine lust mehr den immer wieder zu reparieren, bald kommt ein stromsparender htpc server)

Also ran da und viel Spaß.

Freut mich, dass Ihr weitergemacht habt und das ganze dann fest implementiert werden soll, falls das noch nicht passiert ist! :)