70x tau id trigger default config

[jpavel]

Added default configurations for EDproducers necessary for ConfDB at HLT + added one missing consumes interface

[cmsbuild]

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

70x tau id trigger default config

It involves the following packages:

RecoTauTag/Configuration
RecoTauTag/RecoTau

@nclopezo, @cmsbuild, @anton-a, @thspeer, @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.
@ktf you are the release manager for this.

[ktf]

to pre12 / 71X.

[cmsbuild]

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

[jpavel]

any news on the evaluation of this request? It is already in ConfDB template in CMSSW_7_0_0_pre11_HLT3

[slava77]

looking at it

[slava77]

@nclopezo David, jenkins report here didn't return any comparisons yet.
Probably needs to be restarted.

[slava77]

Isn't the nominal rule for cfi to have the same name as the defined producer?
hpsPFTauProducerSansRefs_cfi.py

https://twiki.cern.ch/twiki/bin/view/CMSPublic/SWGuideAboutPythonConfigFile

[jpavel]

Ah, I did not know about this rule - if such a rule indeed exists, the name in cfi can be changed without trouble.

[slava77]

Hi Pavel

If it's not much trouble to rename, I'd prefer the cfi name matches the defined module instance name.

Thank you.

[jpavel]

Sure, np
On 15/01/14 15:39, slava77 wrote:

Hi Pavel

If it's not much trouble to rename, I'd prefer the cfi name matches
the defined module instance name.

Thank you.

—
Reply to this email directly or view it on GitHub
#1910 (comment).

Dr. Pavel JEZ
postdoctoral researcher
Centre for Cosmology, Particle Physics and Phenomenology-CP3
Université Catholique de Louvain

[anton-a]

@Degano, @nclopezo can you please run jenkins for this PR

[cmsbuild]

Pull request #1910 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-1910/2114/summary.html

[jpavel]

HLT database depends on this PR, so please try to merge it into 70x even if #1817 is not approved in time.

[ktf]

@slava77 @anton-a ?

[anton-a]

looking into it

[anton-a]

+1

[cmsbuild]

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