XENONnT / fuse

XENON Framework for Unified Simulations of Events
BSD 3-Clause "New" or "Revised" License
6 stars 1 forks source link

Wrong dtype for Geant4 time field. #208

Closed HenningSE closed 6 months ago

HenningSE commented 6 months ago

With https://github.com/XENONnT/fuse/pull/195 we changed the dtype for the Geant4 input "t" from float64 to int64. This was discussed here: https://github.com/XENONnT/fuse/pull/195#issuecomment-2063219919. In fact I now realized that this change is wrong and can lead to problems with the upcoming lineage clustering where the exact timing of our energy deposits is of interest. To simulations running with DBSCAN clustering, the effect should be minimal.