Skip to content

Improve NEST GPU Utilization 1/N#14086

Merged
stevehuang52 merged 1 commit intoNVIDIA-NeMo:mainfrom
MahmoudAshraf97:fast_nest
Jul 2, 2025
Merged

Improve NEST GPU Utilization 1/N#14086
stevehuang52 merged 1 commit intoNVIDIA-NeMo:mainfrom
MahmoudAshraf97:fast_nest

Conversation

@MahmoudAshraf97
Copy link
Contributor

@MahmoudAshraf97 MahmoudAshraf97 commented Jul 1, 2025

This PR is among a series aimed to improve the training speed and GPU utilization of NEST models #13619

This PR Simplifies __getitem__ method of LhotseAudioNoiseDataset
Since Lhotse cuts are returned already stacked and collated, there is no need of using AudioNoiseItem to split them and then stack them again in _audio_noise_collate_fn since torch.stack calls are expensive for large tensors, only the noise tensors need to be stacked

cc @pzelasko @stevehuang52

Signed-off-by: Mahmoud Ashraf <hassouna97.ma@gmail.com>
Copy link
Collaborator

@stevehuang52 stevehuang52 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks~

@stevehuang52 stevehuang52 merged commit cabd3e9 into NVIDIA-NeMo:main Jul 2, 2025
126 checks passed
@MahmoudAshraf97 MahmoudAshraf97 deleted the fast_nest branch July 2, 2025 13:19
AmirHussein96 pushed a commit to AmirHussein96/NeMo that referenced this pull request Jul 23, 2025
Signed-off-by: Mahmoud Ashraf <hassouna97.ma@gmail.com>
Signed-off-by: Amir Hussein <amhussein@nvidia.com>
AmirHussein96 pushed a commit to AmirHussein96/NeMo that referenced this pull request Aug 5, 2025
Signed-off-by: Mahmoud Ashraf <hassouna97.ma@gmail.com>
Signed-off-by: Amir Hussein <amhussein@nvidia.com>
AmirHussein96 pushed a commit to AmirHussein96/NeMo that referenced this pull request Aug 5, 2025
Signed-off-by: Mahmoud Ashraf <hassouna97.ma@gmail.com>
Signed-off-by: Amir Hussein <amhussein@nvidia.com>
nasretdinovr pushed a commit to nasretdinovr/NeMo that referenced this pull request Aug 8, 2025
Signed-off-by: Mahmoud Ashraf <hassouna97.ma@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants