Skip to content

Commit 5ad6287

Browse files
committed
fix cout of current areatag
1 parent 7efe1eb commit 5ad6287

4 files changed

Lines changed: 8 additions & 6 deletions

File tree

icaruscode/PMT/Calibration/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ cet_build_plugin( OpHitRecalibrator art::module
7979
lardataobj::RecoBase
8080
lardataobj::RawData
8181
larcore::Geometry_Geometry_service
82+
larreco::PhotonCalibrator # IPhotonCalibrator interface
8283
icaruscode_Timing
8384
)
8485

icaruscode/PMT/Calibration/ICARUSPhotonCalibratorServiceFromDB.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ namespace calib {
5656

5757
ICARUSPhotonCalibratorServiceFromDB(Parameters const& params, art::ActivityRegistry& reg);
5858

59-
virtual std::string getAreaDatabaseTag() const = 0;
59+
std::string getAreaDatabaseTag() const { return fProvider.getAreaDatabaseTag(); }
6060

6161
private:
6262
provider_type const* provider() const override { return &fProvider; }

icaruscode/PMT/Calibration/OpHitRecalibrator_module.cc

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ class icarus::OpHitRecalibrator : public art::SharedProducer
113113

114114
/// Pointers to the online corrections services
115115
icarusDB::PMTTimingCorrections const &fPMTTimingCorrectionsService;
116-
icarusDB::PhotonCalibratorFromDB const &fPhotonCalibratorService;
116+
calib::IPhotonCalibrator const &fPhotonCalibratorService;
117117

118118
/// Pointer to the provider for the old pmt corrections
119119
std::unique_ptr<icarusDB::PMTTimingCorrectionsProvider> fOldTimingProvider;
@@ -129,7 +129,7 @@ icarus::OpHitRecalibrator::OpHitRecalibrator(fhicl::ParameterSet const &config,
129129
fSPEArea{config.get<double>("SPEArea", -1.)},
130130
fVerbose{config.get<bool>("Verbose", false)},
131131
fPMTTimingCorrectionsService{*(lar::providerFrom<icarusDB::IPMTTimingCorrectionService const>())},
132-
fPhotonCalibratorService{*(lar::providerFrom<icarusDB::ICARUSPhotonCalibratorServiceFromDB const>())},
132+
fPhotonCalibratorService{*(lar::providerFrom<calib::IPhotonCalibratorService const>())},
133133
fOldTimingProvider{std::make_unique<icarusDB::PMTTimingCorrectionsProvider>(config.get<fhicl::ParameterSet>("OldTimingDBTags"))}
134134
{
135135
async<art::InEvent>();
@@ -166,8 +166,9 @@ icarus::OpHitRecalibrator::OpHitRecalibrator(fhicl::ParameterSet const &config,
166166

167167
if (fRecalibratePE && fUseGainDatabase)
168168
{
169-
mf::LogInfo("OpHitRecalibrator") << "Re-calibration of PE (gain) enabled:\n"
170-
<< "AreaTag: " << fPhotonCalibratorService.getAreaDatabaseTag();
169+
art::ServiceHandle<calib::ICARUSPhotonCalibratorServiceFromDB const> serv; //really sorry about this
170+
mf::LogInfo("OpHitRecalibrator") << "Re-calibration of PE (gain) from database enabled:\n"
171+
<< "AreaTag: " << serv->getAreaDatabaseTag();
171172
}
172173

173174
// Consumes

icaruscode/PMT/Calibration/PhotonCalibratorFromDB.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ class icarusDB::PhotonCalibratorFromDB: public calib::IPhotonCalibrator {
149149
std::uint64_t RunToDatabaseTimestamp( unsigned int run ) const;
150150

151151
/// Get current area database tag
152-
std::string getAreaDatabaseTag() const override { return fAreaTag; }
152+
std::string getAreaDatabaseTag() const { return fAreaTag; }
153153

154154
private:
155155

0 commit comments

Comments
 (0)