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

Fix tests for EuclideanKeyedVectors.similarity_matrix. Fix #1961 #1984

Merged

Conversation

Witiko
Copy link
Contributor

@Witiko Witiko commented Mar 17, 2018

If the tests pass, then this resolves #1961, and #1966.

@Witiko Witiko force-pushed the fix-similarity-matrix-unit-tests branch from 609fce3 to 32faeaa Compare March 17, 2018 19:04
@Witiko Witiko force-pushed the fix-similarity-matrix-unit-tests branch from 32faeaa to 65cba0d Compare March 17, 2018 19:17
@Witiko
Copy link
Contributor Author

Witiko commented Mar 17, 2018

AppVeynor is experiencing some unrelated issues with the Python 2.7 tests. Other than that, we should be ready to merge.

@menshikh-iv
Copy link
Contributor

@Witiko this connected with fresh setuptools release, related issue piskvorky/smart_open#176. I'll resolve it ASAP.

@menshikh-iv
Copy link
Contributor

thank you @Witiko 👍

@menshikh-iv menshikh-iv changed the title Fix similarity matrix unit tests Fix tests for EuclideanKeyedVectors.similarity_matrix. Fix #1961 Mar 18, 2018
@menshikh-iv menshikh-iv merged commit a781b40 into piskvorky:develop Mar 18, 2018
@Witiko Witiko deleted the fix-similarity-matrix-unit-tests branch March 18, 2018 13:46
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.

similarity_matrix: unit tests not run
2 participants