MVA xml file reading in createGBRForest is longest time in initializing PAT job

[Dr15Jones]

Profiling workflow 11834.21 step 4 showed that createGBRForest was the largest contributor to the time it takes to startup the job.

            2.1  .........       1.14 / 1.19         PhotonMVAEstimator::PhotonMVAEstimator(edm::ParameterSet const&) [464]
            8.3  .........       4.54 / 4.54         ElectronMVAEstimatorRun2::init(std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > const&) [102]
[50]       12.4       6.84       0.00 / 6.84       createGBRForest(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > >&)

In other cases ( #34908 ) this was solved by converting the xml files into ROOT files. It looks like it would be good to do the same for the Electron and Photon ID (?) cases as well.

[Dr15Jones]

assign reconstruction

[cmsbuild]

New categories assigned: reconstruction

@jpata,@slava77,@clacaputo you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cmsbuild]

A new Issue was created by @Dr15Jones Chris Jones.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[makortel]

Does this issue have any overlap with #37867?

[Dr15Jones]

Does this issue have any overlap with #37867?

I think you are right.

[jpata]

-reconstruction

• duplicate of convert MVA models for electronMVAValueMapProducer from XML to ROOT #37867

[makortel]

@cmsbuild, please close