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

Support for multipoint type not working #458

Closed
artemp opened this issue Oct 11, 2011 · 7 comments
Closed

Support for multipoint type not working #458

artemp opened this issue Oct 11, 2011 · 7 comments

Comments

@artemp
Copy link
Member

artemp commented Oct 11, 2011

see: http://lists.berlios.de/pipermail/mapnik-users/2009-June/002007.html

reported for shapefiles only so far.

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] Looks like we 1) don't catch multipoints in the shape plugin and 2) get caught in an infinite loop where we never reach the end of the file right here: http://trac.mapnik.org/browser/trunk/plugins/input/shape/shape_featureset.cpp#L106

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] I also noticed that the ogr pluging can read multipoint only if the 'multiple_geometries" flag is passed - which, if I understand correctly 'explodes' each point into its own geometry. If that flag is not passed then the points are lost/ignored.

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] see r1458 where I added workaround for ogr driver.

I still consider this a blocker until we avoid the infinite loop in the shapefile plugin.

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] pushing to 0.7.0

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] pushing back to 0.7.0 as I've just run into another situation where this same loop is not exited.

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[springmeyer] r1559 fixed multipoint reading, but it looks like we still are missing proper handling of multipointzm. will attach testing shapefile next.

@artemp
Copy link
Member Author

artemp commented Oct 11, 2011

[artem] Fixed in 0.7 branch. Both multipoint and multipointzm (with or without index) are handled correctly.

@artemp artemp closed this as completed Oct 11, 2011
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

1 participant