Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

n_similarity() in word2vec and doc2vec raises ValueError if an empty list is passed #761

Closed
wants to merge 19 commits into from

Conversation

droudy
Copy link
Contributor

@droudy droudy commented Jun 28, 2016

Fixes #743

@@ -6,6 +6,7 @@ Changes
- In hdpmodel and dtmmodel
- NOT BACKWARDS COMPATIBLE!
* Added random_state parameter to LdaState initializer and check_random_state() (@droudy, #113)
* `n_similarity()` raises `ValueError` if an empty list is passed to it in word2vec, doc2vec (@droudy, #761)
Copy link
Owner

@piskvorky piskvorky Jun 30, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tmylk now that CHANGELOG is markdown (rather than plaintext), let's add links as true links: people's profiles, links to pull requests, links to supporting research articles...

@tmylk
Copy link
Contributor

tmylk commented Jul 1, 2016

@droudy Could you please rebase your branch so that the merge conflicts are resolved? Thanks

@droudy
Copy link
Contributor Author

droudy commented Jul 1, 2016

@tmylk I have resolved merge conflicts

Travis build fails for python 2.6, build stalls at test.test_wikicorpus.TestWikiCorpus

@tmylk
Copy link
Contributor

tmylk commented Oct 4, 2016

Needs rebase and then can be merged

@tmylk
Copy link
Contributor

tmylk commented Oct 4, 2016

Fixed in #883

@tmylk tmylk closed this Oct 4, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants