seominjoon / denspi

Real-Time Open-Domain Question Answering with Dense-Sparse Phrase Index (DenSPI)
https://nlp.cs.washington.edu/denspi
Apache License 2.0
200 stars 26 forks source link

Quick Q: Does DenSPI depend on CoreNLP? #11

Closed waalge closed 4 years ago

waalge commented 4 years ago

Its in the installer here

but its commented out in the Dockerfile.

I haven't seen any other reference to it.

jhyuklee commented 4 years ago

Hi @waalge , the DrQA installation requires the CoreNLP installation. You might not need that if you don't want to use the CoreNLP tokenizer in DrQA.

waalge commented 4 years ago

Thanks for your reply, Ok. Yeah, I understood that its interchangeable with spacy. Although its not immediate whether this gives comparable results.