Coverity fixes
[u/mrichter/AliRoot.git] / MUON / AliMUONPadStatusMaker.cxx
index c13bb14..59d4c9f 100644 (file)
 #include "AliMUON2DStoreValidator.h"
 #include "AliMUONCalibParamNI.h"
 #include "AliMUONCalibrationData.h"
+#include "AliMUONLogger.h"
+#include "AliMUONRecoParam.h"
 #include "AliMUONStringIntMap.h"
+#include "AliMUONTrackerData.h"
 #include "AliMUONVCalibParam.h"
 
 #include "AliMpArea.h"
 #include "AliMpArrayI.h"
+#include "AliMpCDB.h"
 #include "AliMpConstants.h"
 #include "AliMpDDLStore.h"
 #include "AliMpDEManager.h"
 #include "AliMpDetElement.h"
-#include "AliMpHVNamer.h"
+#include "AliMpDCSNamer.h"
+#include "AliMpManuIterator.h"
 #include "AliMpManuUID.h"
 
 #include "AliCDBEntry.h"
 #include <Riostream.h>
 #include <TArrayI.h>
 #include <TExMap.h>
+#include <TFile.h>
+#include <TKey.h>
 #include <TMap.h>
+#include <TROOT.h>
 #include <TString.h>
+#include <TSystem.h>
 
 /// \cond CLASSIMP
 ClassImp(AliMUONPadStatusMaker)
@@ -60,31 +69,44 @@ ClassImp(AliMUONPadStatusMaker)
 
 //_____________________________________________________________________________
 AliMUONPadStatusMaker::AliMUONPadStatusMaker(const AliMUONCalibrationData& calibData)
-: fCalibrationData(calibData),
-  fGainA0Limits(0,1E30),
-  fGainA1Limits(-1E-30,1E30),
-  fGainThresLimits(0,4095),
-  fHVSt12Limits(0,5000),
-  fHVSt345Limits(0,5000),
-  fPedMeanLimits(0,4095),
-  fPedSigmaLimits(0,4095),
-  fStatus(new AliMUON2DMap(true)),
-  fHV(new TExMap),
-  fPedestals(calibData.Pedestals()),
-  fGains(calibData.Gains())
+: fkCalibrationData(calibData),
+fGainA1Limits(0,1E30),
+fGainA2Limits(-1E-30,1E30),
+fGainThresLimits(0,4095),
+fPedMeanLimits(0,4095),
+fPedSigmaLimits(0,4095),
+fManuOccupancyLimits(0,1.0),
+fBuspatchOccupancyLimits(0,1.0),
+fDEOccupancyLimits(0,1.0),
+fStatus(new AliMUON2DMap(true)),
+fHV(0x0),
+fPedestals(calibData.Pedestals()),
+fGains(calibData.Gains()),
+fTrackerData(0x0)
 {
-   /// ctor
-    AliInfo(Form("ped store %s gain store %s",
-                 fPedestals->ClassName(),
-                 fGains->ClassName()));
+  /// ctor
+  if ( calibData.OccupancyMap() )
+  {
+    /// create a tracker data from the occupancy map
+    fTrackerData = new AliMUONTrackerData("OCC","OCC",*(calibData.OccupancyMap()));
+  }     
+  if ( calibData.HV() )
+  {
+    /// Only create the fHV internal store if there are some HV values available
+    fHV = new TExMap;
+  }
+  
+  SetHVLimit(-1,0.0);
 }
 
 //_____________________________________________________________________________
 AliMUONPadStatusMaker::~AliMUONPadStatusMaker()
 {
   /// dtor.
   delete fStatus;
   delete fHV;
+  delete fTrackerData;
 }
 
 //_____________________________________________________________________________
