-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 asr warnings #10469
Fix asr warnings #10469
Conversation
Signed-off-by: Nithin Rao Koluguri <nithinraok>
Signed-off-by: Nithin Rao Koluguri <nithinraok>
Signed-off-by: nithinraok <nithinraok@users.noreply.github.com>
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.
Very nice, thanks for the amp cleanup !
Signed-off-by: Nithin Rao Koluguri <nithinraok>
Signed-off-by: Nithin Rao Koluguri <nithinraok>
Signed-off-by: nithinraok <nithinraok@users.noreply.github.com>
…o fix_asr_warnings
Signed-off-by: Nithin Rao Koluguri <nithinraok>
Signed-off-by: nithinraok <nithinraok@users.noreply.github.com>
Signed-off-by: Nithin Rao Koluguri <nithinraok>
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.
LGTM, thanks for the fix~!
Signed-off-by: Nithin Rao Koluguri <nithinraok>
[🤖]: Hi @nithinraok 👋, I just wanted to let you know that, you know, a CICD pipeline for this PR just finished successfully ✨ So it might be time to merge this PR or like to get some approvals 🚀 But I'm just a 🤖 so I'll leave it you what to do next. Have a great day! //cc @ko3n1g |
In CI-CD stage for more than a week. Merging as it was failing for LLM issues unrelated to this PR. |
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com>
* check * 1 * added to examples/asr * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) * Apply isort and black reformatting * check * 1 * added to examples/asr * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) * Apply isort and black reformatting * tested on V100 * Apply isort and black reformatting * replace cuda for jit scripted modules * device type fix * revert diar for CI --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao <nithinrao.koluguri@gmail.com> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com> Signed-off-by: Lifu Zhang <tomzhanglf@gmail.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com> Signed-off-by: Lifu Zhang <tomzhanglf@gmail.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com> Signed-off-by: Hainan Xu <hainanx@nvidia.com>
* check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * check Signed-off-by: Nithin Rao Koluguri <nithinraok> * 1 Signed-off-by: Nithin Rao Koluguri <nithinraok> * added to examples/asr Signed-off-by: Nithin Rao Koluguri <nithinraok> * deprecates cuda.amp.autocast to replace with amp.autocast(**Args) Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * tested on V100 Signed-off-by: Nithin Rao Koluguri <nithinraok> * Apply isort and black reformatting Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> * replace cuda for jit scripted modules Signed-off-by: Nithin Rao Koluguri <nithinraok> * device type fix Signed-off-by: Nithin Rao Koluguri <nithinraok> * revert diar for CI Signed-off-by: Nithin Rao Koluguri <nithinraok> --------- Signed-off-by: Nithin Rao Koluguri <nithinraok> Signed-off-by: nithinraok <nithinraok@users.noreply.github.com> Co-authored-by: Nithin Rao Koluguri <nithinraok> Co-authored-by: nithinraok <nithinraok@users.noreply.github.com>
What does this PR do ?
Replaces deprecated cuda.amp.autocast with amp.autocast(**Args)
Collection: All (majorly ASR, TTS, Audio)
Changelog
GitHub Actions CI
The Jenkins CI system has been replaced by GitHub Actions self-hosted runners.
The GitHub Actions CI will run automatically when the "Run CICD" label is added to the PR.
To re-run CI remove and add the label again.
To run CI on an untrusted fork, a NeMo user with write access must first click "Approve and run".
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