[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2017-01-23 Thread sachingoel0101
Github user sachingoel0101 commented on the issue: https://github.com/apache/flink/pull/757 I have opened a PR https://github.com/apache/flink/pull/3192 as per the discussion here to add the Lloyd's algorithm first. Please review that first and then I will rebase this work on top of

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2017-01-17 Thread thvasilo
Github user thvasilo commented on the issue: https://github.com/apache/flink/pull/757 Sure @sachingoel0101 feel free to split up the PRs to reduce overhead. For added initialization schemes let me throw [this recent

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2017-01-17 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/757 @sachingoel0101 no problem been there ;) That would be good thnx! --- 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

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2017-01-17 Thread sachingoel0101
Github user sachingoel0101 commented on the issue: https://github.com/apache/flink/pull/757 @skonto Been a bit busy. My apologies. I was working on this again some time back and would like to split this into two PRs. One for K means itself, another for adding initialization

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2017-01-17 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/757 @sachingoel0101 @tillrohrmann ? --- 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

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2016-10-07 Thread sachingoel0101
Github user sachingoel0101 commented on the issue: https://github.com/apache/flink/pull/757 I'll update based on your comments in a few days. ^^ On Oct 8, 2016 06:21, "Stavros Kontopoulos" wrote: > @sachingoel0101

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2016-10-07 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/757 @sachingoel0101 @tillrohrmann any plans for this PR? --- 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

[GitHub] flink issue #757: [FLINK-2131][ml]: Initialization schemes for k-means clust...

2016-09-30 Thread skonto
Github user skonto commented on the issue: https://github.com/apache/flink/pull/757 any progress with this PR? --- 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