Merge pull request #31431 from guitargeek/GsfElectronProducer_unused_…

…parameters Remove unused MVA parameters from GsfElectronProducer
cms-sw · Sep 11, 2020 · 49597b4 · 49597b4
2 parents 725d87a + f9ac387
commit 49597b4
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/RecoEgamma/EgammaElectronAlgos/interface/GsfElectronAlgo.h b/RecoEgamma/EgammaElectronAlgos/interface/GsfElectronAlgo.h
@@ -137,10 +137,6 @@ class GsfElectronAlgo {
     bool isEndcaps;
     bool isFiducial;
 
-    // BDT output (if available)
-    double minMVA;
-    double minMvaByPassForIsolated;
-
     // transverse impact parameter wrt beam spot
     double maxTIP;
 

diff --git a/RecoEgamma/EgammaElectronProducers/plugins/GsfElectronProducer.cc b/RecoEgamma/EgammaElectronProducers/plugins/GsfElectronProducer.cc
@@ -205,8 +205,6 @@ void GsfElectronProducer::fillDescriptions(edm::ConfigurationDescriptions& descr
     psd0.add<bool>("isFiducial", false);
     psd0.add<bool>("seedFromTEC", true);
     psd0.add<double>("maxTIP", 999999999.0);
-    psd0.add<double>("minMVA", -0.4);
-    psd0.add<double>("minMvaByPassForIsolated", -0.4);
     // preselection parameters
     desc.add<edm::ParameterSetDescription>("preselection", psd0);
   }
@@ -271,8 +269,6 @@ namespace {
         .isBarrel = pset.getParameter<bool>("isBarrel"),
         .isEndcaps = pset.getParameter<bool>("isEndcaps"),
         .isFiducial = pset.getParameter<bool>("isFiducial"),
-        .minMVA = pset.getParameter<double>("minMVA"),
-        .minMvaByPassForIsolated = pset.getParameter<double>("minMvaByPassForIsolated"),
         .maxTIP = pset.getParameter<double>("maxTIP"),
         .seedFromTEC = pset.getParameter<bool>("seedFromTEC"),
     };