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

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

Open
ilich917 opened this issue Nov 13, 2021 · 5 comments
Open

Comments

@ilich917
Copy link

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
Copy link
Owner

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

@ilich917
Copy link
Author

ilich917 commented Nov 13, 2021 via email

@dmmiller612
Copy link
Owner

Do you have a stack trace of the error?

@phil-scholarcy
Copy link

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
Copy link

PR here for full spacy 3 compatibility including the latest coreference module: #142

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants