Skip to content

fix for issue #4051 #4054

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

Merged

Conversation

tttthomasssss
Copy link
Contributor

replaced X = np.asarray(X) with check_array(...) in affinity_propagation_.py, related issues #4051, #4052

@amueller
Copy link
Member

amueller commented Jan 6, 2015

Thanks for the fix :) Not sure what the best way to do the testing is. Or I wouldn't mind merging this one and testing everything via #4052 if @jnothman consents.

@jnothman
Copy link
Member

jnothman commented Jan 6, 2015

Why would I refuse?

On 7 January 2015 at 08:03, Andreas Mueller notifications@github.com
wrote:

Thanks for the fix :) Not sure what the best way to do the testing is.
Or I wouldn't mind merging this one and testing everything via #4052
#4052 if @jnothman
https://github.com/jnothman consents.


Reply to this email directly or view it on GitHub
#4054 (comment)
.

@amueller
Copy link
Member

amueller commented Jan 6, 2015

Ok then :) thanks for the fix @tttthomasssss!

amueller added a commit that referenced this pull request Jan 6, 2015
@amueller amueller merged commit 43e8c37 into scikit-learn:master Jan 6, 2015
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

Successfully merging this pull request may close these issues.

3 participants