Project

General

Profile

Support #24611

simultaneous use of auxdetgeometry service with geometry service

Added by Christopher Hilgenberg 9 months ago. Updated 25 days ago.

Status:
Feedback
Priority:
High
Assignee:
-
Category:
Geometry
Target version:
develop/HEAD
Start date:
07/08/2020
Due date:
% Done:

0%

Estimated time:
Experiment:
ICARUS
Co-Assignees:
Duration:

Description

We need to access AuxDetGeometryCore via the auxdetgeometry service. At the same time, we need to access GeometryCore via geometry service. My understanding is that the auxdetgeometry service has problems in this use case. What course of action is recommended?

History

#1 Updated by Gianluca Petrillo 9 months ago

  • Co-Assignees deleted (Adi Ashkenazi)

#2 Updated by Kyle Knoepfel 9 months ago

  • Status changed from New to Feedback

What problems did you encounter while trying to use both in the same job? We are not aware of any issues.

#3 Updated by Gianluca Petrillo 9 months ago

That is because there is a poor-man synchronisation attempt between the two services.
One is a fork of the other, and both attempt to initialise a geometry representation from a GDML file.
Since this happens via ROOT... ça va sans dire. ROOT TGeoManager hooks gGeoManager (if I remember the name right), and g stands for "ROOT loves g lobal state".
There was also the request at the time (LArIAT, I think) that AuxDetGeometry service could be loaded without Geometry service.
My dream at the time was to have a GeometrySource which managed the shared resource, and the other two services would interface with it as needed... never happened.

#4 Updated by Kyle Knoepfel 9 months ago

Thanks, Gianluca, for the info. Chris, can you send us a FHiCL file, which would load both services according to your desired use case? That will help us explore the entanglements and pursue a solution to any problems that arise.

#5 Updated by Kyle Knoepfel 8 months ago

  • Priority changed from High to Normal
  • Assignee set to Kyle Knoepfel

#6 Updated by Biswaranjan Behera 3 months ago

  • Priority changed from Normal to High

Kyle Knoepfel wrote:

Thanks, Gianluca, for the info. Chris, can you send us a FHiCL file, which would load both services according to your desired use case? That will help us explore the entanglements and pursue a solution to any problems that arise.

Dear Kyle,

Sorry for the delay on replying back.

In /icarus/app/users/bbehera/devs/srcs/icarusalg/icarusalg/Geometry/geometry_icarus.fcl,
I have added these line in order to use AuxdetGeometry.

  1. CRT Auxiliary detector service configuration

icarus_auxdetgeo: {
Name: "icarus_splitwires"
GDML: "icarus_complete_splitwire_newcrtgeo_channelmaptest_15Jan2021.gdml"
ROOT: "icarus_complete_splitwire_newcrtgeo_channelmaptest_15Jan2021.gdml"
}
icarus_auxdetgeometry_helper: {
service_provider: "CRTGeometryHelper"
}

  1. geometry configuration bundle

icarus_crt_channelmapping_services: {
AuxDetGeometry: @local::icarus_auxdetgeo
AuxDetExptGeoHelperInterface: @local::icarus_auxdetgeometry_helper
} # icarus_crt_channelmapping_services

and my g4.fcl file is located here.
/icarus/app/users/bbehera/work/crt/chmap/g4.fcl

input to this fhicl file is located here: /icarus/app/users/bbehera/work/crt/chmap/gen.root

gdml file can be found in this location:
/icarus/app/users/bbehera/devs/srcs/icarusalg/icarusalg/Geometry/gdml/icarus_complete_splitwire_newcrtgeo_channelmaptest_15Jan2021.gdml
/icarus/app/users/bbehera/devs/srcs/icarusalg/icarusalg/Geometry/gdml/icarus_complete_splitwire_newcrtgeo_channelmaptest_15Jan2021_nowires.gdml

Please let me know if I could provide more information.

Thank you.

With regards,
Bishu

#7 Updated by Gianluca Petrillo 2 months ago

I have taken a quick look at AuxDetGeometry. It seems that the functionality carried by it is not overwhelming...
I can imagine the possibility of transferring it into Geometry.
In fact, only two channel mapping methods (PositionToAuxDetSensitive() and PositionToAuxDetChannel()) are unique to AuxDetGeometry.

#8 Updated by Kyle Knoepfel 2 months ago

  • Assignee deleted (Kyle Knoepfel)

Gianluca, your proposal sounds sensible. Are you willing to make a PR?

#9 Updated by Kyle Knoepfel about 2 months ago

Gianluca, any progress or thoughts on this?

#10 Updated by Erica Snider about 1 month ago

Gianluca??

#11 Updated by Kyle Knoepfel 25 days ago

We will schedule a meeting to discuss this issue (in particular) but also in the context of experiments' broader geometry needs.

Also available in: Atom PDF