-
Notifications
You must be signed in to change notification settings - Fork 483
ITS: perVertex do low multiplicty vertices first #14402
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
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
DeltaRof is currently being studied in dedicated studies on MC, please check if you suspect this might break it. |
|
Actually, looking at it again, I think what I said was completely unfounded and will have no effect on this at all. |
|
@f3sch did you check if now low-mult. vertices not stealing tracks from the high mult. ones? |
|
@shahor02 no, I did not. Max wanted to study this now. |
|
Error while checking build/O2/fullCI_slc9 for 9687dd3 at 2025-06-14 15:53: Full log here. |
|
I used an implementation I have locally of the inverted ordering, and studied the perROF vs perPV vs perPV inverted order (from low mult to high mult) performance in a Pb-Pb 38kHz simulation. Here the results: The per PV with inverted order shows worse performance than the other two, so I won't discuss it further. The per PV vs per ROF comparison show very similar performance, with per ROF having slightly better average track length for good tracks and the per Vertex finding more good tracks in absolute value but slightly worse in proportion to the total number of tracks. Here also the comparison of the number of cluster populations:
to me from these numbers is very difficult to assess if the perPV will improve or worsen the global tracking performance. @shahor02 if you have any suggestions for further checks, I can try them. |
|
Thanks @mpuccio ! From your statistics I would assume you have ~150 collisions, at 38kHz you should have ~35 ROFs with >1 collisions and they will be mostly low mult., hardly one can see the effect reported by Igor: For that one would need to process order of magnitude larger stat and check only ROFs with |
|
For my own education how sure are we, that this stealing comes only from the perVertex and is not some convolution with deltaROF=1, which was from what I can see also enabled in this production? |
|
Well, in 16kHz data I posted above, the ROFs neighbouring with the one having collisions are empty most of the time (except the noise) so the deltaROF should not matter. Also, we had test productions |
|
Hi @shahor02, Overall, the difference is not striking even without the pt cuts, so indeed more statistics will be required. However, with the pt cuts the perPV was finding more good tracks than the perROF in absolute terms and the difference between the number of fakes is smaller. Still, margins are thin and I will wait for the new, larger MC with only central Pb-Pb collisions. P.S. the total number of good cluster attachments is the number of clusters for which the MC label matches the one of the track they are assigned to. |
|
This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days. |
|
This PR did not have any update in the last 30 days. Is it still needed? Unless further action in will be closed in 5 days. |



Uh oh!
There was an error while loading. Please reload this page.