Skip to content

Commit 5bba0ac

Browse files
committed
fix coordinate ordering
1 parent 844a72b commit 5bba0ac

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

Core/include/Acts/Seeding2/CylindricalSpacePointGrid2.hpp

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ namespace Acts::Experimental {
2222

2323
/// A cylindrical space point grid used for seeding in a cylindrical detector
2424
/// geometry.
25-
/// The grid is defined in cylindrical coordinates (phi, r, z) and allows for
26-
/// efficient access to space points based on their azimuthal angle, radial
27-
/// distance, and z-coordinate.
25+
/// The grid is defined in cylindrical coordinates (phi, z, r) and allows for
26+
/// efficient access to space points based on their azimuthal angle,
27+
/// z-coordinate, and radial distance.
2828
class CylindricalSpacePointGrid2 {
2929
public:
3030
/// Space point index type used in the grid.
@@ -102,7 +102,7 @@ class CylindricalSpacePointGrid2 {
102102

103103
/// Get the bin index for a space point given its azimuthal angle, radial
104104
/// distance, and z-coordinate.
105-
/// @param position The position of the space point in (phi, r, z) coordinates
105+
/// @param position The position of the space point in (phi, z, r) coordinates
106106
/// @return The index of the bin in which the space point is located, or
107107
/// `std::nullopt` if the space point is outside the grid bounds.
108108
std::optional<std::size_t> binIndex(const Vector3& position) const {
@@ -114,19 +114,19 @@ class CylindricalSpacePointGrid2 {
114114
/// Get the bin index for a space point given its azimuthal angle, radial
115115
/// distance, and z-coordinate.
116116
/// @param phi The azimuthal angle of the space point in radians
117-
/// @param r The radial distance of the space point from the origin
118117
/// @param z The z-coordinate of the space point
118+
/// @param r The radial distance of the space point from the origin
119119
/// @return The index of the bin in which the space point is located, or
120120
/// `std::nullopt` if the space point is outside the grid bounds.
121-
std::optional<std::size_t> binIndex(float phi, float r, float z) const {
122-
return binIndex(Vector3(phi, r, z));
121+
std::optional<std::size_t> binIndex(float phi, float z, float r) const {
122+
return binIndex(Vector3(phi, z, r));
123123
}
124124

125125
/// Insert a space point into the grid.
126126
/// @param index The index of the space point to insert
127127
/// @param phi The azimuthal angle of the space point in radians
128-
/// @param r The radial distance of the space point from the origin
129128
/// @param z The z-coordinate of the space point
129+
/// @param r The radial distance of the space point from the origin
130130
/// @return The index of the bin in which the space point was inserted, or
131131
/// `std::nullopt` if the space point is outside the grid bounds.
132132
std::optional<std::size_t> insert(SpacePointIndex index, float phi, float r,
@@ -136,7 +136,7 @@ class CylindricalSpacePointGrid2 {
136136
/// @return The index of the bin in which the space point was inserted, or
137137
/// `std::nullopt` if the space point is outside the grid bounds.
138138
std::optional<std::size_t> insert(const ConstSpacePointProxy2& sp) {
139-
return insert(sp.index(), sp.phi(), sp.r(), sp.z());
139+
return insert(sp.index(), sp.phi(), sp.z(), sp.r());
140140
}
141141

142142
/// Fill the grid with space points from the container.

0 commit comments

Comments
 (0)