You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2017/09/18 16:25:12 UTC

[38/50] lucene-solr:feature/autoscaling_72: SOLR-11351: make LTRScoringModel model more extensible

SOLR-11351: make LTRScoringModel model more extensible


Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/0c831b99
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/0c831b99
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/0c831b99

Branch: refs/heads/feature/autoscaling_72
Commit: 0c831b9970550716e800478fe77ec05eecc05d57
Parents: 1544839
Author: Christine Poerschke <cp...@apache.org>
Authored: Mon Sep 11 12:35:27 2017 -0700
Committer: Christine Poerschke <cp...@apache.org>
Committed: Mon Sep 11 12:47:35 2017 -0700

----------------------------------------------------------------------
 solr/CHANGES.txt                                                 | 2 ++
 .../ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java  | 4 +++-
 2 files changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/0c831b99/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 519852e..5d243c5 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -203,6 +203,8 @@ Other Changes
 
 * SOLR-11132: Refactor common getSortField logic in various FieldTypes (Jason Gerlowski, hossman)
 
+* SOLR-11351: Make LTRScoringModel model more extensible. (Christine Poerschke)
+
 ==================  7.0.0 ==================
 
 Versions of Major Components

http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/0c831b99/solr/contrib/ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java
----------------------------------------------------------------------
diff --git a/solr/contrib/ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java b/solr/contrib/ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java
index 9edcfe5..2b4f575 100644
--- a/solr/contrib/ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java
+++ b/solr/contrib/ltr/src/java/org/apache/solr/ltr/model/LTRScoringModel.java
@@ -80,7 +80,7 @@ public abstract class LTRScoringModel {
   protected final List<Feature> features;
   private final List<Feature> allFeatures;
   private final Map<String,Object> params;
-  private final List<Normalizer> norms;
+  protected final List<Normalizer> norms;
 
   public static LTRScoringModel getInstance(SolrResourceLoader solrResourceLoader,
       String className, String name, List<Feature> features,
@@ -123,6 +123,8 @@ public abstract class LTRScoringModel {
    * {@link ModelException} if they do not make sense.
    */
   protected void validate() throws ModelException {
+    final List<Feature> features = getFeatures();
+    final List<Normalizer> norms = getNorms();
     if (features.isEmpty()) {
       throw new ModelException("no features declared for model "+name);
     }