NeMo
NeMo copied to clipboard
RADTTS model changes to accommodate export with batch size > 1
What does this PR do ?
Add a one line overview of what this PR aims to accomplish.
Collection: [Note which collection this PR will affect]
Changelog
- Add specific line by line info of high level changes in this PR.
Usage
- You can potentially add a usage example below
# Add a code snippet demonstrating how to use this
Before your PR is "Ready for review"
Pre checks:
- [ ] Make sure you read and followed Contributor guidelines
- [ ] Did you write any new necessary tests?
- [ ] Did you add or update any necessary documentation?
- [ ] Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
- [ ] Reviewer: Does the PR have correct import guards for all optional libraries?
PR Type:
- [ ] New Feature
- [x] Bugfix
- [ ] 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.
Additional Information
- Related to # (issue)
This pull request introduces 5 alerts and fixes 3 when merging ee20fb79707370df9d6c57080ceb1f13aad2a788 into f1825bc4b724b78c2d6ca392b616e8dc9a8cde04 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 7090757946e08d7a43717453d3d5bd4753d44a2a into f1825bc4b724b78c2d6ca392b616e8dc9a8cde04 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging a95ced2d0b6ec2b0e6bd97d998af63902cb78edb into f1825bc4b724b78c2d6ca392b616e8dc9a8cde04 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 5cf6852c382fd47d59e12c4e015046d649568633 into 5d5c0fbff7ebe5c5991fb99a1cfeb57d5014ad42 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 79d056830c543e1554d231dfff6beadb92eeaeae into 3cdff5b88b5eb8c9bee857702e1887be842a3861 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 732a7dd802caad4c58420ca63dd2e9c188ed914d into a39fc925a63b5836626485d25876b4f12ff1c967 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 29bc7f138a2b98fac63b85a484c0a4e77935dd11 into a39fc925a63b5836626485d25876b4f12ff1c967 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 206b71819f22493e5f45069c24f590a2bb8bb1e5 into 971485ce7fedd7a6d16966f00583fe2736129f52 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging 830c4fbd84184aef8c86d6afcf407139cad53639 into 73fcfd7cdf69f375d6398a2930f9d1c03c96db39 - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging b405d2d88e2a2f9c9993c25bb0399e107c80119f into fc76ae71de658d7e6d369c33f1b38af3934bba9e - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
This pull request introduces 5 alerts and fixes 3 when merging c1da4efed63f7293f945175afa0d41e1c1e9cd67 into fc76ae71de658d7e6d369c33f1b38af3934bba9e - view on LGTM.com
new alerts:
- 5 for Unused import
fixed alerts:
- 2 for Unused local variable
- 1 for Except block handles 'BaseException'
In general, the PR LGTM. But I want @MikyasDesta to review/approve
The PR LGTM. We also trained a model and results are good.