<div dir="ltr">In view of Markus M explanation, +1 for RC2 today rather than tomorrow.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 16 January 2015 at 20:31, Markus Metz <span dir="ltr"><<a href="mailto:markus.metz.giswork@gmail.com" target="_blank">markus.metz.giswork@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Jan 16, 2015 at 1:27 PM, Moritz Lennert<br>
<<a href="mailto:mlennert@club.worldonline.be">mlennert@club.worldonline.be</a>> wrote:<br>
> On 16/01/15 12:15, Martin Landa wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> 2015-01-16 12:13 GMT+01:00 Markus Metz <<a href="mailto:markus.metz.giswork@gmail.com">markus.metz.giswork@gmail.com</a>>:<br>
>><br>
>>> Can we please get RC2 out soon? In the last days I have fixed numerous<br>
>>> bugs in the vector library and changed/restored the basic vector IO<br>
>>> interface, it is now more similar to G6 and it needed some code clean<br>
>>> up.<br>
><br>
><br>
> Thanks for all this work ! Could you explain a bit more on what types of<br>
> bugs this fixed ?<br>
<br>
</span>The first set of bugs was related to vector topology.<br>
<br>
Bug 1 affected point-in-polygon tests, a basic geometry function. The<br>
affected functions were Vect_point_in_area(),<br>
Vect_point_in_area_outer_ring() and Vect_point_in_island() which<br>
returned sometimes the wrong result (point outside instead of inside<br>
or vice versa). This in turn affected the functions<br>
Vect_attach_centroids(), Vect_attach_isle() and Vect_attach_isles()<br>
which are needed to update topology when boundaries are added, deleted<br>
or modified.<br>
<br>
Bug 2 was in Vect_attach_centroids() andVect_attach_isles(). Centroids<br>
and isles were not properly reattached when boundaries are added,<br>
deleted or modified. These bugs still need to be fixed in G6.<br>
<br>
Bugs 1 + 2 meant that (re)attaching centroids and isles during vector<br>
modification was not working well, a fairly important feature for<br>
modifying vector topology.<br>
<br>
The modifications related to basic vector IO fixed some bugs<br>
introduced in G7. Some functions were only working with topology, even<br>
though equivalent functions not requiring topology are available. A<br>
newly introduced test prevented access to the non-topological<br>
variants. Further on, some function definitions were changed such that<br>
new arguments were introduced that were not used/not needed. I have<br>
syncronized the IO interface and updated the documentation. It is now<br>
more similar to G6 and some functions have become non-topological<br>
equivalents (interesting for large point clouds).<br>
<span class=""><br>
><br>
>><br>
>> I agree, but would suggest to wait at least one/two week(s), probably<br>
>> more bugfixes will be collected.<br>
>><br>
><br>
> As these seem to be modifications in fundamental library functions, I would<br>
> plead for getting RC2 out more quickly than foreseen, i.e. I'd plead for 1<br>
> week, not 2. That way these modifications will get a bit more testing before<br>
> the final release.<br>
<br>
</span>I would plead for 1 day rather than 1 week.<br>
<span class=""><br>
><br>
> This is one example of why the proposed release procedure [1] contains this:<br>
><br>
> "Any backports during the soft freeze should be announced on the developers<br>
> mailing list with 24 hours advance to allow possible discussion."<br>
><br>
> Maybe this should be extended to "Any backports or extensive bug fixes<br>
> during..." ?<br>
><br>
> If these changes had been announced, we could have delayed RC1 for a few<br>
> days...<br>
<br>
</span>I discovered the bugs only in the last days and tried to get them<br>
fixed as soon as possible, but some of the bugs were rather obscure<br>
and I had no idea how quickly I would be able to find their reason and<br>
fix them. The fixes are all thoroughly tested (I guess I have never<br>
before tested vector topology so thoroughly...).<br>
<br>
Markus M<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Moritz<br>
><br>
> [1] <a href="http://trac.osgeo.org/grass/wiki/RFC/4_ReleaseProcedure" target="_blank">http://trac.osgeo.org/grass/wiki/RFC/4_ReleaseProcedure</a><br>
_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">----</div>
</div>