[Xapian-devel] Merging of the TfIdf patch

[aarsh shah]

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.

-Regards
-Aarsh


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.
>
> -Regards
> -Aarsh
>
>
>
> 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
>> do
>> > 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:
>>
>> http://people.csail.mit.edu/jrennie/ecoc-svm/smart.html
>>
>> But TfIdfWeight requires them to be in upper-case, while SMART documents
>> them as being lowercase.  Is there a reason for that?
>>
>> Cheers,
>>     Olly
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.xapian.org/pipermail/xapian-devel/attachments/20130404/ca9115bb/attachment.htm>