Cut based discriminator against electron (AgainstElectron2)

[mbluj]

This request is to add a new cut-based tau discriminator against electron (PFRecoTauDiscriminationAgainstElectron2) which is expected to be used at HLT. The discriminator will be also a simple (and less performant) alternative for a MVA-based discriminant at offline.

[cmsbuild]

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

Cut based discriminator against electron (AgainstElectron2)

It involves the following packages:

RecoTauTag/RecoTau

@nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please review it and eventually sign? Thanks.
@jpavel this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
@nclopezo, @ktf you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[mbluj]

Hello,

for reference I am linking presentations introducing the discriminant:
https://indico.cern.ch/event/313614/contribution/5/material/slides/1.pdf
https://indico.cern.ch/event/315877/contribution/1/material/slides/0.pdf

Michal

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3671/1273/summary.html

[cmsbuild]

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

[anton-a]

Remove hardcoded numbers from the code.

[anton-a]

Remove root type floats

[anton-a]

My understanding is that this discriminator is targeting use in HLT. Please make timing tests and compare the performance with the current methods.

[mbluj]

Hi Anton,

the correction will be introduced to the code, thank you for spotting this.

What concerns comparison with existing methods used at offline you can find it in this plot: http://anayak.web.cern.ch/anayak/TauTrigger/AntiEleStudies/EffVsFakeRate_Comparison.pdf A green point corresponds with working point of the new discriminator which should be compared with red curve for current offline cut-based discriminant. Two things should be noted:

• new discriminator has only one WP defined - other WPs can be defined too.
• there is not another against-electron discriminant which can work at HLT, even current offline cut-based one uses mva built in PFCandidate (giving electron-likeness) which is not available at HLT.
  Finally, we will certainty preform timing tests, but it should be noted that tau discriminators are run always at the end of the trigger chain after iterative tracking is run. Therefore, even given tau discriminator is not very fast it does not contribute significantly to total execution time of path in each it is executed as the time is dominated by tracking.

Michal

[anton-a]

Thanks, Michal.
Please make changes to the code as soon as you can so that it can be included in the next pre-release.

[mbluj]

Hi Anton,

code with requested changes has just pushed to the cut-based-againstElectron2 branch on mbluj/cmssw.

Michal

[cmsbuild]

Pull request #3671 was updated. @nclopezo, @cmsbuild, @anton-a, @thspeer, @slava77, @Degano can you please check and sign again.

[cmsbuild]

+1
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3671/1357/summary.html

[cmsbuild]

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

[anton-a]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_1_X IBs unless changes (tests are also fine). @nclopezo, @ktf can you please take care of it?