@@ -92,62 +114,65 @@ TString
 AliMUONPadStatusMaker::AsString(Int_t status)
 {
   /// return a human readable version of the integer status
+  
+  if ( status == 0 ) 
+  {
+    return "Brave New World";
+  }
+  
   Int_t pedStatus;
   Int_t gainStatus;
   Int_t hvStatus;
+  Int_t occStatus;
+  
+  DecodeStatus(status,pedStatus,hvStatus,gainStatus,occStatus);
+  
+  TString s;
+  
+  if ( pedStatus & kPedMeanZero ) s += "& Ped Mean is Zero ";
+  if ( pedStatus & kPedMeanTooLow ) s += "& Ped Mean Too Low ";
+  if ( pedStatus & kPedMeanTooHigh ) s += "& Ped Mean Too High ";
+  if ( pedStatus & kPedSigmaTooLow ) s += "& Ped Sigma Too Low ";
+  if ( pedStatus & kPedSigmaTooHigh ) s += "& Ped Sigma Too High ";
+  if ( pedStatus & kPedMissing ) s += "& Ped is missing ";
+  
+       if ( gainStatus & kGainA1TooLow ) s+="& Gain A1 is Too Low ";
+       if ( gainStatus & kGainA1TooHigh ) s+="& Gain A1 is Too High ";
+       if ( gainStatus & kGainA2TooLow ) s+="& Gain A2 is Too Low ";
+       if ( gainStatus & kGainA2TooHigh ) s+="& Gain A2 is Too High ";
+       if ( gainStatus & kGainThresTooLow ) s+="& Gain Thres is Too Low ";
+       if ( gainStatus & kGainThresTooHigh ) s+="& Gain Thres is Too High ";
+       if ( gainStatus & kGainMissing ) s+="& Gain is missing ";
+       
+       if ( hvStatus & kHVError ) s+="& HV is on error ";
+       if ( hvStatus & kHVTooLow ) s+="& HV is Too Low ";
+       if ( hvStatus & kHVTooHigh ) s+="& HV is Too High ";
+       if ( hvStatus & kHVChannelOFF ) s+="& HV has channel OFF ";
+       if ( hvStatus & kHVSwitchOFF ) s+="& HV has switch OFF ";
+       if ( hvStatus & kHVMissing ) s+="& HV is missing ";
+
+  if ( occStatus & kManuOccupancyTooHigh ) s+="& manu occupancy too high ";
+  if ( occStatus & kManuOccupancyTooLow ) s+="& manu occupancy too low ";
+  if ( occStatus & kBusPatchOccupancyTooHigh ) s+="& bus patch occupancy too high ";
+  if ( occStatus & kBusPatchOccupancyTooLow ) s+="& bus patch occupancy too low ";
+  if ( occStatus & kDEOccupancyTooHigh ) s+="& DE occupancy too high ";
+  if ( occStatus & kDEOccupancyTooLow ) s+="& DE occupancy too low ";
+  
+  if ( s[0] == '&' ) s[0] = ' ';
   
-  DecodeStatus(status,pedStatus,hvStatus,gainStatus);
-  
-//  /// Gain status
-//  enum EGainStatus
-//  {
-//    kGainOK = 0,
-//    kGainA0TooLow = (1<<1),
-//    kGainA0TooHigh = (1<<2),
-//    kGainA1TooLow = (1<<3),
-//    kGainA1TooHigh = (1<<4),
-//    kGainThresTooLow = (1<<5),
-//    kGainThresTooHigh = (1<<6),
-//    
-//    kGainMissing = kMissing // please always use last bit for meaning "missing"
-//  };
-//  
-//  /// Pedestal status
-//  enum EPedestalStatus
-//  {
-//    kPedOK = 0,
-//    kPedMeanZero = (1<<1),
-//    kPedMeanTooLow = (1<<2),
-//    kPedMeanTooHigh = (1<<3),
-//    kPedSigmaTooLow = (1<<4),
-//    kPedSigmaTooHigh = (1<<5),
-//    
-//    kPedMissing = kMissing // please always use last bit for meaning "missing"
-//  };
-//  
-  TString s("PED ");
-  
-  if ( pedStatus == 0 ) s+= " OK";
-  if ( pedStatus & kPedMeanZero ) s += " Mean is Zero. ";
-  if ( pedStatus & kPedMeanTooLow ) s += " Mean Too Low. ";
-  if ( pedStatus & kPedMeanTooHigh ) s += " Mean Too High. ";
-  if ( pedStatus & kPedSigmaTooLow ) s += " Sigma Too Low. ";
-  if ( pedStatus & kPedSigmaTooHigh ) s += " Sigma Too High. ";
-  if ( pedStatus & kPedMissing ) s += " is missing.";
-  
-//  /// HV Error
-//  enum EHVError 
-//  {
-//    kHVOK = 0,
-//    kHVError = (1<<0),
-//    kHVTooLow = (1<<1),
-//    kHVTooHigh = (1<<2),
-//    kHVChannelOFF = (1<<3),
-//    kHVSwitchOFF = (1<<4),
-//    
-//    kHVMissing = kMissing // please always use last bit for meaning "missing"
-//  };
+  return s;
+}
 
