<p dir="ltr">^I'm now looking to start writing documentation for BM25+ weighting function. Would it be a good idea to start already and alongside make changes to the PR as required after its review? </p>
<p dir="ltr">Thanks,<br>
Vivek</p>
<div class="gmail_quote">On Jun 11, 2016 3:07 AM, "Vivek Pal" <<a href="mailto:vivekpal.dtu@gmail.com">vivekpal.dtu@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello everyone,<div><br></div><div>I have been working on adding support for BM25+ weighting function from the last couple of weeks. Initially, I considered modifying bm25weight.cc to add support for BM25+ function without disturbing functionalities of BM25. But that didn't work out very well. A day or two was spent trying to refactor and debug the same code.</div><div><br></div><div>Later, I took another approach following the suggestions from James and implemented a new sub class (BM25PlusWeight) for BM25+ weighting function. All tests are passing. I've have pushed the changes to update the pull request: <a href="https://github.com/xapian/xapian/pull/104" target="_blank">https://github.com/xapian/xapian/pull/104</a> </div><div><br></div><div>Next, I'll be refactoring the code, improving test coverage in bm25plusweight.cc. and writing the documentation for BM25+.  By the mid-end of the next week, I'll begin working on the implementation next weighting function as per project timeline i.e. PL2+ weighting function.</div><div><br></div><div>Regards,</div><div>Vivek</div></div>
</blockquote></div>