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

sent2vec in Gensim #1376

Open
bhargavvader opened this issue May 31, 2017 · 19 comments
Open

sent2vec in Gensim #1376

bhargavvader opened this issue May 31, 2017 · 19 comments
Labels
difficulty hard Hard issue: required deep gensim understanding & high python/cython skills help wanted wishlist Feature request

Comments

@bhargavvader
Copy link
Contributor

The EPFL ML group have come up with a sentence embedding technique, sent2vec. The repo with the code can be found here.

It would be interesting to have this feature in Gensim, either with a wrapper or otherwise.

@bhargavvader
Copy link
Contributor Author

@menshikh-iv could you please add the appropriate labels for this?

@menshikh-iv menshikh-iv added the wishlist Feature request label Jun 1, 2017
@menshikh-iv
Copy link
Contributor

@bhargavvader of course :)

@souravsingh
Copy link
Contributor

I am interested in working on the issue.

@colmantse
Copy link

I am also interested, but I am new on GitHub. Would be great if this could be my first contribution.

@martinjaggi
Copy link

happy to help if any questions should come up. note that our current code is very close to fasttext (though unsupervised), so integration should be not too hard

@simonhessner
Copy link

Just wanted to ask for updates regarding sent2vec in gensim. Is somebody currently working on it?

@menshikh-iv
Copy link
Contributor

@simonhessner this is almost done in #1619, will be merged soon (I hope).

@SriniNaga
Copy link

@menshikh-iv Is this feature(Sent2Vec) available in Gensim now?

@menshikh-iv
Copy link
Contributor

@SriniNaga sorry, still not merged, will be available in 3.7.0

@wandabwa2004
Copy link

@menshikh-iv any timelines for version 3.7.0?

@menshikh-iv
Copy link
Contributor

@wandabwa2004 no, this is still not scheduled

@prerna135 what's a status of #1619?

  • Build still failing
  • Performance measurement (in comparison to other any2vec models)
  • Memory consumption (we discuss that this is too high)

You promised that you'll have a look and fix if needed in September, what's going on?

@piskvorky
Copy link
Owner

@prerna135 ping.

@prerna135
Copy link
Contributor

@menshikh-iv @piskvorky Sent you an email regarding this.

@cstayyab
Copy link

cstayyab commented Mar 3, 2019

@menshikh-iv @piskvorky any updates on this issue and the merge conflict?

@piskvorky
Copy link
Owner

@cstayyab It's up to @prerna135 .

@tmthyjames
Copy link

any update on this? love the blog post on it!

gensim is awesome! keep up the great work.

@MarcosGrzeca
Copy link

News?

Thank you :)

@0x01h
Copy link

0x01h commented Oct 22, 2019

Any improvements on this issue?

@mpenkov
Copy link
Collaborator

mpenkov commented Dec 21, 2019

No. The original author has abandoned the PR, and I don't have the bandwidth to take it on myself right now. If anyone is able and willing to push it over the line, please go ahead by all means.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty hard Hard issue: required deep gensim understanding & high python/cython skills help wanted wishlist Feature request
Projects
None yet
Development

No branches or pull requests