Skip to content

Conversation

@FabianPfeufferCAP
Copy link

Add a description

Added additional documentation on how the boundary between lanes at different heights should be modeled.

Some questions to ask:
What is this change?

  • Additional documentation.

What does it fix?

  • Missing documentation on how to model lane boundaries.

Take this checklist as orientation for yourself, if this PR is ready for the Change Control Board:

  • My suggestion follows the style and contributors guidelines.
  • I have taken care about the documentation.
  • I have done the DCO signoff.
  • My changes generate no errors when passing CI tests.
  • I have successfully implemented and tested my fix/feature locally.
  • Appropriate reviewer(s) are assigned.

@stefancyliax stefancyliax added this to the V3.5.0 milestone May 30, 2022
@stefancyliax stefancyliax added the ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB. label May 30, 2022
@stefancyliax
Copy link
Contributor

OSI CCB 30.05.2022:

  • Content is fine. Will be merged and included in 3.5.

@clemenshabedank clemenshabedank added the RoadModel4.0 A label to collect road model topics for OSI 4.0 label May 30, 2022
Fabian Pfeuffer and others added 2 commits May 30, 2022 16:51
* Added additional documentation on how the boundary between lanes at
  different heights should be modeled.

Signed-off-by: Fabian Pfeuffer <Fabian.Pfeuffer@partner.bmw.de>
Co-authored-by: Stefan Cyliax <stefan.cyliax@asam.net>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
@pmai pmai force-pushed the update/lane/lane_boundary_doc branch from a6e057c to 9a02801 Compare May 30, 2022 14:52
@pmai pmai merged commit f836f2a into OpenSimulationInterface:master May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB. RoadModel4.0 A label to collect road model topics for OSI 4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants