change the naming convention of CaloMETs

[TaiSakuma]

This PR changes the naming convention of CaloMETs. In particular, this PR

• renames met caloMet
• renames metNoHF caloMetBE
• renames metHO caloMetBEFO
• renames metNoHFHO caloMetBEO

caloMet, caloMetBE, caloMetBEFO are event contents of RECO and AOD.

We announced this change at the JMAR meeting on 24-Apr-2014:
https://indico.cern.ch/event/304145/contribution/0

We discussed this change at the RECO meeting on 17-Apr-2014: https://indico.cern.ch/event/313948/contribution/18

[cmsbuild]

A new Pull Request was created by @TaiSakuma (Tai Sakuma) for CMSSW_7_1_X.

change the naming convention of CaloMETs

It involves the following packages:

HLTriggerOffline/Higgs
HLTriggerOffline/SUSYBSM
RecoMET/Configuration
RecoMET/METProducers

@thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please review it and eventually sign? Thanks.
@nhanvtran 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.
@nclopezo, @ktf 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
I ran the usual tests and I found errors in the following unit tests:

---> test testRecoMETMETProducers had ERRORS

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

[cmsbuild]

Pull request #3801 was updated. @thspeer, @danduggan, @rovere, @cmsbuild, @anton-a, @nclopezo, @deguio, @slava77, @Degano, @ojeda can you please check and sign again.

[cmsbuild]

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

[cmsbuild]

-1
I ran the usual tests and I found errors in the following unit tests:

---> test testRecoMETMETProducers had ERRORS

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

[TaiSakuma]

I updated this PR as we disscussed at #3938.
I also rebased onto 7_1_X.

[cmsbuild]

Pull request #3801 was updated. @ojeda, @danduggan, @rovere, @monttj, @cmsbuild, @anton-a, @thspeer, @deguio, @slava77, @vadler, @Degano, @nclopezo can you please check and sign again.

[slava77]

71X is now closed for new features.
Do we really need this change in 71X?
Tai, can we just proceed to having it in 72X?

[vadler]

@TaiSakuma : Ah, I'm very, very sorry, but I think I made a mistake here :-( I got confused by other cases, where changes in the event content hit future intagration of PAT work flows into RelVal production with cmsDriver.py (s. #3936 ).
The proper solution would be to comment that line in PhysicsTools/PatAlgos/test/patTuple_addJets_cfg.py rather than changing it until we base the PAT tests on the new event content.
Again, I'm very sorry for the additional work.

[TaiSakuma]

@slava77, it doesn't have to be 71X. I will make a PR for 72X.

[TaiSakuma]

@vadler, that is fine. I will comment out the line in new PR for 72X.