[live] [merged] Anzeige inaktiver Timer

Hallo zusammen,

ich nutze selbst ein paar Patches, die vielleicht für den einen oder anderen von euch von Interesse sein könnten. Markus hat vorgeschlagen, solche Patches hier im Forum in jeweils einem Thread kurz vorzustellen und euer Votum einzuholen, ob sie ins Git sollen:

• 0010 Added icon and indication of inactive timers.zip

Bislang zeigt Live in allen Übersichten jeden Timer mit einer grünen "Lampe" an, egal, ob der Timer aktiv ist oder auf inaktiv gestellt wurde. Dies betrifft die Programmübersichten, die Zeitleiste, die Suchergebnisse von Suchtimern und die Detailanzeigen. In der Zeitleiste werden Sendungen, für die ein Timer gesetzt ist, hellrot unterlegt, wobei auch hier nicht zwischen aktiv und inaktiv unterschieden wird.

Der Patch zeigt an allen Stellen für anstelle der grünen für aktive Timer eine hellblaue Lampe für inaktive Timer an. In der Zeitleiste werden die entsprechenden Sendungen statt hellrot für aktive nun hellgrau für inaktive Timer unterlegt.

Weil pro Beitrag immer nur fünf Dateien hochgeladen werden können, findet ihr die Screenshots weiter unten.

Lasst man die Farbwahl auf euch wirken, vor allem bezüglich der Unterscheidbarkeit von grün zu hellblau. Falls ihr den inaktiven Timer mehr Richtung dunkelgrau haben möchtet – kein Problem. Mir war er dann nur zu "unauffällig".

Viele Grüße

Stefan

Zeitleiste

Timer

wird (mit Patch Nr. 8 für das Timer-Popup) zu:

Suchtimer

mit den Details des ersten Timers:

Ich habe die Farbe nur per Hue geändert. Den Kreis mit einem hinsichtlich der Schattierung perfekten Grau zu füllen, ist etwas aufwändiger – war jetzt aber gar nicht so schlimm:

• 0010a Added icon and indication of inactive timers.zip

Damit sieht es dann zum Beispiel so aus:

Quote from wmautner

Danke - bei mir ist das Grau aber (Standardpalette "Veltliner")

nicht da:

Die Timer-Anzeigen sind nicht Teil der Themes, sondern finden sich alle im Verzeichnis live/img im Git-Verzeichnis. Von dort wird die Grafik auch geladen:

<img src="img/<$ en.timer_is_active ? "record_timer.png" : "record_inactive_timer.png" $>"

Schau also bitte einmal, ob du eine Datei live/img/record_inactive_timer.png findest oder kopiere die beigefügte Grafik () einmal dorthin und rufe danach make install-web auf. Beim Browser hilft zudem oftmals ein vollständiger Reload (Shift+Relload beim Firefox) oder das Leeren des Caches.

Viele Grüße
Stefan

Schön, dass es doch noch geklappt hat.

Ich habe den Patch nochmals gecheckt, das Image sollte drin sein. Kann aber sein, dass sich Git mit Binaries in Patches etwas schwer tut. Mir ist das dieser Tage bei irgendeiner Git-Operation (die Details sind mir leider schon entfallen) etwas Ähnliches passiert. Auch da musste ich das Image nochmals per Hand einspielen.

Wie von wmautner vorgeschlagen, habe ich den Patch nochmals überarbeitet und gegen den aktuellen Master-Branch (Commit 9c84e44 – "search_epg_images_with_channelid") rebased:

• 00010b Added icon and indication of inactive or recording timers.zip

Laufende Aufnahmen werden jetzt mit einer hell und dunkel werdenden "LED" dargestellt, ähnlich wie in der Timer-Übersicht. Auch werden eventuell vorhandene Aufzeichnungen nicht mehr angezeigt, solange der Timer läuft:

Sobald der Timer nicht mehr läuft, also inaktiv ist:

… oder noch in der Zukunft liegt:

… sind vorhandene Aufnahmen wieder sichtbar. Die Anzeige laufender, aktiver und inaktiver Timer erfolgt auch in der Zeitleiste:

Enjoy & viele Grüße

Stefan

Hier erst einmal das neue Icon:

Was die msgids betrifft, habe ich wohl versehentlich die PO-Dateien nach dem Kompilieren mit eingecheckt. Da sich gegenüber dem Master beim Kompilieren immer solche Verschiebungen ergeben, setze ich diese Dateien normalerweise auf den offiziellen Stand zurück, wenn die Code-Änderungen keine neuen msgids eingeführt haben.

Hier nochmals der Patch ohne die PO-Dateien:

• 00010c Added icon and indication of inactive or recording timers.zip

Viele Grüße

Stefan

Markus müsste die Änderungen vor den Committen und Pushen noch kompilieren, damit daraus resultierende Veränderungen in den PO-Dateien ebenfalls mit im Git landen.

Auch fände ich es gut, wenn die mittlerweile überflüssigen – und deshalb automatisch auskommentierten – msgids am Ende mal entsorgt würden. Ein kleines Skript könnte da helfen, aber notfalls tut es auch die Kommandozeile in der Bash:

for po in po/*.po; do grep -E -av "^#(~ msg|, fuzzy)" $po | sed -rz 's/^\n+//; s/\n+$/\n/g' > $po.$$; mv $po.$$ $po; done

Viele Grüße

Stefan

Hi Markus,

danke wieder einmal für das Update. Im Grunde genommen funktioniert der Patch, doch ist mir ein seltsamer Effekt aufgefallen, den ich mir nicht erklären kann.

Ich habe einen Timer auf 20:10 Uhr (Olympia 2024) gesetzt, der aufgrund des einstellten Vorlauf ab 20:05 Uhr mit der Aufzeichnung beginnen soll:

Wir betrachten im folgenden die Situation um ca. 20:20 Uhr. In der Timer-Übersicht ist alles ganz normal:

In der Programmübersicht wird der Timer weder als aktiv noch als laufend gekennzeichnet und die bereits angefangene Aufzeichnung deshalb mit ausgegeben:

Und in der Zeitleiste wird die Sendung davor, mit der sich der Timer wegen des Vorlaufs überschneidet, als laufend laufend markiert:

Wegen des Limits vom 5 Dateianhängen geht es im nächsten Beitrag weiter…

Eliminiert man jetzt den Vorlauf und setzt den Timer auf eine Zeit von 20:15 oder 20:20 Uhr:

… ist in der Timer-Übersicht natürlich noch alles in Ordnung (siehe oben). In der Programmübersicht wird der Timer nun als laufend gemeldet und die bereits vorhandene Aufzeichnung deshalb nicht dargestellt:

In der Zeitleiste hingegen wird die Sendung davor mit einem aktiven Timer dargestellt, die eigentliche Sendung nun aber mit einem laufenden Timer:

Getestet hatte ich den Patch blöderweise in einem Szenario, bei dem keine Überschneidung von Timern vorgekommen ist. Und dort hat der Patch auch (siehe die Screenshots weiter oben) wie gewünscht funktioniert.

Ich habe mir den Code für die Programmübersicht in recman.cpp (und das zugehörige Pendant in pagelems.ecpp) nochmals angeschaut:

  if (timer) {
    if (timer->Recording()) {
      epg_item.append("&ts=r");
      // do not show a recording that is underway
      recItemFound = false;
    }
    else if (!(timer->Flags() & tfActive))
      epg_item.append("&ts=i");
  }

… und kann dort (wie auch im Code, der diese Kennungen auswertet) keinen logischen Fehler erkennen. Auch der Code für die Zeitleiste:

const cTimer* timer = LiveTimerManager().GetTimer(Event->EventID(), Channel->GetChannelID());
if (!timer)
    en.timer = tState::None;
else if (timer->Recording())
    en.timer = tState::Recording;
else if (timer->Flags() & tfActive)
    en.timer = tState::Active;
else
    en.timer = tState::Inactive;

… ist so wie er sein soll. Die einzige Erklärung die ich habe, wäre, dass die Abfragen der Timer-Stati diese komischen Ergebnisse liefert.

Doch der ziemlich ähnliche Code für die Timer-Übersicht:

if (timer->Recording()) {
    timerStateImg = "arrow_rec.gif";
    timerStateHint = tr("Timer is recording.");
}
else if (timer->Flags() & tfActive) {
    if (timerConflicts.HasConflict(timer->Id() )) {
        timerStateImg = "timerconflict.gif";
        timerStateHint = tr("Timer has a conflict.");
    }
    else {
        timerStateImg = "arrow.png";
        timerStateHint = tr("Timer is active.");
    }
}

… verhält sich so, wie ich es auch für die Programmübersicht und die Zeitleiste erwarten würde.

Hast du dafür vielleicht eine Erklärung? Ich jedenfalls bin einigermaßen ratlos.

Danke & Grüße

Stefan

Ich habe das Ganze um 21:32 Uhr nochmals mit einem anderen Szenario getestet:

Da verhält sich trotz der Überschneidung mit der Sendung davor alles so, wie es soll… komisch, oder?

Tritt das vielleicht nur dann auf, wenn nach dem Start des VDR der Event zum Timer der erste in der Liste ist und:

const cTimer* timer = LiveTimerManager().GetTimer(Event->EventID(), Channel->GetChannelID() );

… falsche Daten liefert – ein Randwertproblem also möglicherweise?

PS – Was mit gerade noch eingefallen ist: Wenn ich den VDR starte und gleich die Programmübersicht aufrufe, erhalte ich immer erst einmal Kanal 1 (Das Erste) angezeigt, nicht den zuletzt einstellten Kanal (n-tv). Kann es sein, dass Live zu diesem Zeitpunkt noch mit seiner Initialisierung beschäftigt ist und solche "frühen" Anfragen den Aufbau der internen Datenbanken von Recording bzw. Event Manager beeinträchtigen?

Danke & Grüße

Stefan