bsunanda: Run2-hcx40 Update the hit validation package to match Phase2 simulation

[bsunanda]

This request replaces PR 11634 which could not be merged with the last updates to this.

[cmsbuild]

A new Pull Request was created by @bsunanda for CMSSW_7_6_X.

bsunanda: Run2-hcx40 Update the hit validation package to match Phase2 simulation

It involves the following packages:

Validation/HcalHits

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

[bsunanda]

I could not push PR 11634 after removing merge conflict. I tried several times and asked for help. So I am making a fresh PR on the top of the latest IB.
@cmsbuild Could you initiate the tests at the earliest

---

From: cmsbuild [notifications@github.com]
Sent: 09 October 2015 09:11
To: cms-sw/cmssw
Cc: Sunanda Banerjee
Subject: Re: [cmssw] bsunanda: Run2-hcx40 Update the hit validation package to match Phase2 simulation (#11711)

A new Pull Request was created by @bsunandahttps://github.com/bsunanda for CMSSW_7_6_X.

bsunanda: Run2-hcx40 Update the hit validation package to match Phase2 simulation

It involves the following packages:

Validation/HcalHits

@cmsbuildhttps://github.com/cmsbuild, @dandugganhttps://github.com/danduggan, @vanbesienhttps://github.com/vanbesien, @deguiohttps://github.com/deguio 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.
@deganohttps://github.com/degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/11711#issuecomment-146774717.

[bsunanda]

@cmsbuild Please initiate the tests

[smuzaffar]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/8722/console

[cmsbuild]

+1
Tested at: 4045e7f
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11711/8722/summary.html

[cmsbuild]

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

[deguio]

+1

[cmsbuild]

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

[davidlange6]

+1