Skip to content

Changing config to better optimize efficiency#271

Merged
toi42 merged 2 commits into
mainfrom
toi_config_change
Jul 16, 2026
Merged

Changing config to better optimize efficiency#271
toi42 merged 2 commits into
mainfrom
toi_config_change

Conversation

@toi42

@toi42 toi42 commented Jul 16, 2026

Copy link
Copy Markdown
Member

This update increases general reconstruction efficiency. The old base efficiency plot is below, followed by the new one on this config, both on a single neutrino event file

Old

muon_ke_tms_enter

New

muon_ke_tms_enter

We also see an increase in the 'good' reco efficiency, which is percentage with correct endpoint and charge id

Old

good_reco_muon_ke_tms_enter

New

good_reco_muon_ke_tms_enter

This does come at the cost of narrowing the scan range of the Hough Transform. It now scans from -1.5 to 1.5 slope, or in other words covers from -55 degrees to 55 degrees. This could be expanded and we could instead scan on finer intervals, but that comes at a performance cost. Before approving, I would like someone else to run this on their own edep sim files and ensure it works properly

@toi42

toi42 commented Jul 16, 2026

Copy link
Copy Markdown
Member Author

This also comes with a much more aggressive A* algorithm, which does sometimes lead to merging erroneous hits into this

entry_285

(The top hits in black are reco hits, extended upwards, and then below that is a shifted Hough line which has has additional hits merged into it)

Future updates will pull this back, based on a to be determined purity metric, trying to gauge how many false hits are in a track

@jdkio jdkio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove .DS_Store from the PR please. Otherwise it looks good. Please make sure to make an issue noting that we need to investigate options for higher angle muons and tracks.

@toi42
toi42 merged commit a5cd908 into main Jul 16, 2026
1 check passed
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