-
Notifications
You must be signed in to change notification settings - Fork 270
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
NER: Add html-like tag in output #262
Conversation
Hello @wannaphongcom! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2019-08-31 10:21:42 UTC |
|
||
return [(self.__pos_tags[i][0], data) for i, data in enumerate(self.__y)] |
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.
It would be good if we have tests for this change.
changes in this commit: - assert the output of the method `get_ner` when argument `tag` is set to True for all 13 tags as described in (https://github.com/wannaphongcom/thai-ner/tree/master/model/1.2) - assert the output of the method `get_ner` when argument `pos` is set to True - asser the output of the method `get_ner` when argument `pos` is set to False
tests/test_tag.py
Outdated
|
||
self.assertEqual( | ||
ner.get_ner( | ||
"อาจารย์เอกพล ประจำคณะวิสกรรมศาสตร์ ", |
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.
typo?
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.
Oh, yes. Thanks for checking the typos.
Apart from the typo, the PR looks good to me. |
As thainer version 1.2 provide different results from previous version
if this pull request is ready, you can merge it. |
Add output like html tag (ThaiNameTagger).