Anzeige der verfügbaren Aufnahmezeit (extrecmenu, live, extpatch)

nvertigo · 6. Dezember 2012

Hallo,

mich störte es schon eine ganze Zeit, dass der vdr (erste OSD-Seite), exrecmenu, live und bestimmt auch noch viele andere, unterschiedliche Mumaßungen über die verbleibende Aufnahmezeit anstellen. Jetzt habe ich mir das mal angesehen. Im vdr haben wir seit (hier brauche ich mal Hilfe, ich bin zu blöd das im HISTORY-File zu finden, obwohl ich sicher bin es vor nicht all zu langer Zeit gelsenen zu haben), die wundervolle Funktion cVideoDiskUsage::FreeMinutes(). kls rechnet sich hier den Wolf, um halbwegs sinnvoll zu raten. Und die Zeit, die hier als int zurück kommt, wird auch im OSD auf der ersten Seite angezeigt.

Ich habe das mal in live und extrecmenu angepasst und habe jetzt überall die gleiche Restzeit. Bei extrecmenu habe ich das Problem, dass ich - wie gesagt - nicht finde, seit wann man diese Funktion verwenden kann, deshalb habe ich - bestimmt fälschlicher Weise - es einfach in den bestehenden konditionalen Code (VDRVERSNUM >= 10727) reingebaut.

Außerdem habe ich noch was im extpatch gefunden. TIMERINFO enthält an der Stelle auch noch veralteten Code. ( helau: ich hoffe Du fühlst Dich zuständig?)

Aber der Reihe nach. live:

Diff

--- pages/recordings.ecpp.orig  2012-12-06 12:39:24.477030379 +0100
+++ pages/recordings.ecpp       2012-12-06 12:42:25.820871277 +0100
@@ -72,7 +72,8 @@


 int FreeMB, UsedMB;
 int Percent = VideoDiskSpace(&FreeMB, &UsedMB);
-int Minutes = int(double(FreeMB) / MB_PER_MINUTE);
+//int Minutes = int(double(FreeMB) / MB_PER_MINUTE);
+int Minutes = cVideoDiskUsage::FreeMinutes();
 int Hours = Minutes / 60;
 Minutes %= 60;
 diskinfo = cString::sprintf("%s %d%%  -  %2d:%02d %s", trVDR("Disk"), Percent, Hours, Minutes, trVDR("free"));

Alles anzeigen

extrecmenu:

Diff

--- mymenurecordings.c.orig     2012-12-06 12:46:37.780639140 +0100
+++ mymenurecordings.c  2012-12-06 13:28:00.519006663 +0100
@@ -650,11 +650,11 @@


 void myMenuRecordings::Title()
 {
-  int freemb=FreeMB();
 #if VDRVERSNUM >= 10727
-  int MBperMinute = Recordings.MBperMinute();
-  int minutes=int(double(freemb)/(MBperMinute>0?MBperMinute:MB_PER_MINUTE));
+//  int MBperMinute = Recordings.MBperMinute();
+  int minutes=cVideoDiskUsage::FreeMinutes();
 #else
+  int freemb=FreeMB();
   int minutes=int(double(freemb)/MB_PER_MINUTE);
 #endif

Alles anzeigen

In menu.c nach anwenden des extpatch von helau:

Diff

--- menu.c.orig 2012-12-06 14:04:24.989429215 +0100
+++ menu.c      2012-12-06 14:06:42.149330922 +0100
@@ -1318,16 +1318,17 @@
 }


 #ifdef USE_TIMERINFO
