Skip to content

Speed up charge geometry loading#231

Merged
mjkramer merged 1 commit intodevelopfrom
feature/geom-load-speed
Jan 7, 2026
Merged

Speed up charge geometry loading#231
mjkramer merged 1 commit intodevelopfrom
feature/geom-load-speed

Conversation

@mjkramer
Copy link
Member

@mjkramer mjkramer commented Jan 7, 2026

This is mainly for ND-LAr. It was taking ten minutes to initialize the pixel coordinates LUT. Now it takes about a minute.

  • Generate the LUT's keys just once, instead of 35 times
  • Add a LUT.set_scalar method optimized for setting one element at a time
  • Avoid a bunch of function calls that just repeatedly calculate "IO groups per module"
  • Add a progress bar since it's still kinda slow

@mjkramer mjkramer force-pushed the feature/geom-load-speed branch from 76c1266 to 32c06e3 Compare January 7, 2026 15:44
@mjkramer mjkramer marked this pull request as ready for review January 7, 2026 15:58
@mjkramer mjkramer merged commit 82784ca into develop Jan 7, 2026
2 checks passed
@mjkramer mjkramer deleted the feature/geom-load-speed branch January 7, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant