Wunsch/Anfrage: Umschalt-Timer "mit Offset" versehen?

[tewsbo]

hallo,
ist es möglich - analog zum aufnahme-timer - für den umschalt-timer auch einen "offset" im image zu implementieren? ist so gemeint, dass wenn ich über epg einen umschalt-timer (z.b. 2min) setze, dass dann der timer in der timer-übersicht gleich mit dem eingestellten offset gespeichert wird (aus 18:00 wird 17:58uhr).

die umschaltung per epg-daten bisher ist nicht unbedingt immer präzise und wenn der film/die sendung schon läuft, dann ist das halt ärgerlich.

bei ner aufnahme kann man ja auch den "offset" setzen, um nix zu verpassen. und warum soll das bei einem umschalttimer nicht auch sinn machen?

wenn ihr die idee natürlich für nicht sinnvoll betrachtet, dann nicht. fände es aber eine nützliche ergänzung.
und gehe mal davon aus, dass das evtl gar nicht so schwer zu proggen sein müsste.

Danke im Voraus für sie sicher kommenden Antworten

[rhabarber1848]

hallo,
ist es möglich - analog zum aufnahme-timer - für den umschalt-timer auch einen "offset" im image zu implementieren?

Soll der Umschalt-offset den gleichen Wert haben wie der Aufnahme-offset?
Dann brächte man keine neue Option in Neutrino.
Kannst Du eigene Images kompilieren um evtl. demnächst einen Patch zu testen?
Ich ändere meine Umschalttimer nach der Erstellung auch oft ab,
deshalb finde ich Deinen Vorschlag sinnvoll.

[tewsbo]

1) Soll der Umschalt-offset den gleichen Wert haben wie der Aufnahme-offset?
2) Dann brächte man keine neue Option in Neutrino.
3) Kannst Du eigene Images kompilieren um evtl. demnächst einen Patch zu testen?
4) Ich ändere meine Umschalttimer nach der Erstellung auch oft ab,
5) deshalb finde ich Deinen Vorschlag sinnvoll.

1) getrennt wäre schöner. wer nutzt einen aufnahme-offset von ca. 2 minuten? ich denke, bei denen, die den rec-offset nutzen, sind die werte größer, sonst macht es ja bei der aufnahme nicht wirklich sinn. beim umschalttimer reichen m.e. 2-5minuten
2) sicher ein argument, aber siehe punkt 1
3) leider nein
4) um wieviel minuten? (siehe punkt 1+2)
5) danke

[rhabarber1848]

4) um wieviel minuten? (siehe punkt 1+2)

1-2 Minuten, genauso wie den Aufnahmetimer.
Aber gut, eine neue Option kostet nicht die Welt
Ich melde mich hier wieder, wenn ich einen Patch habe,
kann aber ein paar Tage dauern.

[tewsbo]

hmm, hab an meiner box einen hdd-recorder, da sind die rec-offsets größer, weil es ja keine rolle spielt und man dadurch sicher nix verpasst.
danke für das interesse und das "angedrohte" engagement.

[FlatTV]

Tester gesucht? machen wir

cu FlatTV

[rhabarber1848]

Bitte testen: timerd_zapto_starttime.diff

Der Patch ist wegen der daemon-Kommunikation größer geworden
als ich dachte, aber ein erster Test zeigt, dass er funktioniert.
Die Umschalt-offset-Einstellung habe ich erstmal beim Aufnahme-
offset untergebracht (Einstellungen, Aufnahme), die Option sollte
im CVS aber woanders liegen, Vorschläge?

[seife]

Warum muss das der Timerd überhaupt wissen? Zieh die X Minuten doch einfach beim Erstellen des Timers ab.

Selbst bei den anderen Timern ist mir nicht klar, warum das unbedingt im timerd gemacht werden muss.

Mir ist aber auch nicht klar, wozu man den timerd überhaupt benötigt und warum das nicht einfach im Neutrino mit drin ist

[rhabarber1848]

Warum muss das der Timerd überhaupt wissen?

Beim Aufnahmetimer funktioniert das genauso, siehe timerd.cpp ab Zeile 298.
Diesen Mechanismus habe ich für das Umschalt-offset kopiert.

Mir ist aber auch nicht klar, wozu man den timerd überhaupt benötigt und warum das nicht einfach im Neutrino mit drin ist

