Backport to 80X: Flip the soft lepton sIP in case the Negative CTagger is used

[smoortga]

This is a backport to CMSSW_8_0_X of the recent commit #15478 (which was in CMSSW_8_1_X) and which was merged today (18 August 2016).

@mverzett, @kovitang and @monttj are interested in following this PR

[cmsbuild]

A new Pull Request was created by @smoortga for CMSSW_8_0_X.

It involves the following packages:

RecoBTag/CTagging
RecoBTag/SecondaryVertex

@cmsbuild, @cvuosalo, @slava77, @davidlange6 can you please review it and eventually sign? Thanks.
@imarches, @acaudron, @JyothsnaKomaragiri, @mverzett, @ferencek, @pvmulder this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

[cvuosalo]

@cmsbuild please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/14592/console

[cmsbuild]

+1
Tested at: 897d1ea
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-15510/14592/summary.html

[cmsbuild]

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

[cvuosalo]

backport #15478

[cvuosalo]

+1

For #15510 897d1ea

Allow sign flipping of the softElectron and softMuon impact parameters for the negative c tagger. There should be no change in monitored RECO quantities.

#15478 is the 81X version of this PR, and it has already been merged.

The code changes are satisfactory, and Jenkins tests against baseline CMSSW_8_0_X_2016-08-18-1100 show no significant differences, as expected.

[davidlange6]

+1

[davidlange6]

+1

[davidlange6]

@smuzaffar - seems the bot is confused here - it thinks there are signatures pending for some reason?

[smuzaffar]

humm, may be due to backport label. I am looking

[cmsbuild]

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

[smuzaffar]

fixed (new labels urgent, bug-fix, backport and new-features were blocking the removal of pending-signatures)