jpountz commented on code in PR #13757:
URL: https://github.com/apache/lucene/pull/13757#discussion_r1755522774


##########
lucene/core/src/java/org/apache/lucene/search/similarities/DFRSimilarity.java:
##########
@@ -94,6 +94,26 @@ public class DFRSimilarity extends SimilarityBase {
    */
   public DFRSimilarity(
       BasicModel basicModel, AfterEffect afterEffect, Normalization 
normalization) {
+    this(basicModel, afterEffect, normalization, true);
+  }
+
+  /**
+   * Creates DFRSimilarity from the three components.
+   *
+   * <p>Note that <code>null</code> values are not allowed: if you want no 
normalization, instead
+   * pass {@link NoNormalization}.
+   *
+   * @param basicModel Basic model of information content
+   * @param afterEffect First normalization of information gain
+   * @param normalization Second (length) normalization
+   * @param discountOverlaps True if overlap tokens (tokens with a position of 
increment of zero)

Review Comment:
   I believe that the end of the sentence is missing? ("are discounted")



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

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