Remove redundant dictionary definition in DataFormats/HGCRecHit

[makortel]

PR description:

The equivalent <class name="edm::SortedCollection<HGCRecHit,edm::StrictWeakOrdering<HGCRecHit> >"/> is already defined above. This duplication was (I'd say accidentally) found in #45423 (comment) .

Resolves cms-sw/framework-team#962

PR validation:

Code compiles, and the to-be-added edmDumpClassVersion succeeds to process the classes_def.xml files.

[cmsbuild]

cms-bot internal usage

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-45551/41033

[cmsbuild]

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

It involves the following packages:

• DataFormats/HGCRecHit (reconstruction, upgrade)

@cmsbuild, @jfernan2, @mandrenguyen, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks.
@apsallid, @bsunanda, @cseez, @felicepantaleo, @hatakeyamak, @lgray, @missirol, @mmusich, @pfs, @rovere, @sethzenz, @vandreev11, @youyingli this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[makortel]

@cmsbuild, please test

[makortel]

type -changes-dataformats

[cmsbuild]

-1

Failed Tests: RelVals-INPUT
Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-62929d/40582/summary.html
COMMIT: de8bdea
CMSSW: CMSSW_14_1_X_2024-07-24-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/45551/40582/install.sh to create a dev area with all the needed externals and cmssw changes.

• DAS Queries: The DAS query tests failed, see the summary page for details.

RelVals-INPUT

• 2500.2452500.245_hcalDPGCalibNANO140Xrun3/step2_hcalDPGCalibNANO140Xrun3.log

Comparison Summary

Summary:

• No significant changes to the logs found
• Reco comparison results: 251 differences found in the comparisons
• DQMHistoTests: Total files compared: 44
• DQMHistoTests: Total histograms compared: 3325254
• DQMHistoTests: Total failures: 2766
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3322468
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 43 files compared)
• Checked 191 log files, 161 edm output root files, 44 DQM output files
• TriggerResults: no differences found

[makortel]

Comparison differences are related to #39803 and #45505. The file open failure in 2500.245 occurs in IBs.

[makortel]

ignore tests-rejected with ib-failure

[srimanob]

+Upgrade

[mandrenguyen]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (test failures were overridden). This pull request will be automatically merged.