Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ToRoadPosition implementation #67

Open
francocipollone opened this issue Sep 15, 2021 · 0 comments
Open

ToRoadPosition implementation #67

francocipollone opened this issue Sep 15, 2021 · 0 comments

Comments

@francocipollone
Copy link
Collaborator

https://github.com/ToyotaResearchInstitute/maliput_multilane/blob/1518673a0df56ee187497fcde079de9353ae3391/src/maliput_multilane/road_geometry.cc#L41-L56

This logic was changed in maliput_malidrive because of two possible scenarios.

  • Let's say that we want to do a ToRoadPosition at the start of lane B. We have lane A and B connected by A-end and B-start.
    doing a ToRoadPosition at the very start of the B lane could lead to ambiguity when:

    • Distance for Lane B is zero and distance for Lane A is less than linear tolerance, let's say half of linear_tolerance
    • Both lateral displacement (R) (which is evaluated if the distance isn't enough, and they are both within lane bounds) is close to zero. However, because of numerical error, the R for Lane A is 1e-7 and the R for Lane B is 1e-6.
    • In this case, Lane A will be selected as the best candidate.
  • The other scenario to have in mind is that the distance of all the lanes in a segment to a particular Inertial position will be the same. Given that the distance field from the RoadPositionResult means the distance from the requested inertial position to an inertial position in the segment volume, not the lane volume. Currently, it isn't an issue but when trying to make the logic more robust because of the first scenario, this case needs to be taken into account.

Related issues in maliput_malidrive backend:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant