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

Copy utt2lang when using utils/copy_data_dir.sh #4789

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

jonghwanhyeon
Copy link
Contributor

Currently utils/copy_data_dir.sh does not copy utt2lang to destination directory. However, utils/subset_data_dir.sh does as follows:

[ -f $srcdir/utt2lang ] &&
utils/filter_scp.pl $destdir/utt2spk <$srcdir/utt2lang >$destdir/utt2lang

Because I encountered this issue when making multi-language and multi-speaker TTS model, I wrote this PR.

@kkm000
Copy link
Contributor

kkm000 commented Aug 31, 2022

LGTM. Nice catch, thank you!

@kkm000 kkm000 merged commit a754c18 into kaldi-asr:master Aug 31, 2022
rkjaran added a commit to tiro-is/kaldi that referenced this pull request Nov 18, 2022
* 'master' of github.com:kaldi-asr/kaldi:
  Update run_blstm.sh (kaldi-asr#4790)
  Provide provision to pass subword separator as argument to make_position_dependent_subword_lexicon.py (kaldi-asr#4794)
  change [utter] to [utterance.] in data_prep.dox (kaldi-asr#4795)
  [egs] Kill feral dupes of --num-threads in few local eg scripts (kaldi-asr#4792)
  [scripts] Move otherwise misleading comment in mkgraph.sh (kaldi-asr#4787)
  [scripts] Copy utt2lang in copy_data_dir.sh (kaldi-asr#4789)
  Fix: ali-to-post piping in post-to-tacc example (kaldi-asr#4788)
  [infra] add cpu-only docker build  (kaldi-asr#4783)
  [infra] docker images automatically using gh (kaldi-asr#4782)
  remove the c++17 removed function random_shuffle
  remove obsoleted and C++17 incompatible unary_function<>, resolves kaldi-asr#4732
  [egs] Set PYTHONUNBUFFERED=1 in all recipes (kaldi-asr#4770)
  Fix typo in gen_cmake_skeleton (kaldi-asr#4779)
  use last segment of the CXX as the ccbin compiler for cuda (kaldi-asr#4778)
  Windows conda fixes (kaldi-asr#4777)
  Replace TravisCI with github actions (kaldi-asr#4776)
  Kaldi recipe for SPGISpeech (kaldi-asr#4772)
  fix portaudio install script, closes kaldi-asr#4755 (kaldi-asr#4773)
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.

2 participants