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

shield symbolizer not on road #1350

Closed
ramunasd opened this issue Jul 27, 2012 · 22 comments
Closed

shield symbolizer not on road #1350

ramunasd opened this issue Jul 27, 2012 · 22 comments
Assignees
Labels
Milestone

Comments

@ramunasd
Copy link
Contributor

bug

Data is correct, style is not changed. Don't know after which commits this happens, i'm using latest version (03422de)

@springmeyer
Copy link
Member

Thanks for the report. Would it be possible for you to isolate that geometry to produce a simple testcase?

@ramunasd
Copy link
Contributor Author

Data is here http://openmap.lt/#l=54.91423,25.77223,15,L
You can browse for more examples and download dierctly from OSM.

@springmeyer
Copy link
Member

Can you still replicate if you step back to 07ec5c3 ?

@springmeyer
Copy link
Member

c7960a2 also might be suspect, so try with the one proceeding: 8cf9a5a

@ramunasd
Copy link
Contributor Author

8cf9a5a renders good

@springmeyer
Copy link
Member

thanks, and if you move ahead one commit, to c7960a2, can you replicate the problem again?

@ramunasd
Copy link
Contributor Author

c7960a2 looks good

@springmeyer
Copy link
Member

okay, going forward, f6fa57d should work, but perhaps 7b10400 does not?

@springmeyer
Copy link
Member

btw, I am unable to replicate this with the osm.xml (master) and mapnik (master) and data downloaded for your example region via http://api.openstreetmap.org/api/0.6/map?bbox=25.663,54.8419,25.9194,54.9695

@ramunasd
Copy link
Contributor Author

f6fa57d renders good, 7b10400 cannot compile:

In file included from include/mapnik/geometry.hpp:27:0,
                 from include/mapnik/feature.hpp:29,
                 from include/mapnik/parse_path.hpp:29,
                 from include/mapnik/symbolizer.hpp:28,
                 from include/mapnik/building_symbolizer.hpp:28,
                 from include/mapnik/rule.hpp:27,
                 from include/mapnik/feature_type_style.hpp:27,
                 from bindings/python/mapnik_style.cpp:29:
include/mapnik/vertex_vector.hpp:116:5: error: declaration of âunsigned int mapnik::vertex_vector<T>::vertex(unsigned int, mapnik::vertex_vector<T>::coord_type_, mapnik::vertex_vector<T>::coord_type_) constâ [-fpermissive]
include/mapnik/vertex.hpp:44:8: error: changes meaning of âvertexâ from âstruct mapnik::vertex<T, 2>â [-fpermissive]
scons: **\* [bindings/python/mapnik_style.os] Error 1
scons: building terminated because of errors.
make: **\* [mapnik] Error 2

@lightmare
Copy link
Contributor

@ramunasd try to cherry-pick 4625e50 , could fix compilation

@ramunasd
Copy link
Contributor Author

Updated to latest 19ed871 and after full compilation mapnik renders good.
Is this some cross linking issue?

@ramunasd ramunasd reopened this Jul 30, 2012
@ramunasd
Copy link
Contributor Author

Something still goes not very well, 19ed871 renders shields not on road - http://osmlt.openmap.lt/14/9366/5185.png

@artemp artemp closed this as completed Jul 30, 2012
@artemp artemp reopened this Jul 30, 2012
@artemp
Copy link
Member

artemp commented Jul 30, 2012

@ramunasd - could you post relevant part of your style, thanks!

@ramunasd
Copy link
Contributor Author

@artemp
Copy link
Member

artemp commented Jul 30, 2012

@ramunasd - thanks!

@springmeyer - well, placement="line" in ShieldSymbolizer fixes this but this is not the end of story -> symbolizer_helpers are in a real big mess at the moment.

@artemp
Copy link
Member

artemp commented Jul 30, 2012

@ramunasd - previous behaviour is restored in 822531a

Note: ShieldSymbolizer placement defaults to placement="point"

@ramunasd
Copy link
Contributor Author

@artemp 822531a renders good, but many tests fails

@artemp
Copy link
Member

artemp commented Jul 31, 2012

@ramunasd - what tests?

@ramunasd
Copy link
Contributor Author

visual tests

@artemp
Copy link
Member

artemp commented Jul 31, 2012

@ramunasd - ok, thanks. BTW, all visual tests are passing for me.

@springmeyer
Copy link
Member

this was fixed fully in 6f17428

springmeyer pushed a commit that referenced this issue Nov 29, 2012
springmeyer pushed a commit that referenced this issue Dec 3, 2012
… for cairo/grid renderers to ensure proper placement on lines (refs #1604,#1350,#1607)
PetrDlouhy pushed a commit to PetrDlouhy/mapnik that referenced this issue Aug 22, 2013
PetrDlouhy pushed a commit to PetrDlouhy/mapnik that referenced this issue Aug 22, 2013
…29d768 for cairo/grid renderers to ensure proper placement on lines (refs mapnik#1604,mapnik#1350,mapnik#1607)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants