Skip to content

fix Chinese docker path#16

Merged
nshmyrev merged 2 commits intoalphacep:masterfrom
naxingyu:fix-cn-docker-path
Nov 5, 2019
Merged

fix Chinese docker path#16
nshmyrev merged 2 commits intoalphacep:masterfrom
naxingyu:fix-cn-docker-path

Conversation

@naxingyu
Copy link

@naxingyu naxingyu commented Nov 5, 2019

don't know if you accept outsider contribution. made it anyway...

@naxingyu
Copy link
Author

naxingyu commented Nov 5, 2019

And if this url is the multi_cn model, consider update it, refer to alumae/kaldi-gstreamer-server#216

@nshmyrev nshmyrev merged commit 23a2c77 into alphacep:master Nov 5, 2019
@nshmyrev
Copy link
Contributor

nshmyrev commented Nov 5, 2019

@naxingyu huge thanks, I actually wanted to give you a proper credit for multi-cn which this model is based on. It is not exactly multi-cn though, it is retrained for 8khz.

@naxingyu
Copy link
Author

naxingyu commented Nov 6, 2019

@nshmyrev Thanks for the credit. I see that 8k is set as sample rate in your code, so that makes more sense...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants