FPGATrackSim: Add unique ID in FPGATrackSim objects
This adds uid
as an extra variable in the FPGATrackSim
objects (hit, (truth) track) for future use.
Somehow related to EFTRACK-555
Merge request reports
Activity
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
- Resolved by Ioannis Maznas
Changes look good to me - I made two groups of suggestions. One group related to usage of
HepMcParticleLink::ExtendedBarCode
that I'd missed replacing in !74355 (merged) and another set related to the fact that the values returned byHepMcParticleLink::id()
should already be valid, but if your plan was to switch the clients to the new method names first, then switch to using the actual id values, then that's fine by me too.
added EFTracking Trigger main review-pending-level-1 labels
CI Result SUCCESS (hash 79a9e6c3)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)added review-approved label and removed review-pending-level-1 label
mentioned in commit adec6d89