Skip to content

Manage extra requires + merge g2p and romanization to one transliterate module#153

Merged
bact merged 14 commits into
PyThaiNLP:devfrom
bact:dev
Nov 9, 2018
Merged

Manage extra requires + merge g2p and romanization to one transliterate module#153
bact merged 14 commits into
PyThaiNLP:devfrom
bact:dev

Conversation

@bact

@bact bact commented Nov 6, 2018

Copy link
Copy Markdown
Member

@coveralls

coveralls commented Nov 6, 2018

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-21.5%) to 72.219% when pulling a47d297 on bact:dev into 580ea00 on PyThaiNLP:dev.

@bact

bact commented Nov 8, 2018

Copy link
Copy Markdown
Member Author

Build fails, basically from missing imports (epitrain).

Another issue related to imports. Have to decided how to deal with "optional packages" that used by some functionalities.

@bact

bact commented Nov 9, 2018

Copy link
Copy Markdown
Member Author

Travis fails because: The job exceeded the maximum log length, and has been terminated. Too many packages to install.

AppVeyor fails because there's no torch PyPI package for Windows.

@bact

bact commented Nov 9, 2018

Copy link
Copy Markdown
Member Author

Test on Windows (AppVeyor) fails due to UnicodeDecodeError in artagger (#155)

@bact bact changed the title Propose to merge g2p and romanization modules to one common transliterate module Manage extra requires + merge g2p and romanization to one transliterate module Nov 9, 2018
@bact
bact requested a review from wannaphong November 9, 2018 04:58
Comment thread setup.py Outdated
Comment thread setup.py
@bact
bact merged commit 694dbf7 into PyThaiNLP:dev Nov 9, 2018
@wannaphong wannaphong mentioned this pull request Nov 9, 2018
@wannaphong wannaphong added this to the 1.8 milestone Nov 10, 2018
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.

3 participants