Better CLI

[p16i]

related to #250

Example usages:

$ ./bin/pythainlp --help
usage: pythainlp [-h] command

positional arguments:
  command     [corpus|tokenization|soundex]

optional arguments:
  -h, --help  show this help message and exit

$  ./bin/pythainlp tokenization word  --text "ผมร<0e31>กประเทศไทย? สามารถ" --engine newmm
Using engine=newmm
ผม|รัก|ประเทศไทย|?| |สามารถ

$ ./bin/pythainlp tokenization syllable  --text "ผมร<0e31>กประเทศไทย? สามารถ"
Using engine=ssg
ผม~รัก~ประ~เทศ~ไทย~? ~สา~มารถ

$ ./bin/pythainlp soundex  --text "บ<0e39>รณการ" --engine lk82
บE419

[pep8speaks]

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-08-23 17:16:55 UTC

[coveralls]

Coverage decreased (-5.3%) to 88.267% when pulling 35e5766 on cli-improvement into 0dd822f on dev.

[wannaphong]

Colab : https://colab.research.google.com/drive/1HQ8cK_SnVaGmxHCblffmZg_nsJVsbks6

[wannaphong]

kenize syllable ?

[p16i]

sry for that. It's just temporary code.

[wannaphong]

@heytitle It's ok. Can you add Part-of-Speech (POS) tags ?

[p16i]

Sure! My local code was outdated, so that was why I didn't implement POS. I'm going to fix it.

[p16i]

POS added.

./bin/pythainlp tagging pos --text "ผม|ไม่|กิน|เผ็ด"

@wannaphongcom What do you think if we slightly change the interface of commands requiring text?

In particular, it would be better if these commands receive a file instead of just a text, and write a result into another file. For example,

./bin/pythainlp tagging pos --input some text.txt \
 --output pos-tagged.txt \
 --engine ... 

./bin/pythainlp tokenization word --input some text.txt \
 --output tokenised-word.txt \
 --engine ... 

IMHO, this is the way I would use CLIs.

[wannaphong]

@heytitle I agree. It will greatly facilitate the use of the CLIs.

[p16i]

Do you happen to know how many people use PyThaiNLP's CLI?

I'm wondering whether we should create another PR for the receiving input as a file feature.

What do you think?

[wannaphong]

@heytitle I not know number of CLI users. I agree. We should create PR for the receiving input as a file feature.

[wannaphong]

Now, Can i merge this pull request?

[p16i]

@wannaphongcom yes