<div dir="ltr">Thanks -- I will try to look at this within the next couple of days.<div><br></div><div>Dan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 27, 2017 at 10:40 AM, Darafei "Komяpa" Praliaskouski <span dir="ltr"><<a href="mailto:komzpa@gmail.com" target="_blank">komzpa@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Dan, <div><br></div><div>I'm sorry to have committed that change too fast. <br>I was under impression that I've addresses all of the comments you've made on the code. Now I see that I replied not to all of them, but just had those hidden by github as underlying code changed according to the comment, and shown only in PR history.<br><br>I've reopened PR so we can continue with review. I hope we can do that without a revert. (I need the math there to work well too, so it's not in my interest to ignore issues :)</div><div><br></div>I'll start thread on size_t for counters on -dev.<br><br><div class="gmail_quote"><div dir="ltr">ср, 27 дек. 2017 г. в 17:25, Daniel Baston <<a href="mailto:dbaston@gmail.com" target="_blank">dbaston@gmail.com</a>>:<br></div></div><div><div class="h5"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span style="font-size:12.8px">Given that this approaches a complete rewrite, I would have appreciated the chance to review this before you committed it to trunk. The PR currently has several open questions. This commit also makes some style changes (size_t instead of int for counters is one) that should be brought up on the dev list, so we can get a ticket to apply them globally if they are desirable.</span><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Dan</div></div><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 26, 2017 at 9:07 AM, Darafei <span dir="ltr"><<a href="mailto:komzpa@gmail.com" target="_blank">komzpa@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Author: komzpa<br>
Date: 2017-12-26 06:07:49 -0800 (Tue, 26 Dec 2017)<br>
New Revision: 16188<br>
<br>
Modified:<br>
   trunk/NEWS<br>
   trunk/doc/reference_measure.<wbr>xml<br>
   trunk/liblwgeom/cunit/cu_<wbr>algorithm.c<br>
   trunk/liblwgeom/lwgeom_<wbr>median.c<br>
Log:<br>
Weight-aware ST_GeometricMedian<br>
<br>
Weight is to be supplied as M ordinate of individual points in MultiPoint.<br>
<br>
Bring lwgeom_median.c cunit test coverage to 100%.<br>
<br>
ST_GeometricMedian(fail_if_<wbr>not_converged=false, max_iter=0)<br>
might give you weighted centroid, but that's not a promise.<br>
<br>
Closes #3954<br>
Closes <a href="https://github.com/postgis/postgis/pull/176" rel="noreferrer" target="_blank">https://github.com/postgis/<wbr>postgis/pull/176</a><br>
<br>
<br>
<br></blockquote></div></div></div></blockquote></div></div></div></div>
</blockquote></div><br></div>