Feature/srepetto add grain variables#84
Conversation
chenel
left a comment
There was a problem hiding this comment.
Overall seems fine. Just one clarification before we proceed (see below)
| kRange, ///< Amount of material traversed by particle | ||
| kMCS, ///< Multiple scattering | ||
| kCalorimetry, ///< Observed energy deposited in active volume | ||
| kEnergyDeposition, ///< From visible light |
There was a problem hiding this comment.
Does this ever include a correction for unobserved energy to get back to a true energy estimator? Or is it literally the sum of the visible energy (which is nearly always an underestimate)? If it contains a correction, then I think it's not any different than kCalorimetry.
There was a problem hiding this comment.
We do not correct for calculating the true energy, but we reconstruct the energy of the particle from the total photons collected in our sensors...if you prefer we can use kCalorimetry, otherwise we can distinguish between the two methods and we can add another variable as that suggested
There was a problem hiding this comment.
Okay---so you're just taking the values read out of the photosensors? If so, then I agree it's different. That's usually a poor energy estimate, so I'm curious why you want to keep it. I don't want this to unnecessarily become a blocker, so if it's an important quantity, that's fine... I'm just trying to understand.
New structure of SRSAND with GRAIN, tracker and ecal classes to distinguish tracks, showers and ecal clusters inside SAND.
Added new variable to calculate the neutrino energy starting from the reconstruction given by GRAIN.