diff --git a/RecoEgamma/EgammaElectronAlgos/src/ElectronSeedGenerator.cc b/RecoEgamma/EgammaElectronAlgos/src/ElectronSeedGenerator.cc index 8ddf20399943c..19e8d37017b6a 100644 --- a/RecoEgamma/EgammaElectronAlgos/src/ElectronSeedGenerator.cc +++ b/RecoEgamma/EgammaElectronAlgos/src/ElectronSeedGenerator.cc @@ -47,6 +47,8 @@ #include #include + +#ifdef COUNT_ElectronSeeds namespace { struct Count { long long s=0; @@ -56,7 +58,7 @@ namespace { Count stcount; } - +#endif @@ -276,16 +278,18 @@ void ElectronSeedGenerator::run // Find the seeds recHits_.clear(); - LogDebug ("run") << "new cluster, calling seedsFromThisCluster"; + LogDebug ("ElectronSeedGenerator") << "new cluster, calling seedsFromThisCluster"; seedsFromThisCluster(sclRefs[i],hoe1s[i],hoe2s[i],out,tTopo); } - LogDebug ("run") << ": For event "<dets().size()]; auto ndets = theTrackerGeometry->dets().size(); - // std::vector away(ndets,false); int iTsos[ndets]; for ( auto & i : iTsos) i=-1; @@ -164,11 +153,10 @@ PixelHitMatcher::compatibleSeeds const GeomDet *geomdet1 = it1->det(); auto ix1 = geomdet1->gdetIndex(); - assert(ix1position().basicVector().dot(xmeas.basicVector()) <0; - if (away[ix1]) continue; + + /* VI: this generates regression (other cut is just in phi). in my opinion it is safe and makes sense + auto away = geomdet1->position().basicVector().dot(xmeas.basicVector()) <0; + if (away) continue; */ const GlobalPoint& hit1Pos = hit_gp_map_[idx1]; @@ -246,7 +234,6 @@ PixelHitMatcher::compatibleSeeds mapTsos2_fast_.clear() ; - stcount.n += result.size(); return result ; }