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.

Sign up for GitHub

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

Jump to bottom

Centralized Track-Cluster Matching #1648

Open
ruse-traveler opened this issue Oct 28, 2024 · 2 comments
Open

Centralized Track-Cluster Matching #1648

ruse-traveler opened this issue Oct 28, 2024 · 2 comments
Assignees
@tlprotzman
Labels
topic: lepton ID Issues pertinent to electron/muon finding topic: particle flow topic: reconstruction

Comments

@ruse-traveler
Copy link
Contributor

One of the key items identified in the July 2024 UGM is an centralized algorithm to produce track-cluster associations (see the discussions at the UGM and in the Aug. 27th, Sep. 16th, and Oct. 14th Reconstruction WG meetings). This algorithm would function as the starting point for both the eID and PF workflows, and is crucial to excising truth information from the existing eID algorithm.

Describe the solution you'd like
An algorithm which takes in tracks and clusters and produces

Describe alternatives you've considered
n/a

Additional context
This falls into the Reconstruction WG's eID/PFA development program
@ruse-traveler ruse-traveler mentioned this issue Oct 29, 2024
Open
@veprbl
Copy link
Member

veprbl commented Dec 12, 2024

It makes sense to start with this by porting https://github.com/eic/snippets/blob/main/Calorimetery/MatchClustersToTrackProjections/MatchProjectionsAndClusters.cxx into an EICrecon algorithm.

@tlprotzman tlprotzman mentioned this issue Jan 1, 2025
Closed
6 tasks
@tlprotzman
Copy link
Contributor

I've started to port that to an algorithm, I'll let you know when things are more tested and ready for review

@ruse-traveler ruse-traveler moved this from Todo to In Progress in ePIC Software and Computing Jan 2, 2025
@tlprotzman tlprotzman mentioned this issue Mar 25, 2025
Merged
7 tasks
github-merge-queue bot pushed a commit that referenced this issue Mar 28, 2025
@tlprotzman @pre-commit-ci @veprbl
Track cluster matching (#1768)
6b203fd 
### Briefly, what does this PR introduce?
A centralized algorithm for cluster-track matching which does not rely
on any truth issue.

### What kind of change does this PR introduce?
- [ ] Bug fix (issue #__)
- [X] New feature (issue #1648 )
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [ ] Changes have been communicated to collaborators

### Does this PR introduce breaking changes? What changes might users
need to make to their code?

### Does this PR change default behavior?
Adds `"TrackClusterMatches` and `BarrelClusters` collections to default
output

---------

Co-authored-by: EIC Container Build Service <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Dmitry Kalinkin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tlprotzman tlprotzman

Labels
topic: lepton ID Issues pertinent to electron/muon finding topic: particle flow topic: reconstruction
Projects
ePIC Software and Computing
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
3 participants
@veprbl @tlprotzman @ruse-traveler