Thanks to visit codestin.com
Credit goes to github.com

Skip to content

cp: Add import guards for mcore lightning module (#14970) into r2.5.0#14982

Merged
chtruong814 merged 2 commits into
r2.5.0from
chruong/import-guard
Oct 27, 2025
Merged

cp: Add import guards for mcore lightning module (#14970) into r2.5.0#14982
chtruong814 merged 2 commits into
r2.5.0from
chruong/import-guard

Conversation

@chtruong814
Copy link
Copy Markdown
Collaborator

What does this PR do ?

cp: Add import guards for mcore lightning module (#14970) into r2.5.0

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)

* Add import guards for mcore lightning module

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

* Apply isort and black reformatting

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

* Add import guard for Apex

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

* Add asr install checker

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: Charlie Truong <[email protected]>
@github-actions github-actions Bot removed the Run CICD label Oct 25, 2025
@chtruong814 chtruong814 merged commit b8e001a into r2.5.0 Oct 27, 2025
266 of 275 checks passed
@chtruong814 chtruong814 deleted the chruong/import-guard branch October 27, 2025 19:29
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