JME bugfixes port of #23499

[rappoccio]

Several JME bugs fixed here:

• AK8 jets needed rawFactor to reapply jet corrections
• AK4 jets needed muon fraction, addressing this issue
• Trigger objects needed AK8 jets, addressing this issue

9.4.x version is #23499
10.1.x version is #23501

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-23557/5147

[cmsbuild]

A new Pull Request was created by @rappoccio for master.

It involves the following packages:

PhysicsTools/NanoAOD

@gpetruc, @cmsbuild, @arizzi, @monttj can you please review it and eventually sign? Thanks.
@gpetruc this is something you requested to watch as well.
@davidlange6, @slava77, @fabiocos you are the release manager for this.

cms-bot commands are listed here

[davidlange6]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/28623/console Started: 2018/06/11 16:43

[cmsbuild]

+1
Tested at: ad00ed5
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-23557/28623/summary.html

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

[cmsbuild]

Comparison job queued.

[cmsbuild]

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

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 114 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 2901844
• DQMHistoTests: Total failures: 8
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2901646
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: -1.64 KiB( 30 files compared)
• DQMHistoSizes: changed ( 4.53,... ): -0.328 KiB AlCaReco/SiStripCalMinBias
• Checked 128 log files, 14 edm output root files, 31 DQM output files

[fabiocos]

@arizzi @gpetruc this is affecting NanoAOD, please have a look

[fabiocos]

@arizzi @gpetruc ping

[fabiocos]

@arizzi @gpetruc could you please check also this proposed update and sign it or comment?

[gpetruc]

will check now, but it would have been much easier if the PR had been done the proper way, i.e. first to cms-nanoAOD starting from the cms-nanoAOD master branch so that we can run the integration check

[gpetruc]

hold
#23446 should go in first

[cmsbuild]

Pull request has been put on hold by @gpetruc
They need to issue an unhold command to remove the hold state or L1 can unhold it for all

[davidlange6]

Your PR is unmergeable. Please have a look and possibly rebase it.