Skip to content
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

fix embedding name for verifying speakers #3578

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

nithinraok
Copy link
Collaborator

Signed-off-by: nithinraok [email protected]

What does this PR do?

Please have one of the following:

  • Fixes verify helper function for speaker verification

Before your PR is "Ready for review"

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or updated any necessary documentation?

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.

@nithinraok nithinraok merged commit 234eb10 into r1.6.1 Feb 1, 2022
@nithinraok nithinraok deleted the fix_verify_helper_function branch February 1, 2022 18:08
ericharper pushed a commit that referenced this pull request Feb 2, 2022
titu1994 pushed a commit that referenced this pull request Feb 2, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix the tutorial notebooks bug (#3465)

* fix checkpoint loading and model config file

Signed-off-by: Yi Dong <[email protected]>

* fix style

Signed-off-by: Yi Dong <[email protected]>

* Fix checkpoint converter in O2 style (#3486)

* Fix checkpoint converter in O2 style

Signed-off-by: Yu Yao <[email protected]>

* Fix style

Signed-off-by: Yu Yao <[email protected]>

Co-authored-by: Yu Yao <[email protected]>
Co-authored-by: Eric Harper <[email protected]>

* Pushing updated WFST Tutorial to r1.6.0 (#3521)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: Yang Zhang <[email protected]>

* WFST tutorial update (#3531)

* Pushing updated WFST Tutorial to r1.6.0

Signed-off-by: tbartley94 <[email protected]>

* Hopefully final corrections to WFST tutorials.

Signed-off-by: tbartley94 <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

* add rank check and barrier

Signed-off-by: ericharper <[email protected]>

* fix embedding name for verifying speakers (#3578)

Signed-off-by: nithinraok <[email protected]>

* Add guards (#3579)

* add guards

Signed-off-by: ericharper <[email protected]>

* remove import

Signed-off-by: ericharper <[email protected]>

* style

Signed-off-by: ericharper <[email protected]>

* raise import error instead of warning

Signed-off-by: ericharper <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Yi Dong <[email protected]>
Co-authored-by: yaoyu-33 <[email protected]>
Co-authored-by: Yu Yao <[email protected]>
Co-authored-by: tbartley94 <[email protected]>
Co-authored-by: Yang Zhang <[email protected]>
Co-authored-by: Nithin Rao <[email protected]>
fayejf pushed a commit that referenced this pull request Mar 2, 2022
fayejf pushed a commit that referenced this pull request Mar 2, 2022
* update branch

Signed-off-by: ericharper <[email protected]>

* Fix the tutorial notebooks bug (#3465)

* fix checkpoint loading and model config file

Signed-off-by: Yi Dong <[email protected]>

* fix style

Signed-off-by: Yi Dong <[email protected]>

* Fix checkpoint converter in O2 style (#3486)

* Fix checkpoint converter in O2 style

Signed-off-by: Yu Yao <[email protected]>

* Fix style

Signed-off-by: Yu Yao <[email protected]>

Co-authored-by: Yu Yao <[email protected]>
Co-authored-by: Eric Harper <[email protected]>

* Pushing updated WFST Tutorial to r1.6.0 (#3521)

Signed-off-by: tbartley94 <[email protected]>

Co-authored-by: Yang Zhang <[email protected]>

* WFST tutorial update (#3531)

* Pushing updated WFST Tutorial to r1.6.0

Signed-off-by: tbartley94 <[email protected]>

* Hopefully final corrections to WFST tutorials.

Signed-off-by: tbartley94 <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

* add rank check and barrier

Signed-off-by: ericharper <[email protected]>

* fix embedding name for verifying speakers (#3578)

Signed-off-by: nithinraok <[email protected]>

* Add guards (#3579)

* add guards

Signed-off-by: ericharper <[email protected]>

* remove import

Signed-off-by: ericharper <[email protected]>

* style

Signed-off-by: ericharper <[email protected]>

* raise import error instead of warning

Signed-off-by: ericharper <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

* update branch

Signed-off-by: ericharper <[email protected]>

Co-authored-by: Yi Dong <[email protected]>
Co-authored-by: yaoyu-33 <[email protected]>
Co-authored-by: Yu Yao <[email protected]>
Co-authored-by: tbartley94 <[email protected]>
Co-authored-by: Yang Zhang <[email protected]>
Co-authored-by: Nithin Rao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants