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

Sentence Similarity web demo is not working #1406

Closed
ibrahimsharaf opened this issue Oct 9, 2017 · 6 comments
Closed

Sentence Similarity web demo is not working #1406

ibrahimsharaf opened this issue Oct 9, 2017 · 6 comments
Labels
examples Code examples in /examples

Comments

@ibrahimsharaf
Copy link

ibrahimsharaf commented Oct 9, 2017

Hi, I am not sure if it's the right repo to report this issue, but I was tinkering around with spaCy sentence similarity demo and it is not working on all my browsers, it keeps loading and giving me this:

Oops, something went wrong. Please try again!

Example query: https://demos.explosion.ai/similarity/?text1=Spain%20&text2=The%20Philippines

@honnibal honnibal added the examples Code examples in /examples label Oct 20, 2017
@lgenerknol
Copy link
Contributor

The cause is GET https://api.explosion.ai/similarity/models returns a 502.
Additionally, the root GET https://api.explosion.ai/ returns the default nginx page.

@msamogh
Copy link

msamogh commented Nov 7, 2017

Any update on this?

I'm unable to run the siamese network provided in the thinc repo's demo folder as well. So hoping for some solution to this to test out how well it works.

@honnibal
Copy link
Member

honnibal commented Nov 7, 2017

Sorry for dragging my feet on this folks.

I was never really thrilled by the results the similarity models were giving, even though the performance on the development data was pretty good. I think the things the models learned really didn't generalise well.

I've now replaced the similarity demo with one that compares the similarity results from spaCy's en_core_web_lg and en_core_web_sm models.

https://demos.explosion.ai/similarity/?text1=This%20is%20a%20sentence&text2=This%20is%20another%20sentence

I'm hoping to revisit the supervised similarity stuff soon. @msamogh I thought I got the siamese example up to date in thinc? When did you try?

@honnibal honnibal closed this as completed Nov 8, 2017
@msamogh
Copy link

msamogh commented Nov 9, 2017

@honnibal I tried it a couple of times in the last month.

There was a one-line change that I had to make in Thinc's source code to get the example running. When I tried recompiling, however, I would run into a circular dependency error involving the spaCy and Thinc versions. So I could never get it to work.

Regarding this:

I was never really thrilled by the results the similarity models were giving, even though the performance on the development data was pretty good. I think the things the models learned really didn't generalise well.

Was it trained on the Quora QA dataset?

@ines
Copy link
Member

ines commented Nov 9, 2017

Was it trained on the Quora QA dataset?

Yes, we trained one model on Quora, and one on StackOverflow (CQADupStack). There was also a baseline model using the English GloVe vectors.

@lock
Copy link

lock bot commented May 8, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Code examples in /examples
Projects
None yet
Development

No branches or pull requests

5 participants