Fix cluster-TP association for TrajectorySeeds in QuickTrackAssociatorByHits #7626
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When cluster-TrackingParticle association was introduced in QuickTrackAssociatorByHits, the necessary "preparation step" was not added for TrajectorySeeds. As a result the TrackerSeedValidator got no associations to TPs for TrajectorySeeds. This is fixed here by making the call to
prepareCluster2TPMap()
, as in the methods making the associations for Tracks.Note: AFAICT 74X is not affected as it seems that the bug got fixed in #3639.
Tested in 620_SLHC20 (that the fix does what it should), and in 620_SLHC23_patch2 and wf 10239 that it doesn't affect standard workflows (as TrackerSeedValidator is not part of them).
@rovere @VinInn