-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
bumped spacy to 3.4 #11598
bumped spacy to 3.4 #11598
Conversation
Replace Coveralls test coverage badge with CodeClimate badge
Update dependencies
…dockerfiles Only Push Docker Base Images for Internal Builds
…files Fix Condition to Detect Forks when Running CI
Add workflow step to install pluggy
🚀 A preview of the docs have been deployed at the following URL: https://11598--rasahq-docs-rasa-v2.netlify.app/docs/rasa |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
pyproject.toml
Outdated
@@ -203,7 +203,7 @@ version = "^0.4.4" | |||
markers = "sys_platform == 'win32'" | |||
|
|||
[tool.poetry.dependencies.spacy] | |||
version = ">=3.1,<3.3" | |||
version = ">=3.1,<3.5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we require min 3.4 on aarch64?
fingerprint_data = { | ||
"node_name": rasa.utils.common.module_path_from_class(graph_component_class), | ||
"component_implementation": inspect.getsource(graph_component_class), | ||
"config": config, | ||
"inputs": inputs, | ||
"dependency_versions": dependency_versions, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will invalidate any existing cache entries right? Not an issue but should prob be mentioned in the changelog entry.
will be merged as part of #11578 |
Proposed changes:
">=3.1,<3.5"
SpacyNLP
component. Does not change behavior as of now, but could have led to bugs if more parameters were added.SpacyNLP
component. Showing that case sensitivity can be disabled and enabled.Status (please check what you already did):
black
(please check Readme for instructions)