Open danpovey opened 8 years ago
also cleanuped->cleaned up
How about adding a final optional 4th argument called lm_dir (the 3rd argument being 'work_dir'), so the user can specify where they want the final LM to be written? This will make life easier for callers, as they won't have to figure out where pocolm would put their stuff.
OK, I will add the work_dir and lm_dir argument. Regards to the wordlist name, I know it is not ideal. But I think we should not remove the final suffix if it is meaningful. For example, we have 3 different wordlist and named them as 'vocab.1', 'vocab.2' and 'vocab.3', they can't be distinguished if we remove the suffix.
OK, don't remove the suffix then.. if people want control they can add the lm_dir argument. Also, you won't have to create a subdirectory 'work' once you add the work_dir argument. Dan
On Sat, Sep 3, 2016 at 10:13 PM, Wang Jian notifications@github.com wrote:
OK, I will add the work_dir and lm_dir argument. Regards to the wordlist name, I know it is not ideal. But I think we should not remove the final suffix if it is meaningful. For example, we have 3 different wordlist and named them as 'vocab.1', 'vocab.2' and 'vocab.3', they can't be distinguished if we remove the suffix.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/danpovey/pocolm/issues/65#issuecomment-244580190, or mute the thread https://github.com/notifications/unsubscribe-auth/ADJVu4t6LbA7jGFFwwCXvmfhZYGdHPH_ks5qmilSgaJpZM4J0Xzp .
Yes, of course.
The usage message of train_lm.py (see below) does not agree with what the program actually does. The usage message suggests the output goes to lm_dir, but it goes to a subdirectory. I think you should rename lm_dir in the args to work_dir. And the usage message should explain what the location of the actual lm_dir output will be. There should be an "epilog" provided to the usage message, with an example usage- preferably a couple of example usages, one with a vocab and one with num-words specified. Also, you are using the 'basename' of the wordlist as part of the name of the lm_dir. What if the wordlist has a suffix, like foo.txt? Then foo.txt will become part of that name. It seems to me not ideal. Maybe strip any final suffix.