-
Notifications
You must be signed in to change notification settings - Fork 28
Add conventional tracking into ai-assisted tracking as complementary #543
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
Conversation
…fter AI-assisted tracking
zieglerv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A study I did on events with multiple AI candidates for a given track shows that a substantial fraction of AI failures comes from choosing the seed with maximum AI probability where AI gets the wrong set of segments. In such cases, some segments are real segments on track and some do not belong to the track. The real segments are near the track trajectory. Your algorithm could be improved by doing the following:
- Read all hits from the Hits bank
- Remake the clusters from these hits
- Look for segments not suggested by AI that are near the AI seed segment list (define a cut based on doca to the AI segments and local angle)
- Using this group of segments find the subgroup of segments yielding the best track
- After this search for tracks using the segments that are not on-track to find possible seeds that have no segments suggested by AI.
|
The update actually covers the issue that you mentioned. |
|
The update is suggested by Raffaella. Add one more bit (bit 13) into status of tracks. It points out if a track is from AI-assisted tracking or conventional tracking. 0 means conventional tracking, and 1 means AI-assisted tracking. |
|
ready? |
|
Looking at the code the list of clusters considered are the clusters which are not on tracks: |
zieglerv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment in the conversation section.
|
Exactly to say, segmentsConv contains segments not on tracks from AI-assisted tracking. |
…543) * As supplementary, take conventional tracking with remaining cluters after AI-assisted tracking * add one more bit into track status to tell if track is from AI-asssisted tracking
…543) * As supplementary, take conventional tracking with remaining cluters after AI-assisted tracking * add one more bit into track status to tell if track is from AI-asssisted tracking
…543) * As supplementary, take conventional tracking with remaining cluters after AI-assisted tracking * add one more bit into track status to tell if track is from AI-asssisted tracking
…543) * As supplementary, take conventional tracking with remaining cluters after AI-assisted tracking * add one more bit into track status to tell if track is from AI-asssisted tracking
…543) * As supplementary, take conventional tracking with remaining cluters after AI-assisted tracking * add one more bit into track status to tell if track is from AI-asssisted tracking
See details for update and validation in the attached document.
AI-assisted Tracking + Conventional Tracking as Complementary.pdf
With the update, as well as previous updates for forward tracking, slope for forward tracking reaches the goal: -0.001/nA, as test by RGA and RGD datasets.
This PR is for both dc2 and dc1. So this branch should also be merged into the dc1 branch.