-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FLINK-1731] [ml] Implementation of Feature K-Means and Test Suite #1
[FLINK-1731] [ml] Implementation of Feature K-Means and Test Suite #1
Conversation
Modified KMeans to use a Seq for model
Merging [FLINK-1731] [ml] Implementation of Feature K-Means and Test Suite from @peedeeX21's fork.
Merging. @peedeeX21 |
@sachingoel0101 nice. thanks. |
No. I faced this same issue yesterday. Apparently, I was unable to see your fork as base fork too. I don't know how but manipulating the URL directly seemed to work. |
oke. that's strange. I guess we are not supposed to work together. :( It works perfectly with my colleagues. Anyway, good luck ... |
Merging this fork into @sachingoel0101 fork for convenience of code review by committers.
See: apache#700