-
Notifications
You must be signed in to change notification settings - Fork 30
reduce string comparisons in ADC decoding, cleanup #1066
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
base: development
Are you sure you want to change the base?
Conversation
eb13585 to
9aff614
Compare
|
Summary of |
9aff614 to
616076c
Compare
|
And when |
|
The LTCC/RF discrepancy is due to |
|
And the If it ends up in |
|
The Maybe/hopefully this will be explained by picking up the pedestal/threshold form |
|
|
Another overlap in |
616076c to
ea3a037
Compare
|
After disabling channels 12-15 on all slot 8s in |
|
|
|
So, after fixing The first 10k events from each "gold" run's .00001 file are identical between the main branch and redecodering2 (which is the sum of pull requests #1064, #1065, #1066): |
ea3a037 to
4684280
Compare
4684280 to
18d66af
Compare
Note, this assumes unique
crate/slot/channelin all/daq/fadcCCDB tables. Now, the first match wins. Previously the last match won. For run 5038, theRF/LTCC::adcbanks have a resulting difference, which will be addressed by fixing CCDB tables to be unique.Note, this was a component of #1051.