Skip to content

Conversation

@baltzell
Copy link
Collaborator

@baltzellbaltzell 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 ...

@baltzellbaltzell changed the title Redecodering2Decoding speedupsJan 7, 2026
@baltzell
Copy link
CollaboratorAuthor

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:(

@baltzellbaltzellforce-pushed the redecodering2 branch 3 times, most recently from d91cf86 to eeb1b82CompareJanuary 12, 2026 15:03
@baltzell
Copy link
CollaboratorAuthor

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

@baltzell
Copy link
CollaboratorAuthor

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

@baltzellbaltzellforce-pushed the redecodering2 branch 2 times, most recently from e457e5b to bdb04f3CompareJanuary 12, 2026 23:02
@baltzell
Copy link
CollaboratorAuthor

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

@baltzell
Copy link
CollaboratorAuthor

Broken into #1060, #1061, #1062, #1063, #1064, #1065, and #1066.

Sign up for freeto 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

@baltzell