Hi Roopa,

If you look at the diff: 

https://github.com/apache/lucene-solr/pull/162/files

I didn't change much in SolrIndexSearcher, you can try to skip the file when 
applying the patch and redo the changes after.

Alternatively, the feature branch is available here: 

https://github.com/bloomberg/lucene-solr/commits/master-solr-8776

you could try to merge with that or cheery-pick my changes. 

Are you interested in reranking the groups or also in reranking the documents 
inside each group? 

Cheers,
Diego


From: solr-user@lucene.apache.org At: 12/09/17 19:07:25To:  
solr-user@lucene.apache.org
Subject: Re: Learning to Rank (LTR) with grouping

Hi I tried to apply this JIRA SOLR-8776 as a patch as this feature is
critical.

Here are the steps I took on my mac:

On branch branch_6_5

Your branch is up-to-date with 'origin/branch_6_5'

patch -p1 -i 162.patch --dry-run


I am getting Failures for certain Hunks

Example:

patching file
solr/core/src/java/org/apache/solr/search/SolrIndexSearcher.java

Hunk #1 FAILED at 1471.


Could you please give your input on how to apply this ticket as a patch for
branch_6_5 ?


Thank you,

Roopa

On Fri, Dec 8, 2017 at 6:52 PM, Roopa Rao <roop...@gmail.com> wrote:

> Hi Diego,
>
> Thank you, I will look into this and see how I could patch this.
>
> Thank you for your quick response,
> Roopa
>
>
> On Fri, Dec 8, 2017 at 5:44 PM, Diego Ceccarelli <
> diego.ceccare...@gmail.com> wrote:
>
>> Hi Roopa,
>>
>> LTR is implemented using RankQuery, and at the moment grouping doens't
>> support RankQuery.
>> I opened a jira item time ago
>> (https://issues.apache.org/jira/browse/SOLR-8776) and I would be happy
>> to receive feedback on that.  You can find the code here
>> https://github.com/apache/lucene-solr/pull/162.
>>
>> Cheers,
>> diego
>>
>> On Fri, Dec 8, 2017 at 9:15 PM, Roopa Rao <roop...@gmail.com> wrote:
>> > Hi,
>> >
>> > I am using grouping and LTR together and the results are not getting
>> > re-rank as it does without grouping.
>> >
>> > I am passing &rq parameter.
>> >
>> > Does LTR work with grouping on?
>> > Solr version 6.5
>> >
>> > Thank you,
>> > Roopa
>>
>
>


Reply via email to