+//_____________________________________________________________________________
+TString
+AliMUONPadStatusMaker::AsCondition(Int_t mask)
+{
+  /// return a human readable version of the mask's equivalent condition
+  
+  TString s(AsString(mask));
+  
+  s.ReplaceAll("&","|");
+  
   return s;
 }
 
@@ -155,12 +180,14 @@ AliMUONPadStatusMaker::AsString(Int_t status)
 Int_t
 AliMUONPadStatusMaker::BuildStatus(Int_t pedStatus, 
                                    Int_t hvStatus, 
-                                   Int_t gainStatus)
+                                   Int_t gainStatus,
+                                   Int_t occStatus)
 {
   /// Build a complete status from specific parts (ped,hv,gain)
   
   return ( hvStatus & 0xFF ) | ( ( pedStatus & 0xFF ) << 8 ) | 
-  ( ( gainStatus & 0xFF ) << 16 );
+  ( ( gainStatus & 0xFF ) << 16 ) |
+  ( ( occStatus & 0xFF ) << 24 ) ;
 }
 
 //_____________________________________________________________________________
@@ -168,10 +195,12 @@ void
 AliMUONPadStatusMaker::DecodeStatus(Int_t status, 
                                     Int_t& pedStatus, 
                                     Int_t& hvStatus, 
-                                    Int_t& gainStatus)
+                                    Int_t& gainStatus,
+                                    Int_t& occStatus)
 {
   /// Decode complete status into specific parts (ped,hv,gain)
   
+  occStatus = ( status & 0xFF000000 ) >> 24;
   gainStatus = ( status & 0xFF0000 ) >> 16;
   pedStatus = ( status & 0xFF00 ) >> 8;
   hvStatus = (status & 0xFF);
@@ -190,18 +219,22 @@ AliMUONPadStatusMaker::HVSt12Status(Int_t detElemId, Int_t sector,
   /// and the switch).
   /// Returns false if hv switch changed during the run.
   
-  AliCodeTimerAuto("")
+  AliCodeTimerAuto("",0)
+  
+  if (!fHV) return kFALSE;
   
   Bool_t error = kFALSE;
   hvChannelTooLow = kFALSE;
   hvChannelTooHigh = kFALSE;
   hvChannelON = kTRUE;
 
-  AliMpHVNamer hvNamer;
+  Int_t chamberId = AliMpDEManager::GetChamberId(detElemId);
   
-  TString hvChannel(hvNamer.DCSHVChannelName(detElemId,sector));
+  AliMpDCSNamer hvNamer("TRACKER");
   
-  TMap* hvMap = fCalibrationData.HV();
+  TString hvChannel(hvNamer.DCSChannelName(detElemId,sector));
+  
+  TMap* hvMap = fkCalibrationData.HV();
   TPair* hvPair = static_cast<TPair*>(hvMap->FindObject(hvChannel.Data()));
   if (!hvPair)
   {
@@ -219,9 +252,8 @@ AliMUONPadStatusMaker::HVSt12Status(Int_t detElemId, Int_t sector,
     }
     else
     {
-      // find out min and max value, and makes a cut
+      // find out min value, and makes a cut
       Float_t hvMin(1E9);
-      Float_t hvMax(0);
       TIter next(values);
       AliDCSValue* val;
       
@@ -229,15 +261,12 @@ AliMUONPadStatusMaker::HVSt12Status(Int_t detElemId, Int_t sector,
       {
         Float_t hv = val->GetFloat();
         hvMin = TMath::Min(hv,hvMin);
-        hvMax = TMath::Max(hv,hvMax);
       }
       
-      float lowThreshold = fHVSt12Limits.X();
-      float highThreshold = fHVSt12Limits.Y();
+      float lowThreshold = HVLimit(chamberId);
             
       if ( hvMin < lowThreshold ) hvChannelTooLow = kTRUE;
-      if ( hvMax > highThreshold ) hvChannelTooHigh = kTRUE;
-      if ( hvMin < 1 ) hvChannelON = kFALSE;
+      if ( hvMin < hvNamer.TrackerHVOFF() ) hvChannelON = kFALSE;
     }
   }
   
@@ -245,6 +274,49 @@ AliMUONPadStatusMaker::HVSt12Status(Int_t detElemId, Int_t sector,
 }
 
 //_____________________________________________________________________________
+Float_t
+AliMUONPadStatusMaker::SwitchValue(const TObjArray& dcsArray)
+{
+  /// Loop over the dcs value for a single switch to decide whether
+  /// we should consider it on or off
+  
+  // we'll count the number of ON/OFF for this pad, to insure
+  // consistency (i.e. if status changed during the run, we should
+  // at least notify this fact ;-) and hope it's not the norm)
+  Int_t nTrue(0);
+  Int_t nFalse(0);
+  TIter next(&dcsArray);
+  AliDCSValue* val;
+  
+  while ( ( val = static_cast<AliDCSValue*>(next()) ) )
+  {
+    if ( val->GetBool() )
+    {
+      ++nTrue;
+    }
+    else
+    {
+      ++nFalse;
+    }
+  }
+  
+  if ( (nTrue>0 && nFalse>0) )
+  {
+    // change of state during the run, consider it off
+    return 0.0;
+  }
+  
+  if ( nFalse ) 
+  {
+    /// switch = FALSE means the HV was flowding up to the PCB.
+    /// i.e. switch = FALSE = ON
+    return 1.0;    
+  }
+  
+  return 0.0;
+}
+
+//_____________________________________________________________________________
 Bool_t 
 AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
                                      Bool_t& hvChannelTooLow,
@@ -257,7 +329,9 @@ AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
   /// Returns false if something goes wrong (in particular if 
   /// hv switch changed during the run).
   
-  AliCodeTimerAuto("")
+  AliCodeTimerAuto("",0)
+  
+  if (!fHV) return kFALSE;
   
   Bool_t error = kFALSE;
   hvChannelTooLow = kFALSE;
@@ -265,11 +339,13 @@ AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
   hvSwitchON = kTRUE;
   hvChannelON = kTRUE;
   
-  AliMpHVNamer hvNamer;
+  AliMpDCSNamer hvNamer("TRACKER");
+  
+  Int_t chamberId = AliMpDEManager::GetChamberId(detElemId);
   
-  TString hvChannel(hvNamer.DCSHVChannelName(detElemId));
+  TString hvChannel(hvNamer.DCSChannelName(detElemId));
   
-  TMap* hvMap = fCalibrationData.HV();
+  TMap* hvMap = fkCalibrationData.HV();
   
   TPair* hvPair = static_cast<TPair*>(hvMap->FindObject(hvChannel.Data()));
   if (!hvPair)
@@ -288,9 +364,8 @@ AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
     }
     else
     {
-      // find out min and max value, and makes a cut
+      // find out min value, and makes a cut
       Float_t hvMin(1E9);
-      Float_t hvMax(0);
       TIter next(values);
       AliDCSValue* val;
       
@@ -298,19 +373,16 @@ AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
       {
         Float_t hv = val->GetFloat();
         hvMin = TMath::Min(hv,hvMin);
-        hvMax = TMath::Max(hv,hvMax);
       }
 
-      float lowThreshold = fHVSt345Limits.X();
-      float highThreshold = fHVSt345Limits.Y();
+      float lowThreshold = HVLimit(chamberId);
 
       if ( hvMin < lowThreshold ) hvChannelTooLow = kTRUE;
-      else if ( hvMax > highThreshold ) hvChannelTooHigh = kTRUE;
-      if ( hvMin < 1 ) hvChannelON = kFALSE;
+      if ( hvMin < hvNamer.TrackerHVOFF() ) hvChannelON = kFALSE;
     }
   }
   
-  TString hvSwitch(hvNamer.DCSHVSwitchName(detElemId,pcbIndex));
+  TString hvSwitch(hvNamer.DCSSwitchName(detElemId,pcbIndex));
   TPair* switchPair = static_cast<TPair*>(hvMap->FindObject(hvSwitch.Data()));
   if (!switchPair)
   {
@@ -328,34 +400,8 @@ AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
     }
     else
     {
-      // we'll count the number of ON/OFF for this pad, to insure
-      // consistency (i.e. if status changed during the run, we should
-      // at least notify this fact ;-) and hope it's not the norm)
-      Int_t nTrue(0);
-      Int_t nFalse(0);
-      TIter next(values);
-      AliDCSValue* val;
-      
-      while ( ( val = static_cast<AliDCSValue*>(next()) ) )
-      {
-        if ( val->GetBool() )
-        {
-          ++nTrue;
-        }
-        else
-        {
-          ++nFalse;
-        }
-      }
-      
-      if ( (nTrue>0 && nFalse>0) )
-      {
-        AliWarning(Form("Status of HV Switch %s changed during this run nTrue=%d nFalse=%d! Will consider it OFF",
-                        hvSwitch.Data(),nTrue,nFalse));
-        error = kTRUE;
-      }
-      
-      if ( nFalse ) hvSwitchON = kFALSE;
+      Float_t sv = SwitchValue(*values);
+      if ( sv < 0.99 ) hvSwitchON = kFALSE;
     }
   }
   return error;
