Aufnahme als ungesehen markieren

Guggst du hier.

Gruß
iNOB

Mach hier ein Ticket auf.

Gruß
iNOB

Moin!

Evtl. wäre es besser rofafor anzusprechen, er ist der Autor des Patches.
Wenn wir bei yaVDR anfangen, Patches zu ändern und das nicht upstream kommt, ist die Änderung beim nächsten Update wieder in Vergessenheit geraten.

http://www.saunalahti.fi/~rahrenbe/vdr/patches/index.php

Lars.

Hallo,

ich habe es bei mir auch rausgenommen. Der Grund für diese Änderung war wohl, das am Ende der Aufnahme (resume auf oder hinter letzter Schnittmarke) die Resume-Position wieder auf Anfang gesetzt wird. Das sollte man dann natürlich mit rausnehmen. Da ich Aufnahmen nur seltenst zweimal schaue und da kann ich dann GRÜN benutzen, um zum Start zu kommen, habe ich es mit folgendem Patch wieder auf Standardverhalten zurückgesetzt:

--- vdr-2.0.2.orig/dvbplayer.c
+++ vdr-2.0.2/dvbplayer.c
@@ -383,15 +383,15 @@ bool cDvbPlayer::Save(void)
             abs(Index - marks.First()->Position()) <= int(round(RESUMEBACKUP * framesPerSecond)))
            Index = 0;
         int backup = int(round(RESUMEBACKUP * framesPerSecond));
-        if (Index >= index->Last() - backup)
-           Index = 0;
-        else {
+//        if (Index >= index->Last() - backup)
+//           Index = 0;
+//        else {
            Index -= backup;
            if (Index > 0)
               Index = index->GetNextIFrame(Index, false);
            else
               Index = 0;
-           }
+//           }
         if (Index >= 0)
            return index->StoreResume(Index);
         }
--- vdr-2.0.2.orig/recording.h
+++ vdr-2.0.2/recording.h
@@ -134,7 +134,7 @@ public:
   int FileSizeMB(void) const;
        ///< Returns the total file size of this recording (in MB), or -1 if the file
        ///< size is unknown.
-  bool IsNew(void) const { return GetResume() < 0; }
+  bool IsNew(void) const { return GetResume() <= 0; }
   bool IsEdited(void) const;
   bool IsPesRecording(void) const { return isPesRecording; }
   bool IsOnVideoDirectoryFileSystem(void) const;

Übrigens habe ich beim Stöbern im Code gesehen, dass die Resume-Datei im Standard-Edit-Recording-Menü gelöscht werden kann. Mit ExtRecMenü will es mir nicht gelingen. Dort sollte man eigentlich mit KEY_INFO reinkommen.

Tschüß Frank

Zitat von FrankJepsen

habe ich es mit folgendem Patch wieder auf Standardverhalten zurückgesetzt:

A bit cleaner approach is just to revert the original patch:

patch -p1 -R < vdr-2.x.x-resumereset.patch

Hello,

Zitat von rofafor

A bit cleaner approach is just to revert the original patch:

Code

patch -p1 -R < vdr-2.x.x-resumereset.patch

You are right of cause. But in yaVDR all this patches are integrated as one patch. The reason I posted was mainly to remind others that there is a reason for the patch and that it makes no sense to revert just half of it.

Perhaps should have made it configurable via setup to make everyone happy.

Bye Frank

How about a patch like this:

@@ -374,11 +374,16 @@
   if (index) {
      int Index = ptsIndex.FindIndex(DeviceGetSTC());
      if (Index >= 0) {
-        Index -= int(round(RESUMEBACKUP * framesPerSecond));
-        if (Index > 0)
-           Index = index->GetNextIFrame(Index, false);
-        else
-           Index = 0;
+        int backup = int(round(RESUMEBACKUP * framesPerSecond));
+        if (Index >= index->Last() - backup)
+           Index = index->GetNextIFrame(0, false);
+        else {
+           Index -= backup;
+           if (Index > 0)
+              Index = index->GetNextIFrame(Index, false);
+           else
+              Index = 0;
+           }
         if (Index >= 0)
            return index->StoreResume(Index);
         }

and leave IsNew() as it is.
This would rewind just after the beginning.

Bye Frank

Vom patch bearbeiten und Pakete neu basteln wollte ich mangels Erfahrung gerne die Finger weglassen. Eine Suche hier im Forum hat mich zumindest auch nicht schlauer gemacht (was sicher einfach gewesen wäre... :-)).

Ist denn die Wahrscheinlichkeit gegeben das das alte Verhalten im yaVDR wieder einzug hält?

Kann man nicht im Live-Plugin den Wiedergabestatus zurücksetzen für jede beliebige Aufnahme, dort, wo man auch den Namen ändern kann?

Dort gibt es eine Checkbox namens "Wiedergabeposition löschen".

Gruß
hepi

Zitat von hepi

Dort gibt es eine Checkbox namens "Wiedergabeposition löschen".

Der Möglichkeiten sind viele.

Mir erschließt sich allerdings der Sinn des Patches nicht. Wenn ich eine gesehene Aufnahme nochmal ab Anfang sehen will, muss ich doch nur GRÜN statt OK drück und gut.

Den Ungesehen Status durch GRÜN EXIT zu setzen ist nicht nur einfach, sondern entbehrt auch nicht einer gewissen Logik. Ich denke Klaus hat sich bei dem <= in IsNew schon was gedacht.

Quick und Dirty, wie Rofafor richtig anmerkte sollte es bei yaVDR so klappen:

cd /usr/src
apt-get source vdr
apt-get build-dep vdr
cd vdr-2.0.2


Änderungen in recording.h (<=) und dvbplayer.c (4x //) machen


dpkg-source --commit             (irgendeinen Namen für den Patch eintippen)
dpkg-buildpackage -tc -us -uc
cd ..
dpkg -i vdr-dev_2.0.2-3yavdr0~0.5_all.deb

Tschüß Frank

Zitat von FrankJepsen

Mir erschließt sich allerdings der Sinn des Patches nicht. Wenn ich eine gesehene Aufnahme nochmal ab Anfang sehen will, muss ich doch nur GRÜN statt OK drück und gut.

Das ist so ein klassischer (Groß)eltern-Patch für Leute, die keine Softkeys verstehen können

Zitat von FrankJepsen

Den Ungesehen Status durch GRÜN EXIT zu setzen ist nicht nur einfach, sondern entbehrt auch nicht einer gewissen Logik.

Funktioniert das auch im Zusammenhang mit "extrecmenu" ? Genau das ist mir gestern nicht gelungen.

Gruß
iNOB

Hallo,

da IsNew() in recordings.h ist, muss nach dem Compilieren vom VDR die VDR-DEV installiert werden und das vdr-plugin-extrecmenu neu compiliert werden:

cd /usr/src
apt-get source vdr-plugin-extrecmenu
apt-get build-dep vdr-plugin-extrecmenu
dpkg -i vdr-dev_2.0.2-3yavdr0~0.5_all.deb
cd vdr-plugin-extrecmenu-1.2.2.git20130323
dpkg-buildpackage -tc -us -uc
cd ..
dpkg -i vdr-plugin-extrecmenu-1.2.2.git20130323

Tschüß Frank

Zitat von Olmimoe

Bis jetzt konnte ich eine bereits gesehene Aufnahme duch Abspielen von Anfang (grüne Taste) und sofortiges stoppen als ungesehen markieren. Nach dem Update aus VDR 2.0.2 geht das nicht mehr. Die Aufnahme startet zwar von vorne aber die gesehen Markierung bleibt trotz sofortigem stoppen.

Das das so ging war eher ein Zufall, denn vorgesehene Funktion.

Wenn bei yaVDR nicht (!) "vdr-plugin-extrecmenu" als Aufnahmemenu aktiv ist, sondern das Normale vom VDR, gehst Du auf die gewünschte Aufnahme, drückst die Taste "Info" Deiner FB, nicht (!) blau, und kommst in den Aufnahme-Editor von rofafors "vasarajanauloja" Patchbundle. Dort gibt es u.a. den Punkt "Gespeicherten Zeitpunkt der letzten Wiedergabe löschen". Man kann auch die Schnittmarken löschen und die Aufnahme umbenennen. Was leider noch fehlt ist die Auswahl von Verzeichnissen ähnl. "vdr-plugin-live", da wollte sich rofa evtl. irgendwann mal drum kümmern.

Dieser Aufnahme Editor ist IMHO etwas, was grundsätzlich in den VDR gehören würde und rofa's Patch ist ja nun seit 1.3.x getestet ...

Regards
fnu

PS.: Und ich empfinde die Funktionsweise von rofa's "resumereset.patch" inzwischen als deutlich logischeres Verhalten, z.B. das eine bereits gesehene Aufnahme einfach wieder von vorne beginnt und nicht sinnlos erneut die letzten paar Frames abspielt ...

Zitat von fnu

Was leider noch fehlt ist die Auswahl von Verzeichnissen ähnl. "vdr-plugin-live", da wollte sich rofa irgendwann mal drum kümmern.

Das geht prinzipiell, ist aber nicht ganz intuitiv umgesetzt:
Mit der Info-Taste landet man im Menü:
[Blockierte Grafik: https://dl.dropboxusercontent.com/u/960809/yavdr-recording/move-rec.jpg]
Jetzt drückt man Rot, dann kann man einen Ordner aus der folders.conf des VDR mit OK bzw. Rot wählen und auch neue Verzeichnisse Anlegen, falls nötig (das landet dann in der folders.conf):
[Blockierte Grafik: https://dl.dropboxusercontent.com/u/960809/yavdr-recording/target-rec.jpg]
Und dann muss man mit der Taste Blau das Verschieben bzw. mit Gelb das Kopieren anstoßen:
[Blockierte Grafik: https://dl.dropboxusercontent.com/u/960809/yavdr-recording/copy.jpg]

Zitat von seahawk1986

Das geht prinzipiell, ist aber nicht ganz intuitiv umgesetzt:

Hmm, ja, stimmt. Aber ich würde mir wünschen das wäre auch einfacher/logischer, wenn man hier eines der existierenden Verzeichnisse aus "video.00" auswählen könnte.

So ist das ja schon wieder eine andere Ebene, eigentlich losgelöst vom VDR, ganz ähnlich der unglücklich umgesetzten Funktion der Kanalgruppen.

In "vdr-plugin-live" kann ich genau die Verzeichnisse auswählen die existieren und genutzt werden, mir war sogar so der Patch für diesen Aufnahmeeditor in live war auch von rofa ...

Regard
fnu

Dann lösche ich lieber über die reccmd.conf die resume Datei und lass den resetresume.patch komplett weg. Das Verschieben von x-beliebigen Verzeichnissen mit Hilfe von extrecmenu ist mir da wichtiger, als ein Beizaubern des "NEW"-Symbols über GREEN+EXIT.

Gruß
iNOB