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

mapsforge-map-writer: reference all part elements #1025

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@Gustl22
Contributor

Gustl22 commented Dec 28, 2017

  • reference all part elements (not only the first one; necessary to pass tags on to parts in reader)
  • tag-mapping: enable roof directions

Minimal increased size and processing time

mapsforge-map: reference all part elements
tag-mapping: enable roof directions

@devemux86 devemux86 added this to the 0.10.0 milestone Dec 29, 2017

devemux86 added a commit that referenced this pull request Dec 29, 2017

S3DB (#1025)
- map-writer: reference all part elements
- tag-mapping: enable roof directions
@devemux86

This comment has been minimized.

Show comment
Hide comment
@devemux86

devemux86 Dec 29, 2017

Collaborator

Thanks!
Merged via 5e3194f.

Collaborator

devemux86 commented Dec 29, 2017

Thanks!
Merged via 5e3194f.

@devemux86 devemux86 closed this Dec 29, 2017

@Gustl22 Gustl22 deleted the Gustl22:tag-mapping branch Jan 2, 2018

@Gustl22

This comment has been minimized.

Show comment
Hide comment
@Gustl22

Gustl22 Aug 23, 2018

Contributor

@devemux86: Is it worth to enable all roof directions (NNE, NNW etc.) in tag-mapping? This would add about 350 ways (overpass turbo).

Contributor

Gustl22 commented Aug 23, 2018

@devemux86: Is it worth to enable all roof directions (NNE, NNW etc.) in tag-mapping? This would add about 350 ways (overpass turbo).

@devemux86

This comment has been minimized.

Show comment
Hide comment
@devemux86

devemux86 Aug 24, 2018

Collaborator

If improve rendering for such ways amount, probably worth the effort?

Collaborator

devemux86 commented Aug 24, 2018

If improve rendering for such ways amount, probably worth the effort?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment