]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - EMCAL/AliEMCALCalibTimeDep.cxx
Changes needed for evaporation and fragmentation
[u/mrichter/AliRoot.git] / EMCAL / AliEMCALCalibTimeDep.cxx
index d08ffd409a1497f14d14a0c530085606dafb29ff..1249653413681a7e8e29bc3c28a969db2275f2cb 100644 (file)
@@ -22,7 +22,7 @@
 // class for EMCAL time-dep calibration   
 // - supposed to run in preprocessor
 // we use input from the following sources:
-// AliEMCALBiasAPD (bias values),  AliEMCALCalibMapAPD (APD calibration and location info),
+// AliEMCALCalibTempCoeff (APD temperature coefficients),
 // AliCaloCalibSignal (LED DA), AliEMCALSensorTempArray (ELMB DCS)
 // AliEMCALCalibReference: LED amplitude and temperature info at reference time
 //
 
 #include <iostream>
 #include <TGraphSmooth.h>
+#include <TMath.h>
 #include "AliLog.h"
 #include "AliCDBEntry.h"
 #include "AliCDBManager.h"
 #include "AliEMCALSensorTempArray.h"
 #include "AliCaloCalibSignal.h"
-#include "AliEMCALBiasAPD.h"
-#include "AliEMCALCalibMapAPD.h"
+#include "AliEMCALCalibTempCoeff.h"
 #include "AliEMCALCalibReference.h"
 #include "AliEMCALCalibTimeDepCorrection.h" 
 #include "AliEMCALCalibTimeDep.h"
@@ -53,6 +53,8 @@ const double kTempCoeffP0Factor = -1.381e7; //
 const double kTempCoeffP1Const = -0.023; // 
 const double kTempCoeffP1Factor = -4.966e5; //
  
+const double kTempMaxDiffMedian = 2; // Temperature values should not be further away from median value within SM when considered in the average calc.
+
 const double kErrorCode = -999; // to indicate that something went wrong
 
 using namespace std;
@@ -66,17 +68,22 @@ AliEMCALCalibTimeDep::AliEMCALCalibTimeDep() :
   fEndTime(0),
   fMinTemp(0),
   fMaxTemp(0),
+  fMinTempVariation(0),
+  fMaxTempVariation(0),
+  fMinTempValid(15),
+  fMaxTempValid(35),
   fMinTime(0),
   fMaxTime(0),
   fTemperatureResolution(0.1), // 0.1 deg C is default
+  fMaxTemperatureDiff(5), // 5 deg C is default max diff relative to reference 
   fTimeBinsPerHour(2), // 2 30-min bins per hour is default
   fHighLowGainFactor(16), // factor ~16 between High gain and low gain
   fTempArray(NULL),
   fCalibSignal(NULL),
-  fBiasAPD(NULL),
-  fCalibMapAPD(NULL),
+  fCalibTempCoeff(NULL),
   fCalibReference(NULL),
-  fCalibTimeDepCorrection(NULL)
+  fCalibTimeDepCorrection(NULL),
+  fVerbosity(0)
 {
   // Constructor
 }
@@ -89,17 +96,22 @@ AliEMCALCalibTimeDep::AliEMCALCalibTimeDep(const AliEMCALCalibTimeDep& calibt) :
   fEndTime(calibt.GetEndTime()),
   fMinTemp(calibt.GetMinTemp()),
   fMaxTemp(calibt.GetMaxTemp()),
+  fMinTempVariation(calibt.GetMinTempVariation()),
+  fMaxTempVariation(calibt.GetMaxTempVariation()),
+  fMinTempValid(calibt.GetMinTempValid()),
+  fMaxTempValid(calibt.GetMaxTempValid()),
   fMinTime(calibt.GetMinTime()),
   fMaxTime(calibt.GetMaxTime()),
   fTemperatureResolution(calibt.GetTemperatureResolution()),
+  fMaxTemperatureDiff(calibt.GetMaxTemperatureDiff()),
   fTimeBinsPerHour(calibt.GetTimeBinsPerHour()),
   fHighLowGainFactor(calibt.GetHighLowGainFactor()),
   fTempArray(calibt.GetTempArray()),
   fCalibSignal(calibt.GetCalibSignal()),
-  fBiasAPD(calibt.GetBiasAPD()),
-  fCalibMapAPD(calibt.GetCalibMapAPD()),
+  fCalibTempCoeff(calibt.GetCalibTempCoeff()),
   fCalibReference(calibt.GetCalibReference()),
-  fCalibTimeDepCorrection(calibt.GetCalibTimeDepCorrection())
+  fCalibTimeDepCorrection(calibt.GetCalibTimeDepCorrection()),
+  fVerbosity(calibt.GetVerbosity())
 {
   // copy constructor
 }
@@ -130,16 +142,21 @@ void  AliEMCALCalibTimeDep::Reset()
   fEndTime = 0;
   fMinTemp = 0;
   fMaxTemp = 0;
+  fMinTempVariation = 0;
+  fMaxTempVariation = 0;
+  fMinTempValid = 15; 
+  fMaxTempValid = 35;
   fMinTime = 0;
   fMaxTime = 0;
   fTemperatureResolution = 0.1; // 0.1 deg C is default
+  fMaxTemperatureDiff = 5; // 5 deg C is default max diff relative to reference 
   fTimeBinsPerHour = 2; // 2 30-min bins per hour is default
   fTempArray = NULL;
   fCalibSignal = NULL;
-  fBiasAPD = NULL;
-  fCalibMapAPD = NULL;
+  fCalibTempCoeff = NULL;
   fCalibReference = NULL;
   fCalibTimeDepCorrection = NULL;
+  fVerbosity = 0;
   return;
 }
 
@@ -152,11 +169,18 @@ void  AliEMCALCalibTimeDep::PrintInfo() const
   cout << " VARIABLE DUMP: " << endl
        << " GetStartTime() " << GetStartTime() << endl
        << " GetEndTime() " << GetEndTime() << endl
+       << " GetMinTime() " << GetMinTime() << endl
+       << " GetMaxTime() " << GetMaxTime() << endl
        << " GetMinTemp() " << GetMinTemp() << endl
-       << " GetMaxTemp() " << GetMaxTemp() << endl;
+       << " GetMaxTemp() " << GetMaxTemp() << endl
+       << " GetMinTempVariation() " << GetMinTempVariation() << endl
+       << " GetMaxTempVariation() " << GetMaxTempVariation() << endl
+       << " GetTemperatureResolution() " << GetTemperatureResolution() << endl;
   // run ranges
   cout << " RUN INFO: " << endl
+       << " runnumber " << GetRunNumber() << endl
        << " length (in hours) " << GetLengthOfRunInHours() << endl
+       << " length (in bins) " << GetLengthOfRunInBins() << endl
        << " range of temperature measurements (in hours) " << GetRangeOfTempMeasureInHours()
        << " (in deg. C) " << GetRangeOfTempMeasureInDegrees()
        << endl;
@@ -205,42 +229,6 @@ void AliEMCALCalibTimeDep::Initialize(Int_t run,
   return;
 }
 
-//________________________________________________________________
-Double_t AliEMCALCalibTimeDep::GetTemperature(UInt_t timeStamp) const
-{// return estimate for all SuperModules and sensors, that had data 
-
-  // first convert from seconds to hours..
-  Double_t timeHour = (timeStamp - fStartTime) * kSecToHour;
-
-  Double_t average = 0;
-  int n = 0;
-
-  for (int i=0; i<fTempArray->NumSensors(); i++) {
-    
-    AliEMCALSensorTemp *st = fTempArray->GetSensor(i);
-
-    // check if we had valid data for the time that is being asked for
-    if ( timeStamp>=st->GetStartTime() && timeStamp<=st->GetEndTime() ) {
-      AliSplineFit *f = st->GetFit();
-      if (f) { // ok, looks like we have valid data/info
-       // let's check what the expected value at the time appears to be
-       Double_t val = f->Eval(timeHour);
-       average += val;
-       n++;
-      }
-    } // time
-  } // loop over fTempArray
-  
-  if (n>0) { // some valid data was found
-    average /= n;
-    return average;
-  }
-  else { // no good data
-    return kErrorCode;
-  }
-
-}
-
 //________________________________________________________________
 Double_t AliEMCALCalibTimeDep::GetTemperatureSM(int imod, UInt_t timeStamp) const
 {// return estimate for this one SuperModule, if it had data 
@@ -248,8 +236,8 @@ Double_t AliEMCALCalibTimeDep::GetTemperatureSM(int imod, UInt_t timeStamp) cons
   // first convert from seconds to hours..
   Double_t timeHour = (timeStamp - fStartTime) * kSecToHour;
 
-  Double_t average = 0;
   int n = 0;
+  Double_t valArr[8]={0}; // 8 sensors per SM
 
   for (int i=0; i<fTempArray->NumSensors(); i++) {
     
@@ -262,9 +250,13 @@ Double_t AliEMCALCalibTimeDep::GetTemperatureSM(int imod, UInt_t timeStamp) cons
        if (f) { // ok, looks like we have valid data/info
          // let's check what the expected value at the time appears to be
          Double_t val = f->Eval(timeHour);
-         cout << " i " << i << " val " << val << endl;
-         average += val;
-         n++;
+         if ( fVerbosity > 0 ) {
+           cout << " sensor i " << i << " val " << val << endl;
+         }
+         if (val>fMinTempValid && val<fMaxTempValid && n<8) { 
+           valArr[n] = val;
+           n++;
+         }
        }
       } // time
     }
@@ -272,8 +264,24 @@ Double_t AliEMCALCalibTimeDep::GetTemperatureSM(int imod, UInt_t timeStamp) cons
   } // loop over fTempArray
   
   if (n>0) { // some valid data was found
-    average /= n;
-    return average;
+    Double_t median = TMath::Median(n, valArr);
+    Double_t average = 0;
+    Int_t nval = 0;
+    for (int is=0; is<n; is++) {
+      if (TMath::Abs(valArr[is] - median) < kTempMaxDiffMedian) {
+       average += valArr[is];
+       nval++;
+      }
+    }
+    //cout << " n " << n << " nval " << nval << " median " << median << endl;
+    if (nval >  0) {
+      average /= nval;
+      //cout << " average " << average << endl; 
+      return average;
+    }
+    else { // this case should not happen, but kept for completeness (coverity etc)
+      return median;
+    }
   }
   else { // no good data
     return kErrorCode;
@@ -281,67 +289,44 @@ Double_t AliEMCALCalibTimeDep::GetTemperatureSM(int imod, UInt_t timeStamp) cons
 
 }
 
-//________________________________________________________________
-Double_t AliEMCALCalibTimeDep::GetTemperatureSMSensor(int imod, int isens, UInt_t timeStamp) const
-{// return estimate for this one SuperModule and sensor, if it had data 
-
-  // first convert from seconds to hours..
-  Double_t timeHour = (timeStamp - fStartTime) * kSecToHour;
-
-  for (int i=0; i<fTempArray->NumSensors(); i++) {
-    
-    AliEMCALSensorTemp *st = fTempArray->GetSensor(i);
-    int module = st->GetSector()*2 + st->GetSide();
-    if ( module == imod && st->GetNum()==isens ) { // right module, and sensor
-      // check if we had valid data for the time that is being asked for
-      if ( timeStamp>=st->GetStartTime() && timeStamp<=st->GetEndTime() ) {
-       AliSplineFit *f = st->GetFit();
-       if (f) { // ok, looks like we have valid data/info
-         // let's check what the expected value at the time appears to be
-         Double_t val = f->Eval(timeHour);
-
-         return val; // no point to move further in for loop, we have found the sensor we were looking for
-       }
-      } // time
-    }
-    
-  } // loop over fTempArray
-  
-  // if we made it all here, it means that we didn't find the sensor we were looking for
-  // i.e. no good data
-  return kErrorCode;
-
-}
-
 //________________________________________________________________
 Int_t AliEMCALCalibTimeDep::CalcCorrection()
 { // OK, this is where the real action takes place - the heart of this class..
   /* The philosophy is as follows:
-     0. Init corrections to 1.0 values
-     1: if we have LED info for the tower, use it
-     2. if not 1, we rely on LED info averaged over strip
-     3. if not 2 either, we try to use temperature info + APD bias and calibration info
+     0. Init corrections to 1.0 values, and see how many correction bins we need
+     1. Check how large temperature variations we have through the run - do we really need all the correction bias (otherwise adjust to single bin)
+     2. try to use temperature info + APD temperature coefficient info, to estimate correction.
+     For now (from Dec 2009), we do not use LED info.
    */
 
   // 0: Init
   // how many SuperModules do we have?
   Int_t nSM = fCalibReference->GetNSuperModule();
   // how many time-bins should we have for this run?
-  Int_t nBins = (Int_t) GetLengthOfRunInBins(); // round-down (from double to int)
+  Int_t nBins = (Int_t) (GetLengthOfRunInBins() + 1); // round-up (from double to int; always at least 1)
   Int_t binSize = (Int_t) (3600/fTimeBinsPerHour); // in seconds
+
+  // 1: get info on how much individual sensors might have changed during
+  // the run (compare max-min for each sensor separately)
+  if (fMaxTempVariation < fTemperatureResolution) {
+    nBins = 1; // just one bin needed..
+  }
+  if (nBins == 1) {    
+    binSize = fEndTime - fStartTime;
+  }
+  if (fVerbosity > 0) {
+    cout << " nBins " << nBins << " binSize " << binSize << endl;
+  }
+
   // set up a reasonable default (correction = 1.0)
+  fCalibTimeDepCorrection = new AliEMCALCalibTimeDepCorrection(nSM);
   fCalibTimeDepCorrection->InitCorrection(nSM, nBins, 1.0);
   fCalibTimeDepCorrection->SetStartTime(fStartTime);
   fCalibTimeDepCorrection->SetNTimeBins(nBins);
   fCalibTimeDepCorrection->SetTimeBinSize(binSize);
 
-  // 1+2: try with LED corrections
-  Int_t nRemaining = CalcLEDCorrection(nSM, nBins);
-
-  // 3: try with Temperature, if needed
-  if (nRemaining>0) {
-    nRemaining = CalcTemperatureCorrection(nSM, nBins);
-  }
+  // 2: try with Temperature correction 
+  Int_t nRemaining = CalcTemperatureCorrection(nSM, nBins, binSize);
 
   return nRemaining;
 }
@@ -356,8 +341,10 @@ Double_t AliEMCALCalibTimeDep::GetTempCoeff(Double_t IDark, Double_t M) const
   Double_t dP1 = kTempCoeffP1Const + kTempCoeffP1Factor * IDark;
  
   Double_t dTC = dP0 + dP1*M;
+  // from % numbers to regular ones..:
+  dTC *= 0.01;
 
-  return dTC;
+  return TMath::Abs(dTC); // return the absolute value, to avoid any sign confusion
 }
 
 /* Next come the methods that do the work in picking up all the needed info..*/
@@ -388,6 +375,8 @@ Int_t AliEMCALCalibTimeDep::ScanTemperatureInfo()
 
   fMinTemp = 999; // init to some large value (999 deg C)
   fMaxTemp = 0;
+  fMinTempVariation = 999; // init to some large value (999 deg C)
+  fMaxTempVariation = 0;
   fMinTime = 2147483647; // init to a large value in the far future (0x7fffffff), year 2038 times..
   fMaxTime = 0;
 
@@ -396,18 +385,40 @@ Int_t AliEMCALCalibTimeDep::ScanTemperatureInfo()
   for (int i=0; i<fTempArray->NumSensors(); i++) {
     
     AliEMCALSensorTemp *st = fTempArray->GetSensor(i);
+    if ( st->GetStartTime() == 0 ) { // no valid data
+      continue;
+    }
 
     // check time ranges
     if (fMinTime > st->GetStartTime()) { fMinTime = st->GetStartTime(); }
     if (fMaxTime < st->GetEndTime()) { fMaxTime = st->GetEndTime(); }
-
     // check temperature ranges
-    TGraph *g = st->GetGraph();
-    if (g) { // ok, looks like we have valid data/info
-      // let's check what the expected value at the time appears to be
-      if (fMinTemp > g->GetMinimum()) { fMinTemp = g->GetMinimum(); }
-      if (fMaxTemp < g->GetMaximum()) { fMaxTemp = g->GetMaximum(); }
-      n++;
+    AliSplineFit *f = st->GetFit();
+
+    if (f) { // ok, looks like we have valid data/info
+      int np = f->GetKnots();
+      Double_t *y0 = f->GetY0();
+      // min and max values within the single sensor
+      Double_t min = 999;
+      Double_t max = 0;
+      int nval = 0;
+      for (int ip=0; ip<np; ip++) { 
+       if (y0[ip]>fMinTempValid && y0[ip]<fMaxTempValid) {
+         if (min > y0[ip]) { min = y0[ip]; }
+         if (max < y0[ip]) { max = y0[ip]; }
+         nval++;
+       }
+      }
+      if (nval>0) {
+       if (fMinTemp > min) { fMinTemp = min; }
+       if (fMaxTemp < max) { fMaxTemp = max; }
+       Double_t variation = max - min;
+       if (fMinTempVariation > variation) { fMinTempVariation = variation; }
+       if (fMaxTempVariation < variation) { fMaxTempVariation = variation; }
+       
+       n++;
+      }
     }
   } // loop over fTempArray
   
@@ -430,11 +441,11 @@ void AliEMCALCalibTimeDep::GetCalibSignalInfo()
   }
 
   if (fCalibSignal) { 
-    AliInfo( Form("CalibSignal: NEvents %d NAcceptedEvents %d Entries %d AvgEntries LEDRefEntries %d LEDRefAvgEntries %d",
+    AliInfo( Form("CalibSignal: NEvents %d NAcceptedEvents %d Entries %lld AvgEntries LEDRefEntries %lld LEDRefEntries %lld, LEDRefAvgEntries %lld",
                  fCalibSignal->GetNEvents(), fCalibSignal->GetNAcceptedEvents(),
                  fCalibSignal->GetTreeAmpVsTime()->GetEntries(),
                  fCalibSignal->GetTreeAvgAmpVsTime()->GetEntries(),
-                  fCalibSignal->GetTreeLEDAmpVsTime()->GetEntries(),
+                 fCalibSignal->GetTreeLEDAmpVsTime()->GetEntries(),
                  fCalibSignal->GetTreeLEDAvgAmpVsTime()->GetEntries() ) );               
   }
   else {
@@ -445,39 +456,20 @@ void AliEMCALCalibTimeDep::GetCalibSignalInfo()
 }
 
 //________________________________________________________________
-void AliEMCALCalibTimeDep::GetBiasAPDInfo() 
-{
-  // pick up Preprocessor output, based on fRun (most recent version)
-  AliCDBEntry* entry = AliCDBManager::Instance()->Get("EMCAL/Calib/BiasAPD", fRun);
-  if (entry) {
-    fBiasAPD = (AliEMCALBiasAPD *) entry->GetObject();
-  }
-
-  if (fBiasAPD) { 
-    AliInfo( Form("BiasAPD: NSuperModule %d ", fBiasAPD->GetNSuperModule() ) );
-  }
-  else {
-    AliWarning( Form("AliEMCALBiasAPD not found!") );
-  }
-  
-  return;
-}
-
-//________________________________________________________________
-void AliEMCALCalibTimeDep::GetCalibMapAPDInfo() 
+void AliEMCALCalibTimeDep::GetCalibTempCoeffInfo() 
 {
   // pick up Preprocessor output, based on fRun (most recent version)
-  AliCDBEntry* entry = AliCDBManager::Instance()->Get("EMCAL/Calib/MapAPD", fRun);
+  AliCDBEntry* entry = AliCDBManager::Instance()->Get("EMCAL/Calib/TempCoeff", fRun);
   // stored object should be a TTree; read the info
   if (entry) {
-    fCalibMapAPD = (AliEMCALCalibMapAPD *) entry->GetObject();
+    fCalibTempCoeff = (AliEMCALCalibTempCoeff *) entry->GetObject();
   }
 
-  if (fCalibMapAPD) { 
-    AliInfo( Form("CalibMapAPD: NSuperModule %d ", fCalibMapAPD->GetNSuperModule() ) );
+  if (fCalibTempCoeff) { 
+    AliInfo( Form("CalibTempCoeff: NSuperModule %d ", fCalibTempCoeff->GetNSuperModule() ) );
   }
   else {
-    AliWarning( Form("AliEMCALCalibMapAPD not found!") );
+    AliWarning( Form("AliEMCALCalibTempCoeff not found!") );
   }
   
   return;
@@ -487,7 +479,7 @@ void AliEMCALCalibTimeDep::GetCalibMapAPDInfo()
 void AliEMCALCalibTimeDep::GetCalibReferenceInfo() 
 {
   // pick up Preprocessor output, based on fRun (most recent version)
-  AliCDBEntry* entry = AliCDBManager::Instance()->Get("EMCAL/Calib/MapAPD", fRun);
+  AliCDBEntry* entry = AliCDBManager::Instance()->Get("EMCAL/Calib/Reference", fRun);
   if (entry) {
     fCalibReference = (AliEMCALCalibReference *) entry->GetObject();
   }
@@ -580,10 +572,10 @@ Int_t AliEMCALCalibTimeDep::CalcLEDCorrection(Int_t nSM, Int_t nBins)
   TTree *treeAvg = fCalibSignal->GetTreeAvgAmpVsTime();
   TTree *treeLEDRefAvg = fCalibSignal->GetTreeAvgAmpVsTime();
 
-  int iChannelNum; // for regular towers
-  int iRefNum; // for LED
-  double dHour;
-  double dAvgAmp;
+  int iChannelNum = 0; // for regular towers
+  int iRefNum = 0; // for LED
+  double dHour = 0;
+  double dAvgAmp = 0;
 
   treeAvg->SetBranchAddress("fChannelNum", &iChannelNum);
   treeAvg->SetBranchAddress("fHour", &dHour);
@@ -814,7 +806,7 @@ Int_t AliEMCALCalibTimeDep::CalcLEDCorrectionStripBasis(Int_t nSM, Int_t nBins)
 }
 
 //________________________________________________________________
-Int_t AliEMCALCalibTimeDep::CalcTemperatureCorrection(Int_t nSM, Int_t nBins) 
+Int_t AliEMCALCalibTimeDep::CalcTemperatureCorrection(Int_t nSM, Int_t nBins, Int_t binSize
 { // OK, so we didn't have valid LED data that allowed us to do the correction only 
   // with that info.
   // So, instead we'll rely on the temperature info and try to do the correction
@@ -828,25 +820,25 @@ Int_t AliEMCALCalibTimeDep::CalcTemperatureCorrection(Int_t nSM, Int_t nBins)
 
   Double_t dTempCoeff[AliEMCALGeoParams::fgkEMCALCols][AliEMCALGeoParams::fgkEMCALRows];
   memset(dTempCoeff, 0, sizeof(dTempCoeff));
-  Float_t gainM = 0; 
   Double_t correction = 0;
-  Double_t secondsPerBin = (3600/fTimeBinsPerHour);
+  Double_t secondsPerBin = (Double_t) binSize;
 
   for (int i = 0; i < nSM; i++) {
-    AliEMCALSuperModuleCalibTimeDepCorrection * dataCalibTimeDepCorrection = fCalibTimeDepCorrection->GetSuperModuleCalibTimeDepCorrectionNum(iSM);
+    AliEMCALSuperModuleCalibTimeDepCorrection * dataCalibTimeDepCorrection = fCalibTimeDepCorrection->GetSuperModuleCalibTimeDepCorrectionNum(i);
     iSM = dataCalibTimeDepCorrection->GetSuperModuleNum();
 
     AliEMCALSuperModuleCalibReference * dataCalibReference = fCalibReference->GetSuperModuleCalibReferenceNum(iSM);
-    AliEMCALSuperModuleCalibMapAPD * dataCalibMapAPD = fCalibMapAPD->GetSuperModuleCalibMapAPDNum(iSM);
-    AliEMCALSuperModuleBiasAPD * dataBiasAPD = fBiasAPD->GetSuperModuleBiasAPDNum(iSM);
-    
-    // first calculate the M=Gain values, and TemperatureCoeff, for all towers in this SuperModule, from BiasAPD and CalibMapAPD info
+    AliEMCALSuperModuleCalibTempCoeff * dataCalibTempCoeff = fCalibTempCoeff->GetSuperModuleCalibTempCoeffNum(iSM);
+
+    // first get CalibTempCoeff info
     for (iCol = 0; iCol < AliEMCALGeoParams::fgkEMCALCols; iCol++) {
       for (iRow = 0; iRow < AliEMCALGeoParams::fgkEMCALRows; iRow++) {
-       AliEMCALCalibMapAPDVal * mapAPD = dataCalibMapAPD->GetAPDVal(iCol, iRow);
-       gainM =  fCalibMapAPD->GetGain(mapAPD->GetPar(0), mapAPD->GetPar(1), mapAPD->GetPar(2), 
-                                      dataBiasAPD->GetVoltage(iCol, iRow));
-       dTempCoeff[iCol][iRow] = GetTempCoeff(mapAPD->GetDarkCurrent(), gainM);
+
+       dTempCoeff[iCol][iRow] = dataCalibTempCoeff->GetTC(iCol, iRow);
+       if (fVerbosity > 1) {
+         cout << " iSM " << iSM << " iCol " << iCol << " iRow " << iRow 
+              << " dTempCoeff " << dTempCoeff[iCol][iRow] << endl; 
+       }
       }
     }
     
@@ -863,24 +855,53 @@ Int_t AliEMCALCalibTimeDep::CalcTemperatureCorrection(Int_t nSM, Int_t nBins)
     if (nVal>0) {
       referenceTemperature /= nVal; // valid values exist, we can look into corrections
       
+      Double_t dSMTemperature = 0;
       for (int j = 0; j < nBins; j++) {
-
        // what is the timestamp in the middle of this bin? (0.5 is for middle of bin)
        UInt_t timeStamp = fStartTime + (UInt_t)((j+0.5)*secondsPerBin);
       // get the temperature at this time; use average over whole SM for now (TO BE CHECKED LATER - if we can do better with finer grained info)
-       Double_t dSMTemperature = GetTemperatureSM(iSM, timeStamp); 
-
-       Double_t temperatureDiff = referenceTemperature - dSMTemperature; // old vs new
-       // if the new temperature is higher than the old/reference one, then the gain has gone down 
-       if (fabs(temperatureDiff)>fTemperatureResolution) { 
-         // significant enough difference that we need to consider it
+       Double_t oldSMTemperature = dSMTemperature;
+       dSMTemperature = GetTemperatureSM(iSM, timeStamp); 
+       if (j>0 && (dSMTemperature==kErrorCode)) { 
+         // if we have previous values, and retrieval of values failed - use that instead (hopefully good)
+         dSMTemperature = oldSMTemperature; 
+       }
+       Double_t temperatureDiff = referenceTemperature - dSMTemperature; // ref - new
+       if (fVerbosity > 0) {
+         cout << " referenceTemperature " << referenceTemperature
+              << " dSMTemperature " << dSMTemperature
+              << " temperatureDiff " << temperatureDiff
+              << endl;
+       }
+       // if the new temperature is higher than the old/reference one (diff<0), then the gain has gone down 
+       // if the new temperature is lower than the old/reference one (diff>0), then the gain has gone up
+       // dTempCoeff is a (unsigned) factor describing how many % the gain 
+       // changes with a degree change.  
+       // i.e. the product temperatureDiff * dTempCoeff increase when the gain goes up
+       // The correction we want to keep is what we should multiply our ADC value with as a function
+       // of time, i.e. the inverse of the gain change..
+       if ( (TMath::Abs(temperatureDiff)>fTemperatureResolution)
+            && (TMath::Abs(temperatureDiff)<fMaxTemperatureDiff) ) {
+         // significant enough difference that we need to consider it, and also not unreasonably large
 
          // loop over all towers; effect of temperature change will depend on gain for this tower
          for (iCol = 0; iCol < AliEMCALGeoParams::fgkEMCALCols; iCol++) {
            for (iRow = 0; iRow < AliEMCALGeoParams::fgkEMCALRows; iRow++) {
 
-             correction = temperatureDiff * dTempCoeff[iCol][iRow];
+             // the correction should be inverse of modification in gain: (see discussion above)
+             // modification in gain: 1.0 + (temperatureDiff * dTempCoeff[iCol][iRow])*0.01; 
+             // 1/(1+x) ~= 1 - x for small x, i.e. we arrive at:
+             correction = 1.0 - (temperatureDiff * dTempCoeff[iCol][iRow]); 
              dataCalibTimeDepCorrection->GetCorrection(iCol,iRow)->AddAt(correction, j);
+             if (fVerbosity > 1) {
+               cout << " iSM " << iSM
+                    << " iCol  " << iCol 
+                    << " iRow  " << iRow 
+                    << " j  " << j 
+                    << " correction  " << correction 
+                    << endl;
+             }
            }
          }