<div dir="ltr">Hey,<div style>I have ported a part of VoronoiDiagramBuilder class. It will be over soon. So, I will start writing the test cases for the methods and classes that i have ported so far, in the next week.</div>
<div style><br></div><div style>Thanks,</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 26 June 2013 22:10, Sandro Santilli <span dir="ltr"><<a href="mailto:strk@keybit.net" target="_blank">strk@keybit.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for the report Vishal.<br>
<br>
One thing: before porting more classes I suggest you port or write<br>
the unit tests associated with each class you ported already.<br>
<br>
Same thing for new methods (possibly easier as a unit test for the<br>
class may already be available).<br>
<br>
Every portion of code + docs (doxygen) + test is a good candidate for<br>
merging into the official repository.<br>
<br>
I'm a fan of incremental work :)<br>
<br>
--strk;<br>
<br>
On Sat, Jun 22, 2013 at 12:17:52AM +0530, vishal tiwari wrote:<br>
> *Week 1*<br>
><br>
> *1. Work Done :*<br>
><br>
>    - Have ported the methods that are in<br>
<div class="im">>    geos/triangualte/quadedge/QuadEdgeSubdivion.cpp. The methods are as such:<br>
</div>>       - getVoronoiDiagram : Gets the cells in the Voronoi diagram for a<br>
>       triangulation.<br>
>       - getVoronoiCellPolygons : Gets the polygons for the Voronoi Cells<br>
>       for a triangulation.<br>
>       - getVoronoiCellPolygon : Gets the Voronoi cell around a site<br>
<div class="im">>       specified by the origin of a<br>
</div>> QuadEdge?<<a href="http://trac.osgeo.org/geos/wiki/QuadEdge" target="_blank">http://trac.osgeo.org/geos/wiki/QuadEdge</a>><br>
>       .<br>
>       - getVertexUniqueEdges : Gets a collection of<br>
> QuadEdges?<<a href="http://trac.osgeo.org/geos/wiki/QuadEdges" target="_blank">http://trac.osgeo.org/geos/wiki/QuadEdges</a>> whose<br>
<div class="im">>       origin vertices are a unique set which includes all vertices in the<br>
>       subdivision. The frame vertices can be included if required.<br>
</div>>    - Class TriangleCircumcentreVisitor?<<a href="http://trac.osgeo.org/geos/wiki/TriangleCircumcentreVisitor" target="_blank">http://trac.osgeo.org/geos/wiki/TriangleCircumcentreVisitor</a>><br>
> was<br>
>    also ported.<br>
>    - More methods like geos::geom::Triangle::circumcentre, and<br>
<div class="im">>    geos::geom::closeRing that were required in the above functions were also<br>
>    ported.<br>
><br>
</div>> *2. Problem Faced :*<br>
><br>
>    - I am having some silly problems during compilation which i am working<br>
<div class="im">>    out with strk, it will be solved soon.<br>
><br>
</div>> *3. Work to be done in the next week :*<br>
><br>
>    - Solve the errors that are coming as of now.<br>
>    - Port the classes left, and see the working of the methods ported.<br>
>    - Start with the port of class<br>
> VoronoiDiagramBuilder?<<a href="http://trac.osgeo.org/geos/wiki/VoronoiDiagramBuilder" target="_blank">http://trac.osgeo.org/geos/wiki/VoronoiDiagramBuilder</a>><br>
>    .<br>
><br>
><br>
> Regards,<br>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org">geos-devel@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/geos-devel" target="_blank">http://lists.osgeo.org/mailman/listinfo/geos-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Vishal Tiwari<div>Undergraduate</div><div>Computer Science and Engineering</div><div>Lab for Spatial Informatics</div><div>IIIT-Hyderabad</div></div>

</div>