search

key4hep / EDM4hep

Generic event data model for HEP collider experiments
https://cern.ch/edm4hep
Apache License 2.0
25 stars 35 forks source link

`radiusOfInnerMostHit` could be removed from `Track` #319

Closed tmadlener closed 3 months ago

tmadlener commented 3 months ago

The Track currently has a member radiusOfInnerMostHit, which could be removed, as it is technically easily computable from the attached TrackerHits. The only / main(?) reason to keep it would be to keep this information around even if the TrackerHits are dropped (as is the case for example in DSTs for ILD).

gaede commented 3 months ago

We (BH, JMC, FG) believe that this should be droped. (If really needed later, adding it is easy w/ schema evolution).

Zehvogel commented 3 months ago

This was quite useful information on DST files (no-hits) imo...

tmadlener commented 3 months ago

Removing it for now, since it's mixing analysis and reconstruction level quantities too much. We have a similar discussion for other items, e.g. #322