Skip to content

Conversation

@baltzell
Copy link
Collaborator

@baltzell baltzell commented Jan 13, 2026

Note, this assumes unique crate/slot/channel in all /daq/fadc CCDB tables. Now, the first match wins. Previously the last match won. For run 5038, the RF/LTCC::adc banks have a resulting difference, which will be addressed by fixing CCDB tables to be unique.

Note, this was a component of #1051.

@baltzell baltzell added the speed label Jan 13, 2026
@baltzell baltzell mentioned this pull request Jan 13, 2026
@baltzell baltzell changed the title reduce string comparisons, cleanup Reduce string comparisons in ADC decoding, cleanup Jan 13, 2026
@baltzell baltzell marked this pull request as ready for review January 13, 2026 20:43
@baltzell baltzell changed the title Reduce string comparisons in ADC decoding, cleanup fix: reduce string comparisons in ADC decoding, cleanup Jan 13, 2026
@baltzell baltzell force-pushed the decoder-tt-adc branch 2 times, most recently from eb13585 to 9aff614 Compare January 21, 2026 01:21
@baltzell
Copy link
Collaborator Author

baltzell commented Jan 29, 2026

Summary of hipo-diff on gold runs for PRs #1064, #1065, and #1066:

Run Period     , Gold Run , Redecoderingdiff
rga_spring2018 , 4013     , RF/LTCC::adc
rga_fall2018   , 5038     , RF/LTCC::adc
rgk_2018       , 5700     , RF/LTCC::adc
rgb_spring2019 , 6302     , RF/BAND/LTCC::adc
rga_spring2019 , 6666     , RF/LTCC::adc
rgb_fall2019   , 11567    , RF/LTCC::adc
rgf_2020       , 12933    , RF/LTCC::adc (and HEL::scaler errors, unreleated(?))
rgm_2021       , 15833    , RF/BAND::adc
rgc_spring2022 , 16600    , RF/LTCC::adc
rgc_fall2022   , 17407    , RF/LTCC::adc (and HEL::scaler errors, unrelated(?))
rgc_spring2023 , 17800    , RF/LTCC::adc and RAW::tdc
rgd_2023       , 18779    , BAND/RF/LTCC::adc
rgk_2024       , 19877    , BAND/RF/LTCC::adc
rge_2024       , 20522    , BAND/RF/LTCC::adc and RAW::tdc
rgl_2025       , 23050    , RF/BAND::adc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants