From f0c27e396187f3bf10c7512119e0ddb74553d6a0 Mon Sep 17 00:00:00 2001
From: Sam Harper <sam.j.harper@gmail.com>
Date: Sun, 20 Oct 2019 15:14:51 +0100
Subject: [PATCH 1/4] zeroing for HGCal clusters

---
 .../plugins/EgammaHLTClusterShapeProducer.cc                 | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
index a10a0fbc7bf2c..f02904408f6f9 100644
--- a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
+++ b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
@@ -80,6 +80,11 @@ void EgammaHLTClusterShapeProducer::produce(edm::StreamID sid,
 
   for (unsigned int iRecoEcalCand = 0; iRecoEcalCand < recoecalcandHandle->size(); iRecoEcalCand++) {
     reco::RecoEcalCandidateRef recoecalcandref(recoecalcandHandle, iRecoEcalCand);
+    if(recoecalcandref->superCluster()->seed()->seed().det()==DetId::HGCalEE ){ //HGCAL, skip for now
+      clshMap.insert(recoecalcandref, 0);
+      clsh5x5Map.insert(recoecalcandref, 0);
+      continue;
+    }
 
     std::vector<float> vCov;
     double sigmaee;

From 885ce1c0475829be90e5aac52e36010c0b7c6d74 Mon Sep 17 00:00:00 2001
From: Sam Harper <sam.j.harper@gmail.com>
Date: Fri, 25 Oct 2019 14:05:24 +0100
Subject: [PATCH 2/4] now explicitly checking for ECAL rather than not HGCAL2

---
 .../plugins/EgammaHLTClusterShapeProducer.cc                   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
index f02904408f6f9..cf98fd124ad04 100644
--- a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
+++ b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
@@ -68,6 +68,7 @@ void EgammaHLTClusterShapeProducer::fillDescriptions(edm::ConfigurationDescripti
 void EgammaHLTClusterShapeProducer::produce(edm::StreamID sid,
                                             edm::Event& iEvent,
                                             const edm::EventSetup& iSetup) const {
+
   // Get the HLT filtered objects
   edm::Handle<reco::RecoEcalCandidateCollection> recoecalcandHandle;
   iEvent.getByToken(recoEcalCandidateProducer_, recoecalcandHandle);
@@ -80,7 +81,7 @@ void EgammaHLTClusterShapeProducer::produce(edm::StreamID sid,
 
   for (unsigned int iRecoEcalCand = 0; iRecoEcalCand < recoecalcandHandle->size(); iRecoEcalCand++) {
     reco::RecoEcalCandidateRef recoecalcandref(recoecalcandHandle, iRecoEcalCand);
-    if(recoecalcandref->superCluster()->seed()->seed().det()==DetId::HGCalEE ){ //HGCAL, skip for now
+    if(recoecalcandref->superCluster()->seed()->seed().det()!=DetId::Ecal ){ //HGCAL, skip for now
       clshMap.insert(recoecalcandref, 0);
       clsh5x5Map.insert(recoecalcandref, 0);
       continue;

From 76fa396c307caf40b4501135011736d398d1bc22 Mon Sep 17 00:00:00 2001
From: Sam Harper <sam.j.harper@gmail.com>
Date: Mon, 14 Dec 2020 14:02:37 +0100
Subject: [PATCH 3/4] code formats

---
 .../plugins/EgammaHLTClusterShapeProducer.cc                   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
index cf98fd124ad04..20fd7810259ae 100644
--- a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
+++ b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
@@ -68,7 +68,6 @@ void EgammaHLTClusterShapeProducer::fillDescriptions(edm::ConfigurationDescripti
 void EgammaHLTClusterShapeProducer::produce(edm::StreamID sid,
                                             edm::Event& iEvent,
                                             const edm::EventSetup& iSetup) const {
-
   // Get the HLT filtered objects
   edm::Handle<reco::RecoEcalCandidateCollection> recoecalcandHandle;
   iEvent.getByToken(recoEcalCandidateProducer_, recoecalcandHandle);
@@ -81,7 +80,7 @@ void EgammaHLTClusterShapeProducer::produce(edm::StreamID sid,
 
   for (unsigned int iRecoEcalCand = 0; iRecoEcalCand < recoecalcandHandle->size(); iRecoEcalCand++) {
     reco::RecoEcalCandidateRef recoecalcandref(recoecalcandHandle, iRecoEcalCand);
-    if(recoecalcandref->superCluster()->seed()->seed().det()!=DetId::Ecal ){ //HGCAL, skip for now
+    if (recoecalcandref->superCluster()->seed()->seed().det() != DetId::Ecal) {  //HGCAL, skip for now
       clshMap.insert(recoecalcandref, 0);
       clsh5x5Map.insert(recoecalcandref, 0);
       continue;

From 58cf00828e36aa3f113e75d405e2849047117264 Mon Sep 17 00:00:00 2001
From: Sam Harper <sam.j.harper@gmail.com>
Date: Tue, 15 Dec 2020 22:51:35 +0100
Subject: [PATCH 4/4] small commit to reset bot

---
 .../EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc  | 1 -
 1 file changed, 1 deletion(-)

diff --git a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
index 20fd7810259ae..1df919b981982 100644
--- a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
+++ b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTClusterShapeProducer.cc
@@ -12,7 +12,6 @@
 //
 //
 
-// user include files
 #include "FWCore/Framework/interface/Frameworkfwd.h"
 #include "FWCore/Framework/interface/global/EDProducer.h"
 #include "FWCore/Framework/interface/Event.h"