Aufzeichnungslänge im Display falsch (IMON-Plugin)

P3f · 12. Januar 2011

Hallo Zusammen,

ich setze in meinen Antec Fusion Gehäuse das iMON LCD ein und bin begeistert von dem vdr-plugin-imonlcd, welches im GIT-Repository von VDR-Developer vorhanden ist. (http://projects.vdr-developer.org/wiki/plg-imonlcd)

Nun ist es ja so, das wir langsam stabil auf HD umstellen, und auch da hat sich ja viel Gutes getan. Zum Thema Berechnung der Aufzeichnunslänge sind ja einige Aktivitäten gelaufen und auch seitens KLS bereits Patche gegen 1.7.16 bereitgestellt worden. (Thread). Auch wurde bereits ein Patch für das extrecmenu (der bereits im GIT nun enthalten ist!) erstellt (selber Thread, oben).

Es ist mir heute aufgefallen, das bei HD-Aufnahmen im Display die falsche Aufnahmenlänge angezeigt wird. Eine Aufnahme die laut VDR-Menu korrekt 38 Minuten anzeigt, zeigt bei Wiedergabe im Display 76 Minuten an. Ebenso wird die Laufzeit in 2-3 Sekundenschritte erhöht.

Denke, das im IMON-Plugin also die Zeitberechnung angepasst werden muß. Nur bin ich leider kein Programmierer und finde nicht die korrekte Stelle bzw. den richtigen Rechnungsfaktor. IMHO nach sollte in der /usr/local/src/VDR/PLUGINS/src/imonlcd/watch.c ab Zeile 758 die Berechnungsänderung [Motto: Recording->FramesPerSecond()] sein.

Code

const char * ciMonWatch::FormatReplayTime(int current, int total) const
{
     static char s[32]; 
#if VDRVERSNUM >= 10701
     int cs = (current / DEFAULTFRAMESPERSECOND);
     int ts = (total / DEFAULTFRAMESPERSECOND);
     bool g = (((current / DEFAULTFRAMESPERSECOND) / 3600) > 0)
                  || (((total / DEFAULTFRAMESPERSECOND) / 3600) > 0);
#else     int cs = (current / FRAMESPERSEC);
     int ts = (total / FRAMESPERSEC);
     bool g = (((current / FRAMESPERSEC) / 3600) > 0)
             || (((total / FRAMESPERSEC) / 3600) > 0);
#endif

Alles anzeigen

Kann sich jemand das mal anschauen? Soll ich im Bugtracker von vdr-developer einen Eintrag machen?

Danke für Eure Hilfe,

Pit

TheChief · 12. Januar 2011

zum Thema Aufzeichnungslänge gabs schon mal eienn Thread, da ging es aber um den VDR selbst. Ich glaube da wurde "DEFAULTFRAMESPERSECOND" gegen was anderes ersetzt. Muss mal gucken, ob ich den Fred finde.

FireFly · 12. Januar 2011

Du bist auf dem richtigen Weg: im Prinzip muss nur das DEFAULTFRAMESPERSEC durch Recording->FramesPerSecond() ersetzt werden - aber evtl. steht die Variable Recording (also ein Member der Klasse cRecording, wie auch immer das heißt) in der Klasse ciMonWatch nicht zur Verfügung. Man könnte das dann auch noch als Parameter übergeben

P3f · 12. Januar 2011

Hallo Firefly,

das mit dem "stumpfen" ersetzen hatte ich auch gedacht und bereits versucht. Leider ist dein 2. Satz (Vermutung) richtig, denn das Plugin kompiliert nicht mehr.

Da ich mir aber nicht sicher bin/war, habe ich hier den Patch-Hilfe-Gesuch gestellt.

Gruß,

Pit

anbr · 13. Januar 2011

Wie wäre es mit folgendem Patch :

Diff

diff --git a/watch.c b/watch.c
index 5806871..4e96c85 100644
--- a/watch.c
+++ b/watch.c
@@ -284,10 +284,7 @@ void ciMonWatch::Action(void)
           if(m_eWatchMode != eLiveTV) {
             current = 0;
             total = 0;
-      	    if (ReplayPosition(current,total)
-                 && theSetup.m_nRenderMode == eRenderMode_DualLine) {
-              bReDraw |= ReplayTime(current,total);
-            }
+            bReDraw |= ReplayTime(current,total);
           }
         }
 
@@ -741,29 +738,26 @@ eReplayState ciMonWatch::ReplayMode() const
   return eReplayNone;
 }
 
