You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@predictionio.apache.org by ta...@apache.org on 2017/07/15 16:57:30 UTC

incubator-predictionio git commit: Fix use of case class in classification examples

Repository: incubator-predictionio
Updated Branches:
  refs/heads/develop 4e2abe5c5 -> fdcd9218c


Fix use of case class in classification examples

Closes #410


Project: http://git-wip-us.apache.org/repos/asf/incubator-predictionio/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-predictionio/commit/fdcd9218
Tree: http://git-wip-us.apache.org/repos/asf/incubator-predictionio/tree/fdcd9218
Diff: http://git-wip-us.apache.org/repos/asf/incubator-predictionio/diff/fdcd9218

Branch: refs/heads/develop
Commit: fdcd9218c3758bd1ee1c9811fc6950e7a7aaa400
Parents: 4e2abe5
Author: Naoki Takezoe <ta...@apache.org>
Authored: Sun Jul 16 01:56:45 2017 +0900
Committer: Naoki Takezoe <ta...@apache.org>
Committed: Sun Jul 16 01:56:45 2017 +0900

----------------------------------------------------------------------
 .../src/main/scala/DataSource.scala             |  2 +-
 .../add-algorithm/src/main/scala/Engine.scala   | 22 +++++++++---------
 .../src/main/scala/Evaluation.scala             |  2 +-
 .../src/main/scala/NaiveBayesAlgorithm.scala    |  2 +-
 .../src/main/scala/PrecisionEvaluation.scala    |  2 +-
 .../src/main/scala/RandomForestAlgorithm.scala  |  2 +-
 .../src/main/scala/DataSource.scala             |  2 +-
 .../src/main/scala/Engine.scala                 | 24 ++++++++++----------
 .../src/main/scala/Evaluation.scala             |  2 +-
 .../src/main/scala/NaiveBayesAlgorithm.scala    |  2 +-
 .../src/main/scala/PrecisionEvaluation.scala    |  2 +-
 11 files changed, 32 insertions(+), 32 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/DataSource.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/DataSource.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/DataSource.scala
index d800282..355a5a8 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/DataSource.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/DataSource.scala
@@ -119,7 +119,7 @@ class DataSource(val dsp: DataSourceParams)
         new TrainingData(trainingPoints),
         new EmptyEvaluationInfo(),
         testingPoints.map {
-          p => (new Query(p.features(0), p.features(1), p.features(2)), new ActualResult(p.label))
+          p => (Query(p.features(0), p.features(1), p.features(2)), ActualResult(p.label))
         }
       )
     }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/Engine.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/Engine.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/Engine.scala
index 1e26024..63a11c7 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/Engine.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/Engine.scala
@@ -20,19 +20,19 @@ package org.apache.predictionio.examples.classification
 import org.apache.predictionio.controller.EngineFactory
 import org.apache.predictionio.controller.Engine
 
-class Query(
-  val attr0 : Double,
-  val attr1 : Double,
-  val attr2 : Double
-) extends Serializable
+case class Query(
+  attr0 : Double,
+  attr1 : Double,
+  attr2 : Double
+)
 
-class PredictedResult(
-  val label: Double
-) extends Serializable
+case class PredictedResult(
+  label: Double
+)
 
-class ActualResult(
-  val label: Double
-) extends Serializable
+case class ActualResult(
+  label: Double
+)
 
 object ClassificationEngine extends EngineFactory {
   def apply() = {

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/Evaluation.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/Evaluation.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/Evaluation.scala
index 3bc3399..53f9229 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/Evaluation.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/Evaluation.scala
@@ -31,7 +31,7 @@ case class Accuracy()
 
 object AccuracyEvaluation extends Evaluation {
   // Define Engine and Metric used in Evaluation
-  engineMetric = (ClassificationEngine(), new Accuracy())
+  engineMetric = (ClassificationEngine(), Accuracy())
 }
 
 object EngineParamsList extends EngineParamsGenerator {

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/NaiveBayesAlgorithm.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/NaiveBayesAlgorithm.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/NaiveBayesAlgorithm.scala
index ce7bda0..0ac5e5b 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/NaiveBayesAlgorithm.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/NaiveBayesAlgorithm.scala
@@ -51,7 +51,7 @@ class NaiveBayesAlgorithm(val ap: AlgorithmParams)
     val label = model.predict(Vectors.dense(
       Array(query.attr0, query.attr1, query.attr2)
     ))
-    new PredictedResult(label)
+    PredictedResult(label)
   }
 
 }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/PrecisionEvaluation.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/PrecisionEvaluation.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/PrecisionEvaluation.scala
index cd91a1e..019e2d8 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/PrecisionEvaluation.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/PrecisionEvaluation.scala
@@ -40,5 +40,5 @@ case class Precision(label: Double)
 }
 
 object PrecisionEvaluation extends Evaluation {
-  engineMetric = (ClassificationEngine(), new Precision(label = 1.0))
+  engineMetric = (ClassificationEngine(), Precision(label = 1.0))
 }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/add-algorithm/src/main/scala/RandomForestAlgorithm.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/add-algorithm/src/main/scala/RandomForestAlgorithm.scala b/examples/scala-parallel-classification/add-algorithm/src/main/scala/RandomForestAlgorithm.scala
