-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Sortformer Diarizer 4spk v1 model PR Part 2: Unit-tests for Sortformer Diarizer. #11336
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_01
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
…to sortformer/pr_01
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_01
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_01
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_01
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_01
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
Signed-off-by: tango4j <tango4j@users.noreply.github.com>
Signed-off-by: taejinp <tango4j@gmail.com>
…to sortformer/pr_02
beep boop 🤖: 🚨 The following files must be fixed before merge! Your code was analyzed with PyLint. The following annotations have been identified:
Thank you for improving NeMo's documentation! |
beep boop 🤖: 🙏 The following files have warnings. In case you are familiar with these, please try helping us to improve the code base. Your code was analyzed with PyLint. The following annotations have been identified:
Thank you for improving NeMo's documentation! |
sortformer_diar_model = sortformer_model.eval() | ||
confdict = sortformer_diar_model.to_config_dict() | ||
sampling_rate = confdict['preprocessor']['sample_rate'] | ||
target_frame_count = int(sample_len // frame_length) |
Check notice
Code scanning / CodeQL
Unused local variable Note test
What does this PR do ?
This is the second PR that deals with unit tests for
Sortformer Diarizer Model, 4 speaker limit, v1 model.
Previous PR:
This is PR 2nd out of 3.
1st PR: #11336
3rd PR:
Sortformer Paper:
Sortformer Paper Link
Overview:
NeMo/tests/collections/asr
toNeMo/tests/collections/speaker_tasks
.Collection: ASR
Changelog
Updated the
test_data.tar.gz
file in https://github.com/NVIDIA/NeMo/releases/tag/v1.0.0rc1Usage
GitHub Actions CI
Added the new CIs for Sortformer Diarizer.
Before your PR is "Ready for review"
Pre checks:
PR Type:
If you haven't finished some of the above items you can still open "Draft" PR.
Who can review?
Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.
Additional Information