JEC consumes for btag DQM

[acaudron]

Addapt btag DQM packages to JEC consumes.

[cmsbuild]

A new Pull Request was created by @acaudron for CMSSW_7_5_X.

JEC consumes for btag DQM

It involves the following packages:

DQMOffline/RecoB
Validation/RecoB

@cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks.
@ferencek, @acaudron, @rociovilar, @pvmulder, @imarches 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.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[cmsbuild]

+1
Tested at: b55c19a
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8952/4522/summary.html

[cmsbuild]

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

The workflows 140.53 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons

[acaudron]

Is there an issue or are we waiting for something ? All the tests are fine so I'm not sure why we need to wait for going ahead with this PR.

[wmtan]

@davidlange6 @nclopezo @franzoni @deguio
Please expedite this PR. It is critical for the consumes interface for multithreading.
It's been sitting there for 6 days with no progress.

[wmtan]

@davidlange6 @nclopezo @franzoni @deguio
Please expedite this PR. It is critical for the consumes interface for multithreading.
It's now been sitting there for 10 days unsigned with no progress.

[deguio]

+1
looks ok to me

[davidlange6]

@acaudron - what is the need for these lines?

@slava77 , @cvuosalo

[acaudron]

@davidlange6 these lines complete what have been done in the PR#8817 by @wddgit
Basically I need to have JEC for CHS jets for data (so including Residual correction) so I replace for data the Corrector Chain sequence to include this correction based on a suggestion from @slava77

[slava77]

yes, residual corrections are available only in data GTs.
So, this is the only place where we can turn them on.

From RECO configuration perspective, I wish that instead dummy pass-through corrections were available in MC GT as well and we didn't have to modify configurations.

[davidlange6]

@acaudron, @slava77 - Thanks. @slava77, could you request that we start getting GTs like that, at the next ORP if not next AlCa meeting?

[davidlange6]

+1