Skip to content

Commit e1500fb

Browse files
committed
Require adaptive_max_radius from YAML config, remove all 1000.0 hardcoded fallbacks
- adaptive_max_radius must now always be specified in YAML config - Removed all hardcoded 1000.0 fallback values - computeMaxBurninRadius() now throws errors instead of returning 1000.0 - Clear error messages if adaptive_max_radius is missing from config
1 parent 0a0eb5a commit e1500fb

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

src/ompl/geometric/planners/rrt/src/MAB_RRT.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -551,11 +551,10 @@ double ompl::geometric::MAB_RRT::computeMaxBurninRadius() const
551551
}
552552
}
553553

554-
// If we couldn't determine bounds, return a large default
554+
// If we couldn't determine bounds, throw error
555555
if (minExtent == std::numeric_limits<double>::infinity())
556556
{
557-
OMPL_WARN("Could not determine max radius from bounds, using default");
558-
return 1000.0;
557+
throw std::runtime_error("Could not determine max radius from bounds - adaptive_max_radius must be specified in YAML config");
559558
}
560559

561560
return minExtent;

0 commit comments

Comments
 (0)