Skip to content

PrLongLivedTracking: Correctly update the X-projection before calculating the MVA value for a Track

Christoph Hasse requested to merge FixPrLongLivedTrackingXProj into master

This adds an update of the x-projection before adding it to the deviation variable used for the MVA.

Problem in previous version:
The call to hit->projection() did not return the correct x-projection associated with the track we are currently calculating the MVA value for.
This is because the track only owns the pointer to the hit, so a different track candidate can later on pick said hit up again and update the projection.
Therefor a call to hit->projection in line 526 will return a value that was actually calculated with the last track candidate that used this hit.

To check I ran the reco over 300 Events and did not see any significant changes in the timing due the additional update call.
The number of reconstructed tracks is now 39106 vs previously 39061.

Hi @decianm, as discussed just "pinging" you on here so you can have a look :) Thanks!

Merge request reports

Loading