Skip to content

Update 2.2 branche from dev branche #482

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.

Sign up for GitHub

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

Jump to bottom
Merged
wannaphong merged 87 commits into from
Sep 17, 2020
Merged

Update 2.2 branche from dev branche #482

wannaphong merged 87 commits into from
Sep 17, 2020

Conversation

@wannaphong
Copy link
Member

@wannaphong wannaphong commented Sep 17, 2020

Update 2.2 branche from dev branche

wannaphong and others added 30 commits July 18, 2020 22:53
@wannaphong
Add provinces_all
0cbf3ce
@wannaphong
Update provinces
6cfb947
@bact
Update corpus.rst
1fa019e
@wannaphong
Update common.py
80e4d8b
@wannaphong
Update common.py
362eb2d
@p16i
helper function tone marks and above vowels
88a9488
@wannaphong
Add pythainlp.util.display_thai_char docs
506c1ca
@p16i
fix lint
90d8ed6
@p16i
Merge branch 'display-thai-char' of github.com:PyThaiNLP/pythainlp in…
7e9852d 
…to display-thai-char
@wannaphong
Fix PyTorch Install for GitHub Actions
af5c4b9
@p16i
Merge pull request #463 from PyThaiNLP/display-thai-char
e6e01c5 
helper function tone marks and above vowels
@wannaphong
Add LST20 postag model
90436b6
@wannaphong
Add lst20_tag_signs and lst20_tag_to_text
286e5b5
@wannaphong
Add lst20_ud
60ece5b
@wannaphong
Add pos_tag_sents docs
008f1c1
@wannaphong
Merge pull request #458 from PyThaiNLP/Add-provinces
313aa92 
Add provinces
@bact
Fix type hinting, clean code, remove thailand_provinces_th.txt
baba302
@bact
Fix PEP8
8b4beac
@wannaphong
Add docs
7e36766
@wannaphong
Update tag.rst
9f58ea7
@wannaphong
Update tag.rst
d39f7b0
@wannaphong
Update tag.rst
67393f5
@bact
Merge pull request #466 from PyThaiNLP/Add-provinces
2a853fa 
provinces(): Fix type hinting, clean code, remove thailand_provinces_th.txt
@bact
Update tag.rst
46b7416
@bact
Update pos_tag.py
be8f432 
Use list comprehension in _orchid_to_ud and _lst20_to_ud
@bact
Update perceptron.py
37bcfe4 
Use list comprehension for _postag_clean
@bact
Fix typo and format code
0841183
@bact
Edit function docstring
c42b1c9
@bact
Use list comprehension
ef2e39e
@bact
Add test cases
7e18f2c
bact and others added 23 commits August 23, 2020 10:30
@bact
Add test case load model file that does not exist
aad2507
@bact
Add MIT License information for _tag_perceptron.py
9cc4dad
@bact
Fix PEP8, add corpus details
89d4282
@bact
Merge pull request #470 from PyThaiNLP/add-perceptron-tagger
f98d18c 
Port PerceptronTagger to PyThaiNLP
@krissdap
Fix typo in README.md
9341a8d
@bact
Update README.md
0b2927f
@wannaphong
Merge pull request #472 from krissdap/dev
7c4bd52 
Fix typo in README.md
@bact
Update README.md
38f8089
@bact
Update README.md
579da03
@bact
Update README.md
5b8ad72
@bact
Add family names
04ff603
@bact
Merge pull request #476 from PyThaiNLP/remotes/origin/add-family-names
d3a7bee 
Add family names
@wannaphong
Update tag.rst
2e8933a
@wannaphong
Merge pull request #478 from PyThaiNLP/update-postag-docs (build and …
261f032 
…deploy docs)

Update ORCHID POS tags Docs
@bact
Fix remove_repeat_vowels() bug that remove spaces between vowel
830edab 
It was mistakenly convert string like "ะ า" to "ะา".
@bact
Merge pull request #481 from PyThaiNLP/hotfix-bug-normalize
bdafa57 
Fix remove_repeat_vowels() bug that remove spaces between vowel
@bact
Bump version: 2.2.3 → 2.2.4-dev0
bb0c1a4
@bact
Bump version: 2.2.4-dev0 → 2.2.4-beta0
ced2e3d
@bact
Bump version: 2.2.4-beta0 → 2.2.4
4d00fc4
@wannaphong
Update perceptron.py
7dd3cda
@wannaphong
Merge branch 'dev' of https://github.com/PyThaiNLP/pythainlp into dev
ab5a0e7
@wannaphong
Update lst20_tagger version
19c68be
@wannaphong
Update perceptron.py
45e98a3
@pep8speaks
Copy link

pep8speaks commented Sep 17, 2020 •
edited
Loading

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

Line 48:80: E501 line too long (80 > 79 characters)
Line 122:40: E203 whitespace before ':'
Line 171:32: E251 unexpected spaces around keyword / parameter equals
Line 171:34: E222 multiple spaces after operator
Line 171:34: E251 unexpected spaces around keyword / parameter equals

Line 42:80: E501 line too long (101 > 79 characters)
Line 42:90: E251 unexpected spaces around keyword / parameter equals
Line 42:92: E251 unexpected spaces around keyword / parameter equals

Comment last updated at 2020-09-17 09:40:03 UTC

@wannaphong wannaphong added this to the 2.2 milestone Sep 17, 2020
@coveralls
Copy link

coveralls commented Sep 17, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.4%) to 95.399% when pulling 008470a on dev into ea97162 on 2.2.

@wannaphong wannaphong merged commit 1c5818f into 2.2 Sep 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2.2

Development

Successfully merging this pull request may close these issues.

7 participants

@wannaphong @pep8speaks @coveralls @bact @p16i @krissdap