-
Notifications
You must be signed in to change notification settings - Fork 483
TRD: add optional(default=off) extra table for PID studies #14009
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
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
300a7db to
bb91f20
Compare
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
|
@jgrosseo is it okay to retroactively add |
a3fef38 to
fa4a199
Compare
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Note to self: think about inverse-log encoding pid signal and give 8 more bits to dEdx. Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
|
This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days. |
This introduces a new table to be used for TRD pid studies and parameterisation via special requested passes.
Leaving as draft for now until we agree on exact table information.
Feature requested by Anton Andronic & Kangkan Goswami, discussion is via email.
From local test via
which looks reasonable to me:

Also includes a small cleanup to not output extra fit tables if not requested.