New training for anti-e tau-Id (94X)

[mbluj]

This PR adds configuration with payloads of a new training of anti-e tau-Id, backport of #25486 to 94X.

[cmsbuild]

A new Pull Request was created by @mbluj for CMSSW_9_4_X.

It involves the following packages:

RecoTauTag/Configuration
RecoTauTag/RecoTau

@perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

• Backported from New training for anti-e tau-Id #25486

[cmsbuild]

Pull request #25488 was updated. @perrotta, @cmsbuild, @slava77 can you please check and sign again.

[perrotta]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32250/console Started: 2018/12/18 14:43

[cmsbuild]

+1
Tested at: b6c8adb
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32250/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32250/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 27
• DQMHistoTests: Total histograms compared: 2721493
• DQMHistoTests: Total failures: 111
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2721220
• DQMHistoTests: Total skipped: 162
• DQMHistoTests: Total Missing objects: 0

[perrotta]

This is an almost identical backport of #25486 from the master into 94X: it just misses to backport the two files:

• RecoTauTag/RecoTau/python/tools/runTauIdMVA.py
• RecoTauTag/RecoTau/test/runDeepTauIDsOnMiniAOD.py
  This makes sense, because runTauIdMVA.py was not yet updated and moved into the tools folder in 94X.

Even if not including the tool to run and test the new ID in 94X, however this PR sets up for it in that release. Full backport to 94X will be completed when DNN-tauID (and tau tools) will be finally merged to that release.

[perrotta]

@mbluj (maybe you already told us, but I forgot it, sorry for that...): is there any plan to update this PR once #25385 will get merged in 94X?

[perrotta]

backport of #25486

[mbluj]

@mbluj (maybe you already told us, but I forgot it, sorry for that...): is there any plan to update this PR once #25385 will get merged in 94X?

Yes, the plan is either to update this PR or prepare a new one on top of it depending what is more convenient, i.e. in which order this PR and #25385 are merged.

[cmsbuild]

Pull request #25488 was updated. @perrotta, @cmsbuild, @slava77 can you please check and sign again.

[perrotta]

please test
(now #25621 is in the 94X IBs)

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/32548/console Started: 2019/01/12 09:59

[cmsbuild]

+1
Tested at: 5eddab6
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32548/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
10ed8a1
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32548/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32548/git-merge-result

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25488/32548/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 27
• DQMHistoTests: Total histograms compared: 2721493
• DQMHistoTests: Total failures: 118
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2721213
• DQMHistoTests: Total skipped: 162
• DQMHistoTests: Total Missing objects: 0

[perrotta]

+1

• Identical backport of New training for anti-e tau-Id #25486
• A new anti-e tau-ID defined is added to tau tools for user convenience, but disabled by default: non difference expected in regular workflows and reco sequences
• Jenkins tests pass and show no differences

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_9_4_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_10_5_X is complete. This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[fabiocos]

code-checks

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25488/8034

• This PR adds an extra 28KB to repository

[fabiocos]

+1