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.

Sign up for GitHub

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

Jump to bottom

False positive in check_asam_xodr_junctions_connection_one_connection_element with OpenDRIVE 1.8 #117

Closed
MHoehndorf opened this issue Oct 31, 2024 · 2 comments
Closed

False positive in check_asam_xodr_junctions_connection_one_connection_element with OpenDRIVE 1.8 #117

MHoehndorf opened this issue Oct 31, 2024 · 2 comments
Labels
isState:Integrated An issue that has been integrated and has an integration reviewer isType:Bug An issue that contains contradictions or errors in the standard.

Comments

@MHoehndorf
Copy link

Describe the bug
When checking the opendrive_connection_warning.zip we receive the following warnings:
grafik
According to OpenDrive 1.8 Spec chapter 12.4 "Each connecting road shall be associated with at most one element per incoming road." we think that this warning should not occur as the incoming roads are different.

To Reproduce
Steps to reproduce the behavior:

  1. validate provided file

Expected behavior
For OpenDrive 1.7 the behaviour is correct, but for OpenDrive 1.8 no warning should occur.
@MHoehndorf MHoehndorf added the isType:Bug An issue that contains contradictions or errors in the standard. label Oct 31, 2024
@hoangtungdinh hoangtungdinh mentioned this issue Oct 31, 2024
Merged
@MHoehndorf MHoehndorf changed the title opendrive_connection_warning False positive in check_asam_xodr_junctions_connection_one_connection_element with OpenDRIVE 1.8 Nov 1, 2024
@hoangtungdinh hoangtungdinh mentioned this issue Nov 7, 2024
Open
@AsamDiegoSanchez AsamDiegoSanchez added the isState:Accepted An issue that has been accepted by the group, which needs to be assigned to a responsible label Nov 18, 2024
@andreaskern74
Copy link
Collaborator

Fixed with merge of #118

@andreaskern74 andreaskern74 added isState:Integrated An issue that has been integrated and has an integration reviewer and removed isState:Accepted An issue that has been accepted by the group, which needs to be assigned to a responsible labels Nov 20, 2024
@andreaskern74
Copy link
Collaborator

The solution of this issue introduced the following problem: asam-ev/qc-framework#206

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
isState:Integrated An issue that has been integrated and has an integration reviewer isType:Bug An issue that contains contradictions or errors in the standard.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@andreaskern74 @MHoehndorf @AsamDiegoSanchez