Skip to content

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