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

TestRandCentroids fails #7

Closed
bobhancock opened this issue Jul 31, 2012 · 4 comments
Closed

TestRandCentroids fails #7

bobhancock opened this issue Jul 31, 2012 · 4 comments
Assignees

Comments

@bobhancock
Copy link
Owner

./km_test.go:85: cannot use DataCentroids literal (type DataCentroids) as type CentroidChooser in array element:
DataCentroids does not implement CentroidChooser (wrong type for ChooseCentroids method)
have ChooseCentroids(_matrix.DenseMatrix, int) (_matrix.DenseMatrix, error)
want ChooseCentroids(*matrix.DenseMatrix, int) *matrix.DenseMatrix

@ghost ghost assigned danielhfrank Jul 31, 2012
@danielhfrank
Copy link
Collaborator

I'll see if I can duplicate this on my home machine - in the meantime, could I ask you for the testSetSmall.txt file? Or, alternatively, for a script to generate it? I've been unable to run the full test suite without it

@bobhancock
Copy link
Owner Author

I had these in my ignore file. I'll add them to the repository this
afternoon.
On Jul 31, 2012 2:20 PM, "Dan Frank" <
reply@reply.github.com>
wrote:

I'll see if I can duplicate this on my home machine - in the meantime,
could I ask you for the testSetSmall.txt file? Or, alternatively, for a
script to generate it? I've been unable to run the full test suite without
it


Reply to this email directly or view it on GitHub:
#7 (comment)

@bobhancock
Copy link
Owner Author

I fixed this. Instead of returning an error, which invalidates the interface, if k > row then k = row.

@danielhfrank
Copy link
Collaborator

Thanks for updating! Sorry I didn't get to it sooner. I'd had trouble
reproducing, for whatever reason

On Aug 28, 2012, at 3:27 PM, Bob Hancock notifications@github.com wrote:

I fixed this. Instead of returning an error, which invalidates the
interface, if k > row then k = row.


Reply to this email directly or view it on
GitHubhttps://github.com//issues/7#issuecomment-8103919.

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

No branches or pull requests

2 participants