Skip to content

Add an sklearn wrapper for the Doc2Vec model#1481

Closed
oxymor0n wants to merge 1 commit into
piskvorky:developfrom
pagerinc:develop
Closed

Add an sklearn wrapper for the Doc2Vec model#1481
oxymor0n wants to merge 1 commit into
piskvorky:developfrom
pagerinc:develop

Conversation

@oxymor0n
Copy link
Copy Markdown

  • TaggedDocumentTransformer transform tokenized documents to TaggedDocuments
  • SklD2VModel transforms TaggedDocuments to doc2vec vectors

 * TaggedDocumentTransformer transform tokenized documents to TaggedDocuments
 * SklD2VModel transforms TaggedDocuments to doc2vec vectors
@oxymor0n
Copy link
Copy Markdown
Author

oh oops sorry wasn't aware of #1462

@oxymor0n
Copy link
Copy Markdown
Author

In any case, this is a homebrewed wrapper that we used at @pagerinc and thought you guys were missing. It's not very different than your version in #1462, but I think it still has some value since the code is more concise and the tests are (somewhat) different

@menshikh-iv
Copy link
Copy Markdown
Contributor

@oxymor0n thank you for your work, we looked carefully at your code and changed several things in #1462.
Now Doc2Vec wrapper merged from #1462, for this reason I close this PR.

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.

2 participants