Default python values for new parameters introduced with PR 5255

[jpavel]

A bugfix: #5255 introduced new parameters for the discriminationByIsolation EDProducer, but did not define their default value in python. This fix is important for HLT.

@mbluj

[jpavel]

@slava77 @StoyanStoynev Is there automatic forward port to 73x, or should I make new PR ?

[cmsbuild]

A new Pull Request was created by @jpavel (Pavel Jez) for CMSSW_7_2_X.

Default python values for new parameters introduced with PR 5255

It involves the following packages:

RecoTauTag/RecoTau

@cmsbuild, @nclopezo, @StoyanStoynev, @slava77 can you please review it and eventually sign? Thanks.
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]

No auto forward ports, Giulio made few "by hand" and I understood these were exceptions not to happen again (though it did happen at least once again). Thus you should make a new one for 73X.

[davidlange6]

auto-forward port means the porting happens without asking.. There is still the forward port that creates a pull request… but indeed, make a PR for 73x as a general rule

On Sep 25, 2014, at 8:36 AM, StoyanStoynev notifications@github.com
wrote:

No auto forward ports, Giulio made few "by hand" and I understood these were exceptions not to happen again (though it did happen at least once again). Thus you should make a new one for 73X.

—
Reply to this email directly or view it on GitHub.

[StoyanStoynev]

+1
Tested e426c96 on top of CMSSW_7_2_X_2014-09-22-0200. No effect on standard RECO (short matrix and ttbar samples with PU - upto 200 events) and none expected (affecting only a customization running).

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests. @nclopezo can you please take care of it?