-
Notifications
You must be signed in to change notification settings - Fork 124
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
[cli] support diar #246
[cli] support diar #246
Conversation
先不要合入,有几点需要确认:
|
感觉命令行比较好?设置一些默认的,不过在readme 里边写清楚;现在这个效果咋样? |
没有大规模测试,现在的默认参数和example/voxconverse/v2一致,效果应该和v2的结果一致 |
TODO:
|
基本是从v2改过来的,可以直接拿一两条voxconverse的音频,结果基本能对上就可以了,不一定要测DER |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO:不建议从v2里面单独拎几个func出来新建utils/diar_utils.py,更没必要单独新建utils/cluster.py,还是直接import v2里面的func即可
这样的话打pip包会不方便。 原则上wespeaker/下边放代码,example下放私有的代码,之前diar相关的代码都放在example下了,要不趁机重构下 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done!
wespeaker --task diarization --audio_file /Users/user01/Downloads/voxconverse_test_wav/aepyx.wav --language english