Skip to content

Decoding speedups#979

Closed
baltzell wants to merge 15 commits into
developmentfrom
redecodering
Closed

Decoding speedups#979
baltzell wants to merge 15 commits into
developmentfrom
redecodering

Conversation

@baltzell

@baltzell baltzell commented Dec 4, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Zero change to resulting output data
  • More caching, more hashing, less looping
  • Replace string comparisons with DetectorType comparisons
  • Replace string IndexedTable access with column indices
  • Add Clas12Reader CLARA service

See #949 for finer commits (had trouble rebasing across submodule changes).

Decoding for RG-D LD2 is now down to ~3 ms per event on ifarm23, my laptop, and GitLab CI jobs at JLab.

@baltzell baltzell changed the title Redecodering Decoding speedups Dec 4, 2025
@baltzell
baltzell marked this pull request as ready for review December 4, 2025 00:55
@baltzell baltzell mentioned this pull request Dec 4, 2025
Closed
@baltzell baltzell added the speed label Dec 4, 2025
@baltzell
baltzell force-pushed the redecodering branch 3 times, most recently from 7ae2467 to 92e90f5 Compare December 8, 2025 00:59
@baltzell
baltzell marked this pull request as draft December 10, 2025 23:08
@baltzell baltzell closed this Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raffaelladevita raffaelladevita Awaiting requested review from raffaelladevita raffaelladevita is a code owner
@c-dilks c-dilks Awaiting requested review from c-dilks c-dilks is a code owner
@zieglerv zieglerv Awaiting requested review from zieglerv zieglerv is a code owner
@whit2333 whit2333 Awaiting requested review from whit2333 whit2333 is a code owner

Assignees

No one assigned

Labels

speed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@baltzell