next-exp / nexus

Geant4 simulation framework of the NEXT Collaboration
5 stars 55 forks source link

Change default PMT binning? #180

Open gondiaz opened 1 year ago

gondiaz commented 1 year ago

Full simulation default PMT time binning is 100 ns, instead of the 25 ns which is used in the experiment. Is there any reason for this? If not, I would suggest to change it since it may cause some confusion in the IC processing: you will get 100 ns instead of 25 ns waveforms as you will likely want.

https://github.com/next-exp/nexus/blob/4b3d3d592d5fe0eb09d3dae75669602546d2f44a/source/geometries/PmtR11410.cc#L49

paolafer commented 1 year ago

The original reason was that we used to rebin later to 100 ns. If now it's not the case anymore, we can change the default value to 25 ns.