Skip to content

Commit

Permalink
fr fr fr fr this time
Browse files Browse the repository at this point in the history
  • Loading branch information
sanatd33 committed Apr 24, 2024
1 parent 9e36251 commit 3a78242
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ bool LinePivotPathPlanner::is_done() const {
return false;
}
return abs(cached_angle_change_.value()) <
degrees_to_radians(static_cast<float>(is_done_angle_change_thresh_));
degrees_to_radians(static_cast<float>(kIsDoneAngleChangeThresh));
}

LinePivotPathPlanner::State LinePivotPathPlanner::next_state(const PlanRequest& plan_request) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class LinePivotPathPlanner : public PathPlanner {
Trajectory pivot(const PlanRequest& request);

// TODO(Kevin): ros param this
static constexpr double is_done_angle_change_thresh_{1.0};
static constexpr double kIsDoneAngleChangeThresh{1.0};

enum State { LINE, PIVOT };

Expand Down
2 changes: 1 addition & 1 deletion soccer/src/soccer/planning/planner/pivot_path_planner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ bool PivotPathPlanner::is_done() const {
return false;
}
bool val = abs(cached_angle_change_.value()) <
degrees_to_radians(static_cast<float>(is_done_angle_change_thresh_));
degrees_to_radians(static_cast<float>(kIsDoneAngleChangeThresh));
return val;
}

Expand Down
2 changes: 1 addition & 1 deletion soccer/src/soccer/planning/planner/pivot_path_planner.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,6 @@ class PivotPathPlanner : public PathPlanner {
std::optional<double> cached_angle_change_;

// TODO(Kevin): ros param this
static constexpr double is_done_angle_change_thresh_{1.0};
static constexpr double kIsDoneAngleChangeThresh{1.0};
};
} // namespace planning

0 comments on commit 3a78242

Please sign in to comment.