Added Run3 electron MVA ID weight files from EGM POG

[Prasant1993]

PR Description:
From EGM POG
Electron MVA IDs have been optimized for Run3.
The .root version of weight files have been added.

[cmsbuild]

A new Pull Request was created by @Prasant1993 (Prasant Kumar Rout) for branch master.

@smuzaffar, @aandvalenzuela, @iarspider, @clacaputo, @cmsbuild, @mandrenguyen can you please review it and eventually sign? Thanks.
@Sam-Harper, @jainshilpi, @valsdav, @lgray, @missirol, @sobhatta, @afiqaize, @wrtabb, @a-kapoor, @sameasy, @varuns23, @ram1123 this is something you requested to watch as well.
@perrotta, @dpiparo, @rappoccio you are the release manager for this.
cms-bot commands are listed here

[rappoccio]

please test

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-b9320c/29484/summary.html
COMMIT: f132a26
CMSSW: CMSSW_13_0_X_2022-12-06-1100/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-data/RecoEgamma-ElectronIdentification/27/29484/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• ROOTFileChecks: Some differences in event products or their sizes found
• Reco comparison results: 11948 differences found in the comparisons
• DQMHistoTests: Total files compared: 48
• DQMHistoTests: Total histograms compared: 3421214
• DQMHistoTests: Total failures: 6992
• DQMHistoTests: Total nulls: 12
• DQMHistoTests: Total successes: 3414188
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: -6.326 KiB( 47 files compared)
• DQMHistoSizes: changed ( 11634.0,... ): -0.774 KiB Physics/NanoAODDQM
• DQMHistoSizes: changed ( 13234.0,... ): -0.454 KiB Physics/NanoAODDQM
• Checked 206 log files, 158 edm output root files, 48 DQM output files
• TriggerResults: found differences in 2 / 46 workflows

[a-kapoor]

@cms-sw/reconstruction-l2 @cms-sw/orp-l2 Hi, can we get a go-ahead for this? This is needed for e/gamma IDs in 12_6.
Thanks.

[mandrenguyen]

+1

[a-kapoor]

@perrotta Just need your approval. :-)

[rappoccio]

+1

[makortel]

Many runTheMatrix workflows are failing in CMSSW_13_0_X_2022-12-08-2300 because #40230 was merged but this one wasn't. I suppose the bot didn't merge the PR yet because of the externals-l2 signature is still missing (@smuzaffar @iarspider @aandvalenzuela).

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will be automatically merged.

[smuzaffar]

I have merged this ( and cmsdist PR to update it in cmssw) now and restated an IB

[rappoccio]

Thanks @smuzaffar ! Sorry, I should have pinged you yesterday to preempt the IB failures.

[rappoccio]

Hi, @smuzaffar the IBs in 12_6 are broken looking for this, I think. Maybe I'm mistaken, but I thought this was decoupled from the CMSSW release? Do we need to make a separate 12_6 branch here? Thanks.

[smuzaffar]

@rappoccio , no we do not need 12.6.X branch here. We just need to backport the cmsdist PR cms-sw/cmsdist#8222 for 12.6.X. I will open it and merge it for next 12.6.X

[smuzaffar]

cms-sw/cmsdist#8225 is merged now for 12.6.X