]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - MUON/AliMUONPadStatusMaker.cxx
Change names of sections to avoid confusion with MCH DA sections
[u/mrichter/AliRoot.git] / MUON / AliMUONPadStatusMaker.cxx
index f6e1d7093d4b84d838276ef0961f7f647ce28a92..accac6795883aa12e5797e673a9cc084a13a2e20 100644 (file)
 
 // $Id$
 
+//-----------------------------------------------------------------------------
 /// \class AliMUONPadStatusMaker
 ///
 /// Make a 2DStore of pad statuses, using different sources of information,
 /// like pedestal values, gain values, and HV values.
 ///
 /// \author Laurent Aphecetche
+//-----------------------------------------------------------------------------
 
 #include "AliMUONPadStatusMaker.h"
 
-#include "AliCDBEntry.h"
-#include "AliCDBManager.h"
-#include "AliDCSValue.h"
-#include "AliLog.h"
+#include "AliQA.h"
+
 #include "AliMUON2DMap.h"
 #include "AliMUON2DStoreValidator.h"
 #include "AliMUONCalibParamNI.h"
 #include "AliMUONCalibrationData.h"
-#include "AliMUONHVNamer.h"
+#include "AliMUONStringIntMap.h"
 #include "AliMUONVCalibParam.h"
+#include "AliMUONVTrackerData.h"
+
 #include "AliMpArea.h"
+#include "AliMpArrayI.h"
 #include "AliMpConstants.h"
-#include "AliMpDEIterator.h"
+#include "AliMpDDLStore.h"
 #include "AliMpDEManager.h"
-#include "AliMpIntPair.h"
-#include "AliMpManuList.h"
-#include "AliMpMotifMap.h"
-#include "AliMpMotifPosition.h"
-#include "AliMpPCB.h"
-#include "AliMpPad.h"
-#include "AliMpSector.h"
-#include "AliMpSectorSegmentation.h"
-#include "AliMpSegmentation.h"
-#include "AliMpSlat.h"
-#include "AliMpSlatSegmentation.h"
-#include "AliMpStationType.h"
-#include "AliMpVPadIterator.h"
+#include "AliMpDetElement.h"
+#include "AliMpDCSNamer.h"
+#include "AliMpManuUID.h"
+
+#include "AliCDBEntry.h"
+#include "AliCDBManager.h"
+#include "AliCodeTimer.h"
+#include "AliDCSValue.h"
+#include "AliLog.h"
+
 #include <Riostream.h>
+#include <TArrayI.h>
+#include <TExMap.h>
+#include <TFile.h>
+#include <TKey.h>
 #include <TMap.h>
-#include <TStopwatch.h>
+#include <TROOT.h>
 #include <TString.h>
-
+#include <TSystem.h>
 
 /// \cond CLASSIMP
 ClassImp(AliMUONPadStatusMaker)
@@ -63,115 +67,188 @@ ClassImp(AliMUONPadStatusMaker)
 
 //_____________________________________________________________________________
 AliMUONPadStatusMaker::AliMUONPadStatusMaker(const AliMUONCalibrationData& calibData)
-: fCalibrationData(calibData),
-  fPedMeanLimits(0,4095),
-  fPedSigmaLimits(0,4095),
-  fHVSt12Limits(0,5000),
-  fHVSt345Limits(0,5000)
+: fkCalibrationData(calibData),
+fGainA1Limits(0,1E30),
+fGainA2Limits(-1E-30,1E30),
+fGainThresLimits(0,4095),
+fHVSt12Limits(0,5000),
+fHVSt345Limits(0,5000),
+fPedMeanLimits(0,4095),
+fPedSigmaLimits(0,4095),
+fManuOccupancyLimits(0,0.1),
+fStatus(new AliMUON2DMap(true)),
+fHV(new TExMap),
+fPedestals(calibData.Pedestals()),
+fGains(calibData.Gains()),
+fTrackerData(0x0)
 {
-   /// ctor
-}
-
-//_____________________________________________________________________________
-AliMUONPadStatusMaker::~AliMUONPadStatusMaker()
-{
-  /// dtor.
-}
-
-//_____________________________________________________________________________
-AliMUONVStore*
-AliMUONPadStatusMaker::Combine(const AliMUONVStore& store1,
-                               const AliMUONVStore& store2,
-                               Int_t binShift) const
-{
-  /// Combine two status containers into one, shifting store2 status bits
-  /// to the left by binShift before making an OR with store1.
-  
-  TStopwatch timer;
-  timer.Start(kTRUE);
-  
-  AliMUONVStore* combined = static_cast<AliMUONVStore*>(store1.Clone());
+  /// ctor
+  AliDebug(1,Form("ped store %s gain store %s",
+                  fPedestals->ClassName(),
+                  fGains->ClassName()));
   
-  TIter next(store1.CreateIterator());
-  AliMUONVCalibParam* param1;
+  TString qaFileName(AliQA::GetQADataFileName("MUON",calibData.RunNumber()));
   
-  while ( ( param1 = static_cast<AliMUONVCalibParam*>(next()) ) )
+  // search the QA file in memory first.
+  TFile* f = static_cast<TFile*>(gROOT->GetListOfFiles()->FindObject(qaFileName.Data()));
+
+  if (!f)
   {
-    Int_t detElemId = param1->ID0();
-    Int_t manuId = param1->ID1();
-    AliMUONVCalibParam* param2 = static_cast<AliMUONVCalibParam*>(store2.FindObject(detElemId,manuId));
-    if (!param2)
+    // then tries to open it
+    if ( gSystem->AccessPathName(qaFileName.Data()) == kFALSE ) 
     {
-      AliWarning(Form("Could not get statuses for store2 for DE %d ManuId %d. Marking as missing.",
-                    detElemId,manuId));
-      param2 = static_cast<AliMUONVCalibParam*>(param1->Clone());
-      for ( Int_t manuChannel = 0; manuChannel < param2->Size(); ++manuChannel )
+      f = TFile::Open(qaFileName.Data());
+      if ( f )
       {
-        param2->SetValueAsInt(manuChannel,0,kMissing);
-      }      
-    }
-    AliMUONVCalibParam* paramCombined = static_cast<AliMUONVCalibParam*>(combined->FindObject(detElemId,manuId));
-    if (!paramCombined)
-    {
-      paramCombined = static_cast<AliMUONVCalibParam*>(param2->Clone());
-      combined->Add(paramCombined);
+        AliDebug(1,Form("Got %s from disk",qaFileName.Data()));
+      }
     }
+  }
+  else
+  {
+    AliDebug(1,Form("Got %s from memory",qaFileName.Data()));
+  }
+  
+  if (f)
+  {
+    TDirectory* d = gDirectory;
+    
+    f->cd("MUON/Raws");
+    
+    TIter next(gDirectory->GetListOfKeys());
+    TKey* key;
     
-    for ( Int_t manuChannel = 0; manuChannel < param1->Size(); ++manuChannel )
+    while ( ( key = static_cast<TKey*>(next()) ) && !fTrackerData )
     {
-      if ( AliMpManuList::DoesChannelExist(detElemId, manuId, manuChannel) )
+      TString name(key->GetName());
+      
+      if ( name.Contains("CALZ") )
       {
-        Int_t status1(param1->ValueAsInt(manuChannel));
-        Int_t status2(param2->ValueAsInt(manuChannel));
-        
-        Int_t status = status1 | (status2 << binShift);
-        
-        paramCombined->SetValueAsInt(manuChannel,0,status);
+        fTrackerData = dynamic_cast<AliMUONVTrackerData*>(key->ReadObj());
       }
+      
+    }
+    
+    gDirectory = d;
+    
+    if ( fTrackerData ) 
+    {
+      AliInfo(Form("Will make a cut on MANU occupancy from TrackerData=%s",fTrackerData->GetName()));
+    }
+    else
+    {
+      AliWarning(Form("Found a QA file = %s, but could not get the expected TrackerData in there... (probably not a serious problem though)",
+                      f->GetName()));
     }
   }
-  
-  AliInfo("Timer:");
-  StdoutToAliInfo(timer.Print(););
-  
-  return combined;
+  else
+  {
+    AliWarning("Did not find QA file, so will not use manu occupancy as a criteria");
+  }
 }
 
 //_____________________________________________________________________________
-AliMUONVStore* 
-AliMUONPadStatusMaker::GeneratePadStatus(Int_t value)
+AliMUONPadStatusMaker::~AliMUONPadStatusMaker()
 {
-  /// Generate a "fake" store, with all (detElemId,manuId) present,
-  /// and containing all the same value
+  /// dtor.
+  delete fStatus;
+  delete fHV;
+}
+
+//_____________________________________________________________________________
+TString
+AliMUONPadStatusMaker::AsString(Int_t status)
+{
+  /// return a human readable version of the integer status
+  
+  Int_t pedStatus;
+  Int_t gainStatus;
+  Int_t hvStatus;
+  Int_t otherStatus;
+  
+  DecodeStatus(status,pedStatus,hvStatus,gainStatus,otherStatus);
+  
+  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 ( otherStatus & kManuOccupancyTooHigh ) s+="& manu occupancy too high ";
+  if ( otherStatus & kManuOccupancyTooLow ) s+="& manu occupancy too low ";
   
-  AliMUONVStore* store = new AliMUON2DMap(true);
+  if ( s[0] == '&' ) s[0] = ' ';
   
-  TList* list = AliMpManuList::ManuList();
+  return s;
+}
+
+//_____________________________________________________________________________
+TString
+AliMUONPadStatusMaker::AsCondition(Int_t mask)
+{
+  /// return a human readable version of the mask's equivalent condition
   
-  AliMpIntPair* pair;
+  TString s(AsString(mask));
   
-  TIter next(list);
+  s.ReplaceAll("&","|");
   
-  while ( ( pair = static_cast<AliMpIntPair*>(next()) ) ) 
-  {
-    Int_t detElemId = pair->GetFirst();
-    Int_t manuId = pair->GetSecond();
-    AliMUONVCalibParam* param = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,value);
-    store->Add(param);
-  }
+  return s;
+}
+
+//_____________________________________________________________________________
+Int_t
+AliMUONPadStatusMaker::BuildStatus(Int_t pedStatus, 
+                                   Int_t hvStatus, 
+                                   Int_t gainStatus,
+                                   Int_t otherStatus)
+{
+  /// Build a complete status from specific parts (ped,hv,gain)
   
-  delete list;
+  return ( hvStatus & 0xFF ) | ( ( pedStatus & 0xFF ) << 8 ) | 
+  ( ( gainStatus & 0xFF ) << 16 ) |
+  ( ( otherStatus & 0xFF ) << 24 ) ;
+}
+
+//_____________________________________________________________________________
+void
+AliMUONPadStatusMaker::DecodeStatus(Int_t status, 
+                                    Int_t& pedStatus, 
+                                    Int_t& hvStatus, 
+                                    Int_t& gainStatus,
+                                    Int_t& otherStatus)
+{
+  /// Decode complete status into specific parts (ped,hv,gain)
   
-  return store;
+  otherStatus = ( status & 0xFF000000 ) >> 24;
+  gainStatus = ( status & 0xFF0000 ) >> 16;
+  pedStatus = ( status & 0xFF00 ) >> 8;
+  hvStatus = (status & 0xFF);
 }
 
 //_____________________________________________________________________________
 Bool_t 
-AliMUONPadStatusMaker::GetSt12Status(const TMap& hvMap,
-                                     Int_t detElemId, Int_t sector,
-                                     Bool_t& hvChannelTooLow,
-                                     Bool_t& hvChannelTooHigh,
-                                     Bool_t& hvChannelON) const
+AliMUONPadStatusMaker::HVSt12Status(Int_t detElemId, Int_t sector,
+                                    Bool_t& hvChannelTooLow,
+                                    Bool_t& hvChannelTooHigh,
+                                    Bool_t& hvChannelON) const
 {
   /// Get HV status for one HV sector of St12
   
@@ -179,16 +256,19 @@ AliMUONPadStatusMaker::GetSt12Status(const TMap& hvMap,
   /// and the switch).
   /// Returns false if hv switch changed during the run.
   
+  AliCodeTimerAuto("")
+  
   Bool_t error = kFALSE;
   hvChannelTooLow = kFALSE;
   hvChannelTooHigh = kFALSE;
   hvChannelON = kTRUE;
+
+  AliMpDCSNamer hvNamer("TRACKER");
   
-  AliMUONHVNamer hvNamer;
-  
-  TString hvChannel(hvNamer.DCSHVChannelName(detElemId,sector));
+  TString hvChannel(hvNamer.DCSChannelName(detElemId,sector));
   
-  TPair* hvPair = static_cast<TPair*>(hvMap.FindObject(hvChannel.Data()));
+  TMap* hvMap = fkCalibrationData.HV();
+  TPair* hvPair = static_cast<TPair*>(hvMap->FindObject(hvChannel.Data()));
   if (!hvPair)
   {
     AliError(Form("Did not find expected alias (%s) for DE %d",
@@ -232,29 +312,32 @@ AliMUONPadStatusMaker::GetSt12Status(const TMap& hvMap,
 
 //_____________________________________________________________________________
 Bool_t 
-AliMUONPadStatusMaker::GetSt345Status(const TMap& hvMap,
-                                      Int_t detElemId, Int_t pcbIndex,
-                                      Bool_t& hvChannelTooLow,
-                                      Bool_t& hvChannelTooHigh,
-                                      Bool_t& hvChannelON,
-                                      Bool_t& hvSwitchON) const
+AliMUONPadStatusMaker::HVSt345Status(Int_t detElemId, Int_t pcbIndex,
+                                     Bool_t& hvChannelTooLow,
+                                     Bool_t& hvChannelTooHigh,
+                                     Bool_t& hvChannelON,
+                                     Bool_t& hvSwitchON) const
 {
   /// For a given PCB in a given DE, get the HV status (both the channel
   /// and the switch).
   /// Returns false if something goes wrong (in particular if 
   /// hv switch changed during the run).
   
+  AliCodeTimerAuto("")
+  
   Bool_t error = kFALSE;
   hvChannelTooLow = kFALSE;
   hvChannelTooHigh = kFALSE;
   hvSwitchON = kTRUE;
   hvChannelON = kTRUE;
   
-  AliMUONHVNamer hvNamer;
+  AliMpDCSNamer hvNamer("TRACKER");
+  
+  TString hvChannel(hvNamer.DCSChannelName(detElemId));
   
-  TString hvChannel(hvNamer.DCSHVChannelName(detElemId));
+  TMap* hvMap = fkCalibrationData.HV();
   
-  TPair* hvPair = static_cast<TPair*>(hvMap.FindObject(hvChannel.Data()));
+  TPair* hvPair = static_cast<TPair*>(hvMap->FindObject(hvChannel.Data()));
   if (!hvPair)
   {
     AliError(Form("Did not find expected alias (%s) for DE %d",
@@ -288,13 +371,13 @@ AliMUONPadStatusMaker::GetSt345Status(const TMap& hvMap,
       float highThreshold = fHVSt345Limits.Y();
 
       if ( hvMin < lowThreshold ) hvChannelTooLow = kTRUE;
-      if ( hvMax > highThreshold ) hvChannelTooHigh = kTRUE;
+      else if ( hvMax > highThreshold ) hvChannelTooHigh = kTRUE;
       if ( hvMin < 1 ) hvChannelON = kFALSE;
     }
   }
   
-  TString hvSwitch(hvNamer.DCSHVSwitchName(detElemId,pcbIndex));
-  TPair* switchPair = static_cast<TPair*>(hvMap.FindObject(hvSwitch.Data()));
+  TString hvSwitch(hvNamer.DCSSwitchName(detElemId,pcbIndex));
+  TPair* switchPair = static_cast<TPair*>(hvMap->FindObject(hvSwitch.Data()));
   if (!switchPair)
   {
     AliError(Form("Did not find expected alias (%s) for DE %d PCB %d",
@@ -345,304 +428,222 @@ AliMUONPadStatusMaker::GetSt345Status(const TMap& hvMap,
 }
 
 //_____________________________________________________________________________
-AliMUONVStore* 
-AliMUONPadStatusMaker::MakeGainStatus(const AliMUONVStore& /*gainValues*/) const
-{
-  /// FIXME: to be implemented
-  AliWarning("Not implemented yet");
-  return 0x0;
-}
-
-//_____________________________________________________________________________
-AliMUONVStore* 
-AliMUONPadStatusMaker::MakeHVStatus(const TMap& hvValues) const
+Int_t
+AliMUONPadStatusMaker::HVStatus(Int_t detElemId, Int_t manuId) const
 {
-  /// Scrutinize HV values and deduce an HV status for each pad
-  
-  TStopwatch timerSt12;
-  TStopwatch timerSt345;
-  
-  timerSt12.Start(kTRUE);
-  timerSt12.Stop();
-  timerSt345.Start(kTRUE);
-  timerSt345.Stop();
+  /// Get HV status of one manu
   
-  AliMUONHVNamer hvNamer;
+  AliCodeTimerAuto("")
   
-  AliMpDEIterator deIt;
+  if ( !fkCalibrationData.HV() ) return kMissing;
+
+  Long_t lint = fHV->GetValue(AliMpManuUID::BuildUniqueID(detElemId,manuId));
   
-  deIt.First();
+  if ( lint ) 
+  {
+    return (Int_t)(lint - 1);
+  }
+
+  Int_t status(0);
   
-  AliMUONVStore* hv = new AliMUON2DMap(kTRUE);
+  AliMpDCSNamer hvNamer("TRACKER");
   
-  while ( !deIt.IsDone() )
+  switch ( AliMpDEManager::GetStationType(detElemId) )
   {
-    Int_t detElemId = deIt.CurrentDEId();
-    
-    switch ( AliMpDEManager::GetStationType(detElemId) )
+    case AliMp::kStation12:
     {
-      case AliMp::kStation1:
-      case AliMp::kStation2:
-        timerSt12.Start(kFALSE);
-        for ( int sector = 0; sector < 3; ++sector)
-        {
-          AliDebug(1,Form("detElemId %5d sector %d",detElemId,sector));
-
-          Bool_t hvChannelTooLow, hvChannelTooHigh, hvChannelON;
-          Bool_t error = GetSt12Status(hvValues,
-                                       detElemId,sector,
-                                       hvChannelTooLow,hvChannelTooHigh,
-                                       hvChannelON);
-          Int_t status = 0;
-          if ( error ) status |= kHVError;
-          if ( hvChannelTooLow ) status |= kHVTooLow;
-          if ( hvChannelTooHigh ) status |= kHVTooHigh; 
-          if ( !hvChannelON ) status |= kHVChannelOFF;
-          SetStatusSt12(*hv,detElemId,sector,status);
-          
-        }
-          timerSt12.Stop();
-        break;
-      case AliMp::kStation345:
+      int sector = hvNamer.ManuId2Sector(detElemId,manuId);
+      if ( sector >= 0 ) 
       {
-        timerSt345.Start(kFALSE);
-        for ( Int_t pcbIndex = 0; pcbIndex < hvNamer.NumberOfPCBs(detElemId); ++pcbIndex)
-        {
-          AliDebug(1,Form("detElemId %5d pcbIndex %d",detElemId,pcbIndex));
-          Bool_t hvChannelTooLow, hvChannelTooHigh, hvChannelON,hvSwitchON;
-          Bool_t error = GetSt345Status(hvValues,
-                                        detElemId,pcbIndex,
-                                        hvChannelTooLow,hvChannelTooHigh,
-                                        hvChannelON,hvSwitchON);
-          Int_t status = 0;
-          if ( error ) status |= kHVError;
-          if ( hvChannelTooLow ) status |= kHVTooLow;
-          if ( hvChannelTooHigh ) status |= kHVTooHigh; 
-          if ( !hvSwitchON ) status |= kHVSwitchOFF; 
-          if ( !hvChannelON) status |= kHVChannelOFF;
-          SetStatusSt345(*hv,detElemId,pcbIndex,status);
-        }
-        timerSt345.Stop();
+        Bool_t hvChannelTooLow, hvChannelTooHigh, hvChannelON;
+        Bool_t error = HVSt12Status(detElemId,sector,
+                                    hvChannelTooLow,
+                                    hvChannelTooHigh,
+                                    hvChannelON);
+        if ( error ) status |= kHVError;
+        if ( hvChannelTooLow ) status |= kHVTooLow;
+        if ( hvChannelTooHigh ) status |= kHVTooHigh; 
+        if ( !hvChannelON ) status |= kHVChannelOFF;
+        // assign this status to all the other manus handled by the same HV channel
+        SetHVStatus(detElemId,sector,status);
       }
-        break;
-      default:
-        break;
     }
-    deIt.Next();
+      break;
+    case AliMp::kStation345:
+    {
+      int pcbIndex = hvNamer.ManuId2PCBIndex(detElemId,manuId);
+      if ( pcbIndex >= 0 ) 
+      {
+        Bool_t hvChannelTooLow, hvChannelTooHigh, hvChannelON,hvSwitchON;
+        Bool_t error = HVSt345Status(detElemId,pcbIndex,
+                                     hvChannelTooLow,hvChannelTooHigh,
+                                     hvChannelON,hvSwitchON);
+        if ( error ) status |= kHVError;
+        if ( hvChannelTooLow ) status |= kHVTooLow;
+        if ( hvChannelTooHigh ) status |= kHVTooHigh; 
+        if ( !hvSwitchON ) status |= kHVSwitchOFF; 
+        if ( !hvChannelON) status |= kHVChannelOFF;
+        // assign this status to all the other manus handled by the same HV channel
+        SetHVStatus(detElemId,pcbIndex,status);
+      }
+    }
+      break;
+    default:
+      break;
   }
   
-  AliInfo("St12 timer:");
-  StdoutToAliInfo(timerSt12.Print(););
-  AliInfo("St345 timer:");
-  StdoutToAliInfo(timerSt345.Print(););
-  
-  return hv;
+  return status;
+}
+
+//_____________________________________________________________________________
+AliMUONVCalibParam* 
+AliMUONPadStatusMaker::Neighbours(Int_t detElemId, Int_t manuId) const
+{
+  /// Get the neighbours parameters for a given manu
+  AliMUONVStore* neighbourStore = fkCalibrationData.Neighbours();
+  return static_cast<AliMUONVCalibParam*>(neighbourStore->FindObject(detElemId,manuId));
 }
 
 //_____________________________________________________________________________
 AliMUONVStore* 
-AliMUONPadStatusMaker::MakePedestalStatus(const AliMUONVStore& pedValues) const
+AliMUONPadStatusMaker::NeighboursStore() const
 {
-  /// Assign a pedestal status to each pad
-  
-  TStopwatch timer;
-  
-  timer.Start(kTRUE);
+  /// Return the store containing all the neighbours
+  return fkCalibrationData.Neighbours();
+}
+
+//_____________________________________________________________________________
+AliMUONVCalibParam*
+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
   
-  AliMUONVStore* pedStatuses = new AliMUON2DMap(kTRUE);
+  AliMUONVCalibParam* param = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,-1);
+  fStatus->Add(param);
+
+  AliMUONVCalibParam* pedestals = static_cast<AliMUONVCalibParam*>(fPedestals->FindObject(detElemId,manuId));
+
+  AliMUONVCalibParam* gains = static_cast<AliMUONVCalibParam*>(fGains->FindObject(detElemId,manuId));
   
-  TIter next(pedValues.CreateIterator());
-  AliMUONVCalibParam* pedestals;
-  Int_t nofManus(0);
+  Int_t hvStatus = HVStatus(detElemId,manuId);
+
+  Int_t otherStatus = OtherStatus(detElemId,manuId);
   
-  while ( ( pedestals = static_cast<AliMUONVCalibParam*>(next() ) ) )
+  for ( Int_t manuChannel = 0; manuChannel < param->Size(); ++manuChannel )
   {
-    Int_t detElemId = pedestals->ID0();
-    Int_t manuId = pedestals->ID1();
-    ++nofManus;
-    for ( Int_t manuChannel = 0; manuChannel < pedestals->Size(); ++manuChannel )
+    Int_t pedStatus(0);
+    
+    if (pedestals) 
     {
-      Int_t status(0);
-      if ( AliMpManuList::DoesChannelExist(detElemId, manuId, manuChannel) )
-      {
-        Float_t pedMean = pedestals->ValueAsFloat(manuChannel,0);
-        Float_t pedSigma = pedestals->ValueAsFloat(manuChannel,1);
-        if ( pedMean < fPedMeanLimits.X() ) status |= kPedMeanTooLow;
-        if ( pedMean > fPedMeanLimits.Y() ) status |= kPedMeanTooHigh;
-        if ( pedSigma < fPedSigmaLimits.X() ) status |= kPedSigmaTooLow;
-        if ( pedSigma > fPedSigmaLimits.Y() ) status |= kPedSigmaTooHigh;
-        if ( pedMean == 0 ) status |= kPedMeanZero;
-        
-        AliMUONVCalibParam* vStatus = 
-          static_cast<AliMUONVCalibParam*>(pedStatuses->FindObject(detElemId,manuId));
-        if ( !vStatus ) 
-        {
-          vStatus = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,0);
-          pedStatuses->Add(vStatus);
-        }
-        vStatus->SetValueAsInt(manuChannel,0,status);
-      }
+      Float_t pedMean = pedestals->ValueAsFloatFast(manuChannel,0);
+      Float_t pedSigma = pedestals->ValueAsFloatFast(manuChannel,1);
+      if ( pedMean < fPedMeanLimits.X() ) pedStatus |= kPedMeanTooLow;
+      else if ( pedMean > fPedMeanLimits.Y() ) pedStatus |= kPedMeanTooHigh;
+      if ( pedSigma < fPedSigmaLimits.X() ) pedStatus |= kPedSigmaTooLow;
+      else if ( pedSigma > fPedSigmaLimits.Y() ) pedStatus |= kPedSigmaTooHigh;
+      if ( pedMean == 0 ) pedStatus |= kPedMeanZero;
+    }
+    else
+    {
+      pedStatus = kPedMissing;
+    }
+    
+    Int_t gainStatus(0);
+  
+    if ( gains ) 
+    {
+      Float_t a0 = gains->ValueAsFloatFast(manuChannel,0);
+      Float_t a1 = gains->ValueAsFloatFast(manuChannel,1);
+      Float_t thres = gains->ValueAsFloatFast(manuChannel,2);
+  
+      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;
+    }
+    else
+    {
+      gainStatus = kGainMissing;
     }
+        
+    Int_t status = BuildStatus(pedStatus,hvStatus,gainStatus,otherStatus);
+      
+    param->SetValueAsIntFast(manuChannel,0,status);
   }
   
-  AliInfo(Form("%d manus checked in :",nofManus));
-  StdoutToAliInfo(timer.Print(););
-  return pedStatuses;  
+  return param;
 }
 
 //_____________________________________________________________________________
-AliMUONVStore* 
-AliMUONPadStatusMaker::MakeStatus() const
+Int_t 
+AliMUONPadStatusMaker::OtherStatus(Int_t detElemId, Int_t manuId) const
 {
-  /// Read ped, gains and hv values from CDB, apply some Q&A and produces
-  /// a combined status for each pad.
-
-  TMap* hvValues = fCalibrationData.HV();
-  AliMUONVStore* hvStatus(0x0);
-  
-  if (!hvValues)
-  {
-    AliError("Could not get HV values from CDB. Will create dummy ones and mark those as missing");
-    hvStatus = GeneratePadStatus(kHVMissing);
-  }
-  else
+  /// Get the "other" status for a given manu
+  if ( fTrackerData ) 
   {
-    hvStatus = MakeHVStatus(*hvValues);
+    Double_t occ = fTrackerData->Manu(detElemId,manuId,2);
+    if ( occ < fManuOccupancyLimits.X() )
+    {
+      return kManuOccupancyTooLow;
+    }
+    if ( occ > fManuOccupancyLimits.Y() )
+    {
+      return kManuOccupancyTooHigh;
+    }
   }
-  
-  AliMUONVStore* pedValues = fCalibrationData.Pedestals();
-  AliMUONVStore* pedStatus(0x0);
+  return 0;
+}
 
-  if (!pedValues)
-  {
-    AliError("Could not get pedestals values from CDB. Will create dummy ones and mark those as missing");
-    pedStatus = GeneratePadStatus(kPedMissing);
-  }
-  else
-  {
-    pedStatus = MakePedestalStatus(*pedValues);
-  }
-  
-  // FIXME: should do the same for gains as for hv and ped.    
-  
-  AliMUONVStore* status = Combine(*hvStatus,*pedStatus,8);
-  
-  delete hvStatus;
-  delete pedStatus;
-  
-  // Insure we get all channels there (some or even all can be bad, but they
-  // must be there somehow).
+//_____________________________________________________________________________
+AliMUONVCalibParam* 
+AliMUONPadStatusMaker::PadStatus(Int_t detElemId, Int_t manuId) const
+{
+  /// Get the status container for a given manu
   
-  AliMUON2DStoreValidator validator;
-        
-  TObjArray* a = validator.Validate(*status);
-    
-  if (a) 
+  AliMUONVCalibParam* param = static_cast<AliMUONVCalibParam*>(fStatus->FindObject(detElemId,manuId));
+  if (!param)
   {
-    // this should not happen.
-    AliError("Status store not complete. Crash to follow soon...");
-    StdoutToAliError(a->Print(););
-    AliFatal("this should not happen at all!");
-    delete status;
-    status = 0x0;
+    // not already there, so compute it now
+    AliCodeTimerAuto("ComputeStatus");
+    param = ComputeStatus(detElemId,manuId);
   }
-    
-  return status;
+  return param;
 }
 
 //_____________________________________________________________________________
-void
-AliMUONPadStatusMaker::SetStatusSt12(AliMUONVStore& hvStatus,
-                                     Int_t detElemId, 
-                                     Int_t isector,
-                                     Int_t status) const
+Int_t 
+AliMUONPadStatusMaker::PadStatus(Int_t detElemId, Int_t manuId, Int_t manuChannel) const
 {
-  /// Flag all pads of detElemId (for St12) as bad.
-  
-  // FIXME: need a way to iterator on pads over a given HV sector for St12... 
-  // we currently suppose that one sector is about a third of the chamber...
-  // FIXME !! This has to be checked very carefully...
+  /// Get the status for a given channel
   
-  const AliMp::CathodType kCathodes[] = { AliMp::kCath0, AliMp::kCath1 };
-  
-  for ( Int_t icathode = 0; icathode < 2; ++icathode )
+  AliMUONVCalibParam* param = static_cast<AliMUONVCalibParam*>(fStatus->FindObject(detElemId,manuId));
+  if (!param)
   {
-    const AliMpSectorSegmentation* seg = 
-    static_cast<const AliMpSectorSegmentation*>(AliMpSegmentation::Instance()->GetMpSegmentation(detElemId,kCathodes[icathode]));
-    const AliMpSector* sector = seg->GetSector();
-    AliMpMotifMap* mMap = sector->GetMotifMap();
-    TArrayI a;
-    
-    mMap->GetAllMotifPositionsIDs(a);
-    
-    TVector2 dim = seg->Dimensions();
-    Double_t x = dim.X()*2;
-    Double_t xmin = isector*x/3.0;
-    Double_t xmax = xmin + x/3.0;   
-    
-    for ( Int_t i = 0; i < a.GetSize(); ++i ) 
-    {
-      AliMpMotifPosition* pos = mMap->FindMotifPosition(a[i]);
-      Int_t manuId = pos->GetID();
-      TVector2 position = pos->Position();
-      if ( position.X() >= xmin && position.X() <= xmax) 
-      {
-        AliMUONVCalibParam* dead =
-        static_cast<AliMUONVCalibParam*>(hvStatus.FindObject(detElemId,manuId));
-        if (!dead)
-        {
-          dead = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,status);
-          hvStatus.Add(dead);
-        }        
-        else
-        {
-          // FIXME: this should really not happen, if we'd know really the
-          // relationship between manuId and HV sector...
-          // For the time being, let's leave it like that, for testing
-          // purposes only. For production, this will have to be fixed.
-          AliWarning("Please fixme.");
-        }
-      }
-    }
-  }  
+    // not already there, so compute it now
+    param = ComputeStatus(detElemId,manuId);
+  }
+  return param->ValueAsInt(manuChannel,0);
 }
 
 //_____________________________________________________________________________
 void
-AliMUONPadStatusMaker::SetStatusSt345(AliMUONVStore& hvStatus,
-                                      Int_t detElemId, Int_t pcbIndex,
-                                      Int_t status) const
+AliMUONPadStatusMaker::SetHVStatus(Int_t detElemId, Int_t index, Int_t status) const
 {
-  /// Flag all pads of pcbIndex-th PCB of detElemId (for St345) as bad.
+  /// Assign status to all manus in a given HV "zone" (defined by index, meaning
+  /// is different thing from St12 and St345)
+  
+  AliCodeTimerAuto("")
+  
+  AliMpDetElement* de = AliMpDDLStore::Instance()->GetDetElement(detElemId);
   
-  const AliMp::CathodType kCathodes[] = { AliMp::kCath0, AliMp::kCath1 };
+  const AliMpArrayI* manus = de->ManusForHV(index);
   
-  for ( Int_t icathode = 0; icathode < 2; ++icathode )
+  for ( Int_t i = 0; i < manus->GetSize(); ++ i ) 
   {
-    const AliMpSlatSegmentation* seg = static_cast<const AliMpSlatSegmentation*>
-    (AliMpSegmentation::Instance()->GetMpSegmentation(detElemId,kCathodes[icathode]));
-    const AliMpSlat* slat = seg->Slat();
-    const AliMpPCB* pcb = slat->GetPCB(pcbIndex);
-
-    for ( Int_t i = 0; i < pcb->GetSize(); ++i ) 
-    {
-      AliMpMotifPosition* pos = pcb->GetMotifPosition(i);
-      Int_t manuId = pos->GetID();
-      AliMUONVCalibParam* dead = 
-        static_cast<AliMUONVCalibParam*>(hvStatus.FindObject(detElemId,manuId));
-      if (dead)
-      {
-        AliError(Form("dead is not null as expected from DE %d manuId %d",
-                      detElemId,manuId));
-      }
-      if (!dead)
-      {
-        dead = new AliMUONCalibParamNI(1,AliMpConstants::ManuNofChannels(),detElemId,manuId,status);
-        hvStatus.Add(dead);
-      }
-    }    
+    Int_t manuId = manus->GetValue(i);
+    fHV->Add(AliMpManuUID::BuildUniqueID(detElemId,manuId),status + 1);
   }
 }
-
-
-