@@ -367,9 +413,9 @@ AliMUONPadStatusMaker::HVStatus(Int_t detElemId, Int_t manuId) const
 {
   /// Get HV status of one manu
   
-  AliCodeTimerAuto("")
+  AliCodeTimerAuto("",0)
   
-  if ( !fCalibrationData.HV() ) return kMissing;
+  if ( !fHV ) return kMissing;
 
   Long_t lint = fHV->GetValue(AliMpManuUID::BuildUniqueID(detElemId,manuId));
   
@@ -380,12 +426,11 @@ AliMUONPadStatusMaker::HVStatus(Int_t detElemId, Int_t manuId) const
 
   Int_t status(0);
   
-  AliMpHVNamer hvNamer;
+  AliMpDCSNamer hvNamer("TRACKER");
   
   switch ( AliMpDEManager::GetStationType(detElemId) )
   {
-    case AliMp::kStation1:
-    case AliMp::kStation2:
+    case AliMp::kStation12:
     {
       int sector = hvNamer.ManuId2Sector(detElemId,manuId);
       if ( sector >= 0 ) 
@@ -435,7 +480,7 @@ AliMUONVCalibParam*
 AliMUONPadStatusMaker::Neighbours(Int_t detElemId, Int_t manuId) const
 {
   /// Get the neighbours parameters for a given manu
-  AliMUONVStore* neighbourStore = fCalibrationData.Neighbours();
+  AliMUONVStore* neighbourStore = fkCalibrationData.Neighbours();
   return static_cast<AliMUONVCalibParam*>(neighbourStore->FindObject(detElemId,manuId));
 }
 
@@ -444,7 +489,7 @@ AliMUONVStore*
 AliMUONPadStatusMaker::NeighboursStore() const
 {
   /// Return the store containing all the neighbours
-  return fCalibrationData.Neighbours();
+  return fkCalibrationData.Neighbours();
 }
 
 //_____________________________________________________________________________
@@ -454,22 +499,16 @@ AliMUONPadStatusMaker::ComputeStatus(Int_t detElemId, Int_t manuId) const
   /// Compute the status of a given manu, using all available information,
   /// i.e. pedestals, gains, and HV
   
-//  AliCodeTimerAuto("")
-  
-//  AliCodeTimerStart("Param creation");
   AliMUONVCalibParam* param = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,-1);
   fStatus->Add(param);
-//  AliCodeTimerStop("Param creation");
-  
-//  AliCodeTimerStart("FindObject");
+
   AliMUONVCalibParam* pedestals = static_cast<AliMUONVCalibParam*>(fPedestals->FindObject(detElemId,manuId));
 
   AliMUONVCalibParam* gains = static_cast<AliMUONVCalibParam*>(fGains->FindObject(detElemId,manuId));
-//  AliCodeTimerStop("FindObject");
   
   Int_t hvStatus = HVStatus(detElemId,manuId);
 
-//  AliCodeTimerStart("Loop");
+  Int_t occStatus = OccupancyStatus(detElemId,manuId);
   
   for ( Int_t manuChannel = 0; manuChannel < param->Size(); ++manuChannel )
   {
@@ -498,10 +537,10 @@ AliMUONPadStatusMaker::ComputeStatus(Int_t detElemId, Int_t manuId) const
       Float_t a1 = gains->ValueAsFloatFast(manuChannel,1);
       Float_t thres = gains->ValueAsFloatFast(manuChannel,2);
   
-      if ( a0 < fGainA0Limits.X() ) gainStatus |= kGainA0TooLow;
-      else if ( a0 > fGainA0Limits.Y() ) gainStatus |= kGainA0TooHigh;
-      if ( a1 < fGainA1Limits.X() ) gainStatus |= kGainA1TooLow;
-      else if ( a1 > fGainA1Limits.Y() ) gainStatus |= kGainA1TooHigh;
+      if ( a0 < fGainA1Limits.X() ) gainStatus |= kGainA1TooLow;
+      else if ( a0 > fGainA1Limits.Y() ) gainStatus |= kGainA1TooHigh;
+      if ( a1 < fGainA2Limits.X() ) gainStatus |= kGainA2TooLow;
+      else if ( a1 > fGainA2Limits.Y() ) gainStatus |= kGainA2TooHigh;
       if ( thres < fGainThresLimits.X() ) gainStatus |= kGainThresTooLow;
       else if ( thres > fGainThresLimits.Y() ) gainStatus |= kGainThresTooHigh;
     }
@@ -509,28 +548,76 @@ AliMUONPadStatusMaker::ComputeStatus(Int_t detElemId, Int_t manuId) const
     {
       gainStatus = kGainMissing;
     }
-        
-    Int_t status = BuildStatus(pedStatus,hvStatus,gainStatus);
+    
+    Int_t status = BuildStatus(pedStatus,hvStatus,gainStatus,occStatus);
       
     param->SetValueAsIntFast(manuChannel,0,status);
   }
   
-//  AliCodeTimerStop("Loop");
-  
   return param;
 }
 
 //_____________________________________________________________________________
