Skip to content

Commit

Permalink
fix: data is private in MultiTrajectory GrowableColumns
Browse files Browse the repository at this point in the history
When accessing `parameters` or `covariance` through the generic accessors (i.e. not `predicted`, `filtered`, or `smoothed`), the public `col()` function should be used instead of direct access to the private `data` member.
  • Loading branch information
wdconinc authored Jun 24, 2022
1 parent 75f0835 commit 910606f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Core/include/Acts/EventData/MultiTrajectory.ipp
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ inline auto TrackStateProxy<M, ReadOnly>::parameters() const -> Parameters {
idx = data().ipredicted;
}

return Parameters(m_traj->m_params.data.col(idx).data());
return Parameters(m_traj->m_params.col(idx).data());
}

template <size_t M, bool ReadOnly>
Expand All @@ -73,7 +73,7 @@ inline auto TrackStateProxy<M, ReadOnly>::covariance() const -> Covariance {
} else {
idx = data().ipredicted;
}
return Covariance(m_traj->m_cov.data.col(idx).data());
return Covariance(m_traj->m_cov.col(idx).data());
}

template <size_t M, bool ReadOnly>
Expand Down

0 comments on commit 910606f

Please sign in to comment.