-
Notifications
You must be signed in to change notification settings - Fork 157
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
refactor: Update edm4hep IO code to use frame model #2001
refactor: Update edm4hep IO code to use frame model #2001
Conversation
podio switched to recommending this model, and the store based model we used before is deprecated.
Ok I added CMake checks for the podio and edm4hep version. I need to bump our respective versions in our CI images. |
Codecov Report
@@ Coverage Diff @@
## main #2001 +/- ##
=======================================
Coverage 49.83% 49.83%
=======================================
Files 415 415
Lines 23593 23593
Branches 10681 10681
=======================================
Hits 11758 11758
Misses 4330 4330
Partials 7505 7505 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 1bb5274Full report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. just found some leftovers
Co-authored-by: Andreas Stefl <stefl.andreas@gmail.com>
podio switched to recommending this model, and the store based model we used before is deprecated.