-
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
LaneOffsets's functions' continuity is not being checked #147
Comments
Given that this is used quite a lot by RR we probably should allow having discontinuities in the |
Thanks for the analysis. Yes, any increased RR compatibility would be helpful in our transition downstream. |
Interesting finding. I've noticed the large amount of One question:
In which sense? Do you mean the end of one lane with the start of another lane? Or do you refer to the lateral boundary of a lane and its adjacent lanes? |
Exactly. Given that a discontinuous laneOffset could end up in a contiguous Road. |
https://github.com/ToyotaResearchInstitute/maliput_malidrive/blob/941bb30292bd422e543245e7f39b3fc1a90711f1/maliput_malidrive/src/maliput_malidrive/builder/road_curve_factory.cc#L224-L229
Line 228 should have
FillingGapPolicy::kEnsureContiguity
otherwise it isn't being checked.The text was updated successfully, but these errors were encountered: