search

dmmiller612 / bert-extractive-summarizer

Easy to use extractive text summarization with BERT
MIT License
1.39k stars 307 forks source link

Please can you list the requirements of this when this really works? #115

Open ilich917 opened 2 years ago

ilich917 commented 2 years ago

This library definitely is not working now, but I want to run this with previous versions of spacy, python, transformers, etc. Can you give us the requirements please?

dmmiller612 commented 2 years ago

What's not working? There are a few libraries this depends on, and change frequently.

ilich917 commented 2 years ago

hi, its delivering that error of model not finded

On Sat, Nov 13, 2021, 10:50 Derek Miller @.***> wrote:

What's not working? There are a few libraries this depends on, and change frequently.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dmmiller612/bert-extractive-summarizer/issues/115#issuecomment-968071868, or unsubscribe https://github.com/notifications/unsubscribe-auth/APOS4JEKYHY7TDX7ACZY35LULZULJANCNFSM5H6TSVVQ .

dmmiller612 commented 2 years ago

Do you have a stack trace of the error?

phil-scholarcy commented 1 year ago

I'm working on an update to this repo so that it works with the latest spacy and other libraries, and also includes scaffolding for production deployment. https://github.com/scholarcy/bert-extractive-summarizer/tree/removes-coreference-dependency

For now, I'm removing the coref module but I'll add in the new spacy coreference model once I get it working to reconstitute the original sentence with coref resolved (currently it only provides coref clusters).

I'll make a PR next week hopefully

phil-scholarcy commented 1 year ago

PR here for full spacy 3 compatibility including the latest coreference module: https://github.com/dmmiller612/bert-extractive-summarizer/pull/142