Skip to content

Warn in case a dependent library is not found in libcuopt load #375

Merged
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.10from
rgsl888prabhu:fix_python_error
Sep 5, 2025
Merged

Warn in case a dependent library is not found in libcuopt load #375
rapids-bot[bot] merged 1 commit intoNVIDIA:branch-25.10from
rgsl888prabhu:fix_python_error

Conversation

@rgsl888prabhu
Copy link
Copy Markdown
Collaborator

@rgsl888prabhu rgsl888prabhu commented Sep 4, 2025

Description

While loading libcuopt using load, sometimes due to a missing dependent library we get error as libcuopt.so is missing even though libcuopt.so is present but some other dependency is missing.

This PR adds a warning in such cases so it doesn't fail in the corner cases, but provide a better details through a warning.

Issue

closes #340

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@rgsl888prabhu rgsl888prabhu added this to the 25.10 milestone Sep 4, 2025
@rgsl888prabhu rgsl888prabhu self-assigned this Sep 4, 2025
@rgsl888prabhu rgsl888prabhu requested a review from a team as a code owner September 4, 2025 16:22
@rgsl888prabhu rgsl888prabhu added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Sep 4, 2025
@rgsl888prabhu
Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot rapids-bot bot merged commit 8a6f120 into NVIDIA:branch-25.10 Sep 5, 2025
75 checks passed
aliceb-nv pushed a commit that referenced this pull request Sep 22, 2025
While loading libcuopt using load, sometimes due to a missing dependent library we get error as libcuopt.so is missing even though libcuopt.so is present but some other dependency is missing.

This PR adds a warning in such cases so it doesn't fail in the corner cases, but provide a better details through a warning.

## Issue

closes #340

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Trevor McKay (https://github.com/tmckayus)

URL: #375
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
…A#375)

While loading libcuopt using load, sometimes due to a missing dependent library we get error as libcuopt.so is missing even though libcuopt.so is present but some other dependency is missing.

This PR adds a warning in such cases so it doesn't fail in the corner cases, but provide a better details through a warning.

## Issue

closes NVIDIA#340

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - Trevor McKay (https://github.com/tmckayus)

URL: NVIDIA#375
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Misleading failure when dependency of libcuopt is missing

2 participants