Mir ist das auch nicht klar, aber ich kann es nicht ändern, dazu fehlen
mir die Programmierkenntnisse. Sie reichen gerade noch aus, um den
obigen Patch zu schreiben

[seife]

Mach es dir einfacher: Ändere gar nichts am timerd. Konfiguriere irgendwo im neutrino ein g_settings.pre_announce
Diesen Wert ziehst du beim Erstellen der umschalttimer einfach von der Alarm- und announce-Zeit ab. Beispiel:

Code: Alles auswählen

--- src/gui/eventlist.cpp       3 Oct 2009 10:09:29 -0000       1.128
+++ src/gui/eventlist.cpp       3 Oct 2009 23:10:58 -0000
@@ -496,8 +496,8 @@ int EventList::exec(const t_channel_id c
                                }
 //                             timerdclient.addZaptoTimerEvent(channel_id,
                                timerdclient.addZaptoTimerEvent(GET_CHANNEL_ID_FROM_EVENT_ID(evtlist[selected].eventID),
-                                                               evtlist[selected].startTime,
-                                                               evtlist[selected].startTime - ANNOUNCETIME, 0,
+                                                               evtlist[selected].startTime - pre,
+                                                               evtlist[selected].startTime - ANNOUNCETIME - pre, 0,
                                                                evtlist[selected].eventID, evtlist[selected].startTime, 0);
                                UpdateTimerList();
                                paintItem(selected - liststart);

Sowas in der Art.
Nachdem es nur 3 mal addZaptoTimerEvent() gibt, halte ich das für einfacher.

[tewsbo]

Die Umschalt-offset-Einstellung habe ich erstmal beim Aufnahme-
offset untergebracht (Einstellungen, Aufnahme), die Option sollte
im CVS aber woanders liegen, Vorschläge?

klar
entweder
unter > dbox > einstellungen > diverse einstellungen > allgemein
oder
unter > dbox > einstellungen > diverse einstellungen noch einen punkt "Timer(offsets)" o.ä anlegen (unter epg, zapit ...).
dort könnten dann ja auch die aufnahme-offsets "hinwandern"

[rhabarber1848]

Mach es dir einfacher: Ändere gar nichts am timerd.

In meinem Patch habe ich die Funktion CTimerdClient::getOverlappingTimers
noch nicht berücksichtigt.
Muss das Umschalt-offset nicht auch dort Berücksichtigung finden und
somit in timerd.cpp bekannt sein?

[seife]

Eigentlich nicht.
Glaube ich.
Der umschalttime hat ja in dem Sinne keine "Vorlaufzeit" und auch keine Dauer. Wenn du also bei EPG-Startzeit 19:00 auf den "umschalttimer"-Knopf drückst, und neutrino dann einen umschalttimer für 18:55 machen will, aber deine Aufnahme bis 18:57 geplant ist, dann überschneidet sich das auch so.

Ich habe mir aber den Code nicht angeschaut.

[FlatTV]

@rhabarber1848

Mach es dir einfacher: Ändere gar nichts am timerd.

Nur noch mal nagefragt - änderst du am Patch noch etwas?

cu FlatTV

[rhabarber1848]

änderst du am Patch noch etwas?

Erstmal nicht, aber eher aus Zeitmangel.
Außerdem hat Seife mich noch nicht überzeugt
Hat schon jemand meinen Patch getestet?

[FlatTV]

Darum frage ich... bin aber diese Woche nicht @home

cu FlatTV

[tewsbo]

Hat schon jemand meinen Patch getestet?

so, erster test ohne zwischenfälle verlaufen. funktioniert so, wie gewünscht.
evtl kann ja der ort zu einstellen noch optimiert werden.
vorschläge zu neuen orten hab ich ja weiter vorne schon gemacht.
aber immo werden ja eh die menüs aufgeräumt.

also, erst mal ein danke fürs schnelle reagieren und umsetzen.

[FlatTV]

zweiter, Patch funktioniert

cu FlatTV

[rhabarber1848]

Bitte testen: EDIT: Patch ist im CVS

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/1308
http://article.gmane.org/gmane.comp.vid ... x.scm/1309
http://article.gmane.org/gmane.comp.vid ... x.scm/1310
http://article.gmane.org/gmane.comp.vid ... x.scm/1311
http://article.gmane.org/gmane.comp.vid ... x.scm/1312

