[epgsearch] VPS-Timer mit Dauer X Minuten wird ohne Aufnahme gelöscht wenn die Sendung nach X+N Minuten beginnt

Joe_D · July 5, 2025 at 9:55 AM New

Hallo Zusammen,

habe für die Tagesschau einen epgsearch-2.4.3 Timer mit VPS.
Gestern (04.07.) hatte die Tagesschau wegen Fußball 17 Minuten Verspätung.
Da ich zufällig zugegen war konnte ich "Live" beobachten wie der Timer ohne Aufnahme einfach um 20:16 "verschwand"..

Im vdr-Log gibt es nur folgendes:

Code

Jul  4 19:33:20 vdr vdr: [1467] timer 0 (1 2000-2015 VPS 'Tagesschau') set to event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau'
Jul  4 19:33:20 vdr vdr: [1467] timer 0 (1 2000-2015 VPS 'Tagesschau') set to event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau'
Jul  4 19:58:01 vdr vdr: [1407] timer 1 (1 2000-2015 VPS 'Tagesschau') entered VPS margin
Jul  4 20:16:01 vdr vdr: [1407] timer 1 (1 2000-2015 VPS 'Tagesschau') set to no event
Jul  4 20:16:01 vdr vdr: [1407] deleting timer 1 (1 2000-2015 VPS 'Tagesschau')

Ist das bekannt das VPS-Aufnahmen einfach nicht ausgeführt werden wenn die Startzeit nach dem Ende des regulären Timers liegt?
Wird das vom vdr oder von epgsearch ausgelöst?
Sollte nicht mindestens 30 Minuten oder die 1,5fache Sendungslänge auf den Start gewartet werden?

MarkusE · July 6, 2025 at 9:37 AM New

Ich würde sagen, das ist ein Fehler in VDR.

Datei timers.c, Methode bool cTimer::Expired(void):

Code

              if (Schedule) {                                                                                                                                                                                 
                 for (const cEvent *e = FirstEvent; e; e = Schedule->Events()->Next(e)) {                                                                                                                     
                     if (e->Vps() == StartTime()) {                                                                                                                                                           
                        ExpireTime = e->EndTime() + EXPIRELATENCY;                                                                                                                                            
                        dsyslog("timer %s is waiting for next VPS event %s", *ToDescr(), *e->ToDescr());                                                                                                      
                        // no break here - let's play it safe and look at *all* events                                                                                                                        
                        }                                                                                                                                                                                     
                     }                                                                                                                                                                                        
                 }

Der Vergleich e->Vps() == StartTime() ist hier falsch. StartTime() gibt die event->StartTime() zurück. Korrekt wäre der Vergleich mit der Timer->vps_time(), die unabhängig vom Event ist.

kls , kannst Du Dir das bitte mal anschauen?

nobanzai · July 6, 2025 at 1:08 PM New

Quote from Joe_D

Hallo Zusammen,

habe für die Tagesschau einen epgsearch-2.4.3 Timer mit VPS.
Gestern (04.07.) hatte die Tagesschau wegen Fußball 17 Minuten Verspätung.
Da ich zufällig zugegen war konnte ich "Live" beobachten wie der Timer ohne Aufnahme einfach um 20:16 "verschwand"..

Das hatte ich auch schon mehrfach und auch schon hier gepostet.
Bisher wusste niemand eine Lösung.

kls · July 8, 2025 at 11:17 AM New

Quote from MarkusE

Der Vergleich e->Vps() == StartTime() ist hier falsch. StartTime() gibt die event->StartTime() zurück. Korrekt wäre der Vergleich mit der Timer->vps_time(), die unabhängig vom Event ist.

Im Prinzip gebe ich dir Recht (da ist sicher Handlungsbedarf).
Allerdings wurde der von dir zitierte Code eingebaut um sicherzustellen, dass mehrere Events hintereinander mit der gleichen VPS-Zeit aufgezeichnet werden. Es werden hier alle Events nach dem aktuellen untersucht.
Selbst wenn über eine cTimer::VpsTime()-Funktion der "richtige" Event gefunden wird, bringt das hier nichts, wenn der Sender die tatsächliche EndTime() des Events nicht "nachzieht".

Möglicherweise wäre das hier der "richtige" Fix:

Diff

--- timers.c    2025/07/08 07:21:46     5.27
+++ timers.c    2025/07/08 09:10:03
@@ -779,7 +779,7 @@

 bool cTimer::Expired(void) const
 {
-  if (IsSingleEvent() && !Recording()) {
+  if (IsSingleEvent() && !Recording() && (!event || event->RunningStatus() == SI::RunningStatusNotRunning)) {
      time_t Now = time(NULL);
      time_t ExpireTime = StopTimeEvent();
      if (HasFlags(tfVps)) {

Display More

Joe_D Kannst du bitte die Log-Einträge aus #1 um die Zeilen mit
channel 1 (Das Erste HD) event ... status ...
ergänzen? Bitte bis zu der Zeit, wo die Tagesschau tatsächlich geendet hat (falls möglich).

MarkusE · July 8, 2025 at 4:31 PM New

kls , Du hast recht, wir müssen hier 2 Probleme lösen.

a) Der Sender macht keinen Update der event Zeiten, das sollte Dein Fix aus #4 lösen

b) StartTime() gibt bei VPS Timern nicht die VPS Zeit zurück, sondern die event->StartTime(), die manchmal mit der VPS Zeit übereinstimmt. Manchmal aber auch nicht.

b) Könnte mit

Code

              if (Schedule) {
                 time_t vps_time, dummy;
                 CalcStartStopTime(vps_time, dummy);                                                                                                                                                                     
                 for (const cEvent *e = FirstEvent; e; e = Schedule->Events()->Next(e)) {                                                                                                                     
                     if (e->Vps() == vps_time) {                                                                                                                                                           
                        ExpireTime = e->EndTime() + EXPIRELATENCY;                                                                                                                                            
                        dsyslog("timer %s is waiting for next VPS event %s", *ToDescr(), *e->ToDescr());                                                                                                      
                        // no break here - let's play it safe and look at *all* events                                                                                                                        
                        }                                                                                                                                                                                     
                     }                                                                                                                                                                                        
                 }

Display More

gelöst werden. Man könnte auch rückwärts durch Schedule->Events() loopen:

Code

              if (Schedule) {
                 time_t vps_time, dummy;
                 CalcStartStopTime(vps_time, dummy);
                 for (const cEvent *e = Schedule->Events()->Last(); e; e = Schedule->Events()->Prev(e)) {
                     if (e->Vps() == vps_time) {                                                                                                                                                     
                        ExpireTime = e->EndTime() + EXPIRELATENCY;                                                                                                                                            
                        dsyslog("timer %s is waiting for next VPS event %s", *ToDescr(), *e->ToDescr());
                        return ExpireTime <= Now && event->RunningStatus() == SI::RunningStatusNotRunning))
                        }                                                                                                                                                                                     
                     }                                                                                                                                                                                        
                 }

Display More

Das ist jetzt aber Geschmacksache, und kein Fehler.

kls · July 9, 2025 at 1:06 PM New

Joe_D Kannst du bitte die Log-Einträge aus #1 um die Zeilen mit
channel 1 (Das Erste HD) event ... status ...
ergänzen? Bitte bis zu der Zeit, wo die Tagesschau tatsächlich geendet hat (falls möglich).
Ich würde das gerne noch verifizieren, bevor ich das einbaue.

Joe_D · July 9, 2025 at 4:35 PM New

kls .. sieht dann so aus:

Code

Jul  4 18:10:28 vdr vdr: [1440] channel 1 (Das Erste HD) event Fr. 04.07.2025 17:15-20:00 (VPS: 04.07. 17:15) 'UEFA Frauen-Europameisterschaft Schweiz 2025: Dänemark - Schweden' status 0->4
Jul  4 19:33:20 vdr vdr: [1467] timer 0 (1 2000-2015 VPS 'Tagesschau') set to event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau'
Jul  4 19:33:20 vdr vdr: [1467] timer 0 (1 2000-2015 VPS 'Tagesschau') set to event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau'
Jul  4 19:58:01 vdr vdr: [1407] timer 1 (1 2000-2015 VPS 'Tagesschau') entered VPS margin
Jul  4 20:16:01 vdr vdr: [1407] timer 1 (1 2000-2015 VPS 'Tagesschau') set to no event
Jul  4 20:16:01 vdr vdr: [1407] deleting timer 1 (1 2000-2015 VPS 'Tagesschau')
Jul  4 20:17:21 vdr vdr: [1446] channel 1 (Das Erste HD) event Fr. 04.07.2025 17:15-20:00 (VPS: 04.07. 17:15) 'UEFA Frauen-Europameisterschaft Schweiz 2025: Dänemark - Schweden' status 4->1
Jul  4 20:17:21 vdr vdr: [1446] channel 1 (Das Erste HD) event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau' status 0->4
Jul  4 20:32:45 vdr vdr: [1446] channel 1 (Das Erste HD) event Fr. 04.07.2025 20:00-20:15 (VPS: 04.07. 20:00) 'Tagesschau' status 4->1

kls · July 9, 2025 at 4:47 PM New

Danke, das bestätigt meine Vermutung, dann baue ich das mal so ein.

[epgsearch] VPS-Timer mit Dauer X Minuten wird ohne Aufnahme gelöscht wenn die Sendung nach X+N Minuten beginnt

Joe_D July 5, 2025 at 9:56 AM

Participate now!

Tags

Joe_D July 5, 2025 at 9:56 AM

Participate now!

Tags

Joe_D July 5, 2025 at 9:56 AM