Skip to content
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.

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

Add custom help formatter and shorthand of --thres #33

Merged
merged 1 commit into from
Jan 12, 2022
Merged

Add custom help formatter and shorthand of --thres #33

merged 1 commit into from
Jan 12, 2022

Conversation

eggplants
Copy link
Contributor

$ budoux -h
usage: budoux [-h] [-H] [-m JSON] [-d STR] [-t THRES] [-V] [TXT]

BudouX is the successor to Budou,
the machine learning powered line break organizer tool.

positional arguments:
  TXT                      text (default: None)

optional arguments:
  -h, --help               show this help message and exit
  -H, --html               HTML mode (default: False)
  -m JSON, --model JSON    custom model file path (default: /Users/eggplants/prog/budoux/budoux/models/ja-knbc.json)
  -d STR, --delim STR      output delimiter in TEXT mode (default: ---)
  -t THRES, --thres THRES  threshold value to separate chunks (default: 1000)
  -V, --version            show program's version number and exit

@eggplants eggplants changed the title Add custom help formatter and shorthand of -thres Add custom help formatter and shorthand of --thres Jan 11, 2022
@eggplants
Copy link
Contributor Author

Would it be better to write a reference to CLI in the README? (As well javascript/README.md)

@tushuhei tushuhei self-requested a review January 12, 2022 12:15
@tushuhei
Copy link
Member

The change LGTM! Thank you so much.

Would it be better to write a reference to CLI in the README? (As well javascript/README.md)

That would be awesome. Are you interested in writing the draft?

@tushuhei tushuhei merged commit 7fc4eca into google:main Jan 12, 2022
@eggplants eggplants deleted the fix_cli branch January 12, 2022 12:26
@eggplants
Copy link
Contributor Author

@tushuhei @junseinagao could you port this over to Javascript?

@junseinagao
Copy link
Contributor

@eggplants

I'll implement this function to node cli. Give me a few days!

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