Run 2 format of PCC ntupler - version 1

[capalmer85]

No description provided.

[cmsbuild]

A new Pull Request was created by @capalmer85 for CMSSW_5_3_X.

Run 2 format of PCC ntupler - version 1

It involves the following packages:

RecoLuminosity/LumiProducer

@cmsbuild, @cvuosalo, @nclopezo, @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.
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.
@smuzaffar you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

[slava77]

Hi Chris,

Why is the run2 development going to 53X release?

[capalmer85]

Hi Slava,

It was validated on 8 TeV data to give the same results at the previous
version. I wanted them both to be in some official place, then port to
74X. This is what I said I was going to do in 6962.

Is this alright?

Cheers,
Chris

On Sat, Mar 21, 2015 at 3:58 AM, Slava Krutelyov notifications@github.com
wrote:

Hi Chris,

Why is the run2 development going to 53X release?

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

[cmsbuild]

+1
Tested at: f366ce7
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8446/3682/summary.html

[slava77]

The 53X is on a monthly (or less frequent) schedule, and this kind of developments will not trigger a new release by themselves.
This needs to be kept in mind for additions to 53X.

[capalmer85]

That's ok. As long as it is in the queue, I won't worry about it and move
forward.

Thanks,
Chris

On Mon, Mar 23, 2015 at 4:36 AM, Slava Krutelyov notifications@github.com
wrote:

The 53X is on a monthly (or less frequent) schedule, and this kind of
developments will not trigger a new release by themselves.
This needs to be kept in mind for additions to 53X.

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

[slava77]

+1

for #8446 f366ce7

changes are in the analysis-level ntuple code in RecoLuminosity/LumiProducer/test/analysis
the code compiles and runs with the provided test config.
Standard workflows are not affected.

[cmsbuild]

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

[davidlange6]

+1