Github user funes commented on a diff in the pull request:
https://github.com/apache/spark/pull/661#discussion_r12413127
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/linalg/BreezeVectorConversionSuite.scala
---
@@ -55,4 +55,16 @@ class BreezeVectorConversionSuite extends
Github user funes commented on the pull request:
https://github.com/apache/spark/pull/661#issuecomment-42516407
@mengxr Updated the test case.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user funes commented on a diff in the pull request:
https://github.com/apache/spark/pull/661#discussion_r12311565
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Vectors.scala
---
@@ -136,7 +136,7 @@ object Vectors {
new DenseVector(v.toArray
Github user funes closed the pull request at:
https://github.com/apache/spark/pull/661
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
GitHub user funes reopened a pull request:
https://github.com/apache/spark/pull/661
Bug fix of sparse vector conversion
Fixed a small bug caused by the inconsistency of index/data array size and
vector length.
You can merge this pull request into a Git repository by running
GitHub user funes opened a pull request:
https://github.com/apache/spark/pull/661
Bug fix of sparse vector conversion
Fixed a small bug caused by the inconsistency of index/data array size and
vector length.
You can merge this pull request into a Git repository by running