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.

Sign up for GitHub

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

Jump to bottom

double_matrix_to_ndarray falsely assumes square matrices #4

Closed
fracpete opened this issue Oct 22, 2016 · 1 comment
Closed

double_matrix_to_ndarray falsely assumes square matrices #4

fracpete opened this issue Oct 22, 2016 · 1 comment
Assignees
@fracpete
Labels
bug

Comments

@fracpete
Copy link
Owner

The implementation of double_matrix_to_ndarray assumes that the passed Java matrix is a square matrix. This may be true for confusion matrices but it's definitely an error for Classifier.distributions_for_instances. When there are more instances than class labels, distributions_for_instances works but returns a too big numpy array.

Report for Python 2.7 version of the library: fracpete/python-weka-wrapper#48
@fracpete fracpete added the bug label Oct 22, 2016
@fracpete fracpete self-assigned this Oct 22, 2016
@fracpete
Copy link
Owner Author

That's fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fracpete fracpete

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fracpete