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.

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

Fix multilane support in OSMBuilder #189

Closed
agalbachicar opened this issue Feb 21, 2017 · 4 comments
Closed

Fix multilane support in OSMBuilder #189

agalbachicar opened this issue Feb 21, 2017 · 4 comments
Assignees

Comments

@agalbachicar
Copy link
Member

agalbachicar commented Feb 21, 2017

There is an strange case in MCity that produces a single direction road with only one lane and it seems to be a two direction road.

RNDF render in Gazebo plugin can be seen here:
mcitycase

The OSM map can be seen here: https://www.openstreetmap.org/#map=19/42.30069/-83.69846

osm

Take some known samples of multiple lane roads from OSM and validate this.

@agalbachicar
Copy link
Member Author

agalbachicar commented Feb 22, 2017

I've checked this and it's not working properly. I have downloaded from OSM some blocks of BA. It has an avenue with roads towards downtown and the opposite way and they are displayed only from downtown to the suburbs.

Here is the link of the OSM map. And here is the link of the Google Maps map to compare.

And some pictures of the rendered maps:

OpenDrive:
opendrive-liniers

RNDF Plugin:
rndf-liniers

@basicNew
Copy link
Contributor

Yes, I also did some quick tests yesterday and can confirm that. Thanks for the report @agalbachicar !

@basicNew
Copy link
Contributor

@agalbachicar this has not been yet fixed. I'll change the tile and re-open it.

@basicNew basicNew reopened this Feb 23, 2017
@basicNew basicNew changed the title Verify multilane support with OSMBuilder Fix multilane support with OSMBuilder Feb 23, 2017
@basicNew basicNew changed the title Fix multilane support with OSMBuilder Fix multilane support in OSMBuilder Feb 23, 2017
@basicNew basicNew assigned basicNew and unassigned agalbachicar Feb 23, 2017
@basicNew basicNew removed the SP 1 label Feb 23, 2017
@basicNew
Copy link
Contributor

Closing this in favor of #257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants