{"comment":{"created_at":"2011-10-11T16:10:31Z","body":"[springmeyer] Replying to [comment:2 semprebon]:\r\n> Yeah, the problem seems to occur when mapnik is building the bounding box in feature_style_processor.hpp, in that it is basically ignoring the return value from proj_trans.forward or proj_trans.backward. When the projection fails on the layer's envelope, it ends up with a completely bogus bounding box.\r\n>\r\n\r\nReally? I feel like the bounding box of the map has always been generally right. So if a layer's got messed up data/reprojection and shapes fail the bounding box of the map still ends up including at least what was able to be parsed of that shape.  I've not look terribly closely however, so perhaps you mean that the messed up bbox at the layer level is why shapes are being aborted altogether. I think that ideal behavior is to skip rendering of features that have invalid coordinates but to expand the map extent by their set of valid coordinates. Does this match with your goals?\r\n\r\n> By checking the return values and using the corresponding map extent point if the projection fails, I at least got my original problem resolved, although it's still not correctly handling the test case I gave you.\r\n\r\nWhat was your original problem?","updated_at":"2011-10-11T16:10:31Z","url":"https://api.github.com/repos/artemp/mapnik/issues/comments/2366185","id":2366185,"user":{"avatar_url":"https://secure.gravatar.com/avatar/941f68386168a8a6b60f2529a7830af7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/artemp","id":59056,"login":"artemp"}},"action":"created","issue":{"number":308,"created_at":"2011-10-11T16:10:28Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"In the attached file, I have two shape files. The \"fail\" dataset contains Angola, Ziare, and Antartica, and only Ziare is drawn on the map. The \"works\" dataset is the same, but Antarctica has been removed. Now both Ziare and Angola appear. Both should appear when the first shape file is drawn.\r\n\r\nI'm running version 0.6.0 on OS X.","comments":4,"title":"Doesn't display feature if another (possibly invalid) feature is present","updated_at":"2011-10-11T16:10:31Z","url":"https://api.github.com/repos/artemp/mapnik/issues/308","id":1877447,"assignee":null,"milestone":{"number":13,"created_at":"2011-10-11T15:50:48Z","due_on":"1970-01-01T09:21:56Z","title":"Mapnik 2.0","creator":{"avatar_url":"https://secure.gravatar.com/avatar/941f68386168a8a6b60f2529a7830af7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/artemp","id":59056,"login":"artemp"},"url":"https://api.github.com/repos/artemp/mapnik/milestones/13","open_issues":1,"closed_issues":30,"description":"Upcoming Mapnik 2 release (first in series)\r\n\r\nMajor new features:\r\n\r\n  * Expressions support ( dynamic/data-driven symbolizers - #359, #70,#352)\r\n  * SVG Symbol support, including Gradients\r\n  * \"!MetaWriters\": Support for outputting GeoJSON of rendered features (see: MetaWriter)\r\n  * Python3 support (wiki:Python3k)\r\n  * Better raster scaling methods like lanczos, cubic, and higher quality bilinear (#656)\r\n  * New RasterColorizer for color ramping data rasters\r\n  * Support for \"interior\" placement of points and labels (aka point_on_surface) #709\r\n  * Performance improvements under high concurrent load (avoidance of proj4 locking)\r\n  * Addition of a 'mapnik-config' program to query mapnik settings\r\n  * Support for XInclude in stylesheets: #567 ([http://trac.mapnik.org/browser/trunk/tests/data/good_maps/xinclude examples])\r\n  * New plugins: Rasterlite, GEOS, templates/hello_world\r\n  * NUmerous improvements to alpha blending in raster support\r\n  * New usage of boost::spirit 2 for parsing (colors, filters, expressions)\r\n    * This means much 'slower' compiles but faster runtime code\r\n  * Addition of Jpeg reader for reading jpeg symbols and jpeg datasources (in raster.input)\r\n  * Advanced MarkersSymbolizer drawing options\r\n  * memory_datasource support in python (dynamic creation of features)\r\n  * Support for the clang++ compiler\r\n  * Fractional text halos using Freetype2 Stroker\r\n  * Ability to align pattern fills globally (#203)\r\n  * Vastly improved builds on Open Solaris\r\n  * Support for .dbf files over 2GB (#595)\r\n  * Support for triggering line breaks with '\\n' character (#584)\r\n  * Symbols are lazily cached instead of being read from filesystem many times\r\n  * Features are cached when used in more than one style consecutively (optional feature)\r\n  * Added ability to pass scale factor to renderer to scale symbols, fonts, line widths\r\n  * OGCServer removed from source tree, now at https://github.com/mapnik/OGCServer with many bugfixes: \r\n  * WKT parser\r\n\r\nFollowing features will be addressed in subsequent releases:\r\n \r\n  * Polygon hatch fills (#642)\r\n  * Map orientation\r\n  ","state":"open"},"closed_at":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/941f68386168a8a6b60f2529a7830af7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/artemp","id":59056,"login":"artemp"},"html_url":"https://github.com/artemp/mapnik/issues/308","labels":[{"name":"Agg_Renderer","url":"https://api.github.com/repos/artemp/mapnik/labels/Agg_Renderer","color":"FFFFFF"}],"state":"open"},"legacy":{"issue_id":1877447,"comment_id":2366185}}