Fix soft drop jet producer

[ahinzmann]

Fix for the softdrop jet producer.
This is not run in 7_3_X RECO, but will be ran in analyses, so needs to be fixed.
The same fix has been requested for 7_4_X in #6654.

[cmsbuild]

A new Pull Request was created by @ahinzmann for CMSSW_7_3_X.

Fix soft drop jet producer

It involves the following packages:

RecoJets/JetProducers

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
@rappoccio, @yslai, @TaiSakuma, @nhanvtran, @schoef, @mariadalfonso 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 you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[StoyanStoynev]

@ahinzmann this is fixing a problem observed where? What is the nature of the problem (I see some parameters are also changed)?

[ahinzmann]

The configuration parameter changes are to have CMS default parameters when softdrop and nsubjettiness are run in analyses. With the introduction of the new fastjet, the defaults were modified, and now back to what is used in analyses.

[cmsbuild]

+1
Tested at: 14e468c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6655/1202/summary.html

[cmsbuild]

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

[StoyanStoynev]

I see no issues here but is this the only part in #6654 (74X) needed in 73X?

[ahinzmann]

All the other changes in the 74 request affect reco event content. Since the original request for the reco changes was closed in 73, I assume only non reco changes can still go in 73. If I understood correctly this request contains everything that is possible.

[StoyanStoynev]

+1
For 14e468c.
Jenkins show no diffs (as expected) and I also checked in QCD and ttbar samples (including miniAOD in wf 25202) that indeed there is nothing from this PR affecting the event content or performance. My tests were on top of CMSSW_7_3_X_2014-11-25-1400.

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

[davidlange6]

+1