Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migration of pixel code to esConsumes #32093

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
0a0751c
migration to esConsumes in Validation/SiPixelPhase1HitsV
ferencek Nov 3, 2020
e782ea5
migration to esConsumes in Validation/SiPixelPhase1TrackClustersV
ferencek Nov 3, 2020
04f8bc3
unused code cleanup in Validation/SiPixelPhase1HitsV
ferencek Nov 3, 2020
4f1cbdf
migration to esConsumes in SimTracker/SiPixelDigitizer/test
ferencek Nov 3, 2020
9c663fb
semi-random header file cleanup
ferencek Nov 3, 2020
9c1895c
migration to esConsumes in DQM/SiPixelMonitorClient
ferencek Nov 10, 2020
bad6798
migration to esConsumes in DQM/SiPixelMonitorCluster
ferencek Nov 10, 2020
226be17
additional esConsumes updates in DQM/SiPixelMonitorCluster
ferencek Nov 11, 2020
89f2c85
preserve old behavior where edm::EventSetup::tryToGet() is used
ferencek Nov 11, 2020
6c41889
switched from ESHandle to the actual product as data member to avoid …
ferencek Nov 11, 2020
0ad199b
migration to esConsumes in DQM/SiPixelMonitorDigi
ferencek Nov 12, 2020
638e721
migration to esConsumes in DQM/SiPixelMonitorRawData
ferencek Nov 12, 2020
090b248
migration to esConsumes in DQM/SiPixelMonitorRecHit
ferencek Nov 12, 2020
41d416d
simplifying code, adding comment about intentional copy
ferencek Nov 12, 2020
6441e9c
migration to esConsumes in DQM/SiPixelPhase1Common
ferencek Nov 12, 2020
e3fc1a7
migration to esConsumes in DQM/SiPixelMonitorTrack
ferencek Nov 13, 2020
e7fe132
migration to esConsumes in DQM/SiPixelPhase1Track
ferencek Nov 13, 2020
93ccef4
code format updates in DQM/SiPixelMonitorClient
ferencek Nov 16, 2020
a97fe8d
code format updates in DQM/SiPixelMonitorCluster
ferencek Nov 16, 2020
4c019ae
code format updates in DQM/SiPixelMonitorDigi
ferencek Nov 16, 2020
a73ab1f
code format updates in DQM/SiPixelMonitorRecHit
ferencek Nov 16, 2020
57ff993
code format updates in DQM/SiPixelMonitorTrack
ferencek Nov 16, 2020
728164c
code format updates in DQM/SiPixelPhase1Common
ferencek Nov 16, 2020
36798b0
code format updates in DQM/SiPixelPhase1Track
ferencek Nov 16, 2020
91c264f
code format updates in Validation/SiPixelPhase1TrackClustersV
ferencek Nov 16, 2020
f1ef1b3
esConsumes update in DQM/SiPixelMonitorDigi (removing consumes regist…
ferencek Nov 23, 2020
13ea7f7
esConsumes update in DQM/SiPixelMonitorCluster (removing consumes reg…
ferencek Nov 23, 2020
9d2ce2f
esConsumes update in DQM/SiPixelMonitorRecHit (removing consumes regi…
ferencek Nov 23, 2020
a7d8853
esConsumes update in DQM/SiPixelMonitorTrack (removing consumes regis…
ferencek Nov 23, 2020
6800360
migration to esConsumes in CalibTracker/SiPixelConnectivity
ferencek Nov 23, 2020
ba89446
migration to esConsumes in CalibTracker/SiPixelErrorEstimation
ferencek Nov 23, 2020
c991e95
migration to esConsumes in CalibTracker/SiPixelESProducers
ferencek Nov 24, 2020
ab285fe
migration to esConsumes in CalibTracker/SiPixelLorentzAngle
ferencek Nov 24, 2020
a53d431
code format updates in DQM/SiPixelMonitorTrack
ferencek Nov 24, 2020
0a8eca1
code format updates in DQM/SiPixelMonitorRecHit
ferencek Nov 24, 2020
0dd07a3
code format updates in DQM/SiPixelMonitorDigi
ferencek Nov 24, 2020
5fa9e75
code format updates in DQM/SiPixelMonitorCluster
ferencek Nov 24, 2020
b17511b
code format updates in CalibTracker/SiPixelLorentzAngle
ferencek Nov 24, 2020
b409f0b
code format updates in CalibTracker/SiPixelESProducers
ferencek Nov 24, 2020
74a1a86
migration to esConsumes in CalibTracker/SiPixelGainCalibration
ferencek Nov 24, 2020
d5112ee
migration to esConsumes in CalibTracker/SiPixelTools
ferencek Nov 24, 2020
e9320a3
code format updates in CalibTracker/SiPixelTools
ferencek Nov 24, 2020
a424cfd
migration to esConsumes in SiPixelClusterProducer
ferencek Nov 24, 2020
2640878
migration to esConsumes in RecoLocalTracker/SiPixelClusterizer/test
ferencek Nov 24, 2020
2c88809
full migration to consumes in RecoLocalTracker/SiPixelClusterizer/test
ferencek Nov 24, 2020
48adff9
code format updates in RecoLocalTracker/SiPixelClusterizer/test
ferencek Nov 24, 2020
7b3711f
added missing include after updates to SiPixelOfflineCalibAnalysisBase.h
ferencek Nov 24, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
#ifndef SiPixelFedCablingMapBuilder_H
#define SiPixelFedCablingMapBuilder_H

#include "FWCore/Framework/interface/EventSetup.h"
#include "FWCore/Framework/interface/ConsumesCollector.h"
#include "CondFormats/SiPixelObjects/interface/SiPixelFedCablingTree.h"
#include "DataFormats/DetId/interface/DetId.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "DataFormats/TrackerCommon/interface/TrackerTopology.h"
#include "Geometry/Records/interface/TrackerTopologyRcd.h"

#include <vector>
#include <string>
Expand All @@ -14,7 +17,7 @@ class PixelGeomDetUnit;
class SiPixelFedCablingMapBuilder {
public:
//SiPixelFedCablingMapBuilder(const std::string & associatorName);
SiPixelFedCablingMapBuilder(const std::string fileName, const bool phase1 = false);
SiPixelFedCablingMapBuilder(edm::ConsumesCollector&& iCC, const std::string fileName, const bool phase1 = false);
SiPixelFedCablingTree* produce(const edm::EventSetup& setup);

private:
Expand All @@ -27,6 +30,9 @@ class SiPixelFedCablingMapBuilder {
std::string fileName_;
std::string myprint(const PixelGeomDetUnit* pxUnit);
bool phase1_;

edm::ESGetToken<TrackerTopology, TrackerTopologyRcd> trackerTopoToken_;
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomToken_;
};

#endif
Original file line number Diff line number Diff line change
@@ -1,11 +1,8 @@
#include <iostream>

#include "CalibTracker/SiPixelConnectivity/interface/SiPixelFedCablingMapBuilder.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
#include "DataFormats/FEDRawData/interface/FEDNumbering.h"
#include "FWCore/Framework/interface/ESHandle.h"

#include <ostream>
#include "DataFormats/TrackerCommon/interface/PixelEndcapName.h"
Expand All @@ -30,16 +27,19 @@
using namespace std;
using namespace sipixelobjects;

SiPixelFedCablingMapBuilder::SiPixelFedCablingMapBuilder(const string fileName,
SiPixelFedCablingMapBuilder::SiPixelFedCablingMapBuilder(edm::ConsumesCollector&& iCC,
const string fileName,
const bool phase1)
: fileName_(fileName) //, phase1_(phase1) not used anymore
{}
{
trackerTopoToken_ = iCC.esConsumes<TrackerTopology, TrackerTopologyRcd>();
trackerGeomToken_ = iCC.esConsumes<TrackerGeometry, TrackerDigiGeometryRecord>();
}

SiPixelFedCablingTree* SiPixelFedCablingMapBuilder::produce(const edm::EventSetup& setup) {
// Access geometry
edm::LogInfo("read tracker geometry...");
edm::ESHandle<TrackerGeometry> pDD;
setup.get<TrackerDigiGeometryRecord>().get(pDD);
edm::ESHandle<TrackerGeometry> pDD = setup.getHandle(trackerGeomToken_);
edm::LogInfo("tracker geometry read") << "There are: " << pDD->dets().size() << " detectors";

// Test new TrackerGeometry features
Expand Down Expand Up @@ -86,8 +86,7 @@ SiPixelFedCablingTree* SiPixelFedCablingMapBuilder::produce(const edm::EventSetu
edm::LogInfo(" version ") << version << endl;

// Access topology
edm::ESHandle<TrackerTopology> tTopo;
setup.get<TrackerTopologyRcd>().get(tTopo);
edm::ESHandle<TrackerTopology> tTopo = setup.getHandle(trackerTopoToken_);
const TrackerTopology* tt = tTopo.product();

typedef TrackerGeometry::DetContainer::const_iterator ITG;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ void SiPixelFedCablingMapWriter::analyze(const edm::Event& iEvent, const edm::Ev
//std::cout << "-------HERE-----------" << endl;
//cabling = SiPixelFedCablingMapBuilder(pixelToFedAssociator_).produce(iSetup);
//cabling = SiPixelFedCablingMapBuilder(fileName_,phase1_).produce(iSetup);
cabling = SiPixelFedCablingMapBuilder(fileName_).produce(iSetup);
cabling = SiPixelFedCablingMapBuilder(consumesCollector(), fileName_).produce(iSetup);
//std::cout << "-------Print Map ----------- DOES NOT WORK for phase1 " << endl;
edm::LogInfo("PRINTING MAP (Does not work for phase1: ") << cabling->print(3);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@
//

#include "FWCore/Framework/interface/EDAnalyzer.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include <string>
#include <iostream>
#include <fstream>
Expand All @@ -34,6 +36,7 @@ class SiPixelDetInfoFileWriter : public edm::EDAnalyzer {
void analyze(const edm::Event &, const edm::EventSetup &) override;

private:
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomTokenBeginRun_;
std::ofstream outputFile_;
std::string filePath_;
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,8 @@

#include "CalibTracker/SiPixelESProducers/interface/SiPixelDetInfoFileWriter.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"

#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/CommonDetUnit/interface/GeomDet.h"
#include "Geometry/CommonTopologies/interface/PixelTopology.h"
#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
Expand All @@ -21,6 +18,7 @@ using namespace std;
SiPixelDetInfoFileWriter::SiPixelDetInfoFileWriter(const edm::ParameterSet &iConfig) {
edm::LogInfo("SiPixelDetInfoFileWriter::SiPixelDetInfoFileWriter");

trackerGeomTokenBeginRun_ = esConsumes<TrackerGeometry, TrackerDigiGeometryRecord, edm::Transition::BeginRun>();
filePath_ = iConfig.getUntrackedParameter<std::string>("FilePath", std::string("SiPixelDetInfo.dat"));
}

Expand All @@ -32,9 +30,7 @@ void SiPixelDetInfoFileWriter::beginRun(const edm::Run &run, const edm::EventSet
outputFile_.open(filePath_.c_str());

if (outputFile_.is_open()) {
edm::ESHandle<TrackerGeometry> pDD;

iSetup.get<TrackerDigiGeometryRecord>().get(pDD);
edm::ESHandle<TrackerGeometry> pDD = iSetup.getHandle(trackerGeomTokenBeginRun_);

edm::LogInfo("SiPixelDetInfoFileWriter::beginJob - got geometry ") << std::endl;
edm::LogInfo("SiPixelDetInfoFileWriter") << " There are " << pDD->detUnits().size() << " detectors" << std::endl;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,9 @@
#include <iostream>
#include <map>
#include "FWCore/Framework/interface/one/EDAnalyzer.h"
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/Framework/interface/Event.h"
#include "FWCore/Framework/interface/EventSetup.h"
#include "FWCore/Framework/interface/MakerMacros.h"
#include "FWCore/Framework/interface/Frameworkfwd.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "DataFormats/SiPixelDetId/interface/PixelFEDChannel.h"
#include "CalibTracker/Records/interface/SiPixelFEDChannelContainerESProducerRcd.h"

Expand All @@ -25,11 +22,15 @@ class PixelFEDChannelCollectionMapTestReader : public edm::one::EDAnalyzer<> {
// ----------member data ---------------------------
const bool printdebug_;
const std::string formatedOutput_;
edm::ESGetToken<PixelFEDChannelCollectionMap, SiPixelFEDChannelContainerESProducerRcd>
pixelFEDChannelCollectionMapToken_;
};

PixelFEDChannelCollectionMapTestReader::PixelFEDChannelCollectionMapTestReader(edm::ParameterSet const& p)
: printdebug_(p.getUntrackedParameter<bool>("printDebug", true)),
formatedOutput_(p.getUntrackedParameter<std::string>("outputFile", "")) {
formatedOutput_(p.getUntrackedParameter<std::string>("outputFile", "")),
pixelFEDChannelCollectionMapToken_(
esConsumes<PixelFEDChannelCollectionMap, SiPixelFEDChannelContainerESProducerRcd>()) {
edm::LogInfo("PixelFEDChannelCollectionMapTestReader") << "PixelFEDChannelCollectionMapTestReader" << std::endl;
}

Expand All @@ -53,11 +54,9 @@ void PixelFEDChannelCollectionMapTestReader::analyze(const edm::Event& e, const
}

//this part gets the handle of the event source and the record (i.e. the Database)
edm::ESHandle<PixelFEDChannelCollectionMap> PixelFEDChannelCollectionMapHandle;
edm::LogInfo("PixelFEDChannelCollectionMapTestReader") << "got eshandle" << std::endl;

context.get<SiPixelFEDChannelContainerESProducerRcd>().get(PixelFEDChannelCollectionMapHandle);
edm::LogInfo("PixelFEDChannelCollectionMapTestReader") << "got context" << std::endl;
edm::ESHandle<PixelFEDChannelCollectionMap> PixelFEDChannelCollectionMapHandle =
context.getHandle(pixelFEDChannelCollectionMapToken_);
edm::LogInfo("PixelFEDChannelCollectionMapTestReader") << "got eshandle and context" << std::endl;

const PixelFEDChannelCollectionMap* thePixelFEDChannelCollectionMap = PixelFEDChannelCollectionMapHandle.product();
edm::LogInfo("PixelFEDChannelCollectionMapTestReader") << "got SiPixelFEDChannelContainer* " << std::endl;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,12 @@
#include "CalibTracker/SiPixelESProducers/test/SiPixelFakeGainForHLTReader.h"

#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/CommonTopologies/interface/PixelTopology.h"
namespace cms {
SiPixelFakeGainForHLTReader::SiPixelFakeGainForHLTReader(const edm::ParameterSet& conf)
: conf_(conf),
trackerGeomToken_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord>()),
trackerGeomTokenBeginRun_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord, edm::Transition::BeginRun>()),
SiPixelGainCalibrationForHLTService_(conf),
filename_(conf.getParameter<std::string>("fileName")) {}

Expand All @@ -17,14 +18,9 @@ namespace cms {
fFile->cd();

// Get the Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomToken_);
edm::LogInfo("SiPixelFakeGainForHLTReader") << " There are " << tkgeom->dets().size() << " detectors" << std::endl;

// Get the calibrationForHLT data
//iSetup.get<SiPixelGainCalibrationForHLTRcd>().get(SiPixelGainCalibrationForHLT_);
//edm::LogInfo("SiPixelFakeGainForHLTReader") << "[SiPixelFakeGainForHLTReader::analyze] End Reading FakeGainForHLTects" << std::endl;
//SiPixelGainCalibrationForHLTService_.setESObjects(iSetup);

// for(TrackerGeometry::DetContainer::const_iterator it = tkgeom->dets().begin(); it != tkgeom->dets().end(); it++){
// if( dynamic_cast<PixelGeomDetUnit*>((*it))!=0){
// uint32_t detid=((*it)->geographicalId()).rawId();
Expand Down Expand Up @@ -85,17 +81,14 @@ namespace cms {
char name[128];

// Get Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomTokenBeginRun_);

// Get the calibrationForHLT data
//edm::ESHandle<SiPixelGainCalibration> SiPixelGainCalibration_;
//iSetup.get<SiPixelGainCalibrationRcd>().get(SiPixelGainCalibration_);
SiPixelGainCalibrationForHLTService_.setESObjects(iSetup);
edm::LogInfo("SiPixelFakeGainForHLTReader")
<< "[SiPixelFakeGainForHLTReader::beginJob] End Reading FakeGainForHLTects" << std::endl;
// Get the list of DetId's
std::vector<uint32_t> vdetId_ = SiPixelGainCalibrationForHLTService_.getDetIds();
//SiPixelGainCalibrationForHLT_->getDetIds(vdetId_);
// Loop over DetId's
for (std::vector<uint32_t>::const_iterator detid_iter = vdetId_.begin(); detid_iter != vdetId_.end();
detid_iter++) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,9 @@
//
#include "FWCore/Framework/interface/Frameworkfwd.h"
#include "FWCore/Framework/interface/EDAnalyzer.h"
#include "FWCore/Framework/interface/Event.h"
#include "FWCore/Framework/interface/EventSetup.h"
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
//#include "CondFormats/SiPixelObjects/interface/SiPixelGainCalibrationForHLT.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "CalibTracker/SiPixelESProducers/interface/SiPixelGainCalibrationForHLTService.h"

#include "TROOT.h"
Expand All @@ -47,8 +43,8 @@ namespace cms {

private:
edm::ParameterSet conf_;
edm::ESHandle<TrackerGeometry> tkgeom;
//edm::ESHandle<SiPixelGainCalibrationForHLT> SiPixelGainCalibrationForHLT_;
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomToken_;
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomTokenBeginRun_;
SiPixelGainCalibrationForHLTService SiPixelGainCalibrationForHLTService_;

std::map<uint32_t, TH1F*> _TH1F_Pedestals_m;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,13 @@
#include "CalibTracker/SiPixelESProducers/test/SiPixelFakeGainOfflineReader.h"

#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/CommonTopologies/interface/PixelTopology.h"
namespace cms {
SiPixelFakeGainOfflineReader::SiPixelFakeGainOfflineReader(const edm::ParameterSet& conf)
: conf_(conf),
SiPixelGainCalibrationOfflineService_(conf),
trackerGeomToken_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord>()),
trackerGeomTokenBeginRun_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord, edm::Transition::BeginRun>()),
filename_(conf.getParameter<std::string>("fileName")) {}

void SiPixelFakeGainOfflineReader::analyze(const edm::Event& iEvent, const edm::EventSetup& iSetup) {
Expand All @@ -17,14 +18,9 @@ namespace cms {
fFile->cd();

// Get the Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomToken_);
edm::LogInfo("SiPixelFakeGainOfflineReader") << " There are " << tkgeom->dets().size() << " detectors" << std::endl;

// Get the calibrationOffline data
//iSetup.get<SiPixelGainCalibrationOfflineRcd>().get(SiPixelGainCalibrationOffline_);
//edm::LogInfo("SiPixelFakeGainOfflineReader") << "[SiPixelFakeGainOfflineReader::analyze] End Reading FakeGainOfflineects" << std::endl;
//SiPixelGainCalibrationOfflineService_.setESObjects(iSetup);

// for(TrackerGeometry::DetContainer::const_iterator it = tkgeom->dets().begin(); it != tkgeom->dets().end(); it++){
// if( dynamic_cast<PixelGeomDetUnit*>((*it))!=0){
// uint32_t detid=((*it)->geographicalId()).rawId();
Expand Down Expand Up @@ -91,17 +87,14 @@ namespace cms {
char name[128];

// Get Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomTokenBeginRun_);

// Get the calibrationOffline data
//edm::ESHandle<SiPixelGainCalibrationOffline> SiPixelGainCalibrationOffline_;
//iSetup.get<SiPixelGainCalibrationOfflineRcd>().get(SiPixelGainCalibrationOffline_);
SiPixelGainCalibrationOfflineService_.setESObjects(iSetup);
edm::LogInfo("SiPixelFakeGainOfflineReader")
<< "[SiPixelFakeGainOfflineReader::beginJob] End Reading FakeGainOfflineects" << std::endl;
// Get the list of DetId's
std::vector<uint32_t> vdetId_ = SiPixelGainCalibrationOfflineService_.getDetIds();
//SiPixelGainCalibrationOffline_->getDetIds(vdetId_);
// Loop over DetId's
for (std::vector<uint32_t>::const_iterator detid_iter = vdetId_.begin(); detid_iter != vdetId_.end();
detid_iter++) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,9 @@
//
#include "FWCore/Framework/interface/Frameworkfwd.h"
#include "FWCore/Framework/interface/EDAnalyzer.h"
#include "FWCore/Framework/interface/Event.h"
#include "FWCore/Framework/interface/EventSetup.h"
#include "FWCore/Framework/interface/ESHandle.h"
#include "FWCore/ParameterSet/interface/ParameterSet.h"
#include "FWCore/MessageLogger/interface/MessageLogger.h"
//#include "CondFormats/SiPixelObjects/interface/SiPixelGainCalibrationOffline.h"
#include "Geometry/TrackerGeometryBuilder/interface/TrackerGeometry.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "CalibTracker/SiPixelESProducers/interface/SiPixelGainCalibrationOfflineService.h"

#include "TROOT.h"
Expand All @@ -47,10 +43,10 @@ namespace cms {

private:
edm::ParameterSet conf_;
edm::ESHandle<TrackerGeometry> tkgeom;
//edm::ESHandle<SiPixelGainCalibrationOffline> SiPixelGainCalibrationOffline_;
SiPixelGainCalibrationOfflineService SiPixelGainCalibrationOfflineService_;

SiPixelGainCalibrationOfflineService SiPixelGainCalibrationOfflineService_;
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomToken_;
edm::ESGetToken<TrackerGeometry, TrackerDigiGeometryRecord> trackerGeomTokenBeginRun_;
std::map<uint32_t, TH1F*> _TH1F_Pedestals_m;
std::map<uint32_t, TH1F*> _TH1F_Gains_m;
std::string filename_;
Expand Down
19 changes: 7 additions & 12 deletions CalibTracker/SiPixelESProducers/test/SiPixelFakeGainReader.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3,26 +3,24 @@
#include "CalibTracker/SiPixelESProducers/test/SiPixelFakeGainReader.h"

#include "Geometry/CommonDetUnit/interface/PixelGeomDetUnit.h"
#include "Geometry/Records/interface/TrackerDigiGeometryRecord.h"
#include "Geometry/CommonTopologies/interface/PixelTopology.h"
namespace cms {
SiPixelFakeGainReader::SiPixelFakeGainReader(const edm::ParameterSet& conf)
: conf_(conf), SiPixelGainCalibrationService_(conf), filename_(conf.getParameter<std::string>("fileName")) {}
: conf_(conf),
trackerGeomToken_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord>()),
trackerGeomTokenBeginRun_(esConsumes<TrackerGeometry, TrackerDigiGeometryRecord, edm::Transition::BeginRun>()),
SiPixelGainCalibrationService_(conf),
filename_(conf.getParameter<std::string>("fileName")) {}

void SiPixelFakeGainReader::analyze(const edm::Event& iEvent, const edm::EventSetup& iSetup) {
unsigned int nmodules = 0;
uint32_t nchannels = 0;
fFile->cd();

// Get the Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomToken_);
edm::LogInfo("SiPixelFakeGainReader") << " There are " << tkgeom->dets().size() << " detectors" << std::endl;

// Get the calibration data
//iSetup.get<SiPixelGainCalibrationRcd>().get(SiPixelGainCalibration_);
//edm::LogInfo("SiPixelFakeGainReader") << "[SiPixelFakeGainReader::analyze] End Reading FakeGainects" << std::endl;
//SiPixelGainCalibrationService_.setESObjects(iSetup);

// for(TrackerGeometry::DetContainer::const_iterator it = tkgeom->dets().begin(); it != tkgeom->dets().end(); it++){
// if( dynamic_cast<PixelGeomDetUnit*>((*it))!=0){
// uint32_t detid=((*it)->geographicalId()).rawId();
Expand Down Expand Up @@ -89,17 +87,14 @@ namespace cms {
char name[128];

// Get Geometry
iSetup.get<TrackerDigiGeometryRecord>().get(tkgeom);
edm::ESHandle<TrackerGeometry> tkgeom = iSetup.getHandle(trackerGeomTokenBeginRun_);

// Get the calibration data
//edm::ESHandle<SiPixelGainCalibration> SiPixelGainCalibration_;
//iSetup.get<SiPixelGainCalibrationRcd>().get(SiPixelGainCalibration_);
SiPixelGainCalibrationService_.setESObjects(iSetup);
edm::LogInfo("SiPixelFakeGainReader")
<< "[SiPixelFakeGainReader::beginJob] End Reading FakeGainects" << std::endl;
// Get the list of DetId's
std::vector<uint32_t> vdetId_ = SiPixelGainCalibrationService_.getDetIds();
//SiPixelGainCalibration_->getDetIds(vdetId_);
// Loop over DetId's
for (std::vector<uint32_t>::const_iterator detid_iter = vdetId_.begin(); detid_iter != vdetId_.end();
detid_iter++) {
Expand Down
Loading