-
Notifications
You must be signed in to change notification settings - Fork 178
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: Switch all of the Examples to read/write handles #1939
refactor: Switch all of the Examples to read/write handles #1939
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1939 +/- ##
=======================================
Coverage 49.59% 49.59%
=======================================
Files 413 413
Lines 23443 23443
Branches 10666 10666
=======================================
Hits 11627 11627
Misses 4336 4336
Partials 7480 7480 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for dcdd78bFull report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
CI is finally happy. @andiwand or @benjaminhuth maybe you could have a look? |
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.
nice! looks good to me 👍
Follow up to #1908. Switches over the rest of the Examples components. This also makes
WhiteBoard::{get,add}
private and adds the data handles as friends to restrict unregistered white board access.