Changeset 5872 for trunk/MagicSoft

Timestamp:

01/17/05 14:03:02 (20 years ago)

Author:

gaug

Message:

*** empty log message ***

Location:

trunk/MagicSoft/Mars

Files:

• Changelog (modified) (1 diff)
• mjobs/MJCalibrateSignal.cc (modified) (2 diffs)
• mjobs/MJPedestal.cc (modified) (6 diffs)

trunk/MagicSoft/Mars/Changelog

@@ -59 +59 @@
 
    * mjobs/MJPedestal.cc
+   * mjobs/MJCalibrateSignal.cc
      - fixed default extraction ranges.
 

trunk/MagicSoft/Mars/mjobs/MJCalibrateSignal.cc

@@ -72 +72 @@
 #include "MPedCalcFromLoGain.h"
 #include "MExtractor.h"
+#include "MExtractTimeAndCharge.h"
 #include "MFCosmics.h"
 #include "MTaskEnv.h"
@@ -365 +366 @@
             pedlo2.SetExtractor((MExtractTimeAndCharge*)extractor1);
             pedlo3.SetExtractor((MExtractTimeAndCharge*)extractor1);
-            pedlo1.SetExtractWindow(15, 0/*obsolete*/);
-            pedlo2.SetExtractWindow(15, 0/*obsolete*/);
-            pedlo3.SetExtractWindow(15, 0/*obsolete*/);
+            const Int_t win = ((MExtractTimeAndCharge*)extractor1)->GetWindowSizeHiGain();
+            pedlo1.SetExtractWindow(15, win);
+            pedlo2.SetExtractWindow(15, win/*obsolete*/);
+            pedlo3.SetExtractWindow(15, win/*obsolete*/);
         }
         else

trunk/MagicSoft/Mars/mjobs/MJPedestal.cc

@@ -58 +58 @@
 // Other basic classes
 #include "MExtractor.h"
+#include "MExtractTimeAndCharge.h"
 
 // parameter containers
@@ -901 +902 @@
     case kUsePedRun:
         // In case  other than 'fundamental' second argument is obsolete
-        pedcalc.SetExtractWindow(0,14);  // kUsePedRun
+        // pedcalc.SetExtractWindow(0,14);  // kUsePedRun (take default from class)
         taskenv.SetDefault(&pedcalc);
         tlist.AddToList(&taskenv);
@@ -908 +909 @@
     case kUseData:
         // In case  other than 'fundamental' second argument is obsolete
-        pedlogain.SetExtractWindow(15,14); // kUseData
+        // pedlogain.SetExtractWindow(15,14); // kUseData (take default from class)
         taskenv.SetDefault(&pedlogain);
         tlist.AddToList(&taskenv);
@@ -932 +933 @@
     if (fExtractor)
     {
-      const Float_t f = 0.5+fExtractor->GetHiGainFirst();
-      const Float_t n = 0.5+fExtractor->GetNumHiGainSamples();
-      pedcalc.SetExtractWindow((Int_t)f, (Int_t)n);
-      pedlogain.SetExtractWindow((Int_t)(15+f), (Int_t)n);
-            
+
       if (!fExtractor->InheritsFrom("MExtractTimeAndCharge"))
         {
+
+          const Float_t f = 0.5+fExtractor->GetHiGainFirst();
+          const Int_t win = ((MExtractTimeAndCharge*)fExtractor)->GetWindowSizeHiGain();
+          pedcalc.SetExtractWindow((Int_t)f, win);
+          pedlogain.SetExtractWindow((Int_t)(15+f), win);
+
           if (fExtractionType!=kFundamental)
             {
@@ -947 +950 @@
             }
         }
+      else
+        {
+          const Float_t f = 0.5+fExtractor->GetHiGainFirst();
+          const Float_t n = 0.5+fExtractor->GetNumHiGainSamples();
+          pedcalc.SetExtractWindow((Int_t)f, (Int_t)n);
+          pedlogain.SetExtractWindow((Int_t)(15+f), (Int_t)n);
+        }
+      
       
       if (fExtractionType!=kFundamental)
@@ -957 +968 @@
         }
     }
-
+    else
+      {
+        *fLog << warn << GetDescriptor() 
+              << ": WARNING: No extractor has been handed over! " << endl;
+        *fLog << warn << GetDescriptor()
+              << ": WARNING: Taking default window for pedestal extraction. The calculated pedestal RMS will probably not match with future pedestal RMSs from different extraction windows." << endl;
+      }
+    
     /*
     switch (fExtractType)