Skip to content
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

Wrong variable assignment in kprototypes.py #23

Closed
yanxiang007 opened this issue Aug 4, 2016 · 2 comments
Closed

Wrong variable assignment in kprototypes.py #23

yanxiang007 opened this issue Aug 4, 2016 · 2 comments
Labels

Comments

@yanxiang007
Copy link

In line 118, shouldn't "cl_attr_freq, membship = move_point_num"
be
"cl_attr_sum, membship = move_point_num" as it did in line 92?

@nicodv
Copy link
Owner

nicodv commented Aug 5, 2016

Yes, good find.

Fortunately, this bug happens in a part of the code that is almost never reached (only when a cluster becomes empty during iterations).

If you find this out because you reached this code, and can (consistently) reproduce it, I'd love to see the data and include it as a test scenario. Let met know.

@nicodv nicodv closed this as completed in bf1340f Aug 5, 2016
@nicodv nicodv added the bug label Aug 5, 2016
@yanxiang007
Copy link
Author

Actually I was just re-writing the code to understand this algorithm better and happened to find this bug. No data testing involved. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants