changeset 16473:d9697b778be2

Nits suggested by Diego
author gpoirier
date Tue, 13 Sep 2005 11:38:39 +0000
parents ae74a32ca5d9
children 01d27e023ae4
files TOOLS/README TOOLS/psnr-video.sh
diffstat 2 files changed, 38 insertions(+), 38 deletions(-) [+]
line wrap: on
line diff
--- a/TOOLS/README	Tue Sep 13 11:04:20 2005 +0000
+++ b/TOOLS/README	Tue Sep 13 11:38:39 2005 +0000
@@ -260,15 +260,15 @@
 
 Description:  Calculates the PSNR between two existing video files.
               The PSNR is calculated frame by frame.
-              Prints also the overall PSNR.
+              Also prints the overall PSNR.
               The script can be used to:
-               * compare different softwarescalers (should I use
-                 -sws 1 or -sws 2) ?
-               * compare different resolutions (is it better to scale
+               * Compare different softwarescalers (should I use
+                 -sws 1 or -sws 2 ?)
+               * Compare different resolutions (is it better to scale
                  down to 640x360 or to 560x320)
-               * compare different deinterlacers
-               * compare different video codecs
-               * compare video filters (is it better to use -vf hqdn3d
+               * Compare different deinterlacers
+               * Compare different video codecs
+               * Compare video filters (is it better to use -vf hqdn3d
                  or lavcopts:nr=400)
                * [...]
 
@@ -281,7 +281,7 @@
               [<options1>] are additional MPlayer options for <file1>
               [<options2>] are additional MPlayer options for <file2>
 
-              A file called psnr.dat will be created with the following
+              A file called ./psnr.dat will be created with the following
               content:
 
               File;Y;Cb;Cr
@@ -295,7 +295,7 @@
 
 Note:         This script relies on the the tool "pnmpsnr" for the
               frame-by-frame PSNR calculation.
-              Be aware that psnr-video.sh needs a lot of temporal space
+              Be aware that psnr-video.sh needs a lot of temporary space
               inside /temp/.
 
 
--- a/TOOLS/psnr-video.sh	Tue Sep 13 11:04:20 2005 +0000
+++ b/TOOLS/psnr-video.sh	Tue Sep 13 11:38:39 2005 +0000
@@ -1,8 +1,8 @@
 #!/bin/sh
-# Helper script to ease comparing two video files
-# Copyleft 2001 by Matthias Wieser
+# Helper script to ease comparing the PSNR between two video files
+# Copyleft 2005 by Matthias Wieser
 # This file comes under GPL, see http://www.gnu.org/copyleft/gpl.html for more
-# information on it's licensing.
+# information on its licensing.
 
 TEMPDIR="/tmp/psnr_video"
 WORKDIR=`pwd`/
@@ -10,12 +10,12 @@
    echo
    echo "Usage: `basename $0` <file1> <file2> [<frames>] [<options1>] [<options2>]"
    echo
-   echo " <file1> and <file2> are the video files for which the PSNR should be calculated."
+   echo " <file1> and <file2> are the files for which the PSNR should be calculated."
    echo " [<frames>]          is the number of frames to process, starting from frame 1."
-   echo " [<options1>]        are additional MPlayer options for <file1>"
-   echo " [<options2>]        are additional MPlayer options for <file2>"
+   echo " [<options1>]        are additional MPlayer options for <file1>."
+   echo " [<options2>]        are additional MPlayer options for <file2>."
    echo
-   echo " Be aware that `basename $0` needs a lot of temporal space inside /temp/."
+   echo " Be aware that `basename $0` needs a lot of temporary space inside /tmp/."
    echo
    echo "Example:"
    echo "        ./`basename $0` ./orig.avi ./test.avi 250 \"\" \"-vf pp=ac\""
@@ -31,17 +31,17 @@
 if [ $# -ge 3 ]; then
 	LastFrame=$3
 	echo
-	echo "Will process $LastFrame frames"
+	echo "Will process $LastFrame frames."
 fi
 
 if [ $# -ge 4 ]; then
-	FILE1Opts=$4
-	echo "Mplayer options for ${FILE1}: $FILE1Opts"
+	FILE1_Options=$4
+	echo "MPlayer options for ${FILE1}: $FILE1_Options"
 fi
 
 if [ $# -ge 5 ]; then
-	FILE2Opts=$5
-	echo "Mplayer options for ${FILE2}: $FILE2Opts"
+	FILE2_Options=$5
+	echo "Mplayer options for ${FILE2}: $FILE2_Options"
 fi
 
 mkdir -p ${TEMPDIR}/FILE1
@@ -53,13 +53,13 @@
 
 cd ${TEMPDIR}/FILE1
 
-rm *ppm 2> /dev/null
-rm *del 2> /dev/null
+rm -f *ppm
+rm -f *del
 
 if [ $LastFrame -ge 0 ]; then
-	mplayer $FILE1Opts -frames $LastFrame -nosound -vo pnm ${WORKDIR}$FILE1 >/dev/null
+	mplayer $FILE1_Options -frames $LastFrame -nosound -vo pnm ${WORKDIR}$FILE1 >/dev/null
 else
-	mplayer $FILE1Opts -nosound -vo pnm ${WORKDIR}$FILE1 >/dev/null
+	mplayer $FILE1_Options -nosound -vo pnm ${WORKDIR}$FILE1 > /dev/null
 fi
 ###  File 2
 
@@ -71,25 +71,25 @@
 rm *ppm 2> /dev/null
 
 if [ $LastFrame -ge 0 ]; then
-	mplayer $FILE2Opts -frames $LastFrame -nosound -vo pnm ${WORKDIR}$FILE2 >/dev/null
+	mplayer $FILE2_Options -frames $LastFrame -nosound -vo pnm ${WORKDIR}$FILE2 >/dev/null
 else
-	mplayer $FILE2Opts -nosound -vo pnm ${WORKDIR}$FILE2 >/dev/null
+	mplayer $FILE2_Options -nosound -vo pnm ${WORKDIR}$FILE2 >/dev/null
 fi
 
 
 ###  PSNR
 
 echo
-echo "############## Calculation PSNR #################"
+echo "############## PSNR Calculation #################"
 
-if [[ `ls -1 ${TEMPDIR}/FILE1/*ppm|wc -l` = `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` ]]
+if [[ `ls -1 ${TEMPDIR}/FILE1/*ppm | wc -l` = `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` ]]
 then
 	echo 
 else
 	echo "Files have differing numbers of frames!"
-	echo "$FILE1 has `ls -1 ${TEMPDIR}/FILE1/*ppm|wc -l` frames,"
-	echo "$FILE2 has `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` frames."
-	echo "Processing the first `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` frames."
+	echo "$FILE1 has `ls -1 ${TEMPDIR}/FILE1/*ppm | wc -l` frames,"
+	echo "$FILE2 has `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` frames."
+	echo "Processing the first `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` frames."
 	echo
 fi
 
@@ -118,7 +118,7 @@
          echo "$ALL">>../psnr.dat
         ERROR=`echo "scale=30; (e(-1*$Y/10*l(10))+e(-1*$CB/10*l(10))/4+e(-1*$CR/10*l(10))/4)/1.5"|bc -l`
         ERRORSUM=`cat errorsum.del`
-        echo `echo "scale=30; $ERROR + $ERRORSUM"|bc -l` > errorsum.del
+        echo `echo "scale=30; $ERROR + $ERRORSUM" | bc -l` > errorsum.del
     i=$(($i+1))
 	if [[ $i = $LastFrame ]]
 	then
@@ -126,19 +126,19 @@
 	fi
 done
 ERRORSUM=`cat errorsum.del`
-PSNR=`echo "-10*l($ERRORSUM/$i)/l(10)"|bc -l`
+PSNR=`echo "-10*l($ERRORSUM/$i)/l(10)" | bc -l`
 echo "PSNR:;$PSNR">>../psnr.dat
 cd ..
 mv psnr.dat ${WORKDIR}
 
-if [[ `ls -1 ${TEMPDIR}/FILE1/*ppm|wc -l` = `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` ]]
+if [[ `ls -1 ${TEMPDIR}/FILE1/*ppm | wc -l` = `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` ]]
 then
         echo
 else
         echo "Files have differing numbers of frames!"
-        echo "$FILE1 has `ls -1 ${TEMPDIR}/FILE1/*ppm|wc -l` frames,"
-        echo "$FILE2 has `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` frames."
-        echo "Processed the first `ls -1 ${TEMPDIR}/FILE2/*ppm|wc -l` frames."
+        echo "$FILE1 has `ls -1 ${TEMPDIR}/FILE1/*ppm | wc -l` frames,"
+        echo "$FILE2 has `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` frames."
+        echo "Processed the first `ls -1 ${TEMPDIR}/FILE2/*ppm | wc -l` frames."
         echo
 fi