-
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
chore: rename unused params #1952
chore: rename unused params #1952
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1952 +/- ##
=======================================
Coverage 49.51% 49.51%
=======================================
Files 411 411
Lines 23340 23340
Branches 10634 10634
=======================================
Hits 11557 11557
Misses 4332 4332
Partials 7451 7451
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 53b36e3Full report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
Examples/Algorithms/Digitization/include/ActsExamples/Digitization/DigitizationConfig.hpp
Show resolved
Hide resolved
CI is green now. could you take another look @paulgessinger ? |
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 to me
- rename `/*unused*/` params into something more understandable - improve doc
/*unused*/
params into something more understandable