-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[Tutorial][ASR][Fix] Data paths in ASR with NeMo tutorial #4845
Conversation
Signed-off-by: Ante Jukić <[email protected]>
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
@@ -56,10 +56,6 @@ | |||
"BRANCH = 'main'\n", | |||
"!python -m pip install git+https://github.com/NVIDIA/NeMo.git@$BRANCH#egg=nemo_toolkit[all]\n", | |||
"\n", | |||
"## Grab the config we'll use in this example\n", | |||
"!mkdir configs\n", |
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.
Why is this being moved ?
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.
Just convenience. The first cell is mostly about setting up environment in Colab, so I'd usually skip/comment it. However, this bit is necessary anyway, so I moved it to the spot where conf is actually used.
I don't have a strong preference, and we can revert if prefered.
Signed-off-by: Ante Jukić <[email protected]> Signed-off-by: Ante Jukić <[email protected]> Co-authored-by: Ante Jukić <[email protected]> Signed-off-by: George Zelenfroind <[email protected]>
Signed-off-by: Ante Jukić <[email protected]> Signed-off-by: Ante Jukić <[email protected]> Co-authored-by: Ante Jukić <[email protected]> Signed-off-by: Matvei Novikov <[email protected]>
Signed-off-by: Ante Jukić <[email protected]> Signed-off-by: Ante Jukić <[email protected]> Co-authored-by: Ante Jukić <[email protected]> Signed-off-by: Hainan Xu <[email protected]>
Signed-off-by: Ante Jukić <[email protected]> Signed-off-by: Ante Jukić <[email protected]> Co-authored-by: Ante Jukić <[email protected]> Signed-off-by: Hainan Xu <[email protected]>
What does this PR do ?
This PR aims to fix a few small issues in the ASR with NeMo tutorial.
Collection: ASR
Changelog
data_dir
exists and create if necessarydata_dir
topaths2audio_files
Usage
n/a
Before your PR is "Ready for review"
Pre checks:
PR Type:
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
n/a