<div dir="ltr">Hi Daniel, hi Thomas,<div><br></div><div style>I already made some of the changes Thomas suggested. I removed UTFGRID from MS_LAYER_TYPES and the code in mapimageio.c. Any comments is welcome, they help a lot in understanding how you want the code and functions to be/work.</div>
<div style><br></div><div style>Thanks</div><div style><br></div><div style>Francois</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/7/8 Daniel Morissette <span dir="ltr"><<a href="mailto:dmorissette@mapgears.com" target="_blank">dmorissette@mapgears.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Thomas,<br>
<br>
Thank you for your comments on the code. FYI, this François first stab at getting some UTFGrid data out, and the next step was a full code review of what he did to adjust the direction, so your comments come at the right time.<br>

<br>
Without having fully reviewed the code myself yet, I agree with most of your comments, except for the one about "creating a full-fledged outputformat with its own rendererVtable". François' outputformat does have its own vtable (at the bottom of maputfgrid.c), but it tries to wrap calls to mapagg.cpp vtable functions to rely on existing mapagg.cpp code instead of rewriting all rendering methods from scratch.<br>

<br>
Are you suggesting that maputfgrid.c should be reimplementing all methods based on direct calls to AGG? Simple polygon fills are straightforward and could have been done easily without relying on mapagg.cpp, but when it comes to rendering more complex symbology (complex symbols, dash lines, etc.) I would have thought that reusing code from an existing renderer if possible would be a better approach. Or did I misunderstand what you meant? Are you suggesting that we should ignore complex symbology and just support solid fill, solid lines and simple point symbols?<br>

<br>
Thanks<br>
<br>
Daniel<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On 13-07-06 5:50 AM, thomas bonfort wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Francois, welcome to the team!<br>
<br>
Here are a few comments regarding the code you have published in your branch:<br>
<br>
- Why are you adding a UTFGRID entry to MS_LAYER_TYPES? Layer types<br>
should not need to be extended as utgfgrid is an outputformat that<br>
should apply to existing layers.<br>
<br>
- Using aggFakeOutput  inside your UTFGridRenderer does not seem like<br>
the way to go. I believe you should be creating a full-fledged<br>
outputformat with its own rendererVtable<br>
<br>
- why the WITH_AGG_ALIASED option to cmake? You'll need a specific agg<br>
renderer for utfgrid support (which you are correct will need to be<br>
aliased). The only cmake option I would see would be to eventually be<br>
able to disable utfgrid support completely (and given that there<br>
should be no dependencies, I'm not sure that is needed).<br>
<br>
- you must not export a "rasterbuffer" and add code to mapimageio.c<br>
for saving as json. Implement your own saving functions inside the<br>
utfgrid renderervtable. (c.f. the vector vtables in mapcairo.c), and<br>
publicize you do not support a rasterbuffer.<br>
<br>
- you should not be messing with map->cellsize, scale, width, height,<br>
etc... inside mapdraw.c. Your utfRes adjustments should be private to<br>
the utfrenderer.<br>
<br>
cheers,<br>
thomas<br>
<br>
On 5 July 2013 21:36, Francois Desjarlais <<a href="mailto:fdesjarlais1@gmail.com" target="_blank">fdesjarlais1@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi everybody!<br>
Like Daniel said I'm François Desjarlais, 3rd year computer engineering<br>
student. I just wanted to add a little something to what Daniel wrote.<br>
Instead of mapbox example, you can look at my own example on<br>
<a href="http://msgsoc.mapgears.com/projet_utfgrid/testhtmlmapserver.html" target="_blank">http://msgsoc.mapgears.com/<u></u>projet_utfgrid/<u></u>testhtmlmapserver.html</a>. It's a bit<br>
buggy right now but as summer will go on it will get better.<br>
<br>
Also, I have a question. Daniel told me that Steve Lime could answer:<br>
<br>
Steve,<br>
What is UTFITEM function? I read the RFC-93 multiple times and I can't<br>
figure what it does. It's doing the same thing as UTFDATA because datas are<br>
linked with shapes. And if you don't need certain datas you just remove them<br>
from UTFDATA instead of adding syntax with UTFITEM.<br>
<br>
Thanks!<br>
Francois<br>
<br>
<br>
______________________________<u></u>_________________<br>
mapserver-dev mailing list<br>
<a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/mapserver-dev</a><br>
<br>
</blockquote>
______________________________<u></u>_________________<br>
mapserver-dev mailing list<br>
<a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/mapserver-dev</a><br>
<br>
</blockquote>
<br>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Daniel Morissette<br>
<a href="http://www.mapgears.com/" target="_blank">http://www.mapgears.com/</a><br>
Provider of Professional MapServer Support since 2000</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<u></u>_________________<br>
mapserver-dev mailing list<br>
<a href="mailto:mapserver-dev@lists.osgeo.org" target="_blank">mapserver-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/<u></u>mailman/listinfo/mapserver-dev</a><br>
</div></div></blockquote></div><br></div>