Skip to content

Avoid duplication of pylons in enum #497

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

Merged
merged 18 commits into from
May 18, 2021
Merged

Conversation

clemenshabedank
Copy link
Contributor

@clemenshabedank clemenshabedank commented Mar 15, 2021

Removed cones as an example from TYPE_CONSTRUCTION_SITE_ELEMENT because there is a separate TYPE_PYLON.
@kmeids can you have a look please. @ThomasNaderBMW FYI

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.

If you can’t check all of them, please explain why.
If all boxes are checked or commented and you have achieved at least one positive review, you can assign the label ReadyForCCBReview!

@clemenshabedank clemenshabedank added SensorModeling The Group in the ASAM development project working on sensor modeling topics. Bug Problems in the build system, build scripts, etc or faults in the interface. labels Mar 15, 2021
@kmeids
Copy link

kmeids commented Apr 16, 2021

Output from SM meeting 16.04.2021:

  1. No objection on removing "cones" from the description of "TYPE_CONSTRUCTION_SITE_ELEMENT".

@kmeids kmeids added the ReadyForCCBReview Indicates that this MR is ready for a final review and merge by the CCB. label Apr 16, 2021
@kmeids kmeids added ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB. and removed ReadyForCCBReview Indicates that this MR is ready for a final review and merge by the CCB. labels May 12, 2021
@kmeids
Copy link

kmeids commented May 12, 2021

Output CCB 12.05.2021:

  1. @clemenshabedank please do the DCO and merge.

Copy link

@kmeids kmeids left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Output CCB 12.05.2021:

  1. @clemenshabedank Please fix DCO and merge

Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
vkresch and others added 16 commits May 18, 2021 11:38
Signed-off-by: Viktor Kreschenski <viktor.kreschenski@altran.com>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Stefan Cyliax <stefan.cyliax@asam.net>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Stefan Cyliax <stefan.cyliax@asam.net>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
Signed-off-by: Stefan Cyliax <stefan.cyliax@asam.net>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
…ark colors with OpenDRIVE

Signed-off-by: Pierre R. Mai <pmai@pmsf.de>
Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
…nterface/open-simulation-interface into bugfix/sm/pylons

Signed-off-by: Habedank Clemens <qxs2704@europe.bmw.corp>
@clemenshabedank clemenshabedank merged commit aeb8541 into master May 18, 2021
@clemenshabedank clemenshabedank deleted the bugfix/sm/pylons branch May 18, 2021 10:10
@clemenshabedank clemenshabedank added this to the V3.4.0 milestone Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Problems in the build system, build scripts, etc or faults in the interface. ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB. SensorModeling The Group in the ASAM development project working on sensor modeling topics.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants