Skip to content

Commit da3f68c

Browse files
author
Sunanda
committed
Use ESGetToken in some of the codes in SimGeneral/DataMixingModule and SimGeneral/Debugging
1 parent 06235a5 commit da3f68c

File tree

5 files changed

+25
-18
lines changed

5 files changed

+25
-18
lines changed

SimGeneral/DataMixingModule/plugins/DataMixingEMDigiWorker.cc

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,7 @@
44
//
55
//--------------------------------------------
66

7-
#include "CondFormats/DataRecord/interface/EcalGainRatiosRcd.h"
8-
#include "CondFormats/DataRecord/interface/EcalPedestalsRcd.h"
9-
#include "CondFormats/EcalObjects/interface/EcalGainRatios.h"
107
#include "CondFormats/EcalObjects/interface/EcalMGPAGainRatio.h"
11-
#include "CondFormats/EcalObjects/interface/EcalPedestals.h"
128
#include "DataFormats/Common/interface/Handle.h"
139
#include "FWCore/Framework/interface/ESHandle.h"
1410
#include "FWCore/Framework/interface/EventSetup.h"
@@ -59,6 +55,9 @@ namespace edm {
5955
EBDigiCollectionDM_ = ps.getParameter<std::string>("EBDigiCollectionDM");
6056
EEDigiCollectionDM_ = ps.getParameter<std::string>("EEDigiCollectionDM");
6157
ESDigiCollectionDM_ = ps.getParameter<std::string>("ESDigiCollectionDM");
58+
59+
pedToken_ = iC.esConsumes<EcalPedestals, EcalPedestalsRcd>();
60+
grToken_ = iC.esConsumes<EcalGainRatios, EcalGainRatiosRcd>();
6261
}
6362