-bool ciMonWatch::ReplayPosition(int &current, int &total) const
+bool ciMonWatch::ReplayPosition(int &current, int &total, double& dFrameRate) const
 {
   if (m_pControl && ((cControl *)m_pControl)->GetIndex(current, total, false)) {
     total = (total == 0) ? 1 : total;
+#if VDRVERSNUM >= 10703
+    dFrameRate = ((cControl *)m_pControl)->FramesPerSecond();
+#endif
     return true;
   }
   return false;
 }
 
-const char * ciMonWatch::FormatReplayTime(int current, int total) const
+const char * ciMonWatch::FormatReplayTime(int current, int total, double dFrameRate) const
 {
     static char s[32];
-#if VDRVERSNUM >= 10701
-    int cs = (current / DEFAULTFRAMESPERSECOND);
-    int ts = (total / DEFAULTFRAMESPERSECOND);
-    bool g = (((current / DEFAULTFRAMESPERSECOND) / 3600) > 0)
-            || (((total / DEFAULTFRAMESPERSECOND) / 3600) > 0);
-#else
-    int cs = (current / FRAMESPERSEC);
-    int ts = (total / FRAMESPERSEC);
-    bool g = (((current / FRAMESPERSEC) / 3600) > 0)
-            || (((total / FRAMESPERSEC) / 3600) > 0);
-#endif
+
+    int cs = (int)((double)current / dFrameRate);
+    int ts = (int)((double)total / dFrameRate);
+    bool g = ((cs / 3600) > 0) || ((ts / 3600) > 0);
+
     int cm = cs / 60;
     cs %= 60;
     int tm = ts / 60;
@@ -786,13 +780,22 @@ const char * ciMonWatch::FormatReplayTime(int current, int total) const
     return s;
 }
 
