]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - STEER/AliGRPPreprocessor.cxx
correct for omission
[u/mrichter/AliRoot.git] / STEER / AliGRPPreprocessor.cxx
index 9035388040cd6cb009ec847540992790d4efb2ae..becdb0c455442e4ea936416354ef707515f218af 100644 (file)
 
 #include "AliTriggerConfiguration.h"
 #include "AliTriggerRunScalers.h"
+#include "AliTriggerInput.h"
 
 #include "AliCDBMetaData.h"
-#include "AliLog.h"
 #include "AliESDVertex.h"
 #include "AliLHCReader.h"
 #include "AliLHCData.h"
 #include "AliDCSArray.h"
+#include "AliDAQ.h"
+#include "AliLTUConfig.h"
 
+class AliLog;
 class AliDCSValue;
 class AliShuttleInterface;
 
@@ -64,6 +67,7 @@ class AliShuttleInterface;
 #include <AliCDBId.h>
 #include <AliTriggerConfiguration.h>
 #include <AliCTPTimeParams.h>
+#include <AliLHCClockPhase.h>
 
 const Double_t kFitFraction = -1.;                 // Fraction of DCS sensor fits required
 
@@ -71,11 +75,10 @@ ClassImp(AliGRPPreprocessor)
 
 //_______________________________________________________________
 
-  const Int_t AliGRPPreprocessor::fgknDAQLbPar = 8; // num parameters in the logbook for PHYSICS runs, when beamType from DAQ logbook == NULL
-  const Int_t AliGRPPreprocessor::fgknDAQLbParReduced = 7; // num parameters in the logbook for the other cases
+  const Int_t AliGRPPreprocessor::fgknDAQLbPar = 6; // num parameters in the logbook used to fill the GRP object
   const Int_t AliGRPPreprocessor::fgknDCSDP = 48;   // number of dcs dps
   const Int_t AliGRPPreprocessor::fgknDCSDPHallProbes = 40;   // number of dcs dps
-  const Int_t AliGRPPreprocessor::fgknLHCDP = 5;   // number of dcs dps from LHC data
+  const Int_t AliGRPPreprocessor::fgknLHCDP = 6;   // number of dcs dps from LHC data
   const Int_t AliGRPPreprocessor::fgkDCSDPHallTopShift = 4;   // shift from the top to get tp the Hall Probes names in the list of DCS DPs
   const Int_t AliGRPPreprocessor::fgkDCSDPNonWorking = 5; // number of non working DCS DPs
   const char* AliGRPPreprocessor::fgkDCSDataPoints[AliGRPPreprocessor::fgknDCSDP] = {
@@ -180,9 +183,11 @@ ClassImp(AliGRPPreprocessor)
          "LHC_Beam_Energy",
          "LHC_MachineMode",
          "LHC_BeamMode",
-          "dip/acc/LHC/RunControl/BeamType/Beam1.payload",
-          "dip/acc/LHC/RunControl/BeamType/Beam2.payload"
+          "LHC_Beams_Particle_Type",
+         "BPTX_Phase_Shift_B1",
+         "BPTX_Phase_Shift_B2"
   };
+
   const char* kppError[] = {
                    "",
                    "(DAQ logbook ERROR)",
@@ -194,7 +199,9 @@ ClassImp(AliGRPPreprocessor)
                    "(CTP timing ERROR)",
                   "(SPD Mean Vertex ERROR)",
                   "(DCS FXS Error for LHC Data)",
-                  "(LHC Data Error)"
+                  "(LHC Data Error)",
+                  "(LHC Clock Phase Error (from LHC Data))",
+                  "(LTU Configuration Error)"
   };
 
 //_______________________________________________________________
@@ -299,22 +306,27 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
        //=================//
        // DAQ logbook     //
        //=================//
+
+       Log("*************** Processing DAQ logbook");
+
        UInt_t error = 0;
        
        Int_t iDaqLB = ProcessDaqLB(grpobj);
        TString runType = (TString)GetRunType();
        TString beamType = (TString)GetRunParameter("beamType");
-       //if((runType == "PHYSICS" && iDaqLB == fgknDAQLbPar && beamType!="Cosmics") ||  (runType == "PHYSICS" && iDaqLB == fgknDAQLbParReduced && beamType=="Cosmics") || (runType != "PHYSICS" && iDaqLB == fgknDAQLbParReduced)) {
-       if((runType == "PHYSICS" && iDaqLB == fgknDAQLbPar && !beamType.IsNull()) ||  (runType == "PHYSICS" && iDaqLB == fgknDAQLbParReduced && beamType.IsNull()) || (runType != "PHYSICS" && iDaqLB == fgknDAQLbParReduced)) {
-               Log(Form("DAQ Logbook, successful!"));
+       if(iDaqLB == fgknDAQLbPar) {
+               Log(Form("DAQ Logbook, successful! Retrieved %d/%d entries",iDaqLB,fgknDAQLbPar));
        } else {
-               Log(Form("DAQ Logbook, could not get all expected entries!!!"));
+               Log(Form("DAQ Logbook, could not get all expected entries!!! Retrieved only %d/%d entries",iDaqLB,fgknDAQLbPar));
                error |= 1;
        }
 
        //=================//
        // DAQ FXS         //
        //=================//
+
+       Log("*************** Processing DAQ FXS");
+
        UInt_t iDaqFxs = ProcessDaqFxs();
        if( iDaqFxs == 0 ) {
                Log(Form("DAQ FXS, successful!"));
@@ -326,6 +338,9 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
        //=================//
        // DCS FXS         //
        //=================//
+
+       Log("*************** Processing DCS FXS");
+
        UInt_t iDcsFxs = ProcessDcsFxs(partition, detector);
        if( iDcsFxs == 0 ) {
                Log(Form("DCS FXS, successful!"));
@@ -340,6 +355,9 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
        //=================//
        // DCS data points //
        //=================//
+
+       Log("*************** Processing DCS DPs");
+
        Log(Form("Starting DCS Query at %d and finishing at %d",GetStartTimeDCSQuery(),GetEndTimeDCSQuery()));
        Int_t entries = ProcessDcsDPs( valueMap, grpobj );
        Log(Form("entries found = %d (should be %d)",entries, fgknDCSDP-fgkDCSDPNonWorking));
@@ -369,6 +387,8 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
        // Trigger Configuration //
        //=======================//
 
+       Log("*************** Processing Trigger Configuration");
+
        const char * triggerConf = GetTriggerConfiguration();
 
        if (partition.IsNull() && !detector.IsNull()){ // standalone partition
@@ -436,6 +456,7 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
        // Trigger Timing Parameters //
         //===========================//
 
+       Log("*************** Processing Trigger Time Params");
        
        const char * triggerCTPtiming = GetCTPTimeParams();
 
@@ -501,30 +522,127 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
                error |= 32;
        }
 
+        //===========================//
+       // LTU Configuration         //
+        //===========================//
+
+       Log("*************** Processing LTU Configuration");
+       
+       if (partition.IsNull() && !detector.IsNull()){ // standalone partition
+               Log("STANDALONE partition for current run, using LTU configuration dummy value");
+               AliCDBEntry *cdbEntry = GetFromOCDB("CTP","DummyLTUConfig");
+               if (!cdbEntry) {
+                       Log(Form("No dummy LTU Config entry found, going into error..."));
+                       error |= 2048;
+               }
+               else{
+                       TObjArray *ltuConfig = (TObjArray*)cdbEntry->GetObject();
+                       if (!ltuConfig){
+                               Log(Form("dummy LTU Config not found in OCDB entry, going into error..."));
+                               error |= 2048;
+                       }
+                       else {
+                               AliCDBMetaData metadata;
+                               metadata.SetResponsible("Roman Lietava");
+                               metadata.SetComment("LTU Config from dummy entry in OCDB");
+                               if (!Store("CTP","LTUConfig", ltuConfig, &metadata, 0, 0)) {
+                                       Log("Unable to store the dummy LTU Config object to OCDB!");
+                                       error |= 2048;
+                               }
+                       }
+               }
+       }
+
+       else if (!partition.IsNull() && detector.IsNull()){ // global partition
+       
+               Log("GLOBAL partition for current run, getting LTU Config from DAQ Logbook (logbook_detectors table)");
+               UInt_t  detectorMask = (UInt_t)(((TString)GetRunParameter("detectorMask")).Atoi());
+               Printf ("detectormask = %d",detectorMask);
+               TObjArray * ltuarray = new TObjArray();
+               ltuarray->SetOwner(1);
+               Bool_t isLTUok = kTRUE;
+               for(Int_t i = 0; i<AliDAQ::kNDetectors-2; i++){
+                       if ((detectorMask >> i) & 0x1) {
+                               TString det = AliDAQ::OfflineModuleName(i);
+                               TString detCTPName = AliTriggerInput::fgkCTPDetectorName[i];
+                               if (detCTPName == "CTP") {
+                                       detCTPName="TRG"; // converting according to what is found in DAQ logbook_detectors                                     
+                                       Printf("Processing CTP (CTP Detector name %s) --> SKIPPING, CTP does not have any LTU!!!!!!",detCTPName.Data());
+                                       continue;
+                               }                               
+                               Printf("Processing detector %s (CTP Detector name %s)",det.Data(),detCTPName.Data());
+                               TString* ltu = GetLTUConfig(detCTPName.Data());
+                               if (!ltu){
+                                       Log(Form("No LTU Configuration from DAQ logbook for detector %s (BUT it was expected)! The corresponding CDB entry will not be filled!",detCTPName.Data()));
+                                       error |= 2048;
+                                       isLTUok = kFALSE;
+                                       break;
+                               }
+                               else{
+                                       Float_t ltuFineDelay1 = ltu[0].Atof();
+                                       Float_t ltuFineDelay2 = ltu[1].Atof();
+                                       Float_t ltuBCDelayAdd = ltu[2].Atof();
+                                       const char* name = AliDAQ::DetectorName(i);
+                                       AliLTUConfig* ltuConfig = new AliLTUConfig((UChar_t)AliDAQ::DetectorID(name),ltuFineDelay1,ltuFineDelay2,ltuBCDelayAdd);
+                                       ltuarray->AddAtAndExpand(ltuConfig,i);
+                               }                               
+                       }
+               }
+               if (isLTUok){
+                       AliCDBMetaData metadata;
+                       metadata.SetBeamPeriod(0);
+                       metadata.SetResponsible("Roman Lietava");
+                       metadata.SetComment("LTU Configuration for current run");
+                       if (!Store("CTP","LTUConfig", ltuarray, &metadata, 0, 0)) {
+                               Log("Unable to store the LTU Config object to OCDB!");
+                               error |= 2048;
+                       }               
+               }
+               delete ltuarray;
+       }
+
+       else {
+               Log(Form("Incorrect field in DAQ logbook for partition = %s and detector = %s, going into error without trigger timing parameters...",partition.Data(),detector.Data()));
+               error |= 32;
+       }
+
 
        //=================//
        // LHC Data        //
        //=================//
 
-       UInt_t iLHCData = ProcessLHCData(grpobj);
-
-       if( iLHCData == 0 ) {
-               Log(Form("LHC Data from DCS FXS, successful!"));
-       } else  if (iLHCData == 1) {
-               Log(Form("LHC Data, problems with DCS FXS!"));
-               error |= 256;
-       } else if (iLHCData ==3){
-               Log(Form("Problems in storing LHC Data - but not going into Error"));
-       } else if (iLHCData ==4){
-               Log(Form("Problems with LHC Data to be put in /GRP/GRP/LHCData - but not going into Error"));
-       } else{
-               Log(Form("LHC Data problems"));
-               error |= 512;
-       }
+       if (runType == "PHYSICS"){  // processing the LHC file only in PHYSICS runs
+               Log("*************** Processing LHC Data");
+
+               UInt_t iLHCData = ProcessLHCData(grpobj);
+               
+               if( iLHCData == 0 ) {
+                       Log(Form("LHC Data from DCS FXS, successful!"));
+               } else  if (iLHCData == 1) {
+                       Log(Form("LHC Data, problems with DCS FXS!"));
+                       error |= 256;
+               } else  if (iLHCData == 2) {
+                       Log(Form("LHC Data, problems with DAQ_time_start/DAQ_time_end!"));
+                       error |= 512;
+               } else if (iLHCData ==3){
+                       Log(Form("Problems in storing LHC Phase - going into Error"));
+                       error |= 1024;
+               } else if (iLHCData ==4){
+                       Log(Form("Problems with LHC Phase - going into Error"));
+                       error |= 1024;
+               } else{
+                       Log(Form("LHC Data problems"));
+                       error |= 512;
+               }
        
+       }
+
        //==================//
        // SPD Mean Vertex  //
        //==================//
+
+       Log("*************** Processing SPD Mean Vertex");
+
        if (runType == "PHYSICS"){
                UInt_t iSPDMeanVertex = ProcessSPDMeanVertex();
                if( iSPDMeanVertex == 1 ) {
@@ -552,7 +670,7 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
                Log("GRP Preprocessor Success");
                return 0;
        } else {
-               Log( Form("GRP Preprocessor FAILS!!! %s%s%s%s%s%s%s%s%s%s",
+               Log( Form("GRP Preprocessor FAILS!!! %s%s%s%s%s%s%s%s%s%s%s%s",
                          kppError[(error&1)?1:0],
                          kppError[(error&2)?2:0],
                          kppError[(error&4)?3:0],
@@ -562,7 +680,9 @@ UInt_t AliGRPPreprocessor::Process(TMap* valueMap)
                          kppError[(error&64)?7:0],
                          kppError[(error&128)?8:0],
                          kppError[(error&256)?9:0],
-                         kppError[(error&512)?10:0]
+                         kppError[(error&512)?10:0],
+                         kppError[(error&1024)?11:0],
+                         kppError[(error&2048)?12:0]
                          ));
                return error;
        }
@@ -597,186 +717,303 @@ UInt_t AliGRPPreprocessor::ProcessLHCData(AliGRPObject *grpobj)
        if (fileName.Length()>0){
                AliInfo("Got The LHC Data file");
                AliLHCReader lhcReader;
-               TMap* lhcMap = (TMap*)lhcReader.ReadLHCDP(fileName.Data());
-               if (lhcMap) {
-                       Log(Form("LHCData map entries = %d",lhcMap->GetEntries()));
-                       
-                       // Processing data to be put in AliGRPObject
-                       // Energy
-                       TObjArray* energyArray = (TObjArray*)lhcMap->GetValue(fgkLHCDataPoints[0]);
-                       if (energyArray){                       
-                               Float_t energy = ProcessEnergy(energyArray,timeStart,timeEnd);
-                               if (energy != -1) {
-                                       grpobj->SetBeamEnergy(energy);
-                                       grpobj->SetBeamEnergyIsSqrtSHalfGeV(kTRUE);
-                               }
+
+               // Processing data to be put in AliGRPObject
+
+               // Energy
+               Log("*************Energy ");
+               TObjArray* energyArray = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[0]);
+               if (energyArray){                       
+                       Float_t energy = ProcessEnergy(energyArray,timeStart);
+                       if (energy != -1.) {
+                               grpobj->SetBeamEnergy(energy);
+                               grpobj->SetBeamEnergyIsSqrtSHalfGeV(kTRUE);
                        }
-                       else {
-                               AliError("Energy not found in LHC Data file!!!");
-                       }       
-                       Double_t timeBeamMode = 0;
-                       Double_t timeMachineMode = 0;
-                       Bool_t flagBeamMode = kFALSE;  //flag set true if at least one BeamMode measurement is found within DAQ_time_start and DAQ_time_end
-                       Bool_t flagMachineMode = kFALSE;  //flag set true if at least one MachineMode measurement is found within DAQ_time_start and DAQ_time_end
-                       // BeamMode
-                       TObjArray* beamModeArray = (TObjArray*)lhcMap->GetValue(fgkLHCDataPoints[2]);
-                       Int_t nBeamMode = -1;
-                       if (beamModeArray){     
-                               nBeamMode = beamModeArray->GetEntries();        
-                               if (nBeamMode==0){
-                                       AliInfo("Found zero entries for  the Beam Mode, setting it to UNKNOWN");
-                                       grpobj->SetLHCState("UNKNOWN");
-                               }
-                               else{
-                                       if (nBeamMode==1){
-                                               AliDCSArray* beamMode = (AliDCSArray*)beamModeArray->At(0);
-                                               if (beamMode->GetTimeStamp()>=timeStart && beamMode->GetTimeStamp()<=timeEnd){
-                                                       TObjString* beamModeString = beamMode->GetStringArray(0);
-                                                       AliInfo(Form("LHC State (corresponding to BeamMode) = %s",(beamModeString->String()).Data()));
-                                                       grpobj->SetLHCState(beamModeString->String());
+                       delete energyArray;
+               }
+               else {
+                       AliError("Energy not found in LHC Data file!!!");
+               }       
+
+               Double_t timeBeamModeEnd = timeEnd;        // max validity for Beam Mode 
+               Double_t timeMachineModeEnd = timeEnd;     // max validity for Machine Mode
+               Double_t timeBeamEnd = timeEnd;            // max validity for Beam Type
+               Double_t timeBeamModeStart = -1;    // min validity for Beam Mode
+               Double_t timeMachineModeStart = -1; // min validity for Machine Mode
+               Double_t timeBeamStart = -1;        // min validity for Beam Type
+               Int_t indexBeamMode = -1;                  // index of measurement used to set Beam Mode
+               Int_t indexMachineMode = -1;               // index of measurement used to set Beam Mode
+               Int_t indexBeam = -1;                      // index of measurement used to set Beam Mode
+               Bool_t foundBeamModeStart = kFALSE;        // flag to be set in case an entry for the Beam Mode is found before (or at) SOR
+               Bool_t foundMachineModeStart = kFALSE;     // flag to be set in case an entry for the Beam Mode is found before (or at) SOR
+               Bool_t foundBeamStart = kFALSE;            // flag to be set in case an entry for the Beam Mode is found before (or at) SOR
+               Bool_t flagBeamMode = kFALSE;  //flag set true if a changed occurred in BeamMode
+               Bool_t flagMachineMode = kFALSE;  //flag set true if a changed occurred in MachineMode
+               Bool_t flagBeam = kFALSE;  //flag set true if a changed occurred in BeamType
+               
+               // BeamMode
+               Log("*************BeamMode (LHCState) ");
+               TObjArray* beamModeArray = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[2]);
+               Int_t nBeamMode = -1;
+               if (beamModeArray){     
+                       nBeamMode = beamModeArray->GetEntries();        
+                       if (nBeamMode==0){
+                               AliInfo("Found zero entries for the Beam Mode, leaving it empty");
+                       }
+                       else{
+                               for (Int_t iBeamMode = 0; iBeamMode<nBeamMode; iBeamMode++){
+                                       AliDCSArray* beamMode = (AliDCSArray*)beamModeArray->At(iBeamMode);
+                                       if (beamMode){
+                                               if (beamMode->GetTimeStamp()<=timeStart && beamMode->GetTimeStamp()>=timeBeamModeStart){// taking always the very last entry: of two measurements have the same timestamp, the last one is taken
+                                                       timeBeamModeStart = beamMode->GetTimeStamp();
+                                                       indexBeamMode = iBeamMode;
+                                                       foundBeamModeStart = kTRUE;
                                                }
-                                               else{
-                                                       AliInfo("No Beam Mode found within DAQ_time_start and DAQ_time_end, setting it to UNKNOWN");
-                                                       grpobj->SetLHCState("UNKNOWN");
+                                               else {
+                                                       break;
+
                                                }
                                        }
-                                       else {
-                                               for (Int_t iBeamMode = 0; iBeamMode<nBeamMode; iBeamMode++){
-                                                       AliDCSArray* beamMode = (AliDCSArray*)beamModeArray->At(iBeamMode);
-                                                       if (beamMode->GetTimeStamp()>=timeStart && beamMode->GetTimeStamp()<=timeEnd){
-                                                               AliDCSArray* beamMode1 = (AliDCSArray*)beamModeArray->At(iBeamMode+1);
-                                                               if (beamMode1->GetTimeStamp()>=timeStart && beamMode1->GetTimeStamp()<=timeEnd){
-                                                                       timeBeamMode = beamMode1->GetTimeStamp();
-                                                                       AliWarning(Form("The beam mode changed at timestamp %f! Setting it to the first value found and keeping track of the time of the change to set MaxTimeLHCValidity afterward",timeBeamMode));
+                               }
+                               if (!foundBeamModeStart){
+                                       AliInfo("No value for the Beam Mode found before start of run, the Beam Mode will remain empty");
+                               }
+                               else {
+                                       AliDCSArray* beamMode = (AliDCSArray*)beamModeArray->At(indexBeamMode);
+                                       TObjString* beamModeString = beamMode->GetStringArray(0);
+                                       AliInfo(Form("LHC State (corresponding to BeamMode) = %s (set at %f)",(beamModeString->String()).Data(),beamMode->GetTimeStamp()));
+                                       grpobj->SetLHCState(beamModeString->String());
+                                       if (indexBeamMode < nBeamMode-1){
+                                               AliDCSArray* beamMode1 = (AliDCSArray*)beamModeArray->At(indexBeamMode+1);
+                                               if (beamMode1){
+                                                       if (beamMode1->GetTimeStamp()<=timeStart){
+                                                               AliError("you did not choose the correct value! there is still something before (or at) SOR, but later than this!");
+                                                       }
+                                                       else if (beamMode1->GetTimeStamp()>timeStart && beamMode1->GetTimeStamp()<=timeEnd){
+                                                               timeBeamModeEnd = beamMode1->GetTimeStamp();
+                                                               TObjString* beamModeString1 = beamMode1->GetStringArray(0);
+                                                               TString bmString0 = beamModeString->String();
+                                                               TString bmString1 = beamModeString1->String();
+                                                               if (bmString0.CompareTo(bmString1.Data(),TString::kIgnoreCase) == -1){
+                                                                       AliWarning(Form("The beam mode changed from %s to %s during the run at timestamp %f! Setting it to %s and keeping track of the time of the change to set MaxTimeLHCValidity afterward",bmString0.Data(), bmString1.Data(), timeBeamModeEnd, bmString0.Data()));
+                                                                       flagBeamMode = kTRUE;
                                                                }
-                                                               TObjString* beamModeString = beamMode->GetStringArray(0);
-                                                               AliInfo(Form("LHC State (corresponding to BeamMode) = %s",(beamModeString->String()).Data()));
-                                                               grpobj->SetLHCState(beamModeString->String());
-                                                               flagBeamMode = kTRUE;
-                                                               break;
                                                        }
                                                }
-                                               if (!flagBeamMode){
-                                                       AliError("Found values for BeamMode, but none within DAQ_time_start and DAQ_time_end, setting BeamMode to UNKNOWN");
-                                                       grpobj->SetLHCState("UNKONWN");
+                                               else {
+                                                       AliInfo("Invalid pointer for the first entry for Beam Mode after the first valid one, not considering anything after what has already been found");
                                                }
                                        }
                                }
                        }
-                       else{
-                               AliError("Beam mode array not found in LHC Data file!!!");
+                       delete beamModeArray;
+               }
+               else{
+                       AliError("Beam mode array not found in LHC Data file!!!");
+               }
+               
+               // MachineMode
+               Log("*************MachineMode ");
+               TObjArray* machineModeArray = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[1]);
+               Int_t nMachineMode = -1;
+               if (machineModeArray){
+                       nMachineMode = machineModeArray->GetEntries();
+                       if (nMachineMode==0){
+                               AliInfo("No Machine Mode found, leaving it empty");
                        }
-
-                       // MachineMode
-                       TObjArray* machineModeArray = (TObjArray*)lhcMap->GetValue(fgkLHCDataPoints[1]);
-                       Int_t nMachineMode = -1;
-                       if (machineModeArray){
-                               nMachineMode = machineModeArray->GetEntries();
-                               if (nMachineMode==0){
-                                       AliInfo("No Machine Mode found, setting it to UNKNOWN");
-                                       grpobj->SetMachineMode("UNKNOWN");
-                               }
-                               else{
-                                       if (nMachineMode ==1) {
-                                               AliDCSArray* machineMode = (AliDCSArray*)machineModeArray->At(0);
-                                               if (machineMode->GetTimeStamp()>=timeStart && machineMode->GetTimeStamp()<=timeEnd){
-                                                       TObjString* machineModeString = machineMode->GetStringArray(0);
-                                                       AliInfo(Form("Machine Mode = %s",(machineModeString->String()).Data()));
-                                                       grpobj->SetMachineMode(machineModeString->String());
+                       else{
+                               for (Int_t iMachineMode = 0; iMachineMode<nMachineMode; iMachineMode++){
+                                       AliDCSArray* machineMode = (AliDCSArray*)machineModeArray->At(iMachineMode);
+                                       if (machineMode){
+                                               if (machineMode->GetTimeStamp()<=timeStart && machineMode->GetTimeStamp()>=timeMachineModeStart){// taking always the very last entry: of two measurements have the same timestamp, the last one is taken
+                                                       timeMachineModeStart = machineMode->GetTimeStamp();
+                                                       indexMachineMode = iMachineMode;
+                                                       foundMachineModeStart = kTRUE;
                                                }
                                                else{
-                                                       AliInfo("No Machine Mode found within DAQ_time_start and DAQ_time_end, setting it to UNKNOWN");
-                                                       grpobj->SetMachineMode("UNKNOWN");
+                                                       break;
                                                }
                                        }
-                                       else {
-                                               for (Int_t iMachineMode = 0; iMachineMode<nMachineMode; iMachineMode++){
-                                                       AliDCSArray* machineMode = (AliDCSArray*)machineModeArray->At(iMachineMode);
-                                                       if (machineMode->GetTimeStamp()>=timeStart && machineMode->GetTimeStamp()<=timeEnd){
-                                                               AliDCSArray* machineMode1 = (AliDCSArray*)machineModeArray->At(iMachineMode+1);
-                                                               if (machineMode1->GetTimeStamp()>=timeStart && machineMode1->GetTimeStamp()<=timeEnd){
-                                                                       timeMachineMode = machineMode1->GetTimeStamp();
-                                                                       AliWarning(Form("The machine mode changed at timestamp %f! Setting it to the first value found and keeping track of the time of the change to set MaxTimeLHCValidity afterwards",timeMachineMode));
+                               }
+                               if (!foundMachineModeStart){
+                                       AliInfo("No value for the Machine Mode found before start of run, the Machine Mode will remain empty");
+                               }
+                               else {
+                                       AliDCSArray* machineMode = (AliDCSArray*)machineModeArray->At(indexMachineMode);
+                                       TObjString* machineModeString = machineMode->GetStringArray(0);
+                                       AliInfo(Form("MachineMode = %s (set at %f)",(machineModeString->String()).Data(),machineMode->GetTimeStamp()));
+                                       grpobj->SetMachineMode(machineModeString->String());
+                                       if (indexMachineMode < nMachineMode-1){
+                                               AliDCSArray* machineMode1 = (AliDCSArray*)machineModeArray->At(indexMachineMode+1);
+                                               if (machineMode1){
+                                                       if (machineMode1->GetTimeStamp()>timeStart && machineMode1->GetTimeStamp()<=timeEnd){
+                                                               timeMachineModeEnd = machineMode1->GetTimeStamp();
+                                                               TObjString* machineModeString1 = machineMode1->GetStringArray(0);
+                                                               TString mmString0 = machineModeString->String();
+                                                               TString mmString1 = machineModeString1->String();
+                                                               if (mmString0.CompareTo(mmString1.Data(),TString::kIgnoreCase) == -1){
+                                                                       AliWarning(Form("The machine mode changed from %s to %s during the run at timestamp %f! Setting it to %s and keeping track of the time of the change to set MaxTimeLHCValidity afterward",mmString0.Data(),mmString1.Data(),timeMachineModeEnd,mmString0.Data()));
+                                                                       flagMachineMode = kTRUE;
                                                                }
-                                                               TObjString* machineModeString = machineMode->GetStringArray(0);
-                                                               AliInfo(Form("Machine mode = %s",(machineModeString->String()).Data()));
-                                                               grpobj->SetMachineMode(machineModeString->String());
-                                                               flagMachineMode = kTRUE;
-                                                               break;
                                                        }
                                                }
-                                               if (!flagMachineMode){
-                                                       AliError("Found values for MachineMode, but none within DAQ_time_start and DAQ_time_end, setting MachineMode to UNKNOWN");
-                                                       grpobj->SetMachineMode("UNKONWN");
+                                               else {
+                                                       AliInfo("Invalid pointer for the first entry for Machine Mode after the first valid one, not considering anything after what has already been found");
                                                }
                                        }
                                }
                        }
-                       else {
-                               AliError("Machine mode array not found in LHC Data file!!!");
-                       }       
-
-                       // Setting minTimeLHCValidity
-                       if (timeBeamMode!=0 || timeMachineMode!=0){
-                               Double_t minTimeLHCValidity;
-                               if (timeBeamMode == 0){
-                                       minTimeLHCValidity = timeMachineMode;
+                       delete machineModeArray;
+               }
+               else{
+                       AliError("Machine mode array not found in LHC Data file!!!");
+               }
+               
+               // BeamType1 and BeamType2 - both put in the same string
+               Log("*************BeamType ");
+               TObjArray* beamArray = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[3]);
+               if (beamArray){                 
+                       Int_t nBeam = beamArray->GetEntries();
+                       if (nBeam==0){
+                               AliInfo("No Beam Type found, leaving it empty");
+                       }
+                       else{
+                               for (Int_t iBeam = 0; iBeam<nBeam; iBeam++){
+                                       AliDCSArray* beam = (AliDCSArray*)beamArray->At(iBeam);
+                                       if (beam){
+                                               if (beam->GetTimeStamp()<=timeStart && beam->GetTimeStamp()>=timeBeamStart){// taking always the very last entry: of two measurements have the same timestamp, the last one is taken
+                                                       timeBeamStart = beam->GetTimeStamp();
+                                                       indexBeam = iBeam;
+                                                       foundBeamStart = kTRUE;
+                                               }
+                                               else{
+                                                       break;
+                                               }
+                                       }
                                }
-                               else if (timeMachineMode == 0){
-                                       minTimeLHCValidity = timeBeamMode;
+                               if (!foundBeamStart){
+                                       AliInfo("No value for the Beam Type found before start of run, the Machine Mode will remain empty");
                                }
                                else {
-                                       minTimeLHCValidity= TMath::Min(timeBeamMode,timeMachineMode);
+                                       AliDCSArray* beam = (AliDCSArray*)beamArray->At(indexBeam);
+                                       TObjString* beamString = beam->GetStringArray(0);
+                                       TString beamType = beamString->String();
+                                       AliInfo(Form("Beam Type = %s",beamType.Data()));        
+                                       if (beamType.CompareTo("PROTON",TString::kIgnoreCase) == 0){
+                                               AliInfo("Setting beam type to p-p");
+                                               grpobj->SetBeamType("p-p");
+                                       }
+                                       else { // if there is no PROTON beam, we suppose it is Pb, and we put A-A
+                                               AliInfo("Setting beam type to A-A");
+                                               grpobj->SetBeamType("A-A");
+                                       }
+                                       /*
+                                         else if (beamType.CompareTo("LEAD82",TString::kIgnoreCase) == 0){
+                                               AliInfo("Setting beam type to Pb-Pb");
+                                               grpobj->SetBeamType("Pb-Pb");
+                                       }
+                                       else{
+                                               AliError("Beam Type not known, leaving it empty");
+                                       }
+                                       */
+                                       if (indexBeam < nBeam-1){
+                                               AliDCSArray* beam1 = (AliDCSArray*)beamArray->At(indexBeam+1);
+                                               if (beam1){
+                                                       if (beam1->GetTimeStamp()>timeStart && beam1->GetTimeStamp()<=timeEnd){
+                                                               timeBeamEnd = beam1->GetTimeStamp();
+                                                               TObjString* beamString1 = beam1->GetStringArray(0);
+                                                               TString beamType1 = beamString1->String();
+                                                               if (beamType.CompareTo(beamType1.Data(),TString::kIgnoreCase) == -1){
+                                                                       AliWarning(Form("The Beam Type changed from %s to %s during the run at timestamp %f! Setting it to %s and keeping track of the time of the change to set MaxTimeLHCValidity afterward",beamType.Data(),(beamString1->String()).Data(),timeBeamEnd,beamType.Data()));
+                                                                       flagBeam = kTRUE;
+                                                               }
+                                                       }
+                                               }
+                                               else {
+                                                       AliInfo("Invalid pointer for the first entry for Beam Type after the first valid one, not considering anything after what has already been found");
+                                               }
+                                       }
                                }
-                               AliWarning(Form("Setting MaxTimeLHCValidity to %f",minTimeLHCValidity));
-                               grpobj->SetMaxTimeLHCValidity(minTimeLHCValidity);
                        }
-                       
-                       // BeamType1 and BeamType2 
-                       TObjArray* beam1Array = (TObjArray*)lhcMap->GetValue(fgkLHCDataPoints[3]);
-                       if (beam1Array){                        
-                               AliInfo(Form("%d entries for Beam1",beam1Array->GetEntries()));
+                       delete beamArray;
+               }
+               else{
+                       AliError("Beam Type array not found in LHC Data file!!!");
+               }               
+               
+               // Setting minTimeLHCValidity
+               if (timeBeamModeEnd!=0 || timeMachineModeEnd!=0 || timeBeamEnd !=0){
+                       Double_t minTimeLHCValidity;
+                       if (flagBeamMode == kFALSE && flagMachineMode == kFALSE && flagBeam == kTRUE){ // flagBeam only true
+                               minTimeLHCValidity = timeBeamEnd;
                        }
-                       else{
-                               AliError("Beam1 array not found in LHC data file!!!");
-                       }                       
-                       TObjArray* beam2Array = 
-                               (TObjArray*)lhcMap->GetValue(fgkLHCDataPoints[4]);
-                       if (beam2Array){
-                               AliInfo(Form("%d entries for Beam2",beam2Array->GetEntries()));
+                       else if (flagBeamMode == kFALSE && flagMachineMode == kTRUE && flagBeam == kFALSE){ // flagMachineMode only true
+                               minTimeLHCValidity = timeMachineModeEnd;
                        }
-                       else{
-                               AliError("Beam2 array not found in LHC data file!!!");
-                       }       
-                       // Processing data to go to AliLHCData object
-                       AliLHCData* dt = new AliLHCData(lhcMap,timeStart,timeEnd);
-                                                       
-                       // storing AliLHCData in OCDB
-                       if (dt){                        
-                               AliCDBMetaData md;
-                               md.SetResponsible("Ruben Shahoyan");
-                               md.SetComment("LHC data from the GRP preprocessor.");
-                               
-                               Bool_t result = kTRUE;
-                               result = Store("GRP", "LHCData", dt, &md); 
-                               delete dt;
-                               if (result) return 0;
-                               else return 3;
-                       }
-                       else return 4;
-                       delete lhcMap; 
-               }
-               else {
-                       AliError("Cannot read correctly LHCData file");
-                       return 2;
+                       else if (flagBeamMode == kTRUE && flagMachineMode == kFALSE && flagBeam == kFALSE){ // flagBeamMode only true
+                               minTimeLHCValidity = timeBeamModeEnd;
+                       }
+                       else if (flagBeamMode == kFALSE && flagMachineMode == kTRUE && flagBeam == kTRUE){ // flagBeam and flagMachineMode only true
+                               minTimeLHCValidity= TMath::Min(timeBeamEnd,timeMachineModeEnd);
+                       }
+                       else if (flagBeamMode == kTRUE && flagMachineMode == kFALSE && flagBeam == kTRUE){ // flagBeam and flagBeamMode only true
+                               minTimeLHCValidity= TMath::Min(timeBeamEnd,timeBeamModeEnd);
+                       }
+                       else if (flagBeamMode == kTRUE && flagMachineMode == kTRUE && flagBeam == kFALSE){ // flagMachineMode and flagBeamMode only true
+                               minTimeLHCValidity= TMath::Min(timeMachineModeEnd,timeBeamModeEnd);
+                       }
+                       else {
+                               Double_t arrayTimes[3] = {timeBeamModeEnd,timeMachineModeEnd,timeBeamEnd};// flagMachineMode and flagBeamMode and flagBeam 
+                               minTimeLHCValidity= TMath::MinElement(3,arrayTimes);
+                       }
+                       AliWarning(Form("Setting MaxTimeLHCValidity to %f",minTimeLHCValidity));
+                       grpobj->SetMaxTimeLHCValidity(minTimeLHCValidity);
                }
+               
+               // Processing data to go to AliLHCData object
+               AliLHCData* dt = new AliLHCData(fileName.Data(),timeStart,timeEnd);
+               // storing AliLHCData in OCDB
+               if (dt){
+                 AliInfo(Form("Filled %d records to AliLHCData object",dt->GetData().GetEntriesFast()));
+                 AliCDBMetaData md;
+                 md.SetResponsible("Ruben Shahoyan");
+                 md.SetComment("LHC data from the GRP preprocessor.");
+                 Bool_t result = kTRUE;
+                 result = Store("GRP", "LHCData", dt, &md); 
+                 delete dt;
+                 if (!result){
+                       Log(Form("Problems in storing LHC Data - but not going into Error"));
+                 }
+               }
+
+               // processing LHC Phase
+
+               TObjArray *beam1phase = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[4]);
+               TObjArray *beam2phase = lhcReader.ReadSingleLHCDP(fileName.Data(),fgkLHCDataPoints[5]);
+               if (beam1phase == 0x0 || beam2phase == 0x0){
+                       Log(Form("Problems in retrieving LHC Clock data from LHC file"));
+                       return 4;
+               }                       
+               AliLHCClockPhase *phaseObj = ProcessLHCClockPhase(beam1phase,beam2phase,timeEnd);
+               if (phaseObj){
+                       AliInfo(Form("LHC Phase found"));
+                       AliCDBMetaData mdPhase;
+                       mdPhase.SetResponsible("Cvetan Cheshkov");
+                       mdPhase.SetComment("LHC Clock Phase");
+                       Bool_t result = kTRUE;
+                       result = Store("Calib", "LHCClockPhase", phaseObj, &mdPhase); 
+                       delete phaseObj;
+                       if (!result) return 3;
+               }
+               else return 4;
        }
-  
+       
        else {
                AliError("No LHCData file found in DCS FXS");
                return 1;
        }
 
+       return 0;
 }
 
 //_______________________________________________________________
@@ -794,7 +1031,11 @@ UInt_t AliGRPPreprocessor::ProcessSPDMeanVertex()
                        AliInfo("The following sources produced files with the id VertexDiamond from SPD");
                        list->Print();
                        for (Int_t jj=0;jj<list->GetEntries();jj++){
-                               TObjString * str = dynamic_cast<TObjString*> (list->At(jj));
+                               TObjString * str = dynamic_cast<TObjString*> (list->At(jj)); 
+                               if (!str){
+                                       AliError(Form("Expecting a TObjString in the list for the %d-th source, but something else was found.",jj));
+                                       continue;
+                               }
                                AliInfo(Form("found source %s", str->String().Data()));
                                TString fileNameRun = GetForeignFile("SPD", kDAQ, "VertexDiamond", str->GetName());
                                if (fileNameRun.Length()>0){
@@ -856,7 +1097,7 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("Start time not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("Start time not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
 
        if (timeEnd != 0){
@@ -865,36 +1106,22 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("End time not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("End time not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
 
        if (beamEnergy != 0){
-               grpObj->SetBeamEnergy(beamEnergy);
-               Log(Form("Beam Energy for run %d: %f",fRun, beamEnergy));
-               //if ((runType == "PHYSICS" && beamType!="Cosmics")){
-               if ((runType == "PHYSICS" && !beamType.IsNull())){   // if beamType is NOT Null, then we're not in a Cosmics run
-                       nparameter++; // increasing nparameters only in case we're in PHYSICS runs with beamType != NULL
-               }
+               Log(Form("Beam Energy for run %d: %f (NOT USING IT TO FILL THE GRP OBJECT, taking it from the LHC file)",fRun, beamEnergy));
        } 
        else {
-               //if ((runType == "PHYSICS" && beamType!="Cosmics")){
-               if ((runType == "PHYSICS" && !beamType.IsNull())){ // if beamType is NOT Null, then we're not in a Cosmics run
-                       Log(Form("Beam Energy not put in logbook, setting to invalid in GRP entry, and producing an error (beamType = %s, runType = %s)",beamType.Data(), runType.Data()));
-               }
-               else{
-                       Log(Form("Beam Energy not put in logbook, setting to invalid in GRP entry, but not producing any error (beamType = NULL, runType = %s)", runType.Data()));
-               }
+               Log(Form("Beam Energy not put in logbook, but not using it anyway for the GRP object (taking it from the LHC file)"));
        }
 
                
        if (beamType.Length() != 0){
-               grpObj->SetBeamType(beamType);
-               Log(Form("Beam Type for run %d: %s",fRun, beamType.Data()));
-               nparameter++; 
+               Log(Form("Beam Type for run %d: %s (NOT USING IT TO FILL THE GRP OBJECT, taking it from the LHC file)",fRun, beamType.Data()));
        } 
        else {
-               Log(Form("Beam Type not put in logbook, setting to invalid in GRP entry! Not producing any error, considering this as a Cosmics run"));
-               nparameter++;
+               Log(Form("Beam Type not put in logbook, but not using it anyway for the GRP entry (taking it from the LHC file)"));
        }
                
        if (numberOfDetectors != 0){
@@ -903,7 +1130,7 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("Number Of Detectors not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("Number Of Detectors not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
 
        if (detectorMask != 0){
@@ -912,7 +1139,7 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("Detector Mask not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("Detector Mask not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
 
        if (lhcPeriod.Length() != 0) {
@@ -921,7 +1148,7 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("LHCperiod not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("LHCperiod not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
        if (runType.Length() != 0) {
                grpObj->SetRunType(runType);
@@ -929,7 +1156,7 @@ Int_t AliGRPPreprocessor::ProcessDaqLB(AliGRPObject* grpObj)
                nparameter++;
        } 
        else {
-               Log(Form("Run Type not put in logbook, setting to invalid in GRP entry!"));
+               Log(Form("Run Type not put in logbook, setting to invalid in GRP entry, and causing an error!"));
        }
 
        return nparameter;
@@ -942,6 +1169,7 @@ UInt_t AliGRPPreprocessor::ProcessDaqFxs()
        //======DAQ FXS======//
        
        AliRawEventHeaderV3_9::Class()->IgnoreTObjectStreamer(); // to avoid trying reading TObject store in AliRawEventHeaderV3_9 - temporary fix 
+       AliRawEventHeaderV3_11::Class()->IgnoreTObjectStreamer(); // to avoid trying reading TObject store in AliRawEventHeaderV3_11 - temporary fix 
        TList* list = GetFileSources(kDAQ);  
        if (!list) {
                Log("No raw data tag list: connection problems with DAQ FXS logbook!");
@@ -1486,10 +1714,6 @@ AliSplineFit* AliGRPPreprocessor::GetSplineFit(const TObjArray *array, const TSt
        fit->InitKnots(gr,10,10,0.0);
        fit->SplineFit(2);
        fit->Cleanup();
-       if (!fit) {
-               AliWarning(Form("%s: no fit performed",stringID.Data()));
-               return NULL;
-       } 
        return fit;
 }
 
@@ -1570,6 +1794,7 @@ Float_t* AliGRPPreprocessor::ProcessFloatAll(const TObjArray* array)
                        AliError(Form("Error! Float value found in DCS map at %d-th entry is OUT OF RANGE: value = %6.5e",i,v->GetFloat()));
                        if (v->GetFloat() < fminFloat) AliInfo(Form("The value is smaller than %6.5e",fminFloat));
                        if (v->GetFloat() > fmaxFloat) AliInfo(Form("The value is greater than %6.5e",fmaxFloat));
+                       delete [] parameters;
                        return NULL;
                }
                if(((Int_t)(v->GetTimeStamp()) >= (Int_t)GetStartTimeDCSQuery()) &&((Int_t)(v->GetTimeStamp()) <= (Int_t)GetEndTimeDCSQuery())) {
@@ -1750,6 +1975,15 @@ Float_t* AliGRPPreprocessor::ProcessFloatAll(const TObjArray* array)
                        parameters[1] = AliGRPObject::GetInvalidFloat();
        }
 
+       if (arrayValues){
+               delete [] arrayValues;
+       } 
+       if (arrayWeights){
+               delete [] arrayWeights;
+       } 
+       delete [] arrayValuesTruncMean;
+       delete [] arrayWeightsTruncMean;
+
        AliInfo(Form("(weighted) mean = %f ",parameters[0]));
        AliInfo(Form("(weighted) truncated mean = %f ",parameters[1]));
        AliInfo(Form("median = %f ",parameters[2]));
@@ -1940,6 +2174,8 @@ Float_t AliGRPPreprocessor::ProcessInt(const TObjArray* array)
                        arrayValues[i-ientrySOR] = (Float_t)v->GetInt();
                }
                aDCSArrayMean = TMath::Mean(iCountsRun,arrayValues,arrayWeights);
+               delete [] arrayValues;
+               delete [] arrayWeights;
        }
        else if (iCountsRun == 1){
                AliDCSValue* v = (AliDCSValue *)array->At(ientrySOR);
@@ -1955,6 +2191,8 @@ Float_t AliGRPPreprocessor::ProcessInt(const TObjArray* array)
                        AliDebug(2,Form("value0 = %f, with weight = %f",arrayValues[0],arrayWeights[0])); 
                        AliDebug(2,Form("value1 = %f, with weight = %f",arrayValues[1],arrayWeights[1])); 
                        aDCSArrayMean = TMath::Mean(2,arrayValues,arrayWeights);
+                       delete [] arrayValues;
+                       delete [] arrayWeights;
                }
                else{
                        AliError("Cannot calculate mean - only one value collected during the run, but no value before with which to calculate the statistical quantities");
@@ -1983,6 +2221,8 @@ Float_t AliGRPPreprocessor::ProcessInt(const TObjArray* array)
                        AliDebug(2,Form("value0 = %f, with weight = %f",arrayValues[0],arrayWeights[0])); 
                        AliDebug(2,Form("value1 = %f, with weight = %f",arrayValues[1],arrayWeights[1])); 
                        aDCSArrayMean = TMath::Mean(1,arrayValues,arrayWeights);
+                       delete [] arrayValues;
+                       delete [] arrayWeights;
                }
        }
 
@@ -2094,6 +2334,8 @@ Float_t AliGRPPreprocessor::ProcessUInt(const TObjArray* array)
                        arrayWeights[i-ientrySOR] = (Double_t)(timestamp2 - (Int_t)v->GetTimeStamp());
                        arrayValues[i-ientrySOR] = (Float_t)v->GetUInt();
                }
+               delete [] arrayValues;
+               delete [] arrayWeights;
                aDCSArrayMean = TMath::Mean(iCountsRun,arrayValues,arrayWeights);
        }
        else if (iCountsRun == 1){
@@ -2110,6 +2352,8 @@ Float_t AliGRPPreprocessor::ProcessUInt(const TObjArray* array)
                        AliDebug(2,Form("value0 = %f, with weight = %f",arrayValues[0],arrayWeights[0])); 
                        AliDebug(2,Form("value1 = %f, with weight = %f",arrayValues[1],arrayWeights[1])); 
                        aDCSArrayMean = TMath::Mean(2,arrayValues,arrayWeights);
+                       delete [] arrayValues;
+                       delete [] arrayWeights;
                }
                else{
                        AliError("Cannot calculate mean - only one value collected during the run, but no value before with which to calculate the statistical quantities");
@@ -2138,6 +2382,8 @@ Float_t AliGRPPreprocessor::ProcessUInt(const TObjArray* array)
                        AliDebug(2,Form("value0 = %f, with weight = %f",arrayValues[0],arrayWeights[0])); 
                        AliDebug(2,Form("value1 = %f, with weight = %f",arrayValues[1],arrayWeights[1])); 
                        aDCSArrayMean = TMath::Mean(1,arrayValues,arrayWeights);
+                       delete [] arrayValues;
+                       delete [] arrayWeights;
                }
        }
 
@@ -2212,7 +2458,7 @@ Int_t AliGRPPreprocessor::ReceivePromptRecoParameters(UInt_t run, const char* db
        
        // main logbook
        TString sqlQuery;
-       sqlQuery.Form("SELECT DAQ_time_start, run_type, detectorMask, L3_magnetCurrent, Dipole_magnetCurrent FROM logbook WHERE run = %d", run);
+       sqlQuery.Form("SELECT DAQ_time_start, run_type, detectorMask, L3_magnetCurrent, Dipole_magnetCurrent,beamType FROM logbook WHERE run = %d", run);
        TSQLResult* result = server->Query(sqlQuery);
        if (!result)
                {
@@ -2240,12 +2486,16 @@ Int_t AliGRPPreprocessor::ReceivePromptRecoParameters(UInt_t run, const char* db
        TString detectorMaskString(row->GetField(2));
        TString l3CurrentString(row->GetField(3));
        TString dipoleCurrentString(row->GetField(4));
+       TString beamTypeString(row->GetField(5));
        time_t timeStart = (time_t)(timeStartString.Atoi());
        UInt_t detectorMask = (UInt_t)(detectorMaskString.Atoi());
        Float_t l3Current = (Float_t)(TMath::Abs(l3CurrentString.Atof()));
        Float_t dipoleCurrent = (Float_t)(TMath::Abs(dipoleCurrentString.Atof()));
        Char_t l3Polarity = (l3CurrentString.Atof() < 0) ? 1 : 0;
        Char_t dipolePolarity = (dipoleCurrentString.Atof() < 0) ? 1 : 0;
+       if (beamTypeString.CompareTo("Pb-Pb",TString::kIgnoreCase) == 0){
+               beamTypeString="A-A";
+       }
        
        AliGRPObject * grpObj = new AliGRPObject();
        grpObj->SetTimeStart(timeStart); 
@@ -2256,6 +2506,7 @@ Int_t AliGRPPreprocessor::ReceivePromptRecoParameters(UInt_t run, const char* db
        grpObj->SetL3Polarity(l3Polarity);
        grpObj->SetDipolePolarity(dipolePolarity);
        grpObj->SetPolarityConventionLHC();  // after the dipole cables swap we comply with LHC convention
+       grpObj->SetBeamType(beamTypeString);
 
        delete row;
        row = 0;
@@ -2416,241 +2667,8 @@ Int_t AliGRPPreprocessor::ReceivePromptRecoParameters(UInt_t run, const char* db
        
        return lastRun;
 }
-//-----------------------------------------------------------------
-Double_t AliGRPPreprocessor::CalculateMean(TObjArray* array){
-
-       //
-       // Calculating mean over TObjArray from LHC Data
-       //
-
-       TString timeStartString = (TString)GetRunParameter("DAQ_time_start");
-       TString timeEndString = (TString)GetRunParameter("DAQ_time_end");
-       if (timeStartString.IsNull() || timeStartString.IsNull()){
-               if (timeStartString.IsNull()){ 
-                       AliError("DAQ_time_start not set in logbook! Setting statistical values for current DP to invalid");
-               }
-               else if (timeStartString.IsNull()){
-                       AliError("DAQ_time_end not set in logbook! Setting statistical values for current DP to invalid");
-               }
-               return 0;
-       }  
-
-       Int_t timeStart = (Int_t)(timeStartString.Atoi());
-       Int_t timeEnd = (Int_t)(timeEndString.Atoi());
-       timeStart = 1260646960;
-       timeEnd = 1260652740;
-       Double_t* parameters = new Double_t[5];
-       parameters[0] = -1.;
-       parameters[1] = -1.;
-       parameters[2] = -1.;
-       parameters[3] = -1.;
-       parameters[4] = -1.;
-       Int_t iCounts = 0;
-       Int_t iCountsRun = 0;
-       Int_t nCounts = array->GetEntries();
-       printf("ncounts = %d\n",nCounts);
-       Double_t valueBeforeSOR = 0;
-       Double_t valueAfterEOR = 0;
-       Double_t timestampBeforeSOR = -1.;
-       Double_t timestampAfterEOR = -1.;
-       Int_t ientrySOR = -1;
-       Int_t ientryEOR = -1;
-       Double_t* arrayValues = 0x0; 
-       Double_t* arrayWeights = 0x0; 
-       Bool_t truncMeanFlag = kTRUE;  // flag to indicate whether Truncated Mean should be calculated or not
-       Bool_t sdFlag = kTRUE;  // flag to indicate whether SD (wrt Mean/Median) should be calculated or not
-
-       for(Int_t i = 0; i < nCounts; i++) {
-               AliDCSArray *dcs = (AliDCSArray*)array->At(i);
-               if((dcs->GetTimeStamp() >= timeStart) &&(dcs->GetTimeStamp() <= timeEnd)) {
-                       AliDebug(2,Form("%d-th entry = %f at timestamp %f\n",i,(Double_t)(dcs->GetInt(0)),dcs->GetTimeStamp()));
-                       iCounts += 1;
-                       // look for the last value before SOR and the first value before EOR
-                       if ((dcs->GetTimeStamp() >= timeStart) && (dcs->GetTimeStamp() < timeStart)) {
-                               timestampBeforeSOR = dcs->GetTimeStamp();
-                               AliDebug(2,Form("timestamp of last value before SOR = %f, with DAQ_time_start = %d\n",timestampBeforeSOR,timeStart));
-                               valueBeforeSOR = (Double_t)(dcs->GetInt(0));
-                       }
-                       else if ((dcs->GetTimeStamp() <= timeEnd) && (dcs->GetTimeStamp() > timeEnd) && timestampAfterEOR == -1){
-                               timestampAfterEOR = dcs->GetTimeStamp();
-                               valueAfterEOR = (Double_t)(dcs->GetInt(0));
-                               AliDebug(2,Form("timestamp of first value after EOR = %f, with DAQ_time_end = %d\n",timestampAfterEOR,timeEnd));
-                       }
-                       // check if there are DPs between DAQ_time_start and DAQ_time_end
-                       if((dcs->GetTimeStamp() >= timeStart) &&(dcs->GetTimeStamp() <= timeEnd)) {
-                               if (ientrySOR == -1) ientrySOR = i;  // first entry after SOR
-                               if (ientryEOR < i) ientryEOR = i;  // last entry before EOR
-                               AliDebug(2,Form("entry between SOR and EOR\n"));
-                               iCountsRun += 1;
-                       }
-               }
-               else {
-                       printf("DCS values for the parameter outside the queried interval: timestamp = %f\n",dcs->GetTimeStamp());
-               }
-       }
-
-       if (timestampBeforeSOR == -1.){
-               printf("No value found before SOR\n");
-       }
-       if (timestampAfterEOR == -1.){
-               printf("No value found after EOR\n");
-       }
-
-       printf("Number of valid entries (within DCS query interval) = %i, from a total amount of %i entries\n",iCounts,nCounts);
-       printf("Last value before DAQ_time_start (SOR) = %f at timestamp = %f\n",valueBeforeSOR,timestampBeforeSOR);
-       printf("First value after DAQ_time_end (EOR)   = %f at timestamp = %f\n",valueAfterEOR,timestampAfterEOR);
-       printf("Found %d entries between DAQ_time_start (SOR) and DAQ_time_end (EOR)\n",iCountsRun);
-       printf("Index of first entry after DAQ_time_start (SOR) = %d\n ",ientrySOR);
-       printf("Index of first entry before DAQ_time_end (EOR) = %d\n ",ientryEOR);
-
-       Int_t nentriesUsed = 0;
-       if (iCountsRun > 1){
-               printf("Using entries between DAQ_time_start (SOR) and DAQ_time_end (EOR)\n");
-               printf("Calculating (weighted) Mean and Median\n" );
-               arrayValues = new Double_t[iCountsRun]; 
-               arrayWeights = new Double_t[iCountsRun]; 
-               nentriesUsed = iCountsRun;
-               for (Int_t i = ientrySOR; i <= ientryEOR; i++){
-                       AliDCSArray *dcs = (AliDCSArray *)array->At(i);
-                       Double_t timestamp2 = 0;
-                       if (i < ientryEOR){
-                               AliDCSArray *dcs1 = (AliDCSArray *)array->At(i+1);
-                               timestamp2 = dcs1->GetTimeStamp();
-                       }
-                       else {
-                               timestamp2 = (Double_t)timeEnd+1;
-                       }
-                       arrayWeights[i-ientrySOR] = (Double_t)((Double_t)timestamp2 - dcs->GetTimeStamp());
-                       arrayValues[i-ientrySOR] = (Double_t)(dcs->GetInt(0));
-                       printf("Entry %d: value = %f, weight = %f\n",i-ientrySOR,arrayValues[i-ientrySOR],arrayWeights[i-ientrySOR]);
-               }
-               parameters[0] = TMath::Mean(iCountsRun,arrayValues,arrayWeights);
-               parameters[2] = TMath::Median(iCountsRun,arrayValues,arrayWeights);
-       }
-       else if (iCountsRun == 1){
-               AliDCSArray* dcs = (AliDCSArray *)array->At(ientrySOR);
-               nentriesUsed = 2;
-               if (timestampBeforeSOR != -1 && timestampBeforeSOR != (Int_t)dcs->GetTimeStamp()){
-                       printf("Using single entry between DAQ_time_start (SOR) and DAQ_time_end (EOR) and last entry before SOR. Truncated mean won't be calculated.\n");
-                       arrayValues = new Double_t[2];
-                       arrayWeights = new Double_t[2];
-                       arrayValues[0] = valueBeforeSOR;
-                       arrayWeights[0] = (Double_t)(dcs->GetTimeStamp()-(Double_t)timestampBeforeSOR);
-                       arrayValues[1] = (Double_t)(dcs->GetInt(0));
-                       arrayWeights[1] = (Double_t)((Double_t)timeEnd+1-dcs->GetTimeStamp());
-                       printf("value0 = %f, with weight = %f\n",arrayValues[0],arrayWeights[0]); 
-                       printf("value1 = %f, with weight = %f\n",arrayValues[1],arrayWeights[1]); 
-                       parameters[0] = TMath::Mean(2,arrayValues,arrayWeights);
-                       parameters[2] = TMath::Median(2,arrayValues,arrayWeights);
-                       truncMeanFlag = kFALSE;
-               }
-               else{
-                       printf("Cannot calculate mean, truncated mean, median, SD wrt mean, SD wrt median for current DP - only one value collected during the run, but no value before with which to calculate the statistical quantities\n");
-                       parameters[0] = -1;
-                       parameters[1] = -1;
-                       parameters[2] = -1;
-                       parameters[3] = -1;
-                       parameters[4] = -1;
-                       return parameters[0];
-               }
-       }
-       else { // iCountsRun == 0, using only the point immediately before SOR
-               if (timestampBeforeSOR == -1.){
-                       printf("Cannot set mean, truncated mean, median, SD wrt mean, SD wrt median for current DP - no points during the run collected, and point before SOR missing\n");
-                       parameters[0] = -1;
-                       parameters[1] = -1;
-                       parameters[2] = -1;
-                       parameters[3] = -1;
-                       parameters[4] = -1;
-                       return parameters[0];
-               }
-               else {
-                       printf("Using only last entry before SOR. Truncated mean and Standard deviations (wrt mean/median) won't be calculated.\n");
-                       printf("value = %f\n",valueBeforeSOR); 
-                       parameters[0] = valueBeforeSOR;
-                       parameters[2] = valueBeforeSOR;
-                       truncMeanFlag = kFALSE;
-                       sdFlag = kFALSE;
-               }
-       }
-
-       Double_t temp = 0;
-       Double_t temp1 = 0;
-       Double_t sumweights = 0; 
-       Int_t entriesTruncMean = 0;
-       Double_t* arrayValuesTruncMean = new Double_t[nentriesUsed]; 
-       Double_t* arrayWeightsTruncMean = new Double_t[nentriesUsed]; 
-
-       // calculating SD wrt Mean and Median
-       printf("Calculating SD wrt Mean and SD wrt Median\n");
-       if (sdFlag){
-               for (Int_t i =0; i< nentriesUsed; i++){
-                       //printf("Entry %d: value = %f, weight = %f\n",i,arrayValues[i],arrayWeights[i]);
-                       temp += (arrayValues[i]-parameters[2])*(arrayValues[i]-parameters[2]);
-                       temp1 += arrayWeights[i]*(arrayValues[i]-parameters[0])*(arrayValues[i]-parameters[0]);
-                       sumweights += arrayWeights[i];
-               }
-               // setting SD wrt Mean 
-               if (sumweights != 0 ){
-                       parameters[3] = TMath::Sqrt(temp1/sumweights);
-               }
-               else {
-                       printf("Sum of weights to calculate Standard Deviation (wrt mean) <= 0, setting the SD to invalid\n");
-                       parameters[3] = -1;
-               }
-               // setting SD wrt Median
-               if (nentriesUsed != 0){
-                       parameters[4] = TMath::Sqrt(temp/nentriesUsed);
-               }
-               else{
-                       printf("Number of entries used to calculate Standard Deviation (wrt median) <= 0, setting the SD to invalid\n");
-                       parameters[4] = -1;
-               }
-       }
-       else {
-               parameters[3] = -1;
-               parameters[4] = -1;
-       }               
-
-       // calculating truncated mean (this comes afterwards since you need the SD wrt Mean)
-       if (truncMeanFlag){
-               printf("Calculating Truncated Mean\n");
-               for (Int_t i =0; i< nentriesUsed; i++){
-                       //printf("Entry %d: value = %f, weight = %f\n",i,arrayValues[i],arrayWeights[i]);
-                       if ((arrayValues[i]<=parameters[0]+3*parameters[3]) && (arrayValues[i]>=parameters[0]-3*parameters[3])){
-                               arrayValuesTruncMean[entriesTruncMean]=arrayValues[i];
-                               arrayWeightsTruncMean[entriesTruncMean]=arrayWeights[i];
-                               printf("For Truncated Mean: Entry %d: value = %f, weight = %f\n",entriesTruncMean,arrayValuesTruncMean[entriesTruncMean],arrayWeightsTruncMean[entriesTruncMean]);
-                               entriesTruncMean++;                     
-                       }
-                       else{
-                               printf("Discarding entry\n");
-                       }
-               }
-               // setting truncated mean 
-               if (entriesTruncMean >1){
-                       printf("%d entries used for truncated mean\n",entriesTruncMean);
-                       parameters[1] = TMath::Mean(entriesTruncMean,arrayValuesTruncMean,arrayWeightsTruncMean);
-               }
-               else{   
-                       printf("Too few entries (%d) to calculate truncated mean\n",entriesTruncMean);
-                       parameters[1] = -1;
-               }
-       }
-       else{
-                       parameters[1] = -1;
-       }
-       
-       printf("(weighted) mean = %f \n",parameters[0]);
-       printf("(weighted) truncated mean = %f \n",parameters[1]);
-       printf("median = %f \n",parameters[2]);
-       printf("(weighted) standard deviation with (weighted) mean = %f \n",parameters[3]);
-       printf("standard deviation with median = %f \n",parameters[4]);
-       
-       return (parameters[0]);
-}
 //------------------------------------------------------------------------------------------------------
-Float_t AliGRPPreprocessor::ProcessEnergy(TObjArray* array, Double_t timeStart, Double_t timeEnd){
+Float_t AliGRPPreprocessor::ProcessEnergy(TObjArray* const array, Double_t timeStart){
 
        //
        // Method to processo LHC Energy information
@@ -2659,26 +2677,119 @@ Float_t AliGRPPreprocessor::ProcessEnergy(TObjArray* array, Double_t timeStart,
 
        Int_t nCounts = array->GetEntries();
        Float_t energy = -1;
-       Bool_t inRange = kFALSE;
+       Double_t timeEnergy = -1;
+       Int_t indexEnergy = -1;
+       Bool_t foundEnergy = kFALSE;
+
        AliDebug(2,Form("Energy measurements = %d\n",nCounts));
        if (nCounts ==0){
                AliWarning("No Energy values found! Beam Energy remaining invalid!");
        }
        else{
-               for(Int_t i = 0; i < nCounts; i++) {
+               for (Int_t i = 0; i < nCounts; i++){
                        AliDCSArray *dcs = (AliDCSArray*)array->At(i);
-                       if((dcs->GetTimeStamp() >= timeStart) &&(dcs->GetTimeStamp() <= timeEnd)) {
-                               energy = (Float_t)(TMath::Nint(((Double_t)(dcs->GetInt(0)))*120/1000)); // sqrt(s)/2 energy in GeV
-                               AliInfo(Form("Energy value found = %d, converting --> sqrt(s)/2 = %f (GeV)", dcs->GetInt(0),energy));
-                               inRange = kTRUE;
-                               break;
+                       if (dcs){
+                               if (dcs->GetTimeStamp()<=timeStart && dcs->GetTimeStamp()>=timeEnergy){// taking always the very last entry: of two measurements have the same timestamp, the last one is taken
+                                       timeEnergy = dcs->GetTimeStamp();
+                                       indexEnergy = i;
+                                       foundEnergy = kTRUE;
+                               }
+                               else{
+                                       break;
+                               }
                        }
                }
-               if (inRange == kFALSE){
-                               AliInfo("No Energy value found between DAQ_time_start and DAQ_time_end - energy will remain invalid!");
+               if (!foundEnergy){
+                       AliInfo("No value for the Energy found before start of run, the Energy will remain invalid");
+               }
+               else {
+                       AliDCSArray* dcs = (AliDCSArray*)array->At(indexEnergy);
+                       energy = (Float_t)(TMath::Nint(((Double_t)(dcs->GetInt(0)))*120/1000)); // sqrt(s)/2 energy in GeV
+                       AliInfo(Form("Energy value found = %d (at %f), converting --> sqrt(s)/2 = %f (GeV)", dcs->GetInt(0),dcs->GetTimeStamp(),energy));
                }
-       
        }
 
        return energy;
 }
+//------------------------------------------------------------------------------------------------------
+AliLHCClockPhase* AliGRPPreprocessor::ProcessLHCClockPhase(TObjArray *beam1phase,TObjArray *beam2phase, Double_t timeEnd)
+{
+  //
+  // Method to process LHC-Clock Phase data
+  // Only the values between DAQ_time_start and DAQ_time_end are kept
+  //
+  AliLHCClockPhase *phaseObj = new AliLHCClockPhase;
+
+  Bool_t foundBeam1Phase = kFALSE, foundBeam2Phase = kFALSE;
+  const Float_t threshold = 0.050; // we store the measurement only in case they differ with more 50ps from the previous one 
+
+  TString timeCreatedStr = GetRunParameter("time_created");
+  Double_t timeCreated = timeCreatedStr.Atof();
+
+  Int_t nCounts = beam1phase->GetEntries();
+  AliDebug(2,Form("Beam1 phase measurements = %d\n",nCounts));
+  if (nCounts ==0){
+    AliWarning("No beam1 LHC clock phase values found!");
+    delete phaseObj;
+    return NULL;
+  }
+  else{
+    Double_t prevPhase = 0;
+    for (Int_t i = 0; i < nCounts; i++){
+      AliDCSArray *dcs = (AliDCSArray*)beam1phase->At(i);
+      if (dcs){
+             //if (dcs->GetTimeStamp()>=timeStart && dcs->GetTimeStamp()<=timeEnd) {
+             if (dcs->GetTimeStamp()>=timeCreated && dcs->GetTimeStamp()<=timeEnd) {
+         if ((i == 0) || (i == (nCounts-1)) ||
+             !foundBeam1Phase ||
+             (TMath::Abs(dcs->GetDouble(0)-prevPhase) > threshold)) {
+           prevPhase = dcs->GetDouble(0);
+           foundBeam1Phase = kTRUE;
+           AliInfo(Form("B1 Clk Phase = %f at TS = %f",
+                        (Float_t)dcs->GetDouble(0),dcs->GetTimeStamp()));  
+           phaseObj->AddPhaseB1DP((UInt_t)dcs->GetTimeStamp(),(Float_t)dcs->GetDouble(0));
+         }
+       }
+      }
+    }
+    if (!foundBeam1Phase){
+      AliError("No beam1 LHC clock phase values found within the run!");
+      delete phaseObj;
+      return NULL;
+    }
+  }
+
+  nCounts = beam2phase->GetEntries();
+  AliDebug(2,Form("Beam2 phase measurements = %d\n",nCounts));
+  if (nCounts ==0){
+    AliWarning("No beam2 LHC clock phase values found!");
+    delete phaseObj;
+    return NULL;
+  }
+  else{
+    Double_t prevPhase = 0;
+    for (Int_t i = 0; i < nCounts; i++){
+      AliDCSArray *dcs = (AliDCSArray*)beam2phase->At(i);
+      if (dcs){
+       if (dcs->GetTimeStamp()>=timeCreated && dcs->GetTimeStamp()<=timeEnd) {
+         if ((i == 0) || (i == (nCounts-1)) ||
+             !foundBeam2Phase ||
+             (TMath::Abs(dcs->GetDouble(0)-prevPhase) > threshold)) {
+           prevPhase = dcs->GetDouble(0);
+           foundBeam2Phase = kTRUE;
+           AliInfo(Form("B2 Clk Phase = %f at TS = %f",
+                        (Float_t)dcs->GetDouble(0),dcs->GetTimeStamp()));  
+           phaseObj->AddPhaseB2DP((UInt_t)dcs->GetTimeStamp(),(Float_t)dcs->GetDouble(0));
+         }
+       }
+      }
+    }
+    if (!foundBeam2Phase){
+      AliError("No beam2 LHC clock phase values found within the run!");
+      delete phaseObj;
+      return NULL;
+    }
+  }
+
+  return phaseObj;
+}