<div dir="ltr">> <span style="font-size:12.8px">I can't comment on compare views, due to how github works. You can</span><br style="font-size:12.8px"><span style="font-size:12.8px">> open a pull request, which does allow commenting, and just note it</span><br style="font-size:12.8px"><span style="font-size:12.8px">> isn't ready to merge yet</span><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Opened a pull request: </span><span style="font-size:12.8px"><a href="https://github.com/xapian/xapian/pull/115">https://github.com/xapian/xapian/pull/115</a></span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I have made quite of changes since those previous commits. Like I mentioned in previous email,</span></div><div><span style="font-size:12.8px">I tried using another constructor to pass parameters s and delta that control the weights assigned </span></div><div><span style="font-size:12.8px">by Piv+ normalization. That prevented issues faced earlier. All previous tests pass without any serialisation</span></div><div><span style="font-size:12.8px">errors and I've</span><span style="font-size:12.8px"> added new tests as well; all passing. Will try to improve test coverage further now.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks,</span></div><div><span style="font-size:12.8px">Vivek</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px"><br></span></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 31, 2016 at 10:10 PM, James Aylett <span dir="ltr"><<a href="mailto:james-xapian@tartarus.org" target="_blank">james-xapian@tartarus.org</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"><span>On Sat, Jul 30, 2016 at 01:55:11AM +0530, Vivek Pal wrote:<br>
<br>
> I've pushed all the changes I made so far<br>
> <a href="https://github.com/xapian/xapian/compare/master...ivmarkp:piv+?diff=split&name=piv%2B" rel="noreferrer" target="_blank">https://github.com/xapian/xapian/compare/master...ivmarkp:piv+?diff=split&name=piv%2B</a><br>
><br>
> Can you please add some comments on it? Support for normalization weighting<br>
> is complete -- just these issues with serialisation.<br>
<br>
</span>I can't comment on compare views, due to how github works. You can<br>
open a pull request, which does allow commenting, and just note it<br>
isn't ready to merge yet. (That will also get it built by Travis CI,<br>
meaning people don't have to dig up the build failure from the mailing<br>
list to figure out what's going on.)<br>
<div><div><br>
J<br>
<br>
--<br>
  James Aylett, occasional trouble-maker<br>
  <a href="http://xapian.org" rel="noreferrer" target="_blank">xapian.org</a><br>
<br>
</div></div></blockquote></div><br></div></div>