[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/3382#issuecomment-71796637 @fjiang6 It seems that we don't really have a scalable solution at this time. Do you mind closing this PR? It would be great if you can maintain this k-medoids implementation as a package. For example: http://spark-packages.org/package/26 --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3382 --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/3382#issuecomment-64295676 @fjiang6 I made a comment on the JIRA page. I think we need to discuss the complexity before looking into the code. Shall we have the discussion there? Thanks! --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
GitHub user fjiang6 opened a pull request: https://github.com/apache/spark/pull/3382 [SPARK-4510][MLlib]: Add k-medoids Partitioning Around Medoids (PAM) algorithm PAM (k-medoids) including the test case and an example. Passed the style checks Tested and compared with K-Means in MLlib, showing more steady performances. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Huawei-Spark/spark PAM Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/3382.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #3382 commit 95cd43e21a5e4499fd63125dde6973a4271a0de2 Author: Jiang Fan fjia...@gmail.com Date: 2014-11-20T02:59:18Z add PAM algorithm with an example commit 8721fc2d0fead9e72427909d5dab455e7dcd67f9 Author: Jiang Fan fjia...@gmail.com Date: 2014-11-20T03:05:43Z add newline at end of file commit 9b4131a3fee5e9cd5a7ac58c7718b78236412f7e Author: Jiang Fan fjia...@gmail.com Date: 2014-11-20T05:05:06Z add the PAMSuite.scala --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3382#issuecomment-63781094 [Test build #23666 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23666/consoleFull) for PR 3382 at commit [`9b4131a`](https://github.com/apache/spark/commit/9b4131a3fee5e9cd5a7ac58c7718b78236412f7e). * This patch merges cleanly. --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3382#issuecomment-63791400 [Test build #23666 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23666/consoleFull) for PR 3382 at commit [`9b4131a`](https://github.com/apache/spark/commit/9b4131a3fee5e9cd5a7ac58c7718b78236412f7e). * This patch **passes all tests**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * ` case class Params(` * `class PAM (` * `class PAMModel (val clusterCenters: Array[Vector]) extends Serializable ` --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-4510][MLlib]: Add k-medoids Partitionin...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3382#issuecomment-63791407 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23666/ Test PASSed. --- 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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org