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

Integrate the coastline algorithm into the mapfile-writer #185

Closed
mapsforge opened this issue Nov 6, 2014 · 11 comments
Closed

Integrate the coastline algorithm into the mapfile-writer #185

mapsforge opened this issue Nov 6, 2014 · 11 comments

Comments

@mapsforge
Copy link
Owner

From thilo.mu...@gmail.com on October 14, 2011 18:53:28

We decided to move the generation of closed (water and land) polygons from the map library to the mapfile-writer. We should be able to use our existing Java implementation for that, which is currently located in the android.maps package.

As the coastline algorithm is no longer executed by the map renderer, many water areas and islands will be missing until we have fixed this.

Original issue: http://code.google.com/p/mapsforge/issues/detail?id=185

@mapsforge
Copy link
Owner Author

From thilo.mu...@gmail.com on October 24, 2011 22:14:51

This issue blocks issue #117 .

@mapsforge
Copy link
Owner Author

From jjbross@gmail.com on May 11, 2012 11:23:09

Status: Started
Owner: ookami...@gmail.com

@mapsforge
Copy link
Owner Author

From jjbross@gmail.com on September 28, 2012 07:27:53

A new, coastline-aware version of the mapfile-writer is now available from git://git.assembla.com/mapsf_coastline.git. For further details, please consult the message ("Request for comments on new coastline-aware mapfile-writer") posted to the mapsforge-dev mailinglist earlier.

@mapsforge
Copy link
Owner Author

From LudwigBr...@gmail.com on December 19, 2012 01:05:49

Is there anything that stops this integration going forward?

@mapsforge
Copy link
Owner Author

From carlito...@gmail.com on January 14, 2013 12:22:00

Where could I find the coastline-aware version of the mapfile-writer? I've got many bugs with coasts when I changed my maps from 0.2.4 to 0.3.0 :( It was working quite good before...Hopefully you can give me a solution. I've read something related with coastlines, could I add a coastline to my OSM map?

@mapsforge
Copy link
Owner Author

From ookami...@gmail.com on January 14, 2013 18:05:33

The mentioned message with explanations on the coastline-aware version on the mapsforge-dev mailinglist can be found here: https://groups.google.com/d/topic/mapsforge-dev/t-_zxJ0if-s/discussion

@mapsforge
Copy link
Owner Author

From andrewpe...@gmail.com on May 21, 2013 02:57:24

I'll be very pleased if you could help me in loooking for a problem solution. It's a problem of incorrect water draw on the map posted Oct 14, 2011. Could you please tell me when this defect can be submitted approximately?

@mapsforge
Copy link
Owner Author

From LudwigBr...@gmail.com on January 18, 2014 04:31:25

Issue 432 has been merged into this issue.

@mapsforge
Copy link
Owner Author

From jjbross@gmail.com on January 26, 2014 04:03:42

Issue 443 has been merged into this issue.

@ludwigb
Copy link
Collaborator

ludwigb commented Dec 7, 2014

There is now a companion project to mapsforge at https://github.com/mapsforge/mapsforge-mapcreator that includes the water areas correctly by implementing an approach that has been discussed on our mailing list. https://groups.google.com/forum/#!searchin/mapsforge-dev/Missing$20water$3A$20a$20work-around$20solution/mapsforge-dev/x54kHlyKiBM/jUK4oRHvHUEJ.

Using an external coast line set is preferable to duplicating this effort in mapsforge, so I am closing this issue.

@ludwigb ludwigb closed this as completed Dec 7, 2014
@devemux86
Copy link
Collaborator

Thanks Ludwig, I'll check the results.

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

No branches or pull requests

3 participants