-
Notifications
You must be signed in to change notification settings - Fork 174
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
feat: Add csv prototrack writer #2513
feat: Add csv prototrack writer #2513
Conversation
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.
few minor suggestions but nothing critical
Examples/Io/Csv/include/ActsExamples/Io/Csv/CsvProtoTrackWriter.hpp
Outdated
Show resolved
Hide resolved
Examples/Io/Csv/include/ActsExamples/Io/Csv/CsvProtoTrackWriter.hpp
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## main #2513 +/- ##
=======================================
Coverage 49.79% 49.79%
=======================================
Files 466 466
Lines 26326 26326
Branches 12095 12095
=======================================
Hits 13108 13108
Misses 4615 4615
Partials 8603 8603 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Co-authored-by: Andreas Stefl <[email protected]>
This currently still fails the python tests and clang-tidy. |
This is useful when working e.g. with the Exa.TrkX plugin.