Add some tolerance to certain numerical constraints

maliput / maliput_sparse

A maliput backed capable of modeling RoadNetworks from waypoints.

BSD 3-Clause "New" or "Revised" License

0 stars 0 forks source link

Add some tolerance to certain numerical constraints #41

Closed francocipollone closed 2 years ago

francocipollone commented 2 years ago

Summary

Set Up:

backend: maliput_osm
map: y_shape_road.osm
agent: rail agent

I ran into the following error while traversing the lanes with a rail car agent.

ERROR: Failure at /home/franco/maliput_ws_focal/src/maliput_sparse/src/geometry/utility/geometry.cc:317 in GetBoundPointsAtP(): condition 'p <= line_string.length()' failed

We should gracefully deal with numerical comparisons.

francocipollone commented 2 years ago

I'd like to move OpenRangeValidator from maliput_malidrive to maliput::common and use it in maliput_sparse also. wdyt @agalbachicar about moving OpenRangeValidator up to maliput ?

agalbachicar commented 2 years ago

SGTM