6463
// Virtual destructor needed.
@@ -530,8 +529,7 @@ namespace edm {
530529
std::vector<float> pedeStals(3);
531530

532531
// get pedestals
533-
edm::ESHandle<EcalPedestals> pedHandle;
534-
ES.get<EcalPedestalsRcd>().get(pedHandle);
532+
const auto& pedHandle = ES.getHandle(pedToken_);
535533

536534
const EcalPedestalsMap &pedMap = pedHandle.product()->getMap(); // map of pedestals
537535
EcalPedestalsMapIterator pedIter; // pedestal iterator
@@ -556,8 +554,7 @@ namespace edm {
556554
const std::vector<float> DataMixingEMDigiWorker::GetGainRatios(const edm::EventSetup &ES, const DetId &detid) {
557555
std::vector<float> gainRatios(3);
558556
// get gain ratios
559-
edm::ESHandle<EcalGainRatios> grHandle;
560-
ES.get<EcalGainRatiosRcd>().get(grHandle);
557+
const auto& grHandle = ES.getHandle(grToken_);
561558
EcalMGPAGainRatio theRatio = (*grHandle)[detid];
562559

563560
gainRatios[0] = 1.;

SimGeneral/DataMixingModule/plugins/DataMixingEMDigiWorker.h

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,11 @@
3030
#include "DataFormats/EcalDigi/interface/ESDataFrame.h"
3131
#include "DataFormats/EcalDigi/interface/EcalDigiCollections.h"
3232

33+
#include "CondFormats/DataRecord/interface/EcalPedestalsRcd.h"
34+
#include "CondFormats/EcalObjects/interface/EcalPedestals.h"
35+
#include "CondFormats/DataRecord/interface/EcalGainRatiosRcd.h"
36+
#include "CondFormats/EcalObjects/interface/EcalGainRatios.h"
37+
3338
#include <map>
3439
#include <string>
3540
#include <vector>
@@ -105,6 +110,9 @@ namespace edm {
105110
// unsigned int eventId_; //=0 for signal, from 1-n for pileup events
106111

107112
std::string label_;
113+
114+
edm::ESGetToken<EcalPedestals, EcalPedestalsRcd> pedToken_;
115+
edm::ESGetToken<EcalGainRatios, EcalGainRatiosRcd> grToken_;
108116
};
109117
} // namespace edm
110118

SimGeneral/DataMixingModule/plugins/DataMixingHcalDigiWorker.cc

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@
1313
// calibration headers, for future reference
1414
#include "CalibFormats/HcalObjects/interface/HcalCalibrations.h"
1515
#include "CalibFormats/HcalObjects/interface/HcalCoderDb.h"
16-
#include "CalibFormats/HcalObjects/interface/HcalDbRecord.h"
17-
#include "CalibFormats/HcalObjects/interface/HcalDbService.h"
1816
//
1917
//
2018
#include "DataMixingHcalDigiWorker.h"
@@ -285,6 +283,9 @@ namespace edm {
285283
ZDCDigiCollectionDM_ = ps.getParameter<std::string>("ZDCDigiCollectionDM");
286284
QIE10DigiCollectionDM_ = ps.getParameter<std::string>("QIE10DigiCollectionDM");
287285
QIE11DigiCollectionDM_ = ps.getParameter<std::string>("QIE11DigiCollectionDM");
286+
287+
dbToken_ = iC.esConsumes<HcalDbService, HcalDbRecord>();
288+
288289
}
289290

290291
// Virtual destructor needed.
@@ -293,8 +294,7 @@ namespace edm {
293294
void DataMixingHcalDigiWorker::addHcalSignals(const edm::Event &e, const edm::EventSetup &ES) {
294295
// Calibration stuff will look like this:
295296
// get conditions
296-
edm::ESHandle<HcalDbService> conditions;
297-
ES.get<HcalDbRecord>().get(conditions);
297+
const auto& conditions = ES.getHandle(dbToken_);
298298

299299
// fill in maps of hits
300300

@@ -359,8 +359,7 @@ namespace edm {
359359
<< " for bunchcrossing " << bcr;
360360

361361
// get conditions
362-
edm::ESHandle<HcalDbService> conditions;
363-
ES.get<HcalDbRecord>().get(conditions);
362+
const auto& conditions = ES.getHandle(dbToken_);
364363

365364
convertPileupHcalDigis<HBHEDigiCollection>(*ep, HBHEPileInputTag_, mcc, *conditions, HBHEDigiStorage_);
366365
convertPileupHcalDigis<HODigiCollection>(*ep, HOPileInputTag_, mcc, *conditions, HODigiStorage_);
@@ -407,8 +406,7 @@ namespace edm {
407406
}
408407

409408
void DataMixingHcalDigiWorker::putHcal(edm::Event &e, const edm::EventSetup &ES) {
410-
edm::ESHandle<HcalDbService> conditions;
411-
ES.get<HcalDbRecord>().get(conditions);
409+
const auto& conditions = ES.getHandle(dbToken_);
412410

413411
// collection of digis to put in the event
414412
std::unique_ptr<HBHEDigiCollection> HBHEdigis = buildHcalDigis<HBHEDigiCollection>(HBHEDigiStorage_, *conditions);

SimGeneral/DataMixingModule/plugins/DataMixingHcalDigiWorker.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
#include "FWCore/ParameterSet/interface/ParameterSet.h"
2222

2323
#include "CalibFormats/CaloObjects/interface/CaloSamples.h"
24+
#include "CalibFormats/HcalObjects/interface/HcalDbRecord.h"
25+
#include "CalibFormats/HcalObjects/interface/HcalDbService.h"
2426
#include "DataFormats/Common/interface/Handle.h"
2527
#include "DataFormats/HcalDigi/interface/HBHEDataFrame.h"
2628
#include "DataFormats/HcalDigi/interface/HFDataFrame.h"
@@ -114,6 +116,7 @@ namespace edm {
114116
bool DoZDC_;
115117

116118
std::string label_;
119+
edm::ESGetToken<HcalDbService, HcalDbRecord> dbToken_;
117120
};
118121
} // namespace edm
119122

SimGeneral/Debugging/plugins/CaloParticleDebugger.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ class CaloParticleDebugger : public edm::one::EDAnalyzer<> {
7070
edm::EDGetTokenT<std::vector<CaloParticle>> caloParticlesToken_;
7171
edm::EDGetTokenT<std::vector<SimCluster>> simClustersToken_;
7272
std::vector<edm::EDGetTokenT<std::vector<PCaloHit>>> collectionTagsToken_;
73+
edm::ESGetToken<CaloGeometry, CaloGeometryRecord> geomToken_;
7374
int geometryType_ = 0;
7475
// ----------member data ---------------------------
7576
};
@@ -103,6 +104,7 @@ CaloParticleDebugger::CaloParticleDebugger(const edm::ParameterSet& iConfig)
103104
for (auto const& collectionTag : collectionTags_) {
104105
collectionTagsToken_.push_back(iC.consumes<std::vector<PCaloHit>>(collectionTag));
105106
}
107+
geomToken_ = iC.esConsumes<CaloGeometry, CaloGeometryRecord>();
106108
}
107109

108110
CaloParticleDebugger::~CaloParticleDebugger() {}
@@ -282,8 +284,7 @@ void CaloParticleDebugger::fillSimHits(std::map<int, float>& detIdToTotalSimEner
282284
const edm::Event& iEvent,
283285
const edm::EventSetup& iSetup) {
284286
// Taken needed quantities from the EventSetup
285-
edm::ESHandle<CaloGeometry> geom;
286-
iSetup.get<CaloGeometryRecord>().get(geom);
287+
const auto& geom = iSetup.getHandle(geomToken_);
287288
const HGCalGeometry *eegeom = nullptr, *fhgeom = nullptr, *bhgeomnew = nullptr;
288289
const HcalGeometry* bhgeom = nullptr;
289290
const HGCalDDDConstants* hgddd[3];

0 commit comments

Comments
 (0)