-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add association between ReconstructedParticle and Vertex #133
Conversation
What is the use case for this? In principle a Line 382 in b1b0b2b
and Line 399 in b1b0b2b
|
a collection like this is created by LCFIPlus |
Ah yes, I see. I have opened an issue in podio to support arbitrary associations. Not entirely sure about the timeline though ;) In principle I don't see anything that speaks against merging this now and then reworking things once (if) arbitrary associations arrive in podio. |
Co-authored-by: Andre Sailer <andre.philippe.sailer@cern.ch>
I would say we go ahead and merge this, since there are other association collections around already. All those would change once the arbitrary collections are in place. |
8b1109d
to
0ba65d8
Compare
BEGINRELEASENOTES
ENDRELEASENOTES