TrackingCellIDEncodingSvc: add GeoSvcName parameter to allow picking …

…the correct geosvc
key4hep · Sep 13, 2023 · ceb6056 · ceb6056
1 parent 73396e0
commit ceb6056
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/doc/starterkit/k4MarlinWrapperCLIC/Readme.md b/doc/starterkit/k4MarlinWrapperCLIC/Readme.md
@@ -166,6 +166,7 @@ The ``MarlinDD4hep::InitializeDD4hep`` processor can be replaced by the ``k4SimG
 For example:
 
 ```python
+from Configurables import GeoSvc, TrackingCellIDEncodingSvc
 svcList = []
 geoservice = GeoSvc("GeoSvc")
 geoservice.detectors = [os.environ["K4GEO"]+"/CLIC/compact/CLIC_o3_v15/CLIC_o3_v15.xml"]
@@ -175,6 +176,7 @@ svcList.append(geoservice)
 
 cellIDSvc = TrackingCellIDEncodingSvc("CellIDSvc")
 cellIDSvc.EncodingStringParameterName = "GlobalTrackerReadoutID"
+cellIDSvc.GeoSvcName = geoservice.name()
 cellIDSvc.OutputLevel = INFO
 svcList.append(cellIDSvc)
 ```
diff --git a/k4MarlinWrapper/k4MarlinWrapper/TrackingCellIDEncodingSvc.h b/k4MarlinWrapper/k4MarlinWrapper/TrackingCellIDEncodingSvc.h
@@ -32,15 +32,17 @@ class TrackingCellIDEncodingSvc : public extends<Service, IService> {
 public:
   TrackingCellIDEncodingSvc(const std::string& name, ISvcLocator* svc);
 
-  virtual ~TrackingCellIDEncodingSvc();
-  virtual StatusCode initialize() final;
-  virtual StatusCode finalize() final;
+  ~TrackingCellIDEncodingSvc();
+  StatusCode initialize() final;
+  StatusCode finalize() final;
 
   /// Property to configure which DD4hep constant to use for the encodingString
   Gaudi::Property<std::string> m_encodingStringVariable{
       this, "EncodingStringParameterName", "GlobalTrackerReadoutID",
       "The name of the DD4hep constant that contains the Encoding string for tracking detectors"};
 
+  Gaudi::Property<std::string> m_geoSvcName{this, "GeoSvcName", "GeoSvc", "The name of the GeoSvc instance"};
+
 private:
   SmartIF<IGeoSvc> m_geoSvc;
 };

diff --git a/k4MarlinWrapper/src/components/TrackingCellIDEncodingSvc.cpp b/k4MarlinWrapper/src/components/TrackingCellIDEncodingSvc.cpp
@@ -26,12 +26,16 @@
 DECLARE_COMPONENT(TrackingCellIDEncodingSvc);
 
 TrackingCellIDEncodingSvc::TrackingCellIDEncodingSvc(const std::string& name, ISvcLocator* svc)
-    : base_class(name, svc), m_geoSvc(svc->service("GeoSvc")) {}
+    : base_class(name, svc) {}
 
 TrackingCellIDEncodingSvc::~TrackingCellIDEncodingSvc() {}
 
 StatusCode TrackingCellIDEncodingSvc::initialize() {
   try {
+    info() << "Looking for GeoSvc with the name" << m_geoSvcName.value() << endmsg;
+
+    m_geoSvc = serviceLocator()->service(m_geoSvcName);
+
     info() << "Taking the encoding string as specified in dd4hep constant: " << m_encodingStringVariable.value()
            << endmsg;
 

diff --git a/test/gaudi_opts/geoTest_cld.py b/test/gaudi_opts/geoTest_cld.py
@@ -30,6 +30,7 @@
 
 cellIDSvc = TrackingCellIDEncodingSvc("CellIDSvc")
 cellIDSvc.EncodingStringParameterName = "GlobalTrackerReadoutID"
+cellIDSvc.GeoSvcName = geoservice.name()
 cellIDSvc.OutputLevel = DEBUG
 svcList.append(cellIDSvc)