cammiemw commented on a change in pull request #2097:
URL: https://github.com/apache/lucene-solr/pull/2097#discussion_r532933758



##########
File path: lucene/core/src/java/org/apache/lucene/search/IndriScorer.java
##########
@@ -0,0 +1,37 @@
+package org.apache.lucene.search;
+
+import java.io.IOException;
+
+/**
+ * The Indri parent scorer that stores the boost so that 
+ * IndriScorers can use the boost outside of the term.
+ *
+ */
+abstract public class IndriScorer extends Scorer {
+
+       private float boost;

Review comment:
       I did this because I apply the boost in the scorer rather than in the 
similarity (such as in LMDirichletSimilarity), and I divide by the sum of the 
boosts.  I originally did this to exactly match the Indri scores; however, this 
is not a huge priority.  I don't have much issue with dropping this if it 
doesn't fit in the Lucene workflow well.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to