Fixed path names. Removed pt2 and pt3 in unnecessary cases. Added debug print (dump # of HLT passed events). Replace cout to DebugLog. Updated paths (PhotonMET, PureMET, DiPhoton, HighPtPhoton).

[ryonamin]

This is a PR replaced instead of #11047 (There was a problem to be merged).
Also this is included a latest request that corresponds to #11453. ==> Decided to remove this change and make this change at #11151 (see below).
==> In summary, Backport from 76X to 75X (#10907,#10974, #11219).

[cmsbuild]

A new Pull Request was created by @ryonamin (Ryo Yonamine) for CMSSW_7_5_X.

Fixed path names. Removed pt2 and pt3 in unnecessary cases. Added debug print (dump # of HLT passed events). Replace cout to DebugLog. Updated paths (PhotonMET, PureMET, DiPhoton, HighPtPhoton).

It involves the following packages:

HLTriggerOffline/Exotica

@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.

[deguio]

ciao @ryonamin
sorry but I am confused now.
the rebase is fine, but the equivalent of #11453 should go on top of #11151 and not here.
otherwise I fear that as soon as the first one is merged, the second won't merge anymore.
F.

[ryonamin]

Oh, sorry! You're completely correct. I made a mistake. Do you think I can get the change (#11453) back, and add it to #11151?

[deguio]

yep.
go ahead.
thanks,
F.

[cmsbuild]

Pull request #11456 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

[cmsbuild]

Pull request #11456 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

[deguio]

please test

[deguio]

let's see how it goes...
in principle you could have just removed commit 9a5b543 and following.

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

-1
Tested at: c7f8f2f
I found an error when building:

gmake[1]: **\* [CompilePython] Error 1
gmake[1]: Target `PostBuild' not remade because of errors.
gmake[1]: Leaving directory`/tmp/cmsbuild/workspace/ib-any-integration/CMSSW_7_5_X_2015-09-23-2300'
gmake: **\* [src] Error 2
gmake: Target `all' not remade because of errors.
gmake: **\* [There are compilation/build errors. Please see the detail log above.] Error 2


you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11456/8355/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
4de1b8f
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11456/8355/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11456/8355/git-merge-result

[deguio]

typo to be fixed. see:
https://github.com/cms-sw/cmssw/pull/11456/files#diff-e39cf2b11b6cc26b6dc8db0f427a819dR15

could you just remove those two new files? they will be added anyway with PR #11151
thanks,
F.

[cmsbuild]

Pull request #11456 was updated. @cmsbuild, @danduggan, @vanbesien, @deguio can you please check and sign again.

[ryonamin]

I have learned again thanks to you. Sorry for my mess.

[deguio]

please test

[cmsbuild]

The tests are being triggered in jenkins.

[cmsbuild]

+1
Tested at: 375c000
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-11456/8374/summary.html

[cmsbuild]

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

[deguio]

+1

[cmsbuild]

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