Fernsehprogramm DOIF (aktuell und primetime | non blocking | minimaler traffic)

Begonnen von mumpitzstuff, 12 Juni 2020, 21:47:48

Vorheriges Thema - Nächstes Thema

Damian

Die neuen Features hier: https://forum.fhem.de/index.php/topic,112566.msg1069005.html#msg1069005 könnten bei dieser Aufgabe interessant sein. Z. Zt. müssen beim Fernsehprogramm Änderung sowohl im DEF-Bereich als auch im uiTable-Attribut vorgenommen werden. Eleganter wäre es Anpassungen nur an einer Stelle vornehmen zu müssen, statt an zwei.
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

mumpitzstuff


mumpitzstuff

Im letzten Update das DOIF hatte sich leider ein Bug eingeschlichen, der dazu führt, das der Download gar nicht mehr ausgeführt wird. Deshalb wurde der erste Beitrag entsprechend angepasst und enthält nun den Bugfix.

mumpitzstuff

Zitat von: Damian am 01 Juli 2020, 10:21:33
Die neuen Features hier: https://forum.fhem.de/index.php/topic,112566.msg1069005.html#msg1069005 könnten bei dieser Aufgabe interessant sein. Z. Zt. müssen beim Fernsehprogramm Änderung sowohl im DEF-Bereich als auch im uiTable-Attribut vorgenommen werden. Eleganter wäre es Anpassungen nur an einer Stelle vornehmen zu müssen, statt an zwei.

Ich habe es mir übrigens angesehen und würde erst einmal darauf verzichten das zu übernehmen und zwar aus folgenden Gründen:

1.) Für diesen Anwendungsfall sehe ich keinen essentiellen Mehrwert. Lediglich die Formatierungen von TD könnte man generieren lassen und der Anwender müsste dann nicht mehr manuell daran rum fummeln.
2.) Die Generierung der Templates würde in den sub Bereich rein wandern, der aktuell aber ohnehin schon recht groß ist. Die schöne Trennung zwischen Code für die Ansicht und Code zum generieren der Readings würde verloren gehen.
3.) Der Code ist, meiner Meinung nach, für nicht Programmierer nicht mehr lesbar und damit auch nicht mehr einfach vom Anwender anpassbar bzw. erweiterbar. Es ist mit Sicherheit ein mächtiges Werkzeug, ein Großteil der Anwender wird damit aber massiv überfordert sein.

Damian

Zitat von: mumpitzstuff am 14 Juli 2020, 12:08:37
Ich habe es mir übrigens angesehen und würde erst einmal darauf verzichten das zu übernehmen und zwar aus folgenden Gründen:

1.) Für diesen Anwendungsfall sehe ich keinen essentiellen Mehrwert. Lediglich die Formatierungen von TD könnte man generieren lassen und der Anwender müsste dann nicht mehr manuell daran rum fummeln.
2.) Die Generierung der Templates würde in den sub Bereich rein wandern, der aktuell aber ohnehin schon recht groß ist. Die schöne Trennung zwischen Code für die Ansicht und Code zum generieren der Readings würde verloren gehen.
3.) Der Code ist, meiner Meinung nach, für nicht Programmierer nicht mehr lesbar und damit auch nicht mehr einfach vom Anwender anpassbar bzw. erweiterbar. Es ist mit Sicherheit ein mächtiges Werkzeug, ein Großteil der Anwender wird damit aber massiv überfordert sein.

klar, man muss immer schauen, wo es sinnvoll anwendbar ist
Programmierte FHEM-Module: DOIF-FHEM, DOIF-Perl, DOIF-uiTable, THRESHOLD, FHEM-Befehl: IF

Invers

Seit heute erhalte ich eine Fehlermeldung. Ich dachte, ich hätte vielleicht etwas falsch gemacht und versaut und habe deshalb mal den Code neu eingefügt. Der Fehler bleibt aber in beiden Versionen, also in meiner alten Version und in der Version hier aus dem Forum, auch wenn nichts geändert wird.

Die Fehlermeldung lautet:
error Bareword "prime" not allowed while "strict subs" in use at (eval 36221) line 1. in expression: mode (next or prime)

Kannst du mal einen Blick werfen? Ist das bei euch auch so, oder nur bei mir?
Pi3B+ mit SSD/ Bullseye | FB7590 AX | 12 x Dect200 | CUL433+868 | SDuino | HM-LAN | 3 x Heizung FHT + FKontakte | KeyMatic + 4 FB | HM Wandtaster 2-fach m. LED | 6 x Türkont. TFK-TI | HM-Bew.-Melder innen | 3 x Smoked. HM-SEC-SD-2

mumpitzstuff

Nachdem ich ein Update von FHEM gemacht habe, gibt es bei mir das selbe Problem. Das muss ein Bug im DOIF Modul sein.

mumpitzstuff

Problem ist identifiziert und ich behebe es heute Abend. Alternativ kannst du auch lokal bei dir in der uiTable alle Kommentare mit 2 # versehen anstatt nur einem. Also ## anstatt #.

Invers

Habe ich gemacht, funktioniert. War nur eine Stelle. Danke dir.
Pi3B+ mit SSD/ Bullseye | FB7590 AX | 12 x Dect200 | CUL433+868 | SDuino | HM-LAN | 3 x Heizung FHT + FKontakte | KeyMatic + 4 FB | HM Wandtaster 2-fach m. LED | 6 x Türkont. TFK-TI | HM-Bew.-Melder innen | 3 x Smoked. HM-SEC-SD-2

Invers

Es gelingt mir nicht, das Programm in den Flooplan zu übernehmen. Über den Namen des Doif klappt es nicht. Wer sagt mir, wie es geht?
Pi3B+ mit SSD/ Bullseye | FB7590 AX | 12 x Dect200 | CUL433+868 | SDuino | HM-LAN | 3 x Heizung FHT + FKontakte | KeyMatic + 4 FB | HM Wandtaster 2-fach m. LED | 6 x Türkont. TFK-TI | HM-Bew.-Melder innen | 3 x Smoked. HM-SEC-SD-2

Ellert

Zitat von: Invers am 18 August 2020, 20:08:00
Es gelingt mir nicht, das Programm in den Flooplan zu übernehmen. Über den Namen des Doif klappt es nicht. Wer sagt mir, wie es geht?
Das DOIF als weblink anlegen und diesen in den Floorplan einbunden.

Ellert


Invers

Danke, Ellert.
Für den 2. Vorschlag ist es zu spät, da ich schon Vieles im Flurplan angelegt habe und daher diesen beibehalten möchte.

Könntest du mir bitte den Code für den Weblink geben? Ich habe keine Ahnung, wie das geht. Mein altes Klack war html Weblink

mein letzter Versuch erzeugt nur eine leere Box.
defmod WL_KalckTV weblink htmlCode { '<a href="/fhem/floorplan/Klack"><img src="doif_TEST" width="800" height="600" ></a>' }


Pi3B+ mit SSD/ Bullseye | FB7590 AX | 12 x Dect200 | CUL433+868 | SDuino | HM-LAN | 3 x Heizung FHT + FKontakte | KeyMatic + 4 FB | HM Wandtaster 2-fach m. LED | 6 x Türkont. TFK-TI | HM-Bew.-Melder innen | 3 x Smoked. HM-SEC-SD-2


Invers

Danke dir, hat im Prinzip fuinktioniert. Den Rest habe ich dort angefragt.
Pi3B+ mit SSD/ Bullseye | FB7590 AX | 12 x Dect200 | CUL433+868 | SDuino | HM-LAN | 3 x Heizung FHT + FKontakte | KeyMatic + 4 FB | HM Wandtaster 2-fach m. LED | 6 x Türkont. TFK-TI | HM-Bew.-Melder innen | 3 x Smoked. HM-SEC-SD-2