Closed tmadlener closed 3 weeks ago
BEGINRELEASENOTES
Hypothesis
HitLevelData
RecIonizationCluster
SimPrimaryIonizationCluster
TrackerPulse
TimeSeries
RecDqdx
ENDRELEASENOTES
See #322, #323 for discussion on how we arrived at this point.
The changes for RecDqdx are also present in #311, leading to a likely merge conflict depending on which PR is merged first
We keep the TimeSeries as it is used already, e.g. in dual-readout:
https://github.com/HEP-FCC/dual-readout/blob/51f9766bb3f9e80644bb032b98f1f5b1d4cb0327/DRdigi/include/DigiSiPM.h#L31
[x] Downstream checks & fixes
BEGINRELEASENOTES
Hypothesis
,HitLevelData
,RecIonizationCluster
,SimPrimaryIonizationCluster
,TrackerPulse
TimeSeries
RecDqdx
to make it a purely reconstruction level typeENDRELEASENOTES
See #322, #323 for discussion on how we arrived at this point.
The changes for
RecDqdx
are also present in #311, leading to a likely merge conflict depending on which PR is merged firstWe keep the
TimeSeries
as it is used already, e.g. in dual-readout:https://github.com/HEP-FCC/dual-readout/blob/51f9766bb3f9e80644bb032b98f1f5b1d4cb0327/DRdigi/include/DigiSiPM.h#L31
[x] Downstream checks & fixes