-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
fix: add type hints to all component __init__ constructor parameters #3152
Conversation
@@ -44,6 +44,7 @@ class EntityExtractor(BaseComponent): | |||
def __init__( | |||
self, | |||
model_name_or_path: str = "dslim/bert-base-NER", | |||
model_version: Optional[str] = None, |
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.
Let's add a line in the docstring for this param
self, | ||
knowledge_graph: BaseKnowledgeGraph, | ||
model_name_or_path: str, | ||
model_version: Optional[str] = None, |
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.
Let's add a line in the docstring for this param
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.
Right, totally forgot - thanks Branden
@brandenchan should be rebased and ready to go now. Please have another look |
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
… where applicable
Related Issues
__init__
constructor parameters #3144Proposed Changes:
Make sure all components have all
__init__
parameters typedMake sure all components have the
model_version
optional parameter - where applicableHow did you test it?
No additional tests were added, verified schema has changed.
Notes for the reviewer
Please make sure all components now have all
__init__
parameters typedPlease make sure all components now have
model_version
parameter - if applicable