[Reco] Removed unnecessary include statements

[smuzaffar]

This should fix the following private header usage issue #34718 for

   1 src/RecoLocalCalo/HGCalRecProducers/plugins/KernelManagerHGCalCellPositions.h

[smuzaffar]

please test

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34866/24668

• This PR adds an extra 16KB to repository

[cmsbuild]

A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master.

It involves the following packages:

• CondFormats/HGCalObjects (upgrade, db, alca)
• RecoLocalCalo/HGCalRecProducers (upgrade, reconstruction)

@perrotta, @malbouis, @yuanchao, @jpata, @tlampen, @kpedro88, @AdrianoDee, @srimanob, @slava77, @ggovi, @pohsun, @francescobrivio, @tvami can you please review it and eventually sign? Thanks.
@edjtscott, @vandreev11, @sethzenz, @apsallid, @clelange, @felicepantaleo, @rovere, @lgray, @cseez, @bsunanda, @pfs, @lecriste, @hatakeyamak, @tocheng, @mmusich, @ebrondol, @seemasharmafnal this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5eb0eb/17760/summary.html
COMMIT: 14d66e9
CMSSW: CMSSW_12_1_X_2021-08-12-2300/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/34866/17760/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 39
• DQMHistoTests: Total histograms compared: 2999420
• DQMHistoTests: Total failures: 6
• DQMHistoTests: Total nulls: 1
• DQMHistoTests: Total successes: 2999391
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: -0.004 KiB( 38 files compared)
• DQMHistoSizes: changed ( 312.0 ): -0.004 KiB MessageLogger/Warnings
• Checked 165 log files, 37 edm output root files, 39 DQM output files
• TriggerResults: no differences found

[smuzaffar]

please test

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34866/24689

• This PR adds an extra 20KB to repository

[cmsbuild]

Pull request #34866 was updated. @perrotta, @malbouis, @yuanchao, @jpata, @tlampen, @kpedro88, @AdrianoDee, @srimanob, @slava77, @ggovi, @pohsun, @francescobrivio, @tvami can you please check and sign again.

[ggovi]

+1

[tvami]

+alca

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-5eb0eb/17787/summary.html
COMMIT: f87a446
CMSSW: CMSSW_12_1_X_2021-08-15-2300/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/34866/17787/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 6 differences found in the comparisons
• DQMHistoTests: Total files compared: 39
• DQMHistoTests: Total histograms compared: 3000352
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3000329
• DQMHistoTests: Total skipped: 22
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 38 files compared)
• Checked 165 log files, 37 edm output root files, 39 DQM output files
• TriggerResults: no differences found

[slava77]

+reconstruction

for #34866 f87a446

• code changes are technical, in line with the PR description and the follow up review
• jenkins tests pass and comparisons with the baseline show no relevant differences

[slava77]

+1

my previous was ignored

[perrotta]

+1

• @cms-sw/upgrade-l2 this is technical, and quite simple: let try to merge it

[perrotta]

merge