Changeset 7575 for trunk

Timestamp:

03/07/06 10:33:26 (19 years ago)

Author:

tbretz

Message:

*** empty log message ***

Location:

trunk/MagicSoft/Mars

Files:

• Changelog (modified) (1 diff)
• NEWS (modified) (1 diff)
• mcalib/MCalibrateData.cc (modified) (1 diff)

trunk/MagicSoft/Mars/Changelog

@@ -18 +18 @@
 
                                                  -*-*- END OF LINE -*-*-
+   * mcalib/MCalibrateData.cc:
+     - take the bad pixel information into account when calculating the
+       number of saturating pixels!
+
+
  2006/03/06 Thomas Bretz
 

trunk/MagicSoft/Mars/NEWS

@@ -13 +13 @@
 
    - showplot: path inflation for ganymed files was broken
+
+   - callisto: Fixed an important bug. When the number of saturating pixels
+     were calculated the bad pixel information already available from the
+     calibration was ignored. This yields wrong values in case pixels
+     are broken and saturate all the time or randomly.
 
    - star: The time-evolution shown in the effective on-time plot now displays

trunk/MagicSoft/Mars/mcalib/MCalibrateData.cc

@@ -783 +783 @@
             const Float_t nphot    = signal                         * fCalibConsts  [pixidx];
             const Float_t nphotErr = TMath::Sqrt(TMath::Abs(nphot)) * fCalibFFactors[pixidx];
-            
+
             fCerPhotEvt->AddPixel(pixidx, nphot, nphotErr);
 
-            if (sig.GetNumHiGainSaturated() > 0)
-                numsathi++;
-
-            if (sig.GetNumLoGainSaturated() > 0)
-                numsatlo++;
+            if (!(*fBadPixels)[pixidx].IsUnsuitable())
+            {
+                if (sig.GetNumHiGainSaturated() > 0)
+                    numsathi++;
+
+                if (sig.GetNumLoGainSaturated() > 0)
+                    numsatlo++;
+            }
         } /* if (data) */