Die Umschalt-offset-Einstellung habe ich erstmal beim Aufnahme-
offset untergebracht (Einstellungen, Aufnahme)

Vorschlag für das endgültige Menü:
dbox / Einstellungen / Diverse Einstellungen / Timeroptionen

Dort könnten auch die Aufnahme-Korrektur-Einstellungen
liegen, dies sollte aber im Rahmen der Menürestrukturierung
stattfinden, deshalb habe ich den Patch erstmal so eingecheckt,
auch aufgrund der positiven Rückmeldungen.

@seife: Ich kann verstehen, dass Du timerd loswerden willst,
bis es aber soweit ist, sollte imho kein Code am timerd und
dessen Strukturen vorbei programmiert werden, um eine
spätere Umstellung nicht zu verkomplizieren.

[seife]

Jedes nun unnützerweise neu implementierte "Feature" im timerd verkompliziert eine spätere Umstellung.

[tewsbo]

muss mich nochmal zum thema melden.

eine nicht so schöne nebenerscheinung ist mir noch aufgefallen.

und zwar wird beim setzen des werbe-zapping-timers auch dort die "offsetzeit" abgezogen.
denke, weil der zapping-timer auch über die funktion "umschalten" gesetzt und in der timerübersicht als "umschalttimer" zu finden ist.

kann man da noch was dran ändern?

andernfalls muss dann eben die zeit, die zum werbezappen eingegeben wird, manuell um die anzahl minuten erhöht werden, die im timer-offset hinterlegt sind. ist aber sicher nicht die eleganteste lösung.

[seife]

Sagte ich nicht, dass der timerd der falsche Platz ist, um das Ursprüngliche Problem zu lösen?

;-)

Achso: jetzt einen zusätzlichen Timertyp im timerd zu implementieren ist IMHO nicht die korrekte Lösung des Problems.

Stattdessen sollte man wirklich einfach im Neutrino beim Erstellen des Timers den offset abziehen und gut ist. Evtl. auch nur, wenn wirklich aus dem EPG ein Umschalttimer erstellt wird und nicht, wenn er manuell gemacht wird. Wie das jetzt ist, weiss ich gerade nicht auswendig.

[rhabarber1848]

Es kann sein, dass sich der werbezapper und das Dbox2-Plugin
des TVBrowsers (letzteres nutzt yweb) dabei ins Gehege kommen.
Beides sind 3rd-party plugins, daher ist Rücksichtnahme im CVS nicht
zwingend erforderlich. Ich kann nachvollziehen, dass das offset beim
Werbezapper keinen Sinn macht, nur wie soll die Unterscheidung
zum TVBrowser stattfinden? Gibt es überhaupt Sourcecode zum
Werbezapper-Plugin?

Ich halte es für sinnvoll, dass für Umschalttimer, die via TVBrowser
gesetzt werden, ebenfalls das offset berücksichtigt wird. Hier
können Timer gesetzt sein, für die die Dbox noch gar keinen EPG
hat, was dann wohl einer manuellen Eingabe gleich kommt.

[DrStoned]

Gibt es überhaupt Sourcecode zum Werbezapper-Plugin?

Öffentlich nicht, aber SnowHead hat ihn. Ich denke aber, dass der Werbezapper keinen Boxtimer nutzt, sondern die Umschaltung selbst erledigt. Im Werbezapper-Plugin existiert nämlich ein eigener Timer, der im LC-Display runterzählt und dann umschaltet. Soweit ich weiss, ist blockads eine Art Deamon, der dann die Umschaltung erledigt. Ich kann es gerade nicht testen, da ich meine Dbox für ein paar Tage ausgeliehen habe.

Greetz von DrStoned

[tewsbo]

...eine nicht so schöne nebenerscheinung ist mir noch aufgefallen.

und zwar wird beim setzen des werbe-zapping-timers auch dort die "offsetzeit" abgezogen.
denke, weil der zapping-timer auch über die funktion "umschalten" gesetzt und in der timerübersicht als "umschalttimer" zu finden ist.

kann man da noch was dran ändern?...

hochzieh und nochmal nachfrag.

kann man das noch eleganter lösen oder ist das thema erst mal durch? nicht falsch verstehen, kann auch damit leben, dem zapp-timer entsprechend mehr minuten mit auf den weg zu geben.

danke tewsbo.