-#define MB_PER_MINUTE 25.75 // this is just an estimate!
+//#define MB_PER_MINUTE 25.75 // this is just an estimate!
 void cMenuTimers::ActualiseDiskStatus(void)
 {
   if (!actualiseDiskStatus || !Count())
      return;


   // compute free disk space
-  int freeMB, freeMinutes, runshortMinutes;
-  VideoDiskSpace(&freeMB);
-  freeMinutes = int(double(freeMB) * 1.1 / MB_PER_MINUTE); // overestimate by 10 percent
+  int freeMinutes, runshortMinutes;
+//  VideoDiskSpace(&freeMB);
+//  freeMinutes = int(double(freeMB) * 1.1 / MB_PER_MINUTE); // overestimate by 10 percent
+  freeMinutes = cVideoDiskUsage::FreeMinutes();
   runshortMinutes = freeMinutes / 5; // 20 Percent


   // fill entries list

Alles anzeigen

BTW: Ich finde cVideoDiskUsage::FreeMinutes() rät wesentlci begründeter, als die #define MB_PER_MINUTE 25.75 (ja ich weiss: this is just an estimate!). Gerade auch die kleinen Dinge werden immer schöner - danke, Klaus.

Gruß, Ingo

Keine_Ahnung · 6. Dezember 2012

Zitat von nvertigo

deshalb habe ich - bestimmt fälschlicher Weise - es einfach in den bestehenden konditionalen Code (VDRVERSNUM >= 10727) reingebaut.

Code

2012-03-25: Version 1.7.27


- When estimating the remaining disk space (in hours), the average data rate of all
  existing recordings is now taken into account. If this value can't be determined,
  the previous value of 25.75 MB/min is taken.

Also gut geraten

Edit: Doch nicht FreeMinutes() gibts erst ab 28

cu

nvertigo · 6. Dezember 2012

Jau, danke fürs Vorlesen...

Dann könnte man es so machen:

Diff

--- mymenurecordings.c.orig     2012-12-06 12:46:37.780639140 +0100
+++ mymenurecordings.c  2012-12-06 14:49:22.547602735 +0100
@@ -650,14 +650,17 @@


 void myMenuRecordings::Title()
 {
+#ifdef VDRVERSNUM >= 10728
+  int minutes=cVideoDiskUsage::FreeMinutes();
+#else
   int freemb=FreeMB();
-#if VDRVERSNUM >= 10727
+#if VDRVERSNUM == 10727
   int MBperMinute = Recordings.MBperMinute();
   int minutes=int(double(freemb)/(MBperMinute>0?MBperMinute:MB_PER_MINUTE));
 #else
   int minutes=int(double(freemb)/MB_PER_MINUTE);
 #endif
-
+#endif
   stringstream buffer;
   if(MoveCutterThread->IsMoveListEmpty())
     buffer << Icons::MovingRecording();

Alles anzeigen

iNOB · 6. Dezember 2012

Im gepatchten Live Plugin zeigt er mir erst dann die richtige Zeit an (>0:00), wenn ich vorher mit der FB mindestens einmal im Aufnahmenverzeichnis war.

Gruß
iNOB

nvertigo · 6. Dezember 2012

...jetzt weißt Du in welcher Reihenfolge ich getestet habe. Gucke ich mir an. Danke.

Gruß, Ingo

nvertigo · 6. Dezember 2012

Hi iNOB,

danke für den Tip. Jetzt datet extrecmenu auch während des Schneidens up... Bei live müßte mal jemand wegen der VDRVERSNUMs gucken. Nach dem Patch wird das nicht mehr mit vor 1.7.28 bauen.

live:

Diff

diff --git a/pages/recordings.ecpp b/pages/recordings.ecpp
index ebd57e0..17634a2 100644
--- a/pages/recordings.ecpp
+++ b/pages/recordings.ecpp
@@ -17,7 +17,7 @@


 #include "recman.h"


-#define MB_PER_MINUTE 25.75 // this is just an estimate!
+#define MB_PER_MINUTE 50.75 // this is just an estimate!


 using namespace vdrlive;
 using namespace std;
@@ -72,7 +72,14 @@ deletions.clear();


 int FreeMB, UsedMB;
 int Percent = VideoDiskSpace(&FreeMB, &UsedMB);
+#if VDRVERSNUM >= 10728
+int mylastDiskUsageState = -1;
+cVideoDiskUsage::ForceCheck();
+cVideoDiskUsage::HasChanged(mylastDiskUsageState);
+int Minutes = cVideoDiskUsage::FreeMinutes();
+#else
 int Minutes = int(double(FreeMB) / MB_PER_MINUTE);
+#endif
 int Hours = Minutes / 60;
 Minutes %= 60;
 diskinfo = cString::sprintf("%s %d%%  -  %2d:%02d %s", trVDR("Disk"), Percent, Hours, Minutes, trVDR("free"));

Alles anzeigen

exrecmenu:

Diff

diff --git a/mymenurecordings.c b/mymenurecordings.c
index 0a08d12..048fee8 100644
--- a/mymenurecordings.c
+++ b/mymenurecordings.c
@@ -650,14 +650,14 @@ int myMenuRecordings::FreeMB()


 void myMenuRecordings::Title()
 {
+#if VDRVERSNUM == 10727
   int freemb=FreeMB();
-#if VDRVERSNUM >= 10727
   int MBperMinute = Recordings.MBperMinute();
   int minutes=int(double(freemb)/(MBperMinute>0?MBperMinute:MB_PER_MINUTE));
-#else
+#elif VDRVERSNUM <= 10726
+  int freemb=FreeMB();
   int minutes=int(double(freemb)/MB_PER_MINUTE);
 #endif
-
   stringstream buffer;
   if(MoveCutterThread->IsMoveListEmpty())
     buffer << Icons::MovingRecording();
@@ -673,9 +673,23 @@ void myMenuRecordings::Title()
   else
     buffer << trVDR("Recordings");


+#if VDRVERSNUM >= 10728
+  int mylastDiskUsageState = -1;
+  cVideoDiskUsage::ForceCheck();
+  cVideoDiskUsage::HasChanged(mylastDiskUsageState);
+#endif
+
   buffer << " ("
+#if VDRVERSNUM >= 10728
+         << cVideoDiskUsage::FreeMinutes()/60 << ":"
+#else
          << minutes/60 << ":"
+#endif
+#if VDRVERSNUM >= 10728
+         << setw(2) << setfill('0') << cVideoDiskUsage::FreeMinutes()%60 << " "
+#else
          << setw(2) << setfill('0') << minutes%60 << " "
+#endif
          << trVDR("free")
          << ")";

Alles anzeigen

Gruß, Ingo

iNOB · 6. Dezember 2012

Fein. Jetzt funktionierts.

Thx
iNOB

nvertigo · 6. Dezember 2012

Wie gesagt, danke für den Tip. Wäre mir wahrscheinlich erst aufgefallen, wenn ich mich gar nicht mehr an die "kleine" Änderung erinnert hätte, und dann hätte ich mich tot gesucht...

Ich möchte hier mal eine Bitte an alle Plugin-Maintainer und Patch-Maintainer richten: Wenn Ihr irgendwas mit dem Plattenplatz berechnet, und vor allem immer!!! wenn Ihr in Euren Code die Konstante MB_PER_MIN benutzt, dann - bitte,bitte,bitte,bitte - überprüft, ob man das nicht mit den schicken neuen Features ab 1.7.28 einfacher und richtiger gestalten kann. Danke.

Gruß, Ingo

Anzeige der verfügbaren Aufnahmezeit (extrecmenu, live, extpatch)

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema