[Permashift] Tester für (fast) permanenten Timeshift gesucht

Hallo!

Wie vor ein paar Tagen angekündigt habe ich etwas gebastelt, mit dem man im Liveprogramm zurückspulen oder gerade Gesehenes in eine Sofortaufnahme integriert bekommen kann. Ich hatte vor einer (ziemlich langen) Weile gefragt, ob es für VDR permanenten Timeshift gebe. Mir wurde gesagt, dass es zwei Gruppen gibt: die, die's nicht haben wollen, und die, die's nicht programmieren können. Das war mir ehrlich gesagt ein zusätzlicher Ansporn.

An die, die's nicht haben wollen: Das Plugin ist auf kleine Veränderungen am VDR selbst angewiesen. Ich habe die so minimal-invasiv wie irgendmöglich gehalten. Ich freue mich über jeden, der sich den Patch daraufhin anschaut, ob er für Anwender, die permanenten Timeshift nicht haben wollen, etwas verschlechtern könnte.

An die, die's haben wollen: Probiert's aus, und meldet euch, wenn's funktioniert oder wenn's nicht funktioniert. Für Verbesserungs- und Erweiterungsvorschläge bin ich offen (kann aber nichts versprechen).

Noch viel mehr Vorrede und den Patch für VDR und die Quellen des Plugins gibt es hier:
http://ein-eike.de/vdr-plugin-permashift/

Viel Spaß,
Eike

Hallo!

Danke für alle Freuden-Bekundungen!

Zitat von hotzenplotz5

Ein Eike auch wenn ich im moment nicht zum testen komme
besten dank dafür !

ich werde dann selbstverständlich ein paket für yavdr bauen incl. vdr patch sobald es mit vdr-1.7.36 weitergeht.

Das wär natürlich prima!
(Aber bitte nicht, bevor ein paar Leute erfolgreich getestet haben...)

Ich kuck mal, vielleicht komm ich auch dazu, selbst schon mal ein Deb-Paket zu bauen.

Ciao,
Eike

Hallo!

Zitat von mini73

Klaus benutzt ein etwas anderes Einrücken, der eingerückte Block fängt immer unter der Klammer von if/while/for usw. an.

Ich hatte extra versucht, mich im VDR an Klaus' Gewohnheiten zu halten:
Beim if steht die öffnende Klammer in derselben Zeile, ab da wird eingerückt,
und zwar inklusive der schließenden Klammer. Die englische Wikipedia nennt
das "Ratliff style".
Leider krieg ich den Beispiel-Code hier nicht so reingepastet, wie er bei mir aussieht,
aber auf meinem Bildschirm passt's zusammen.
(Beim Plugin hab ich mich dann aber an meine eigenen Vorlieben gehalten,
da stehen zusammengehörige Klammern in derselben Zeile oder in derselben Spalte.)

Zitat von mini73

Außerdem finde ich es schöner, wenn die Parameter im Header und C-File gleich heißen, das passt z.B. nicht bei der oben zitierten Funktion. Ist aber nur rein kosmetisch.

Du hast völlig Recht - wird geändert.

Zitat

Eine Stelle, zu der ich eine Frage habe:

Code

-  if (pauseLive)
-     Goto(0, true);
+  if (pauseLive) {
+    int Current, Total;
+    GetIndex(Current, Total, false);
+    readIndex = max(Total - 1, 0);
+    Goto(readIndex, true);
+    playMode = pmStill;
+    }

Hast
du bei nicht geladenem Plugin getestet, ob readIndex dann wirklich auf 0
ist? Hast du bestimmt, ich frage nur sicherheitshalber.

Nein, das hab ich so leider nicht getestet. Ich hatte ausprobiert,
ob der Eindruck ohne Plugin passt. Wenn man sich die Zahlen ankuckt,
ist der Index aber etwas größer als 0.
Ursprünglich hab ich zusammen mit dem Pause-Flag übergeben, ob es
eine normale oder eine "Recycling"-Pause ist. Um möglichst wenig
Interface-Änderung zu haben, hatte ich das wieder rausgeschmissen.
Da brauche ich wohl doch ein "bool pauseRecycling" oder so.

Zitat

Zu PausePriority:
Der
vdr-default ist 10, d.h. mit deinem Patch wird ein vdr als inaktiv
angesehen, wenn eine Aufnahme unterhalb dieser Priorität ausgeführt
wird. Das ist schlecht. Wenn ich z.B. spontan etwas im EPG sehe, dass
ich gerne aufnehmen möchte, es aber nicht so wichtig ist wie meine
Suchtimer, dann vegebe ich der Einfachheit halber eine 1 als Priorität,
weil ich dann weiß, dass nichts wichtiges blockiert wird. Solche
Aufnahmen kriege ich dann wohl nicht mehr zu Gesicht...
Dies
ist also eine Abweichung vom Standardverhalten des vdr und müsste
korrigiert werden. Vermutlich wäre es besser, wenn du das
recording-control deiner PermaShift-Aufnahme irgendwie markierst und sie
bei der Prüfung cRecordControls::Active explizit ausnimmst. Die
Priorität ist kein guter Ansatz dafür.

Alles anzeigen

Das was verlorengeht, soll natürlich nicht sein!

Ich würde aber trotzdem die Aufnahme-Priorität benutzen wollen,
nur halt mit einem kleineren Wert.

@Klaus:
Was sagt du? Kann man beim auomatischen Runterfahren Aufnahmen ignorieren,
wenn der, der sie erstellt hat, eine besonders geringe Priorität eingestellt hat?
Also zum Beispiel, negative Priorität behindert das Herunterfahren nicht?

Zitat

Ansonsten sieht's schon
gut aus, hab aber noch keinerlei Tests durchgeführt, sondern mir nur den
Patch angesehen. Wenn ich mich also irre, dann korrigiere mich bitte.

Danke für's Code-Review!
Wie immer sehen mehr Augen mehr...

Ciao,
Eike

Hallo!

Zitat von kls

Dazu möchte ich mich weder in der einen noch in der anderen Richtung äußern ;-).

Na gut, dann muss ich mir wohl vorerst selbst was aus den Fingern saugen...

mini73:
Wie wär's mit < TRANSFERPRIORITY (-1), oder, ganz defensiv, <= MINPRIORITY (-99)?

Zitat

Für die Version 2.0 wird sich da auf jeden Fall nichts mehr ändern.

Hast du dir schon eine Meinung gebildet, ob der Patch danach reinkommen könnte?

Ciao,
Eike

Hallo!

Zitat von Santos

habe das Plugin und den Patch auf meinem VDR 1.7.33 mal gebaut.
Die Aufnahme startet sofort, ohne Probleme.
Spulen funktioniert bei mir nicht bei LIVE TV.

Sollte das Spulen funktionieren ?

Das Record Symbol in meinem Display leuchtet.
Könnte man das abändern, dass es nur leuchtet wenn ich selbst eine Aufnahme starte ?

Alles anzeigen

Für's Spulen musst du erst in Pause, dann Play drücken, und dann sollte es gehen.
Klappt das bei dir?
Ich wollte den Patch so minimal wie möglich halten, damit er ohne Probleme für andere
eingebaut werden könnte. Aber da Klaus das derzeit nicht vorhat, kann ich mir auch das
direkte Spulen mal ankucken. Wird aber dauern, mit dem Code zur Steuerung hab ich
mich noch nicht beschäftigt.

Das mit dem Record-Symbol könnte man vermutlich auch hinkriegen, aber ich hab ein
bisschen die Befürchtung, das wird ein Fass ohne Boden: Letztlich ist es ja eine Aufnahme,
und man müsste dann jeder Ecke, die auf Aufnahmen reagiert, einzeln beibringen,
bei zu geringer Priorität nicht anzuschlagen...

Ciao,
Eike

Zitat von googles

Das ist sicher eine gute Lösung weil auch bei starten wegen Timer Aufnahme müsste der User inaktiv sein.
Und wer braucht schon den Livebuffer wenn er nicht vor der glotze sitzt

Das klingt gut; das schau ich mir an!

Ciao,
Eike

Also...

Man könnte im "Houskeeping" bei Benutzer-Inaktivität die Aufnahme abschalten.
Das klappt soweit, der VDR fährt sich dann auch runter.

Damit wäre der VDR-Patch die Stellen los, in denen er nur Aufnahmen ab einer gewissen Priorität
als "Aktivität" rechnet. Bis auf eine Stelle, nämlich die, die das Housekeeping erlaubt. Aber bei
schwacher Hardware möchte man ja auch das nicht während einer Aufnahme machen, unabhängig
von ihrer Priorität.

Hat da jemand eine Idee?
Könnte das Plugin mit einem Timer oder in einem anderen Thread (zwischen langen Sleeps) die
Aktivität abfragen - und vor allem, könnte/dürfte er von da aus (ggf. also aus einem anderen
Thread) seine Aufnahme stoppen?

Ciao,
Eike