Migrated two clients of the CaloDetDescrManager

1. GeometryWriter switched to reading CaloDetDescrManager from CondStore
2. VP1CaloCellSystem switched to retrieving CaloCel ID helper from DetStore (plus minor code cleanup)

This merge request affects 2 packages:

• graphics/GeometryJiveXML
• graphics/VP1/VP1Systems/VP1CaloSystems

Affected files list will not be printed in this case

Adding @rbianchi as watcher

added EventDisplay master review-pending-level-1 labels

CI Result SUCCESS (hash 9821f9d9)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 41008]

Looks good. Approved

Sergei L1

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 995a8cd8

added sweep:ignore label