knod / read_plugin

Chrome plugin for "read"
9 stars 8 forks source link

nlp_compromise fails (for devs only) #84

Closed knod closed 7 years ago

knod commented 7 years ago

nlp_compromise is built to detect English sentences. Since we want the capability to navigate between sentences we need this, but it has a chance of failing. It'd be great to have a fallback, but for the moment we don't have one. Either someone can file an issue on nlp_compromise's page, providing the text causing the error, or we just let that page go.

In future we will either have to help nlp_compromise improve, find a better sentence separating library, create a fallback, or decide not to be able to navigate between sentences.

knod commented 7 years ago

Page: http://www.koutipandoras.gr/article/tsipras-oi-praikophmaties-den-einai-eyprosdektoi-alla-h-ellada-einai-kratos-dikaioy Environment: Chromium v55.0.2883.87 Built on Ubuntu , running on LinuxMint 18.1 (64-bit) Observer: @synapse25 Original issue: #80

knod commented 7 years ago

Switched to own version of sbd package instead (@knod/sbd). New issue for sbd failures.