-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Kaldi submodules #4945
Labels
Comments
I am interested in merging their history into this repo. However, that would mean altering the history of the repo since it does not follow the kaldi coding guidelines. should I change the history or just make a new repo? |
I wouldn't worry too much about the coding guidelines if it's new recipes-- it won't affect anything existing. |
Awesome! Can i ask you for advice on my kaldi-speaker-diarization paper?
…On Tuesday, September 24, 2024, Daniel Povey ***@***.***> wrote:
I wouldn't worry too much about the coding guidelines if it's new
recipes-- it won't affect anything existing.
—
Reply to this email directly, view it on GitHub
<#4945 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABUMNYBTG5B4NC5V5G262WLZYEYUTAVCNFSM6AAAAABORASBFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZQHAYDGMZSG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sure, what do you want to ask? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Iunno if this is well known but i have github.com/judyfong/sid
and github.com/cadia-lvl/kaldi-speaker-diarization they are submodules of kaldi for the ruv-di dataset
The text was updated successfully, but these errors were encountered: