Skip to content

Commit

Permalink
fix: GeoModelHelper update to remove of detail namespace (#3324)
Browse files Browse the repository at this point in the history
The GeoModelPlugin needed an update to the `detail::` removal from the axis.

This is it.
  • Loading branch information
asalzburger authored Jun 23, 2024
1 parent d39b745 commit e8b249c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Plugins/GeoModel/src/detail/GeoModelBinningHelper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,10 @@ Acts::detail::GeoModelBinningHelper::toProtoBinning(
"GeoModelBinningHelper: Invalid number of binning details, at least "
"the axis boundary type and the number of bins are needed.");
}
detail::AxisBoundaryType boundaryType = detail::AxisBoundaryType::Bound;
AxisBoundaryType boundaryType = AxisBoundaryType::Bound;
std::string axisBoundaryToken = binningDetails[0];
if (axisBoundaryToken == "closed") {
boundaryType = detail::AxisBoundaryType::Closed;
boundaryType = AxisBoundaryType::Closed;
} else if (axisBoundaryToken != "bound") {
throw std::invalid_argument(
"GeoModelBinningHelper: Axis boundary type needs to be closed or "
Expand All @@ -46,7 +46,7 @@ Acts::detail::GeoModelBinningHelper::toProtoBinning(
ActsScalar rangeMin = 0.;
ActsScalar rangeMax = 0.;
if (bValue == BinningValue::binPhi &&
boundaryType == detail::AxisBoundaryType::Closed) {
boundaryType == AxisBoundaryType::Closed) {
rangeMin = -M_PI;
rangeMax = M_PI;
} else {
Expand Down

0 comments on commit e8b249c

Please sign in to comment.