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

wrong message for check_asam_xodr_road_lane_link_new_lane_appear #112

Closed
MircoNierenz opened this issue Oct 8, 2024 · 1 comment
Closed

wrong message for check_asam_xodr_road_lane_link_new_lane_appear #112

MircoNierenz opened this issue Oct 8, 2024 · 1 comment
Labels
isType:Bug An issue that contains contradictions or errors in the standard.

Comments

@MircoNierenz
Copy link
Collaborator

Describe the bug
It occurs the Error: "If a new lane appears besides, only the continuing lane shall be connected to the original lane, not the appearing lane."
But the given Lane and Predecessor lane in the Error Message are not even connected to each other, so either the whole Error should not occur or the provided Lanes in the Error message are wrong.

Wrong: Message is wrong (lanes are not even connected)

Correct: No warning output (or different one)

Example: road_lane_link_new_lane_appear_issue.xodr
road_lane_link_new_lane_appear_issue.zip
@MircoNierenz MircoNierenz added the isType:Bug An issue that contains contradictions or errors in the standard. label Oct 8, 2024
hoangtungdinh added a commit that referenced this issue Oct 8, 2024
@hoangtungdinh
Fix issue in new lane appear rule reported in #112
0b15304 
Signed-off-by: hoangtungdinh <[email protected]>
@hoangtungdinh
Copy link
Contributor

Thanks for finding the issue and the example, @MircoNierenz. We found the problem and had a fix for it.

hoangtungdinh added a commit that referenced this issue Oct 8, 2024
@hoangtungdinh
Fix issue in new lane appear rule reported in #112
8e51f3a 
Signed-off-by: hoangtungdinh <[email protected]>
@hoangtungdinh hoangtungdinh mentioned this issue Oct 8, 2024
Merged
@hoangtungdinh hoangtungdinh moved this to Can close in Quality Checker Oct 14, 2024
@AsamDiegoSanchez AsamDiegoSanchez added the isState:Accepted An issue that has been accepted by the group, which needs to be assigned to a responsible label Oct 23, 2024
@andreaskern74 andreaskern74 removed the isState:Accepted An issue that has been accepted by the group, which needs to be assigned to a responsible label Oct 24, 2024
hoangtungdinh added a commit that referenced this issue Oct 30, 2024
@hoangtungdinh
Fix issue in new lane appear rule reported in #112
ac3d24c 
Signed-off-by: hoangtungdinh <[email protected]>
hoangtungdinh added a commit that referenced this issue Oct 31, 2024
@hoangtungdinh
Fix issue in new lane appear rule reported in #112
d716a3e 
Signed-off-by: hoangtungdinh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
isType:Bug An issue that contains contradictions or errors in the standard.
Projects
Quality Checker
Status: Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@andreaskern74 @hoangtungdinh @MircoNierenz @AsamDiegoSanchez