-
Notifications
You must be signed in to change notification settings - Fork 2
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
Spacy manual load package and function.py #1
Comments
Hi, |
We already have your model on preproduction and its on CI/CD already to move to prod, works great with better performance. We deploy models stored on folders and managed by our models manager. Then we moved the python code to our codebase so all works perfectly. |
Awesome, thanks for the update!
==============
Carlos Rodriguez
==============
…On Fri, May 21, 2021 at 9:04 AM Ramon Navarro Bosch < ***@***.***> wrote:
We already have your model on preproduction and its on CI/CD already to
move to prod, works great with better performance. We deploy models stored
on folders and managed by our models manager. Then we moved the python code
to our codebase so all works perfectly.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA2FKPQJ3H25YTQAB3BTNFDTOYAZVANCNFSM45HRDUCA>
.
|
Great job on releasing this package!
We've a small problem with how its packages as it requires the
functions.py
to be accessible on python namespace. In our case the model is not deployed on a python package so it breaks the dependency.Is there any reason to deliver as it is ? None of the other "official" packages has this python package on it.
Maybe a solution would be to provide a python package to load this helper functions (sentence splitter and lemmatizer) would help. You could provide a
spacy-ca
pypi package which is defined on "requirements" section on the meta.json.The text was updated successfully, but these errors were encountered: