- Fix coverity defects - Increase range for the trigger patch amplitude
authormfasel <mfasel@lbl.gov>
Mon, 13 Oct 2014 14:35:08 +0000 (16:35 +0200)
committermverweij <marta.verweij@cern.ch>
Mon, 13 Oct 2014 14:51:22 +0000 (16:51 +0200)
PWGJE/EMCALJetTasks/UserTasks/AliAnalysisTaskPtEMCalTrigger.cxx

index 02fe1db..d3c28d9 100644 (file)
@@ -185,14 +185,14 @@ namespace EMCalTriggerPtAnalysis {
     const TAxis *clusteraxes[4];
     for(int iaxis = 0; iaxis < 4; ++iaxis) clusteraxes[iaxis] = hclusteraxes + iaxis;
     TAxis hpatchenergyaxes[4];
-    DefineAxis(hpatchenergyaxes[0], "energy", "Patch energy (GeV)", 100, 0., 100.);
+    DefineAxis(hpatchenergyaxes[0], "energy", "Patch energy (GeV)", 100, 0., 100);
     DefineAxis(hpatchenergyaxes[1], "eta", "#eta", etabinning);
     DefineAxis(hpatchenergyaxes[2], "phi", "#phi",  20, 0, 2 * TMath::Pi());
     DefineAxis(hpatchenergyaxes[3], "isMain", "Main trigger", 2, -0.5, 1.5);
     const TAxis *patchenergyaxes[4];
     for(int iaxis = 0; iaxis < 4; ++iaxis) patchenergyaxes[iaxis] = hpatchenergyaxes + iaxis;
     TAxis hpatchampaxes[4];
-    DefineAxis(hpatchampaxes[0], "amplitude", "Patch energy (GeV)", 2000, 0., 2000.);
+    DefineAxis(hpatchampaxes[0], "amplitude", "Patch energy (GeV)", 10000, 0., 1000.);
     DefineAxis(hpatchampaxes[1], "eta", "#eta", etabinning);
     DefineAxis(hpatchampaxes[2], "phi", "#phi",  20, 0, 2 * TMath::Pi());
     DefineAxis(hpatchampaxes[3], "isMain", "Main trigger", 2, -0.5, 1.5);
@@ -262,9 +262,9 @@ namespace EMCalTriggerPtAnalysis {
     AliEmcalTriggerPatchInfo *triggerpatch(NULL);
     TIter patchIter(this->fTriggerPatchInfo);
     while((triggerpatch = dynamic_cast<AliEmcalTriggerPatchInfo *>(patchIter()))){
-      double triggerpatchinfo[4] = {triggerpatch->GetPatchE(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1 : 0};
-      double triggerpatchinfoamp[4] = {triggerpatch->GetADCAmp(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1 : 0};
-      double triggerpatchinfoer[4] = {triggerpatch->GetADCAmpGeVRough(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1 : 0};
+      double triggerpatchinfo[4] = {triggerpatch->GetPatchE(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1. : 0.};
+      double triggerpatchinfoamp[4] = {triggerpatch->GetADCAmp(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1. : 0.};
+      double triggerpatchinfoer[4] = {triggerpatch->GetADCAmpGeVRough(), triggerpatch->GetEtaGeo(), triggerpatch->GetPhiGeo(), triggerpatch->IsMainTrigger() ? 1. : 0.};
       if(triggerpatch->IsJetHigh()){
         fHistos->FillTHnSparse("EnergyJetHigh", triggerpatchinfo);
         fHistos->FillTHnSparse("AmplitudeJetHigh", triggerpatchinfoamp);