[postgis-tickets] r16213 - Fix KMeans initialization issue that lost clusters sometimes.

[Darafei]

Author: komzpa
Date: 2018-01-02 06:06:27 -0800 (Tue, 02 Jan 2018)
New Revision: 16213

Modified:
   branches/2.4/NEWS
   branches/2.4/liblwgeom/lwkmeans.c
Log:
Fix KMeans initialization issue that lost clusters sometimes.

Closes #3965
Closes https://github.com/postgis/postgis/pull/179


Modified: branches/2.4/NEWS
===================================================================
--- branches/2.4/NEWS	2018-01-02 13:59:19 UTC (rev 16212)
+++ branches/2.4/NEWS	2018-01-02 14:06:27 UTC (rev 16213)
@@ -1,11 +1,13 @@
 PostGIS 2.4.3
-2017/xx/xx
+2018/xx/xx
 
  * Bug fixes *
   - #3713, Support encodings that happen to output a '\' character
   - #3827, Set configure default to not do interrupt testing,
-          was causing false negatives for many people
+           was causing false negatives for many people
   - #3930, Minimum bounding circle issues on 32-bit platforms
+  - #3965, ST_ClusterKMeans used to lose some clusters on initialization
+           (Darafei Praliaskouski)
 
  * Enhancements *
   - #3944, Update to EPSG register v9.2 (Even Rouault)

Modified: branches/2.4/liblwgeom/lwkmeans.c
===================================================================
--- branches/2.4/liblwgeom/lwkmeans.c	2018-01-02 13:59:19 UTC (rev 16212)
+++ branches/2.4/liblwgeom/lwkmeans.c	2018-01-02 14:06:27 UTC (rev 16213)
@@ -201,6 +201,7 @@
 			if (seen[j] == closest)
 			{
 				closest = (closest + 1) % config.num_objs;
+				j = 0;
 			}
 			else
 			{