key4hep / EDM4hep

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

MCRecoTrackerHitPlaneAssociation can / should be removed #301

Closed tmadlener closed 3 months ago

tmadlener commented 5 months ago

Currently we have two associations for SimTrackerHit to reco level tracker hits:

https://github.com/key4hep/EDM4hep/blob/a296139da4e040c2290de3f00be4506b485ea77c/edm4hep.yaml#L633-L649

I am fairly certain these are leftovers from before the introduction of the TrackerHit interface (#252) and the dedicated MCRecoTrackerHitPlaneAssociation has become obsolete by now. I am not entirely sure whether we can simply drop it transparently.

In any case this should also go in before v1.0.