Changeset 5357

Timestamp:

11/09/04 15:07:50 (20 years ago)

Author:

gaug

Message:

*** empty log message ***

Location:

trunk/MagicSoft/Mars

Files:

• Changelog (modified) (1 diff)
• mpedestal/MPedCalcPedRun.cc (modified) (1 diff)

trunk/MagicSoft/Mars/Changelog

@@ -20 +20 @@
 
                                                  -*-*- END OF LINE -*-*-
+
+ 2004/11/08: Markus Gaug
+
+   * mpedestal/MPedCalcFromLoGain.[h,cc]
+     - corrected the documentation of the indiv. functions
+     - some cosmetic changes
+
+   * mpedestal/MPedCalcPedRun.cc
+     - previously the variable fNumLoGainSize was automatically set to
+       2 in case that it was initialized to 0. This is a bugfix, since 
+       the default uses 0 slices (see e.g. MJPedestal) for the low-gain.
+       The effect is rather small since the ped. RMS PER SLICE does not 
+       change so much by going up 2 slices, maybe 1-2% depending on the 
+       extraction window. The smaller the window, the bigger the 
+       relative error.
+
 
  2004/11/08: Abelardo Moralejo

trunk/MagicSoft/Mars/mpedestal/MPedCalcPedRun.cc

@@ -259 +259 @@
           << int(fWindowSizeLoGain) << " samples " << endl;
     
-
-    if (fWindowSizeHiGain == 0)
-    {
-        *fLog << warn;
-        *fLog << GetDescriptor() << ": HiGain window currently set to 0, will set it to 2 samples " << endl;
-        fWindowSizeHiGain = 2;
-    }
-    
-    if (fWindowSizeLoGain == 0)
-    {
-        *fLog << warn;
-        *fLog << GetDescriptor() << ": LoGain window currently set to 0, will set it to 2 samples " << endl;
-        fWindowSizeLoGain = 2;
-    }
-
+  if (fWindowSizeHiGain == 0)
+    {
+      *fLog << warn;
+      *fLog << GetDescriptor() << ": HiGain window currently set to 0, will set it to 2 samples " << endl;
+      fWindowSizeHiGain = 2;
+    }
+  
   const Byte_t availhirange = (fHiGainLast-fHiGainFirst+1) & ~1;
   const Byte_t availlorange = (fLoGainLast-fLoGainFirst+1) & ~1;