Skip to content

Add missing info to aux tables for GeoModelXml ReadoutGeometry

Nicholas Styles requested to merge nstyles/GeoModel:addSensorInfoToString into master

Add two missing pieces from previous implementation:

  • Common parameters used by all sensor classes/types were missed previously. They are now added to the information for each sensor. This is a bit of a duplication of information since they are the same for all sensors (or at least all sensors of a given class) but it seems better than having to add several additional tables with only one entry which need to be accessed.
  • Adding the sensor type name and level of splitting to the table of Full Phys Vol info. This will allow the correct design to be looked up from a map and associated when creating the DetectorElemenents.

There are still some small differences observed when testing this in Athena, but I suspect that the fixes will be on the Athena side and not the GeoModel side, and will be easier to pin down once this is merged.

Edited by Nicholas Styles

Merge request reports