+Int_t 
+AliMUONPadStatusMaker::OccupancyStatus(Int_t detElemId, Int_t manuId) const
+{
+  /// Get the "other" status for a given manu
+  Int_t rv(0);
+  
+  if ( fTrackerData ) 
+  {
+    const Int_t occIndex = 2;
+    
+    Double_t occ = fTrackerData->DetectionElement(detElemId,occIndex);
+    
+    if ( occ <= fDEOccupancyLimits.X() )
+    {
+      rv |= kDEOccupancyTooLow;
+    } 
+    else if ( occ > fDEOccupancyLimits.Y() )
+    {
+      rv |= kDEOccupancyTooHigh;
+    }
+    
+    Int_t busPatchId = AliMpDDLStore::Instance()->GetBusPatchId(detElemId,manuId);
+    
+    occ = fTrackerData->BusPatch(busPatchId,occIndex);
+
+    if ( occ <= fBuspatchOccupancyLimits.X() )
+    {
+      rv |= kBusPatchOccupancyTooLow;
+    } 
+    else if ( occ > fBuspatchOccupancyLimits.Y() )
+    {
+      rv |= kBusPatchOccupancyTooHigh;
+    }
+    
+    occ = fTrackerData->Manu(detElemId,manuId,occIndex);
+    
+    if ( occ <= fManuOccupancyLimits.X() )
+    {
+      rv |= kManuOccupancyTooLow;
+    } 
+    else if ( occ > fManuOccupancyLimits.Y() )
+    {
+      rv |= kManuOccupancyTooHigh;
+    }
+  }
+  return rv;
+}
+
+//_____________________________________________________________________________
 AliMUONVCalibParam* 
 AliMUONPadStatusMaker::PadStatus(Int_t detElemId, Int_t manuId) const
 {
-  /// Get the status for a given channel
+  /// Get the status container for a given manu
   
   AliMUONVCalibParam* param = static_cast<AliMUONVCalibParam*>(fStatus->FindObject(detElemId,manuId));
   if (!param)
   {
     // not already there, so compute it now
-    AliCodeTimerAuto("ComputeStatus");
+    AliCodeTimerAuto("ComputeStatus",0);
     param = ComputeStatus(detElemId,manuId);
   }
   return param;
@@ -558,7 +645,7 @@ AliMUONPadStatusMaker::SetHVStatus(Int_t detElemId, Int_t index, Int_t status) c
   /// Assign status to all manus in a given HV "zone" (defined by index, meaning
   /// is different thing from St12 and St345)
   
-  AliCodeTimerAuto("")
+  AliCodeTimerAuto("",0)
   
   AliMpDetElement* de = AliMpDDLStore::Instance()->GetDetElement(detElemId);
   
@@ -570,3 +657,123 @@ AliMUONPadStatusMaker::SetHVStatus(Int_t detElemId, Int_t index, Int_t status) c
     fHV->Add(AliMpManuUID::BuildUniqueID(detElemId,manuId),status + 1);
   }
 }
