Skip to content

Conversation

@bact
Copy link
Member

@bact bact commented Nov 3, 2019

No description provided.

bact and others added 21 commits October 17, 2019 14:52
Add pythainlp.util.thai_time
- Remove person names
- Remove some hyphened words
- Remove incomplete words
Remove some words from dictionary
Update tokenizers and test cases
- remove some words in the pattern ไม่+adj
- remove some words in the pattern อย่าง+adv
@pep8speaks
Copy link

Hello @bact! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 128:80: E501 line too long (81 > 79 characters)

Line 166:80: E501 line too long (100 > 79 characters)
Line 182:80: E501 line too long (80 > 79 characters)

Line 169:80: E501 line too long (480 > 79 characters)
Line 170:80: E501 line too long (249 > 79 characters)
Line 171:80: E501 line too long (623 > 79 characters)
Line 172:80: E501 line too long (477 > 79 characters)
Line 173:80: E501 line too long (687 > 79 characters)
Line 174:80: E501 line too long (221 > 79 characters)
Line 175:80: E501 line too long (827 > 79 characters)
Line 176:80: E501 line too long (322 > 79 characters)
Line 177:80: E501 line too long (333 > 79 characters)
Line 178:80: E501 line too long (522 > 79 characters)
Line 179:80: E501 line too long (188 > 79 characters)
Line 180:80: E501 line too long (739 > 79 characters)
Line 181:80: E501 line too long (630 > 79 characters)
Line 182:80: E501 line too long (203 > 79 characters)
Line 183:80: E501 line too long (573 > 79 characters)
Line 184:80: E501 line too long (603 > 79 characters)
Line 185:80: E501 line too long (379 > 79 characters)
Line 186:80: E501 line too long (456 > 79 characters)

@bact bact changed the title Updage changes from dev branch Update to changes from dev branch Nov 3, 2019
@coveralls
Copy link

Coverage Status

Coverage increased (+0.9%) to 90.265% when pulling 0d5ff17 on dev into 7a7fb80 on 2.1.

@bact bact merged commit bdc4a8e into 2.1 Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants