spaCy NER has been used to recognize locations as named entities. The logic involving compound dependencies and proper nouns cannot be completely removed, as spaCy NER was seen failing to recognize entities with 100% accuracy. An alternate fallback involving a new logic of compound dependencies and proper nouns was thus introduced, that comes into play only when NER fails.
Files changed:
chatbot.py
utilities.py
Dependencies if any: #170
Basic tests done to validate:
pytest -v test_chatbot.py executed in local environment. Results attached as screenshots.
Conflicts if any: None
Checklist:
[x] Read the contributing guidelines
[x] All coding conventions are followed
[x] Style guidelines for this project have been followed
[x] 4 space indentation is used
[x] Relevant comments are added
[x] The code has been tested
[x] No new warnings are generated
Screenshots:
Other Information:
skipping Travis CI
BREAKING CHANGE: refactor to set up environment and dependencies
Fixes #169
Description:
spaCy NER has been used to recognize locations as named entities. The logic involving compound dependencies and proper nouns cannot be completely removed, as spaCy NER was seen failing to recognize entities with 100% accuracy. An alternate fallback involving a new logic of compound dependencies and proper nouns was thus introduced, that comes into play only when NER fails.
chatbot.py
utilities.py
pytest -v test_chatbot.py
executed in local environment. Results attached as screenshots.Checklist:
Screenshots:
Other Information:
@vishakha-lall