Skip to content

Conversation

@baltzell
Copy link
Collaborator

@baltzell baltzell commented Jan 7, 2026

Summary:

  • Less string comparisons and looping
  • More hash/index-based access for CCDB tables and banks
  • Use new global TranslationTable class, with additional DetectorType field
  • Reduce EVIO branch/node transversing

Side Effects:

  • Run 5038
    • Row-ordering changed for ECAL::tdc, RAW::vtp, BMT::adc, and RAW::adc
    • CCDB overlap for /daq/fadc/ltcc and /daq/fadc/rf (LTCC::adc and RF::adc)
  • Run 18799
    • CCDB overlap for /daq/tt/band and /daq/tt/raster (BAND::adc and RASTER::adc)

Note, this is the completion of #1008 ...

@baltzell baltzell changed the title Redecodering2 Decoding speedups Jan 7, 2026
@baltzell baltzell added the speed label Jan 7, 2026
@baltzell
Copy link
Collaborator Author

baltzell commented Jan 7, 2026

For 5038, /daq/fadc/ltcc and /daq/fadc/rf CCDB tables have a duplicate crate/slot/channel.

And, hipo-diff also isn't happy with ECAL::tdc, RAW::vtp, BMT::adc, and RAW::adc:(

@baltzell baltzell force-pushed the redecodering2 branch 3 times, most recently from d91cf86 to eeb1b82 Compare January 12, 2026 15:03
@baltzell
Copy link
Collaborator Author

RAW::vtp is only different by row ordering (now it's sorted by crate, the first index).

@baltzell
Copy link
Collaborator Author

ECAL::tdc is also only different by row ordering (now sorted by s/l/c/o).

@baltzell baltzell force-pushed the redecodering2 branch 2 times, most recently from e457e5b to bdb04f3 Compare January 12, 2026 23:02
@baltzell
Copy link
Collaborator Author

And BMT::adc and RAW::adc are also only different by row ordering.

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