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

Update to spacy 3.4.x #79

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ruben-dedoncker
Copy link

Updated the spacy NER model to version 3.4.x

@zhuolisam
Copy link

Hi, can you update the requirements.txt as well?

@simsong
Copy link

simsong commented Sep 13, 2023

I've been reviewing this. The problem with upgrading to Spacy NER model version 3.4 is that the current resume code seems to have its own model bundled in. Do we know what that model is and what would be required to regenerate it?

@ruben-dedoncker
Copy link
Author

I have already updated the requirements.txt as well as updated the bundled model using the available train data. This update works out-of-the-box

@IvoLeist
Copy link

IvoLeist commented Mar 16, 2024

@ruben-dedoncker
thank you for providing publicly a fix how to update spacy to version 3.4.x
I can confirm that your fork runs out of the box 🚀

Since now some time went past since you have added this PR spacy is now at 3.7.4.
I am not (yet) familiar with spacy but I am interested to learn a little bit about it.
If I would like to retrain it so the warning below vanishes how much computing power/time would this require?

UserWarning: [W095] Model 'en_pipeline' (0.0.0) was trained with spaCy v3.4.1 and may not be 100% compatible with the current version (3.7.4). If you see errors or degraded performance, download a newer compatible model or retrain your custom model with the current spaCy version. For more details and available updates, run: python -m spacy validate
  warnings.warn(warn_msg)

@OmkarPathak
thank you for making your resume parser open source. Looks like a really interesting project 🚀

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

Successfully merging this pull request may close these issues.

None yet

4 participants