Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.
/ hmrb Public archive

Commit

Permalink
Suggestion to include the imported Class (#18)
Browse files Browse the repository at this point in the history
Pycharm has consistently removed the import of SpacyCore when optimising imports because it is not used explicitly in the code. This little change fixes that and clarifies where the nlp pipe name comes from. Gentle suggestion. 

(Aside, I've started exploring `hmrb` and really like the ease of constructing the grammar objects so far -- thanks for the package!)
  • Loading branch information
svenski authored Jun 15, 2021
1 parent d1fced0 commit 51715ac
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/advanced.rst
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ Finally, we can add the ``"hmrb"`` pipeline component using our configuration to
"callbacks": {"my_callback": "callbacks.dummy_callback"}
"map_doc": "augmenters.jsonify_span"
}
nlp.add_pipe("hmrb", config=conf)
nlp.add_pipe(SpacyCore.name, config=conf)
Handling Callbacks
Expand Down

0 comments on commit 51715ac

Please sign in to comment.