Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade/include far field current #37

Merged
merged 8 commits into from
Oct 25, 2021
Merged

Upgrade/include far field current #37

merged 8 commits into from
Oct 25, 2021

Conversation

peter-madigan
Copy link
Member

This includes a number of optimizations to enable the use of a current model that covers the full drift along with 3 neighboring pixels.

  • simulate_pixels.py creates a lookup table for event start and end indices to speed up the loop over track segments
  • simulate_pixels.py uses a loop when creating the track -> unique pixel map which uses substantially less GPU memory without any noticeable performance loss
  • simulate_pixels.py uses a fixed max_radius = 3.5 when finding nearby pixels
  • time ticks are now handled in a more coherent fashion under the assumption that only one event is processed at a time, this has a side effect of reducing the GPU memory consumption and allowing for better simulation of time gaps between events
  • a new current model response_44-far_field.npy has been added
  • a new detector configuration module0-far_field.yaml has been added

Note: this also includes the changes made in the upgrade/add-discriminator-threshold-noise branch -- it might be simpler to just merge this one in rather than trying to do both.

@soleti soleti merged commit cc1d884 into DUNE:multi_module Oct 25, 2021
@peter-madigan peter-madigan deleted the upgrade/include-far-field-current branch November 9, 2021 19:46
infophysics pushed a commit to infophysics/larnd-sim that referenced this pull request Jul 14, 2023
…ld-current

Upgrade/include far field current
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.

2 participants