index 5b472e9..76dd7ca 100644
--- a/examples/scala-parallel-classification/add-algorithm/src/main/scala/RandomForestAlgorithm.scala
+++ b/examples/scala-parallel-classification/add-algorithm/src/main/scala/RandomForestAlgorithm.scala
@@ -64,7 +64,7 @@ class RandomForestAlgorithm(val ap: RandomForestAlgorithmParams) // CHANGED
     val label = model.predict(Vectors.dense(
       Array(query.attr0, query.attr1, query.attr2)
     ))
-    new PredictedResult(label)
+    PredictedResult(label)
   }
 
 }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/DataSource.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/DataSource.scala b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/DataSource.scala
index ef28488..cd7519d 100644
--- a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/DataSource.scala
+++ b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/DataSource.scala
@@ -126,7 +126,7 @@ class DataSource(val dsp: DataSourceParams)
         new EmptyEvaluationInfo(),
         testingPoints.map {
           // MODIFIED
-          p => (new Query(p.features(0), p.features(1), p.features(2), p.features(3)), new ActualResult(p.label))
+          p => (Query(p.features(0), p.features(1), p.features(2), p.features(3)), ActualResult(p.label))
         }
       )
     }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Engine.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Engine.scala b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Engine.scala
index ddbf7dd..72b62a1 100644
--- a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Engine.scala
+++ b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Engine.scala
@@ -21,20 +21,20 @@ import org.apache.predictionio.controller.EngineFactory
 import org.apache.predictionio.controller.Engine
 
 // MODIFIED
-class Query(
-  val featureA : Double,
-  val featureB : Double,
-  val featureC : Double,
-  val featureD : Double
-) extends Serializable
+case class Query(
+  featureA : Double,
+  featureB : Double,
+  featureC : Double,
+  featureD : Double
+)
 
-class PredictedResult(
-  val label: Double
-) extends Serializable
+case class PredictedResult(
+  label: Double
+)
 
-class ActualResult(
-  val label: Double
-) extends Serializable
+case class ActualResult(
+  label: Double
+)
 
 object ClassificationEngine extends EngineFactory {
   def apply() = {

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Evaluation.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Evaluation.scala b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Evaluation.scala
index 3bc3399..53f9229 100644
--- a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Evaluation.scala
+++ b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/Evaluation.scala
@@ -31,7 +31,7 @@ case class Accuracy()
 
 object AccuracyEvaluation extends Evaluation {
   // Define Engine and Metric used in Evaluation
-  engineMetric = (ClassificationEngine(), new Accuracy())
+  engineMetric = (ClassificationEngine(), Accuracy())
 }
 
 object EngineParamsList extends EngineParamsGenerator {

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/NaiveBayesAlgorithm.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/NaiveBayesAlgorithm.scala b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/NaiveBayesAlgorithm.scala
index f12d9a3..6625551 100644
--- a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/NaiveBayesAlgorithm.scala
+++ b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/NaiveBayesAlgorithm.scala
@@ -52,7 +52,7 @@ class NaiveBayesAlgorithm(val ap: AlgorithmParams)
       // MODIFIED
       Array(query.featureA, query.featureB, query.featureC, query.featureD)
     ))
-    new PredictedResult(label)
+    PredictedResult(label)
   }
 
 }

http://git-wip-us.apache.org/repos/asf/incubator-predictionio/blob/fdcd9218/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/PrecisionEvaluation.scala
----------------------------------------------------------------------
diff --git a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/PrecisionEvaluation.scala b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/PrecisionEvaluation.scala
index cd91a1e..019e2d8 100644
--- a/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/PrecisionEvaluation.scala
+++ b/examples/scala-parallel-classification/reading-custom-properties/src/main/scala/PrecisionEvaluation.scala
@@ -40,5 +40,5 @@ case class Precision(label: Double)
 }
 
 object PrecisionEvaluation extends Evaluation {
-  engineMetric = (ClassificationEngine(), new Precision(label = 1.0))
+  engineMetric = (ClassificationEngine(), Precision(label = 1.0))
 }