bug fix to runTauIdMVA.py tool (11_1_X)

[mbluj]

PR description:

This PR fixes bugs introduced to runTauIdMVA.py tool during implementation of the new tauID dataformat #28417.

It is backport of #29976 to 11_1_X

PR validation:

This tool is not used in standard workflows except miniAOD (to include deepTauID) which uses its non-affected part. Therefore, the fix was tested with RecoTauTag/RecoTau/test/runDeepTauIDsOnMiniAOD.py test configuration. This test configuration is added to units tests in this PR.

if this PR is a backport please specify the original PR and why you need to backport that PR:

Backport of #29976 (This PR shares development branch with the PR to master)

[cmsbuild]

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

It involves the following packages:

RecoTauTag/RecoTau

@perrotta, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks.
@riga this is something you requested to watch as well.
@silviodonato, @dpiparo you are the release manager for this.

cms-bot commands are listed here

• Backported from bug fix to runTauIdMVA.py tool #29976

[cmsbuild]

Pull request #29978 was updated. @perrotta, @cmsbuild, @fgolf, @slava77, @santocch, @peruzzim can you please check and sign again.

[slava77]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/6610/console Started: 2020/05/27 02:52

[slava77]

+1

for #29978 7aaf1ab

• this is identical to the 11_2_X version (the same topic branch is used)
• this PR should be safe for a backport: the update affects only a part of the code still used for analysis; the part used in production has only a technical "formatting" change (self. was dropped)

[slava77]

backport of #29976

[cmsbuild]

+1
Tested at: 7aaf1ab
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3c6049/6610/summary.html
CMSSW: CMSSW_11_1_X_2020-05-26-1100
SCRAM_ARCH: slc7_amd64_gcc820

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 36
• DQMHistoTests: Total histograms compared: 2780792
• DQMHistoTests: Total failures: 35
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2780707
• DQMHistoTests: Total skipped: 50
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 35 files compared)
• Checked 152 log files, 16 edm output root files, 36 DQM output files

[santocch]

+1

[silviodonato]

Do you have any comments @peruzzim ?

[silviodonato]

merge
cc: @peruzzim