Skip to content

Commit

Permalink
39 add surface id member to edm4eictrackpoint (#41)
Browse files Browse the repository at this point in the history
### Briefly, what does this PR introduce?
This adds two data members to TrackPoint to identify the detector
and surface on which the TrackPoint lies (implemented for track
projections but could be used more generally).

This change replaces the original change which added a single 32-bit
integer. This was done following a discussion in the S&C meeting on
August 23, 2023. Summary of proposed options can be [found
here](https://indico.bnl.gov/event/20352/contributions/80021/attachments/49313/84184/followup_edm4eic_trackpoint_PR%20copy.pdf).

### What kind of change does this PR introduce?
- [x] New feature (issue #39)
  • Loading branch information
tylerkutz authored Aug 31, 2023
1 parent b46ba9a commit de6035f
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions edm4eic.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,8 @@ components:
## A point along a track
edm4eic::TrackPoint:
Members:
- uint64_t surface // Surface track was propagated to (possibly multiple per detector)
- uint32_t system // Detector system track was propagated to
- edm4hep::Vector3f position // Position of the trajectory point [mm]
- edm4eic::Cov3f positionError // Error on the position
- edm4hep::Vector3f momentum // 3-momentum at the point [GeV]
Expand Down

0 comments on commit de6035f

Please sign in to comment.