kaldi-asr / kaldi

kaldi-asr/kaldi is the official location of the Kaldi project.
http://kaldi-asr.org
Other
14.24k stars 5.32k forks source link

Wrong argument in kaldi hand-on document - config option #4368

Closed a2d8a4v closed 3 years ago

a2d8a4v commented 3 years ago

Hello, I found a wrong description when I read this document

Example of configuring grid software with new-style options

the queue.pl -config conf/no_k20.conf –allow-k20 has a wrong argument -config, which should be revised as --config

Thank you for time to reading

danpovey commented 3 years ago

mm, would be nice if you could make a pull request to fix that. source is in src/doc/.

On Sun, Dec 6, 2020 at 4:08 PM ヤンヤン notifications@github.com wrote:

Hello, I found a wrong description when I read this document

Example of configuring grid software with new-style options https://kaldi-asr.org/doc/queue.html#parallelization_common_new_example

the queue.pl -config conf/no_k20.conf –allow-k20 has a wrong argument -config, which should be revised as --config

Thank you for time to reading

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kaldi-asr/kaldi/issues/4368, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZFLO5I3XF2AI4ADP4Q3LTSTM3YNANCNFSM4UPEUNHA .

a2d8a4v commented 3 years ago

OK! I got it! @danpovey And we support you, don't give up!

danpovey commented 3 years ago

Thanks!!

On Sun, Dec 6, 2020 at 9:36 PM ヤンヤン notifications@github.com wrote:

Thank you! @danpovey https://github.com/danpovey And we support you, don't give up!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kaldi-asr/kaldi/issues/4368#issuecomment-739503681, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZFLO7H3576HFFMIMUFYTTSTOCEFANCNFSM4UPEUNHA .

stale[bot] commented 3 years ago

This issue has been automatically marked as stale by a bot solely because it has not had recent activity. Please add any comment (simply 'ping' is enough) to prevent the issue from being closed for 60 more days if you believe it should be kept open.

kkm000 commented 3 years ago

Fixed in #4369.