-
Notifications
You must be signed in to change notification settings - Fork 2
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
Spiral Lane integration #277
Conversation
Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
- Fix heading. - Fix regression tests. - Fix integration tests. Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
c211f7e fixes the ground curve issues. Now, it is loading with the right tolerance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
1fae479 should fix the bazel pipeline. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
🎉 New feature
Solves #265
Summary
Integrates the SpiralGroundCurve into the builder to get RoadGeometries that support that ground geometry.
Note that there are still some issues with tolerances which are underlying to the implementation in #276. The integration is achieved but the tolerance level is extremely high for SmallTownRoads.xodr at the moment.Picture of an intersection with spiral geometries from SmallTownRoads.xodr
Full mesh file:
SmallTownRoadsMesh.zip
Test it
Via unit and integration tests.
Checklist
Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.