ATLASRECTS-7305: fixes for large cluster building

This MR fixes bugs in the large cluster builder tool in EGamma, used to keep rectangles of 7x11 (S2 granularity) cells in AOD (for central electron and photon studies) (Large clusters in the fwd region are also built for the purpose of keeping cells, but the bugs in that part should be harmless.)

This has no effect on reconstructed quantities, only the size of the AllCalo cell container in AOD should increase a bit, and maybe links from clusters to cells will be reshuffled.

Reconstruction/egamma/egammaTools/src/egammaLargeClusterMaker.cxx

@@ -158,14 +158,17 @@ egammaLargeClusterMaker::execute(const EventContext& ctx,
         ATH_MSG_DEBUG(" CLuster is FWD Cluster ");
       }
 
+      // need some positive energy in EME2 or FCAL0 to be a good candidate
+      if (!(cluster->eSample(CaloSampling::EME2)  > 0 ||
+	    cluster->eSample(CaloSampling::FCAL0) > 0))
+	continue;
+
       // check if cluster is in FCAL or EMEC
+      CaloSampling::CaloSample sam = CaloSampling::FCAL0;
       bool in_EMEC = false;
       if (cluster->eSample(CaloSampling::EME2) >
           cluster->eSample(CaloSampling::FCAL0))
         in_EMEC = true;
-
-      CaloSampling::CaloSample sam = CaloSampling::FCAL0;
-      if (in_EMEC) {
         sam = CaloSampling::EME2;
       }
 
@@ -175,8 +178,10 @@ egammaLargeClusterMaker::execute(const EventContext& ctx,
       auto phi = cluster->phiSample(sam);
 
       if ((eta == 0. && phi == 0.) || fabs(eta) > 100) {
-	ATH_MSG_WARNING("Weird input cluster, eta = "
-			<< eta << " phi = " << phi);
+	ATH_MSG_WARNING("Weird input cluster, maxeta = "
+			<< eta << " phi = " << phi
+			<< " Eeme2 = " << cluster->eSample(CaloSampling::EME2)
+			<< " Efcal = " << cluster->eSample(CaloSampling::FCAL0));
         continue;
       }