-bool ciMonWatch::ReplayTime(int current, int total) {
-    const char * sz = FormatReplayTime(current,total);
-    if(!replayTime || strcmp(sz,*replayTime)) {
-      if(replayTime)
-        delete replayTime;
-      replayTime = new cString(sz);
-      return replayTime != NULL;      
+bool ciMonWatch::ReplayTime(int &current, int &total) {
+    double dFrameRate;
+#if VDRVERSNUM >= 10701
+    dFrameRate = DEFAULTFRAMESPERSECOND;
+#else
+    dFrameRate = FRAMESPERSEC;
+#endif
+    if(ReplayPosition(current,total,dFrameRate) 
+      && theSetup.m_nRenderMode == eRenderMode_DualLine) {
+      const char * sz = FormatReplayTime(current,total,dFrameRate);
+      if(!replayTime || strcmp(sz,*replayTime)) {
+        if(replayTime)
+          delete replayTime;
+        replayTime = new cString(sz);
+        return replayTime != NULL;      
+      }
     }
     return false;
 }
diff --git a/watch.h b/watch.h
index 54e3f5b..2b1f61b 100644
--- a/watch.h
+++ b/watch.h
@@ -128,10 +128,10 @@ protected:
   bool Replay();
   bool RenderScreen(bool bRedraw);
   eReplayState ReplayMode() const;
-  bool ReplayPosition(int &current, int &total) const;
+  bool ReplayPosition(int &current, int &total, double& dFrameRate) const;
   bool CurrentTime();
-  bool ReplayTime(int current, int total);
-  const char * FormatReplayTime(int current, int total) const;
+  bool ReplayTime(int& current, int& total);
+  const char * FormatReplayTime(int current, int total, double dFrameRate) const;
 public:
   ciMonWatch();
   virtual ~ciMonWatch();

Alles anzeigen

P3f · 13. Januar 2011

Hallo Hulk,

sieht gut aus mit dem Patch. Die Anzeige ist nun für meine HD-Aufnahmen auch im Display korrekt.

Nebenbei hast Du ja auch noch die Funktionalität eingebaut, das die aktuelle Uhrzeit im Display steht. Gefällt mir gut....

Wenn noch jemand den Patch erfolgreich testet, würde ich mich freuen wenn dieser Patch ins GIT einfließt.

Danke und Gruß,

Pit

P3f · 18. Januar 2011

Hallo Hulk,

jetzt habe ich doch noch weitere Tests gemacht und bei manchen HD-Aufnahmen rechnet er (immer noch) falsch. Also das alte....Doppelte Gesamtspielzeit und es springen immer 3 Sekunden bei der aktuellen Abspielzeit.

Nachdem das ja im Logfile nicht mitgeloggt wird, weiß ich nicht welche Information Du benötigst. Liest Du die Berechnungsgrundlagen aus der "info.vdr" Datei?

Dann würde ich Dir jeweils ein funktionierende bzw. nicht funktionierende Datei geben können. Auffällig ist nur das die HD-Aufnahme auf einem FTA-Sender (z.B. ARD HD) funktioniert, andere nicht....

GRuß,

Pit

FireFly · 18. Januar 2011

Wird die korrekte Länge angezeigt, nachdem der VDR durchgestartet wurde? Und die Länge ist falsch, wenn der VDR die Aufnahme grade gemacht hat? Dann liegt das am VDR und da gibt es hier im Forum einen Patch dafür (ca. 3 Wochen alt, ich weiß aber nicht mehr in welchem Thread das war :()

Edit: der Patch von kls dazu heißt vdr-1.7.16-recordinginfo.diff

P3f · 18. Januar 2011

Hallo Firefly,

nein das hat nichts mit dem VDR-Patch zu tun. Die Anzeige im Aufzeichnungsmenü ist korrekt. Auch ein Neustart verändert nichts an der Displayanzeige (Berechnung im Plugin!!!)

Die Patche sind bei meinem VDR mit includiert. Du meintest bestimmt diesen Thread, wo KLS bereits 2 Patche geliefert hat, Oder?

Gruß,

Pit

anbr · 18. Januar 2011

Sorry, ich sehe keine logisch Ansatz dein Problem auf den Plugincode des Displaytreiber zu reproduzieren.

Bisher wurde eine feste Framerate verwendet, und jetzt wird zyklisch vom aktivem Replaycontrol (cControl) die aktuelle Framerate abgefragt. Ob das Replaycontrol einen korrekten Wert hat kann nur der jeweilige Player wissen.

Alles andere wäre keine saubere Aufgabentrennung.

P3f · 18. Januar 2011

Hi Hulk,

wahrscheinlich verstehe ich deine Antwort nicht. Es wird immer der gleiche (sogar der selbe!) Player verwendet.

Also, gibt es vielleicht noch weitere Berechnungen im "watch.c", die man anpassen müsste? Was mir halt nicht klar ist das die eine funktioniert, die andere nicht.

Habe jetzt sogar eine HD-Aufnahme gefunden, die falsch ist obwohl auf dem selber Sender aufgenommen (ARD HD). Ich hänge nun mal die beiden info-Dateien aus den Aufnahmeverzeichnissen an.

Bisher dachte ich nun, das der Framedetect aus dieser Datei ausgelesen wird und danach die Zeitberechnung stattfindet.

Vielleicht bin ich ja auch auf dem Holzweg ...

Pit

iNOB · 19. Januar 2011

Hast du das markad Plugin laufen? Wenn ja dann bitte updaten auf die letzte Git-Version. Ansonsten wird das nix mit korrekten Aufzeichnugslängen

Gruß
iNOB

P3f · 19. Januar 2011

Hallo iNOB,

ich bin mal deinem Vorschlag gefolgt, aber auch nachdem ich das GIT-Reposity neu geladen habe und dann kompiliert. Sicherheitshalber habe ich auch die Indexdatei neu erzeugen lassen...

Aber leider bewirkt das auch keine korrekte Anzeige im Display. Im VDR-Aufzeichnungsmenu (mit und auch ohne extrecmenu-Patch) wird die korrekte Länge angezeigt.

Also ich vermute immer noch den Fehler im IMONLCD-Plugin. Was auffällig ist, das die HD-Aufnahme > 1 Stunde falsch angezeigt werden. Das wäre meine momentane Vermutung bzw. ist es die einzige Gemeinsamkeit.

Gruß,

P.S: Vor dieser Umstellung hatte ich NOAD angewendet, bin aber nun auf MARKAD umgestiegen, wie von Dir vorgeschlagen. Mal schauen, ob nun auch die Marken einigermaßen sauber gesetzt werden.

Aufzeichnungslänge im Display falsch (IMON-Plugin)

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema