Fix when multiple Hough clusters are processed for the same X-Z track

The variable nXHits should be a const since it is the same for every loop corresponding to a different Hough cluster candidate. If a hit of one of the X layers is removed in the hit cleaning procedure, the count on X hits should be reset for the next Hough cluster candidate.

This bug fix has negligible effect on physics performance, however it does change the references. The efficiency and ghost rates are expected to go up slightly.

Reference updates in Brunel!984 (merged) and Moore!438 (merged)

added bug fix label

added needs ref update label

assigned to @mstahl

Let's get this tested. /cc @lohenry

/ci-test

mentioned in commit Moore@2626d4a4

mentioned in merge request Moore!438 (merged)

mentioned in commit Brunel@b5e2eee3

mentioned in merge request Brunel!984 (merged)

changed the description

approved this merge request

assigned to @rmatev and unassigned @mstahl

resolved all threads

@rmatev do you need anything else to proceed with merging this? Thanks

mentioned in issue Moore#148 (closed)

added Tracking label and removed needs ref update label

changed title from Bug fix: if multiple Hough clusters are processed for the same X-Z track... to Fix when multiple Hough clusters are processed for the same X-Z track

approved this merge request

unassigned @rmatev

merged

mentioned in commit 08cb4926