Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

speaker-identification-with-vad-non-streaming-asr.py Lack of support for sense_voice. #1884

Merged
merged 4 commits into from
Feb 18, 2025

Conversation

VisminLab
Copy link
Contributor

Fix that speaker-identification-with-vad-non-streaming-asr.py does not support the sense_voice ASR model. (#1883)
image

Copy link
Collaborator

@csukuangfj csukuangfj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please first test it locally.

@VisminLab
Copy link
Contributor Author

@csukuangfj I have retested and resubmitted, and the new submission can now run and be used normally. Sorry again.

Copy link
Collaborator

@csukuangfj csukuangfj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Left a minor comment.

…ing-asr.py


Thanks for the suggestion! I agree that this change makes the description clearer.

Co-authored-by: Fangjun Kuang <csukuangfj@gmail.com>
@VisminLab VisminLab requested a review from csukuangfj February 18, 2025 03:42
Copy link
Collaborator

@csukuangfj csukuangfj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution!

@csukuangfj csukuangfj merged commit 4e83b34 into k2-fsa:master Feb 18, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants