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

Add spacy workaround #12

Merged
merged 15 commits into from
Oct 10, 2022
Merged

Add spacy workaround #12

merged 15 commits into from
Oct 10, 2022

Conversation

pszemraj
Copy link
Owner

downloads the spacy en_core_web_sm model if needed to prevent errors

Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
@pszemraj pszemraj added the enhancement New feature or request label Oct 10, 2022
@pszemraj pszemraj self-assigned this Oct 10, 2022
pszemraj and others added 11 commits October 10, 2022 01:34
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
adds the function to auto-download spacy model
add util function to download spacy model if needed
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
Signed-off-by: Peter <74869040+pszemraj@users.noreply.github.com>
@pszemraj
Copy link
Owner Author

CPU test works fine, see notebook here

@pszemraj
Copy link
Owner Author

GPU testing also works, check out the notebook

@pszemraj pszemraj merged commit 54ce2b1 into master Oct 10, 2022
@pszemraj pszemraj deleted the add-spacy-workaround branch October 10, 2022 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants