Hi, In my explorations of Omega codebase, I have found that Omega is currently missing support for newer weighting schemes added in 1.4.1 (BM25+, PL2+, Dir+). I'd submit a PR addressing that but as I think I might be missing something so just wanted to check if there's a particular reason for that? P.S. Finally back after a long week. Been eagerly waiting for a weekend since the application period ended. (last month of each semester has always been engaging actually). Thanks, Vivek
On 8 Apr 2017, at 12:41, Vivek Pal <vivekpal.dtu at gmail.com> wrote:> In my explorations of Omega codebase, I have found that Omega is currently > missing support for newer weighting schemes added in 1.4.1 (BM25+, PL2+, > Dir+). I'd submit a PR addressing that but as I think I might be missing > something so just wanted to check if there's a particular reason for that?Hi, Vivek — there isn't any particular reason that I'm aware of. It's probably worth pointing (in the omegascript documentation) to the part of the getting started guide which talks about the different weighting schemes and what they do [1]. (It doesn't cover Dir+, but it does talk about BM25+ and PL2+.) [1] https://getting-started-with-xapian.readthedocs.io/en/latest/howtos/weighting_scheme.html J -- James Aylett devfort.com — spacelog.org — tartarus.org/james/
> Hi, Vivek — there isn't any particular reason that I'm aware of. It's > probably worth pointing (in the omegascript documentation) to the part of > the getting started guide which talks about the different weighting schemesIf there isn't any reason then I'd like to send in a patch adding support for those weighting schemes in weight.cc and I agree omegascript documentation should also be updated to point to the weighting scheme section in the getting started guide ($set command to be specific if I'm not wrong) which I'll try to incorporate in the same PR. Additionally, omegascript documentation will also require updating to reflect support for newer weighting schemes in $set command.> It doesn't cover Dir+, but it does talk about BM25+ and PL2+.Actually, Dir+ is mentioned under LMWeight weighting scheme section in the last paragraph, although in brief. Thanks, Vivek