add require for one-based in loadLIBSVM

hhbyyh · hhbyyh · commit 5bd1f9a853db · 2015-05-31T19:13:23.000+08:00
diff --git a/mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala b/mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala
@@ -82,6 +82,8 @@ object MLUtils {
           val value = indexAndValue(1).toDouble
           (index, value)
         }.unzip
+        require(indices.size == 0 || indices(0) >= 0,
+          "indices should be one-based in LIBSVM format")
         (label, indices.toArray, values.toArray)
       }
 

Original file line number	Diff line number	Diff line change
`@@ -82,6 +82,8 @@ object MLUtils {`
`82`	`82`	`val value = indexAndValue(1).toDouble`
`83`	`83`	`(index, value)`
`84`	`84`	`}.unzip`
	`85`	`+ require(indices.size == 0 \|\| indices(0) >= 0,`
	`86`	`+ "indices should be one-based in LIBSVM format")`
`85`	`87`	`(label, indices.toArray, values.toArray)`
`86`	`88`	`}`
`87`	`89`