Skip to content

Fix "thainlp tokenize sent" command line - #383

Merged
bact merged 1 commit into
devfrom
verb-commands
May 5, 2020
Merged

Fix "thainlp tokenize sent" command line#383
bact merged 1 commit into
devfrom
verb-commands

Conversation

@bact

@bact bact commented May 5, 2020

Copy link
Copy Markdown
Member

This PR will partially close #342

  • Fix wrong call of SubwordTokenizationApp instead of SentenceTokenizationApp
  • Change default sentence separator from ^ to @@
  • Update documentation

@bact bact self-assigned this May 5, 2020
@bact bact added the bug bugs in the library label May 5, 2020
@bact bact added this to the 2.2 milestone May 5, 2020
@bact bact changed the title Fix thainlp tokenize sent command line Fix "thainlp tokenize sent" command line May 5, 2020
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 90.691% when pulling ca3c68e on verb-commands into c227976 on dev.

@bact
bact merged commit e6378ff into dev May 5, 2020
@p16i
p16i deleted the verb-commands branch May 5, 2020 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug bugs in the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI commands should be verbs/actions

2 participants