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

Netconvert adds sidewalk to edge which does not allow pedestrians #13624

Closed
m-kro opened this issue Jul 14, 2023 · 3 comments
Closed

Netconvert adds sidewalk to edge which does not allow pedestrians #13624

m-kro opened this issue Jul 14, 2023 · 3 comments
Assignees
@m-kro
Labels
a:netconvert bug sumo-user
Milestone
1.19.0

Comments

@m-kro
Copy link
Contributor

m-kro commented Jul 14, 2023

Posted on the mailing list.

Netconvert uses the sidewalk width from the edge type, but the edge does not allow pedestrians.
@m-kro m-kro added this to the 1.19.0 milestone Jul 14, 2023
@m-kro m-kro self-assigned this Jul 14, 2023
m-kro added a commit that referenced this issue Jul 14, 2023
@m-kro
cross-check permissions and edge type lane widths ref #13624
dacd147 
Signed-off-by: m-kro <m.barthauer@t-online.de>
@m-kro m-kro closed this as completed Jul 14, 2023
@behrisch
Copy link
Contributor

I think this is not a proper fix. Specifying a sidewalk width is the correct way to add a sidewalk. If you do not want to use the type information you should not load the types

@behrisch behrisch reopened this Jul 16, 2023
behrisch added a commit that referenced this issue Jul 16, 2023
@behrisch
Revert "cross-check permissions and edge type lane widths ref #13624"
36843e9 
This reverts commit dacd147.
@behrisch
Copy link
Contributor

Maybe we could switch on --ignore-errors.edge-type by default if there is no type file loaded. This would make this use case considerably easier

@m-kro
Copy link
Contributor Author

m-kro commented Jul 17, 2023

@behrisch OK then I'll add that.

m-kro added a commit that referenced this issue Jul 17, 2023
@m-kro
switch on ignore-errors.edge-type for plain XML w/o type file ref #13624
7541a6f 


Signed-off-by: m-kro <m.barthauer@t-online.de>
m-kro added a commit that referenced this issue Jul 17, 2023
@m-kro
update tests ref #21 #13624
88401cb 
Signed-off-by: m-kro <m.barthauer@t-online.de>
m-kro added a commit that referenced this issue Jul 17, 2023
@m-kro
update doc ref #2 #13624
0450cfc 
Signed-off-by: m-kro <m.barthauer@t-online.de>
m-kro added a commit that referenced this issue Jul 17, 2023
@m-kro
update tests ref #21 #13624
1243b45 
Signed-off-by: m-kro <m.barthauer@t-online.de>
@m-kro m-kro closed this as completed Jul 18, 2023
behrisch added a commit that referenced this issue Sep 1, 2023
@behrisch
Revert "update tests ref #21 #13624"
b603b65 
This reverts commit 1243b45.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-kro m-kro

Labels
a:netconvert bug sumo-user
Projects
None yet
Milestone
1.19.0
Development

No branches or pull requests
2 participants
@behrisch @m-kro