Conversation
karpnv
left a comment
There was a problem hiding this comment.
Need to add yaml config example and readme
|
Add a requirements file with all needed dependencies, including NeMo version |
7960435 to
1d1a351
Compare
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
271f1b0 to
f36dd60
Compare
Signed-off-by: Nune <ntadevosyan@nvidia.com>
|
Good morning Nune! |
|
git pull origin mian should fix tests |
karpnv
left a comment
There was a problem hiding this comment.
pull the latest vertion in main to pass the tests
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: George Zelenfroind <gzelenfroind@nvidia.com>
Jorjeous
left a comment
There was a problem hiding this comment.
config.rst
Please fix warnings in doctests related to file mentioned above
(line interval, etc)
Signed-off-by: George Zelenfroind <gzelenfroind@nvidia.com>
Signed-off-by: George Zelenfroind <gzelenfroind@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
Signed-off-by: Nune <ntadevosyan@nvidia.com>
|
I see that now we have some ubuntu related problems |
| nemo_directory (str): Base directory for NeMo framework | ||
| new_manifest_files (str, Optional): New manifest files to add to the training configuration | ||
| new_tarred_audio_filepaths (str, Optional): New tarred audio filepaths to add to the training configuration | ||
| **kwargs: Additional arguments passed to the parent BaseProcessor class |
There was a problem hiding this comment.
Please add example section
| num_gpus (int): Number of GPUs to use | ||
| is_tarred (bool): Whether the audio is tarred | ||
| first_run (bool): Whether this is the first run of pseudo-labeling | ||
| **kwargs: Additional arguments passed to the parent BaseProcessor class |
There was a problem hiding this comment.
Please add example section
like in this PR
https://github.com/NVIDIA/NeMo-speech-data-processor/pull/115/files#diff-424af7846bc8503aed8b73bc8e234ce9b50e7fa7ac6f1408cc5fdeedbd4c61b9
| config_path (str): Path to the YAML configuration file containing IPL settings | ||
| output_manifest_file (str): Path where the output manifest file will be written | ||
| input_manifest_file (str, Optional): Path to the input manifest file | ||
| """ |
There was a problem hiding this comment.
Please add example section
|
We forgot example section... |
Added three processors for training and inference command generation and one NeMo-Run processor specifically for IPL training.
Config file should be provided for NeMo-Run processor with the following parameters.