Skip to content
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

Make remote-mqpu auto-launcher aware of CUDAQ_DYNLIBS env var #2385

Merged
merged 9 commits into from
Dec 11, 2024

Conversation

bmhowe23
Copy link
Collaborator

If auto-launch is used from a Python environment, it is aware of where the custatevec and cutensornet libs are through the CUDAQ_DYNLIBS environment variable, but it was counting on the LD_LIBRARY_PATH to find them instead. This updates the logic to use the CUDAQ_DYNLIBS if the environment variable is present.

@bettinaheim bettinaheim added the bug fix To be listed under Bug Fixes in the release notes label Dec 11, 2024
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Dec 11, 2024
Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Dec 11, 2024
Copy link

copy-pr-bot bot commented Dec 11, 2024

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bettinaheim
Copy link
Collaborator

bettinaheim commented Dec 11, 2024

/ok to test

Command Bot: Processing...

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
Copy link
Collaborator

bettinaheim commented Dec 11, 2024

/ok to test

Command Bot: Processing...

Signed-off-by: Bettina Heim <[email protected]>
@bettinaheim
Copy link
Collaborator

bettinaheim commented Dec 11, 2024

/ok to test

Command Bot: Processing...

Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Dec 11, 2024
@bettinaheim
Copy link
Collaborator

Note that testing this change fully requires a GPU runner, and hence will only happen post-merge as far as the pipelines are concerned (hence the failing codecov check).

Copy link

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Dec 11, 2024
@bettinaheim bettinaheim merged commit c56261b into NVIDIA:main Dec 11, 2024
212 of 213 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 11, 2024
bettinaheim added a commit that referenced this pull request Dec 11, 2024
Signed-off-by: Ben Howe <[email protected]>
Signed-off-by: Bettina Heim <[email protected]>
Co-authored-by: Bettina Heim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix To be listed under Bug Fixes in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants