Patch zu vdrnfofs mit edl support

helmut55 · 16. Dezember 2012

Hallo zusammen

nachdem ich hier link eine vdrnfofs-version mit edl support gefunden habe, habe ich das auch sofort testen müssen.

Dabei habe ich dann festgestellt, dass ohne eine marks-Datei defekte edl-Dateien erstellt werden.
Die edl-Datei ist nicht nur leer, sondern kann gar nicht angefasst werden.
NFS-Export geht dann auch nicht mehr.

Meine Erweiterung erzeugt dann eine edl-Datei mit folgendem Inhalt
00:00:00.000 99:99:99.99 2

Jetzt muss ich nur noch XMBC zum Mitspielen überreden.

Hier der geänderte Abschnitt aus filesystemnodes.py

Code

def edl_content(self):                                                                                                                                                                 
        if not self._edl_content:                                                                                                                                                          
            self._edl_content = '00:00:00.000 99:99:99.99 2'                                                                                                                               
            info_path = self.path + '/marks'                                                                                                                                               
            if not os.path.exists(info_path):                                                                                                                                              
                info_path = self.path + '/marks.vdr'                                                                                                                                       
                if not os.path.exists(info_path):                                                                                                                                          
                    info_path = None                                                                                                                                                       
            if not info_path == None:                                                                                                                                                      
                with open(info_path,'r') as marks_vdr:                                                                                                                                     
                    marks = marks_vdr.readlines()                                                                                                                                          
                matches = []                                                                                                                                                               
                for mark in marks:                                                                                                                                                         
                    a = self.timecode.search(mark)                                                                                                                                         
                    if a:                                                                                                                                                                  
                        matches.append(a.group(0))                                                                                                                                         
                if self.nullcode.match(matches[0]):                                                                                                                                        
                    # starts at 0                                                                                                                                                          
                    start = 0                                                                                                                                                              
                else:                                                                                                                                                                      
                    matches.insert(0,"00:00:00.000")                                                                                                                                       
                    start = 1                                                                                                                                                              
                matches.append("99:99:99.99")                                                                                                                                              
                edl_list = []                                                                                                                                                              
                for i in xrange(0,len(matches)-1,1):                                                                                                                                       
                    if i % 2 == 0:                                                                                                                                                         
                        if start == 1:                                                                                                                                                     
                            mode = "3"                                                                                                                                                     
                        else:                                                                                                                                                              
                            mode = "2"                                                                                                                                                     
                        edl_list.append("%s %s %s"%(matches[i],matches[i+1],mode))                                                                                                         
                    else:                                                                                                                                                                  
                        if start == 1:                                                                                                                                                     
                            mode = "2"                                                                                                                                                     
                        else:                                                                                                                                                              
                            mode = "3"                                                                                                                                                     
                        edl_list.append("%s %s %s"%(matches[i],matches[i+1],mode))                                                                                                         
                self._edl_content = "\n".join(edl_list)                                                                                                                                    
        return self._edl_content

Alles anzeigen

CU
Helmut

seahawk1986 · 16. Dezember 2012

Danke, das ist mir damals mangels entsprechender Aufnahmen mit fehlenden marks-Dateien nicht aufgefallen.

Zitat von helmut55

Jetzt muss ich nur noch XMBC zum Mitspielen überreden.

Was fehlt dir da noch? XBMC genügt es eigentlich wenn die edl genauso benannt ist wie die Videodatei

helmut55 · 16. Dezember 2012

Laut XBMC wicki müssen die actions NextScene/PreviousScene angesteuert werden. Bis jetzt haben meine Änderungen an den XML-Datenen noch nichts gebracht.

Da fällt mir grade ein:
gilt die edl-Funktionalität im Datei-Modus wie auch im Datenbank-Modus?:
Dazu habe ich noch keinen Hinweis gesehen.

CU
Helmut

seahawk1986 · 16. Dezember 2012

Zitat von helmut55

Laut XBMC wicki müssen die actions NextScene/PreviousScene angesteuert werden. Bis jetzt haben meine Änderungen an den XML-Datenen noch nichts gebracht.

Hier klappt es - in der keyboard.xml sieht es so aus, das gleiche lässt sich auch auf die remote.xml usw. übertragen:

Code

<FullscreenVideo>
    <keyboard>
      <f>FastForward</f>
      <r>Rewind</r>
      <period>StepForward</period>
      <comma>StepBack</comma>
      <backspace>Fullscreen</backspace>
      <quote>SmallStepBack</quote>
      <opensquarebracket>BigStepForward</opensquarebracket>
      <closesquarebracket>BigStepBack</closesquarebracket>
      <return>OSD</return>
      <enter>OSD</enter>
<!-- Skip to scene markers -->
      <n>NextScene</n>
      <b>PreviousScene</b>
<!-- Skip to scene markers end -->
      <m>OSD</m>
      <i>Info</i>
      <o>CodecInfo</o>
      <z>AspectRatio</z>
      <t>ShowSubtitles</t>
      <t mod="ctrl">SubtitleAlign</t>
      <l>NextSubtitle</l>
      <left>StepBack</left>
      <right>StepForward</right>
      <up>BigStepForward</up>
      <down>BigStepBack</down>
      <a>AudioDelay</a>
      <escape>Fullscreen</escape>
      <c>Playlist</c>
      <v>XBMC.ActivateWindow(Teletext)</v>
      <up mod="ctrl">SubtitleShiftUp</up>
      <down mod="ctrl">SubtitleShiftDown</down>
      <pageup>SkipNext</pageup>
      <pagedown>SkipPrevious</pagedown>
    </keyboard>
  </FullscreenVideo>

Alles anzeigen

Es spielt auch keine Rolle, ob man den Dateimodus nutzt oder es aus der Datenbank aufruft, hier bindet er die edl brav ein.

tag · 16. Dezember 2012

Moment mal, mit dem Skript stimmt etwas nicht. Die Zeitangaben passen nicht zusammen.
In der EDL stehen, wenn ich das richtig sehe, Zeitstempel mit Sekundenbruchteilen.
In der marks.vdr stehen aber entweder direkt Frame-Nummern (beginnend bei "1") oder Zeitstempel mit Frame-Nummer nach dem Punkt (beginnend bei "0:00:00.1").
Ob sich XBMC aber ausgerechnet daran stört, bezweifle ich. Schlimmstenfalls sind die Sprungmarken halt etwas ungenau ...

seahawk1986 · 16. Dezember 2012

Ja, man bekommt dadurch bei Endmarken einen zu frühen Sprung (meist schlecht, da der Ton oft schon vor der Werbung auf Stereo wechselt und markad das wertet) und hüpft beim Anfang nach der Werbung etwas zu früh rein (was prinzipiell gut ist, da der Ton recht spät wieder auf Mehrkanal umgeschaltet wird) - bliebe ein zusätzliches Auslesen der Framerate aus der info-Datei und der Versuch das umzurechnen, um den i-Frame besser zu treffen - oder gleich zu runden, denn normalerweise mach es sich besser da vor einem Werbeblock ca. 1 Sekunde dazuzugeben und ca. 2s vor dem von Markad angegebenen Ende des Werbeblocks weiterzumachen...

seahawk1986 · 16. Dezember 2012

Wie wäre es so - oder gibt es was besseres um näher an den Zeitcode der i-Frames zu kommen?:

Code

def edl_content(self):
        if not self._edl_content:
            self._edl_content = '00:00:00.000 99:99:99.99 2'
            info_path = self.path + '/info'
            if not os.path.exists(info_path):
                info_path = self.path + '/info.vdr'
                if not os.path.exists(info_path):
                    info_path = None
            info_vdr = InfoVdr(info_path)
            marks_path = self.path + '/marks'
            if not os.path.exists(marks_path):
                marks_path = self.path + '/marks.vdr'
                if not os.path.exists(marks_path):
                    marks_path = None
            if marks_path != None:
                with open(marks_path,'r') as marks_vdr:
                    marks = marks_vdr.readlines()
                matches = []
                for mark in marks:
                    a = self.timecode.search(mark)
                    if a:
                        tc = a.group(0)
                        hms = tc[:-2]
                        ff = tc[-2:]
                        try:
                            tcms = 1000* int(ff)/int(info_vdr['F'])
                            matches.append("%s%3d"%(hms,int(tcms)))
                        except:
                            matches.append(a.group(0))


                if self.nullcode.match(matches[0]):
                    # starts at 0
                    start = 0
                else:
                    matches.insert(0,"00:00:00.000")
                    start = 1
                matches.append("99:99:99.99")
                edl_list = []
                for i in xrange(0,len(matches)-1,1):
                    if i % 2 == 0:
                        if start == 1:
                            mode = "3"
                        else:
                            mode = "2"
                        edl_list.append("%s %s %s"%(matches[i],matches[i+1],mode))
                    else:
                        if start == 1:
                            mode = "2"
                        else:
                            mode = "3"
                        edl_list.append("%s %s %s"%(matches[i],matches[i+1],mode))
                self._edl_content = "\n".join(edl_list)
        return self._edl_content

Alles anzeigen

tag · 24. Dezember 2012

Zitat von seahawk1986

Wie wäre es so - oder gibt es was besseres um näher an den Zeitcode der i-Frames zu kommen?:

Braucht man den Zeitcode überhaupt? Im XBMC-Wiki steht nämlich, dass man Frames auch direkt angeben kann. Ich weiß aber nicht, ob die bei #0 oder #1 anfangen.

Zitat von seahawk1986

Code

tc = a.group(0)
                        hms = tc[:-2]
                        ff = tc[-2:]

Ziemlich fragil, wenn du mich fragst. Ich würde eher den regulären Ausdruck um ein paar Gruppen erweitern. Also zum Beispiel:

Code

MARKS_FORMAT = re.compile(r'^((\d+):(\d+):(\d+)\.?)?(\d*[1-9]\d*)?')
def mark_to_frame(mark, fps):
    m = MARKS_FORMAT.match(mark)
    if not m or not m.group():
        return None
    frame = int(m.group(5)) - 1 if m.group(5) else 0
    if m.group(1):
        hms = map(int, m.group(2, 3, 4))
        frame += (hms[0] * 3600 + hms[1] * 60 + hms[2]) * fps
    return frame

Bei der Frame-Rate musst du aufpassen, weil die bei alten Aufnahmen nicht in der info.vdr steht, sondern fest auf 25,0 gesetzt ist, wenn ich mich nicht irre.

seahawk1986 · 24. Dezember 2012

Zitat von tag

Braucht man den Zeitcode überhaupt?

Die Frage ist halt ob man sich einen Gefallen damit tut die Frames auszurechnen statt hauptsächlich einen String zu kopieren - da wäre man von den Angaben in der info-Datei bis auf die Frames nach dem Timestamp unabhängig (dass die FPS nicht in der info-Datei stehen war AFAIK nur bei der summary.vdr der Fall, die unterstützt der Parser für die Info-Dateien von vdrnfofs aber sowieso nicht).

Wiki und Manpage liefern interessanterweise unterschiedliche Angaben zum Format für die Schnittmarken (lustig, dass das seit 2007 niemandem aufgefallen ist): http://www.vdr-wiki.de/wiki/index.php/Dateiformat

Zitat

marks.vdr bzw. marks enthält die Schnittmarken. Es ist eine ASCII-kodierte Textdatei. In ungeraden Zeilen steht der Zeitpunkt einer Startmarke, in der darauf folgenden Zeile der der Endmarke. Das Format des Zeitpunktes ist h:mm:ss.HH mit h=Stunde, mm=Minute, ss=Sekunde und HH=Hundertstel. Beispiel: 1:03:43.22

Während vdr(5) das sagt:

Zitat

The file marks.vdr (if present in a recording directory) contains the editing marks defined for this recording. Each line contains the definition of one mark in the following format:
hh:mm:ss.ff comment
where hh:mm:ss.ff is a frame position within the recording, given as "hours, minutes, seconds and (optional) frame number". comment can be any string and may be used to describe this mark. If present, comment must be separated from the frame position by at least one blank.

tag · 27. Dezember 2012

Zitat von seahawk1986

Die Frage ist halt ob man sich einen Gefallen damit tut die Frames auszurechnen statt hauptsächlich einen String zu kopieren - da wäre man von den Angaben in der info-Datei bis auf die Frames nach dem Timestamp unabhängig

Dann teilt man eben die errechneten Frames nochmal durch fps und gibt Sekunden aus. Problem gelöst.

Ich habe sicherheitshalber nochmal direkt im VDR-Quelltext nachgeschaut (HMSFToIndex aus recording.c). Und zwar ist sowohl die Zeitangabe als auch der Frame optional. (Den Code aus meinem letzten Beitrag habe ich entsprechend geändert.) Also egal wofür man sich letztendlich entscheidet, irgendetwas muss man immer umrechnen.
Umzurechnen hat aber auch Vorteile. Will man z.B. wissen, ob eine Marke am Anfang ist, muss man lediglich testen, ob der Frame == 0 ist, und braucht keine regulären Ausdrücke dafür.

Zitat von seahawk1986

(dass die FPS nicht in der info-Datei stehen war AFAIK nur bei der summary.vdr der Fall, die unterstützt der Parser für die Info-Dateien von vdrnfofs aber sowieso nicht).

Ich meine die info.vdr von den "alten" Aufnahmen im PS-Format. Da steht definitiv keine Frame-Rate drin.

kris · 27. Dezember 2012

Hi,

coole sache mit dem EDL. Ich habe basierenden auf den aktuellen Erkenntnissen mal ein (D)Patch gebastelt, damit man sich nicht alles zusammensuchen muss. Dieser (D)Patch ist gegen das YaVDR-Repo erstellt worden und eben auch damit kompatibel.

MfG
KRis

Patch zu vdrnfofs mit edl support

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema