Skip to content

Commit 8034579

Browse files
mikefergusonenricosutera
authored andcommitted
fix a few outdated comments in smac planners (ros-navigation#3978)
Signed-off-by: enricosutera <enricosutera@outlook.com>
1 parent ddace10 commit 8034579

File tree

3 files changed

+5
-11
lines changed

3 files changed

+5
-11
lines changed

nav2_smac_planner/include/nav2_smac_planner/a_star.hpp

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,7 @@ class AStarAlgorithm
7272
typedef std::priority_queue<NodeElement, std::vector<NodeElement>, NodeComparator> NodeQueue;
7373

7474
/**
75-
* @brief A constructor for nav2_smac_planner::PlannerServer
76-
* @param neighborhood The type of neighborhood to use for search (4 or 8 connected)
75+
* @brief A constructor for nav2_smac_planner::AStarAlgorithm
7776
*/
7877
explicit AStarAlgorithm(const MotionModel & motion_model, const SearchInfo & search_info);
7978

@@ -196,16 +195,15 @@ class AStarAlgorithm
196195
inline NodePtr getNextNode();
197196

198197
/**
199-
* @brief Get pointer to next goal in open set
198+
* @brief Add a node to the open set
200199
* @param cost The cost to sort into the open set of the node
201200
* @param node Node pointer reference to add to open set
202201
*/
203202
inline void addNode(const float & cost, NodePtr & node);
204203

205204
/**
206205
* @brief Adds node to graph
207-
* @param cost The cost to sort into the open set of the node
208-
* @param node Node pointer reference to add to open set
206+
* @param index Node index to add
209207
*/
210208
inline NodePtr addToGraph(const unsigned int & index);
211209

@@ -218,9 +216,8 @@ class AStarAlgorithm
218216

219217
/**
220218
* @brief Get cost of heuristic of node
221-
* @param node Node index current
222-
* @param node Node index of new
223-
* @return Heuristic cost between the nodes
219+
* @param node Node pointer to get heuristic for
220+
* @return Heuristic cost for node
224221
*/
225222
inline float getHeuristicCost(const NodePtr & node);
226223

nav2_smac_planner/include/nav2_smac_planner/node_basic.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ class NodeBasic
4747
public:
4848
/**
4949
* @brief A constructor for nav2_smac_planner::NodeBasic
50-
* @param cost_in The costmap cost at this node
5150
* @param index The index of this node for self-reference
5251
*/
5352
explicit NodeBasic(const unsigned int index)

nav2_smac_planner/include/nav2_smac_planner/node_lattice.hpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,6 @@ struct LatticeMotionTable
5858
/**
5959
* @brief Initializing state lattice planner's motion model
6060
* @param size_x_in Size of costmap in X
61-
* @param size_y_in Size of costmap in Y
62-
* @param angle_quantization_in Size of costmap in bin sizes
6361
* @param search_info Parameters for searching
6462
*/
6563
void initMotionModel(

0 commit comments

Comments
 (0)