From e46916c6de24ea11e783cad254421c7b22dc63f4 Mon Sep 17 00:00:00 2001 From: yeckang Date: Mon, 26 Feb 2024 07:59:40 +0100 Subject: [PATCH] bug fix : avoid the segmentation fault caused by the 2024 GEMGeometry --- L1Trigger/L1TGEM/plugins/GEMPadDigiProducer.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/L1Trigger/L1TGEM/plugins/GEMPadDigiProducer.cc b/L1Trigger/L1TGEM/plugins/GEMPadDigiProducer.cc index a8194c05e4f01..f29a4ddae7e3a 100644 --- a/L1Trigger/L1TGEM/plugins/GEMPadDigiProducer.cc +++ b/L1Trigger/L1TGEM/plugins/GEMPadDigiProducer.cc @@ -81,7 +81,7 @@ void GEMPadDigiProducer::beginRun(const edm::Run& run, const edm::EventSetup& ev geometry_ = &*hGeom; // check the number of parititions if (geometry_->hasGE21()) { - use16GE21_ = (geometry_->chamber(GEMDetId(1, 1, 2, 1, 1, 0))->nEtaPartitions() == + use16GE21_ = (geometry_->chamber(GEMDetId(1, 1, 2, 2, 16, 0))->nEtaPartitions() == GEMPadDigi::NumberPartitions::GE21SplitStrip); } @@ -211,12 +211,12 @@ void GEMPadDigiProducer::checkGeometry() const { if (geometry_->hasGE21()) { if (!use16GE21_) { // check that GE2/1 has 8-eta partitions - if (geometry_->chamber(GEMDetId(1, 1, 2, 1, 1, 0))->nEtaPartitions() != GEMPadDigi::NumberPartitions::GE21) { + if (geometry_->chamber(GEMDetId(1, 1, 2, 2, 16, 0))->nEtaPartitions() != GEMPadDigi::NumberPartitions::GE21) { edm::LogError("GEMPadDigiProducer") << "GE2/1 geometry (8 partition) appears corrupted"; } } else { // check that GE2/1 has 16-eta partitions - if (geometry_->chamber(GEMDetId(1, 1, 2, 1, 1, 0))->nEtaPartitions() != + if (geometry_->chamber(GEMDetId(1, 1, 2, 2, 16, 0))->nEtaPartitions() != GEMPadDigi::NumberPartitions::GE21SplitStrip) { edm::LogError("GEMPadDigiProducer") << "GE2/1 geometry (16 partition) appears corrupted"; }