+
+//_____________________________________________________________________________
+Double_t
+AliMUONPadStatusMaker::HVLimit(Int_t chamberId) const
+{
+  /// Get HV limit for a given chamber
+  if ( chamberId >=0 && chamberId < 10 ) 
+  {
+    return fHVLimit[chamberId];
+  }
+  return 0.0;
+}
+
+//_____________________________________________________________________________
+void
+AliMUONPadStatusMaker::SetHVLimit(Int_t chamberId, Double_t hv) 
+{
+  /// Set hv limit for a given chamber (or all if chamberId==-1)
+  
+  if ( chamberId == -1 ) 
+  {
+    for ( Int_t i = 0; i < 10; ++i ) 
+    {
+      fHVLimit[i] = hv;
+    }
+  }
+  else if ( chamberId >= 0 && chamberId < 10 ) 
+  {
+    fHVLimit[chamberId]=hv;
+  }
+  else
+  {
+    AliError(Form("chamberId=%d is invalid",chamberId));
+  }
+}
+
+//_____________________________________________________________________________
+void
+AliMUONPadStatusMaker::SetLimits(const AliMUONRecoParam& recoParams) 
+{
+  /// Set the limits from the recoparam
+  
+  for ( int i = 0; i < 10; ++i )
+  {
+    SetHVLimit(i,recoParams.HVLimit(i));
+  }
+  
+  SetPedMeanLimits(recoParams.PedMeanLowLimit(),recoParams.PedMeanHighLimit());
+  SetPedSigmaLimits(recoParams.PedSigmaLowLimit(),recoParams.PedSigmaHighLimit());
+  
+  SetGainA1Limits(recoParams.GainA1LowLimit(),recoParams.GainA1HighLimit());
+  SetGainA2Limits(recoParams.GainA2LowLimit(),recoParams.GainA2HighLimit());
+  SetGainThresLimits(recoParams.GainThresLowLimit(),recoParams.GainThresHighLimit());
+  
+  SetManuOccupancyLimits(recoParams.ManuOccupancyLowLimit(),recoParams.ManuOccupancyHighLimit());
+  SetBuspatchOccupancyLimits(recoParams.BuspatchOccupancyLowLimit(),recoParams.BuspatchOccupancyHighLimit());
+  SetDEOccupancyLimits(recoParams.DEOccupancyLowLimit(),recoParams.DEOccupancyHighLimit());  
+}
+
+//_____________________________________________________________________________
+void 
+AliMUONPadStatusMaker::Report(UInt_t mask)
+{
+  /// Report the number of bad pads, according to the mask,
+  /// and the various reasons why they are bad (with occurence rates)
+  
+  AliInfo("");
+  AliCodeTimerAuto("",0);
+
+  AliMUONLogger log(1064008);
+  
+  Int_t nBadPads(0);
+  Int_t nPads(0);
+  
+  AliMpManuIterator it;
+  
+  Int_t detElemId, manuId;
+  
+  while ( it.Next(detElemId,manuId) )
+  {
+    AliMpDetElement* de = AliMpDDLStore::Instance()->GetDetElement(detElemId);
+    
+    for ( Int_t i = 0; i < AliMpConstants::ManuNofChannels(); ++i )
+    {
+      if ( de->IsConnectedChannel(manuId,i) ) 
+      {
+        ++nPads;
+        
+        Int_t status = PadStatus(detElemId,manuId,i);          
+        
+        if ( mask && ( status & mask) ) // note that if mask == 0, all pads are good...
+        {
+          ++nBadPads;
+          log.Log(AsString(status));
+        }
+      }
+    }
+  }
+  
+  if (!nPads) 
+  {
+    AliError("Got no pad from the iterator ?! That's not normal. Please check !");
+    return;
+  }
+  
+  TString msg;
+  Int_t ntimes;
+  
+  cout << Form("According to mask %x (human readable form below) %6d pads are bad (over a total of %6d, i.e. %7.2f %%)",
+               mask,nBadPads,nPads,nPads ? nBadPads*100.0/nPads : 0.0) << endl;
+  cout << AliMUONPadStatusMaker::AsCondition(mask) << endl;
+  cout << "--------" << endl;
+  
+  while ( log.Next(msg,ntimes) )
+  {
+    cout << Form("The message (%120s) occured %15d times (%7.4f %%)",msg.Data(),ntimes,ntimes*100.0/nPads) << endl;
+  }
+  
+}
+