[Xapian-devel] Merging of the TfIdf patch
aarshkshah1992 at gmail.com
Wed Apr 3 21:39:12 BST 2013
Hello .Is any additional normalization/code needed before the code for
TfIdf can be merged ? Please do let me know.I want to get the code merged
as soon as we can.
On Wed, Apr 3, 2013 at 1:04 AM, aarsh shah <aarshkshah1992 at gmail.com> wrote:
> No Olly,there is no specific reason for that .I just decided to go with
> uppercase strings as that makes them easily visible and conspicuous when
> reading the documentation and also while reading the code.However, the case
> wont make any difference to the code.
> On Tue, Apr 2, 2013 at 8:39 AM, Olly Betts <olly at survex.com> wrote:
>> On Tue, Mar 26, 2013 at 09:29:53PM +0530, aarsh shah wrote:
>> > Hello Guys. I have updated the code,tests,documentation,makefile entries
>> > and the registry entry of the* *TfIdf patch as per the feedback.Please
>> > let me know if any additional changes are required before the patch
>> can be
>> > merged,
>> One question - the 3 character codes that TfIdfWeight takes are (I
>> assume) modelled on those used by SMART:
>> But TfIdfWeight requires them to be in upper-case, while SMART documents
>> them as being lowercase. Is there a reason for that?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Xapian-devel