[HGCal/FW] Add support for HGCal Geom in FW

[Mourtz]

No description provided.

[cmsbuild]

The code-checks are being triggered in jenkins.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-25135/7131

[cmsbuild]

A new Pull Request was created by @Mourtz (Alex Mourtziapis) for master.

It involves the following packages:

.gitignore
Fireworks/Core
Fireworks/Geometry
Fireworks/SimData
Geometry/HGCalGeometry

@smuzaffar, @civanch, @Dr15Jones, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @alja, @kpedro88 can you please review it and eventually sign? Thanks.
@alja 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

[Dr15Jones]

Please remove the .gitignore file from this pull request.

[cmsbuild]

+1
Tested at: 55855fe
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25135/31537/summary.html

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
5fe97dc
291f20c
fe5711c
c395e0c
You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25135/31537/git-log-recent-commits
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-25135/31537/git-merge-result

[cmsbuild]

Comparison job queued.

[kpedro88]

+upgrade
The image in #25135 (comment) looks very nice

[cmsbuild]

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

@slava77 comparisons for the following workflows were not done due to missing matrix map:

• /build/cmsbld/jenkins/workspace/compare-root-files-short-matrix/results/JR-comparison/PR-25135/7.3_CosmicsSPLoose_UP18+CosmicsSPLoose_UP18+DIGICOS_UP18+RECOCOS_UP18+ALCACOS_UP18+HARVESTCOS_UP18

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 32
• DQMHistoTests: Total histograms compared: 3013827
• DQMHistoTests: Total failures: 1
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3013629
• DQMHistoTests: Total skipped: 197
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 31 files compared)
• Checked 134 log files, 14 edm output root files, 32 DQM output files

[Mourtz]

@ianna
The files that you are suggesting me to modify were merged from #25027.

[Mourtz]

@ianna
It shouldn't show any changes in those files....
Do you want me to get them out of the PR?

[Mourtz]

is there any problem with merging the PR?

[ianna]

+1

[fabiocos]

@alja could you please check and sign it in case?

[alja]

+1

[alja]

+1

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @davidlange6, @slava77, @smuzaffar, @fabiocos (and backports should be raised in the release meeting by the corresponding L2)

[Mourtz]

changes of this PR are included in #25284. Keeping only the last PR.