Add low-pT electrons to MINIAOD, update ID, improve end user experience (back port of 31220)

CommonTools/MVAUtils/BuildFile.xml

@@ -1,10 +1,10 @@
-<use   name="CondFormats/EgammaObjects"/>
-<use   name="FWCore/ParameterSet"/>
-<use   name="FWCore/Utilities"/>
-<use   name="CondFormats/DataRecord"/>
-<use   name="roottmva"/>
-<use   name="tinyxml2"/>
-
+<use name="CommonTools/Utils"/>
+<use name="CondFormats/EgammaObjects"/>
+<use name="FWCore/ParameterSet"/>
+<use name="FWCore/Utilities"/>
+<use name="CondFormats/DataRecord"/>
+<use name="roottmva"/>
+<use name="tinyxml2"/>
 <export>
   <lib   name="1"/>
 </export>

CommonTools/MVAUtils/bin/BuildFile.xml

@@ -0,0 +1,4 @@
+<use name="CommonTools/MVAUtils"/>
+
+<bin name="convertXMLToGBRForestROOT" file="convertXMLToGBRForestROOT.cc">
+</bin>

CommonTools/MVAUtils/bin/convertXMLToGBRForestROOT.cc

@@ -0,0 +1,34 @@
+#include "CommonTools/MVAUtils/interface/GBRForestTools.h"
+
+#include "TFile.h"
+
+#include <boost/filesystem.hpp>
+#include <iostream>
+
+int main(int argc, char **argv) {
+  if (argc != 3) {
+    std::cout << "Please pass a (gzipped) BDT weight file and a name for the output ROOT file." << std::endl;
+    return 1;
+  }
+
+  char *inputFileName = argv[1];
+  char *outputFileName = argv[2];
+
+  if (!boost::filesystem::exists(inputFileName)) {
+    std::cout << "Input file " << inputFileName << " does not exists." << std::endl;
+    return 1;
+  }
+
+  if (boost::filesystem::exists(outputFileName)) {
+    std::cout << "Output file " << outputFileName << " already exists." << std::endl;
+    return 1;
+  }
+
+  auto gbrForest = createGBRForest(inputFileName);
+  std::cout << "Read GBRForest " << inputFileName << " successfully." << std::endl;
+
+  TFile{outputFileName, "RECREATE"}.WriteObject(gbrForest.get(), "gbrForest");
+  std::cout << "GBRForest written to " << outputFileName << " successfully." << std::endl;
+
+  return 0;
+}

CommonTools/MVAUtils/src/GBRForestTools.cc

@@ -3,6 +3,8 @@
 #include "FWCore/ParameterSet/interface/FileInPath.h"
 #include "FWCore/Utilities/interface/Exception.h"
 
+#include "TFile.h"
+
 #include <cstdio>
 #include <cstdlib>
 #include <RVersion.h>
@@ -117,6 +119,16 @@ namespace {
   }
 
   std::unique_ptr<GBRForest> init(const std::string& weightsFileFullPath, std::vector<std::string>& varNames) {
+    //
+    // Load weights file, for ROOT file
+    //
+    if (reco::details::hasEnding(weightsFileFullPath, ".root")) {
+      TFile gbrForestFile(weightsFileFullPath.c_str());
+      std::unique_ptr<GBRForest> up(reinterpret_cast<GBRForest*>(gbrForestFile.Get("gbrForest")));
+      gbrForestFile.Close("nodelete");
+      return up;
+    }
+
     //
     // Load weights file, for gzipped or raw xml file
     //

DataFormats/PatCandidates/src/classes_def_objects.xml

@@ -486,6 +486,8 @@
   <class name="std::vector<edm::Ptr<pat::PackedCandidate> >" />
   <class name="edm::Wrapper<edm::Ptr<pat::PackedCandidate> >" />
   <class name="edm::Wrapper<std::vector<edm::Ptr<pat::PackedCandidate> > >" />
+  <class name="edm::ValueMap<edm::Ptr<pat::PackedCandidate> >" />
+  <class name="edm::Wrapper<edm::ValueMap<edm::Ptr<pat::PackedCandidate> > >" />
 
   <class name="std::vector<edm::Ptr<pat::PackedGenParticle> >" />
   <class name="edm::Wrapper<edm::Ptr<pat::PackedGenParticle> >" />

PhysicsTools/PatAlgos/plugins/BuildFile.xml

@@ -11,6 +11,7 @@
   <use   name="HLTrigger/HLTcore"/>
   <use   name="DataFormats/PatCandidates"/>
   <use   name="DataFormats/BTauReco"/>
+  <use   name="DataFormats/Common"/>
   <use   name="DataFormats/JetReco"/>
   <use   name="DataFormats/MuonReco"/>
   <use   name="DataFormats/TrackReco"/>