Skip to content

Conversation

@chtruong814
Copy link
Collaborator

What does this PR do ?

Update numba to numba-cuda and update cuda python bindings usage

  • The cuda part of numba is now in numba-cuda. Use the cuda 13 version
  • Update the cuda bindings references given the cuda 13 version

Related issues:

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 

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:

  • 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
  • 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)

@github-actions github-actions bot added core Changes to NeMo Core ASR labels Nov 1, 2025
@chtruong814 chtruong814 added Run CICD and removed core Changes to NeMo Core ASR Run CICD labels Nov 1, 2025
@github-actions github-actions bot added core Changes to NeMo Core ASR labels Nov 1, 2025
@github-actions github-actions bot removed the Run CICD label Nov 1, 2025
Signed-off-by: Charlie Truong <[email protected]>
chtruong814 and others added 2 commits November 2, 2025 06:10
Signed-off-by: Charlie Truong <[email protected]>
Signed-off-by: Charlie Truong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

[🤖]: Hi @chtruong814 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

//cc @chtruong814 @ko3n1g @pablo-garay @thomasdhc

This reverts commit 44c523b.

Signed-off-by: Charlie Truong <[email protected]>
This reverts commit 3cb0150.

Signed-off-by: Charlie Truong <[email protected]>
@github-actions github-actions bot removed the CI label Nov 3, 2025
@chtruong814 chtruong814 added Run CICD r2.6.0 Cherry-pick label for the r2.6.0 release branch and removed Run CICD labels Nov 3, 2025
@chtruong814 chtruong814 enabled auto-merge (squash) November 3, 2025 17:19
@github-actions github-actions bot removed the Run CICD label Nov 3, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

[🤖]: Hi @chtruong814 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

//cc @chtruong814 @ko3n1g @pablo-garay @thomasdhc

@chtruong814 chtruong814 merged commit 2224289 into main Nov 3, 2025
492 of 501 checks passed
@chtruong814 chtruong814 deleted the chtruong/deps-25.11 branch November 3, 2025 21:38
chtruong814 added a commit that referenced this pull request Nov 3, 2025
)

* Replace numba with numba-cuda[cu13]

Signed-off-by: Charlie Truong <[email protected]>

* Update cuda bindings ref for cuda 13

Signed-off-by: Charlie Truong <[email protected]>

* Apply isort and black reformatting

Signed-off-by: chtruong814 <[email protected]>

* Use numba for mac

Signed-off-by: Charlie Truong <[email protected]>

* Fix numba check

Signed-off-by: Charlie Truong <[email protected]>

* Apply isort and black reformatting

Signed-off-by: chtruong814 <[email protected]>

* Fix cudart.cudaStreamGetCaptureInfo call for cuda 13

Signed-off-by: Charlie Truong <[email protected]>

* Fix call to cuGraphAddNode

Signed-off-by: Charlie Truong <[email protected]>

* Test speech unit tests only

Signed-off-by: Charlie Truong <[email protected]>

* Skip main unit tests

Signed-off-by: Charlie Truong <[email protected]>

* Fix with_conditional_node cuda calls

Signed-off-by: Charlie Truong <[email protected]>

* Revert "Skip main unit tests"

This reverts commit 44c523b.

Signed-off-by: Charlie Truong <[email protected]>

* Revert "Test speech unit tests only"

This reverts commit 3cb0150.

Signed-off-by: Charlie Truong <[email protected]>

---------

Signed-off-by: Charlie Truong <[email protected]>
Signed-off-by: chtruong814 <[email protected]>
Co-authored-by: chtruong814 <[email protected]>
Signed-off-by: NeMo Bot <[email protected]>
chtruong814 added a commit that referenced this pull request Nov 4, 2025
) (#15024)

* Replace numba with numba-cuda[cu13]



* Update cuda bindings ref for cuda 13



* Apply isort and black reformatting



* Use numba for mac



* Fix numba check



* Apply isort and black reformatting



* Fix cudart.cudaStreamGetCaptureInfo call for cuda 13



* Fix call to cuGraphAddNode



* Test speech unit tests only



* Skip main unit tests



* Fix with_conditional_node cuda calls



* Revert "Skip main unit tests"

This reverts commit 44c523b.



* Revert "Test speech unit tests only"

This reverts commit 3cb0150.



---------

Signed-off-by: Charlie Truong <[email protected]>
Signed-off-by: chtruong814 <[email protected]>
Signed-off-by: NeMo Bot <[email protected]>
Co-authored-by: chtruong814 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASR core Changes to NeMo Core force-run-all r2.6.0 Cherry-pick label for the r2.6.0 release branch skip-linting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants