Skip to content

Commit

Permalink
refactor: remove obsolete assignments due to `calculateTrackQuantitie…
Browse files Browse the repository at this point in the history
…s()` (#2487)

With `calculateTrackQuantities(track);` we reset `track.nMeasurements()` and `track.nHoles()`, no need to set them before.
  • Loading branch information
AJPfleger committed Sep 28, 2023
1 parent ffc9e12 commit 753dbec
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions Core/include/Acts/TrackFitting/KalmanFitter.hpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// This file is part of the Acts project.
//
// Copyright (C) 2016-2019 CERN for the benefit of the Acts project
// Copyright (C) 2016-2023 CERN for the benefit of the Acts project
//
// This Source Code Form is subject to the terms of the Mozilla Public
// License, v. 2.0. If a copy of the MPL was not distributed with this
Expand Down Expand Up @@ -1150,8 +1150,6 @@ class KalmanFitter {
track.covariance() = params.covariance().value();
track.setReferenceSurface(params.referenceSurface().getSharedPtr());
}
track.nMeasurements() = kalmanResult.measurementStates;
track.nHoles() = kalmanResult.measurementHoles;

calculateTrackQuantities(track);

Expand Down Expand Up @@ -1289,8 +1287,6 @@ class KalmanFitter {
track.covariance() = params.covariance().value();
track.setReferenceSurface(params.referenceSurface().getSharedPtr());
}
track.nMeasurements() = kalmanResult.measurementStates;
track.nHoles() = kalmanResult.measurementHoles;

calculateTrackQuantities(track);

Expand Down

0 comments on commit 753dbec

Please sign in to comment.