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 several coding errors in the doc2vec-lee examples #1841

Merged
merged 1 commit into from
Jan 16, 2018
Merged

Fix several coding errors in the doc2vec-lee examples #1841

merged 1 commit into from
Jan 16, 2018

Conversation

PeterHamilton
Copy link
Contributor

This change fixes a couple different coding errors found in the code snippets included in the doc2vec-lee tutorial, including array index out-of-bounds selection and passing test document words to vector inferencing.

This change fixes a couple different coding errors found in the
code snippets included in the doc2vec-lee tutorial, including
array index out-of-bounds selection and passing test document
words to vector inferencing.
@menshikh-iv
Copy link
Contributor

Good catch, thanks @PeterHamilton, congratz with your first contribution 🥇

@menshikh-iv menshikh-iv merged commit 1fbc8b4 into piskvorky:develop Jan 16, 2018
@PeterHamilton
Copy link
Contributor Author

@menshikh-iv Thanks. Happy to help!

sj29-innovate pushed a commit to sj29-innovate/gensim that referenced this pull request Feb 21, 2018
This change fixes a couple different coding errors found in the
code snippets included in the doc2vec-lee tutorial, including
array index out-of-bounds selection and passing test document
words to vector inferencing.
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