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

[FEA] pylibcugraph error handling should include detailed error message from the C API #2430

Closed
rlratzel opened this issue Jul 19, 2022 · 0 comments · Fixed by #2467
Closed
Assignees
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Milestone

Comments

@rlratzel
Copy link
Contributor

pylibcugraph APIs use a utility to check for success after calling C APIs, but do not extract the detailed error message available. Having additional details would be very helpful in debugging problems, such as the one reported here.

There's already a FIXME to address this here.

cc @ChuckHastings


For Developers below this line

  • [] Code passes CI
  • [] Code in pylibcugraph Python layer
  • [] Test cases at each layer (for MG tests, a note in the PR description indicating the new/existing MG tests were run and passed)
@rlratzel rlratzel added ? - Needs Triage Need team to review and classify improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 19, 2022
@rlratzel rlratzel added this to the 22.08 milestone Jul 19, 2022
@rapids-bot rapids-bot bot closed this as completed in cdc563f Aug 4, 2022
@rapids-bot rapids-bot bot closed this as completed in #2467 Aug 4, 2022
@kingmesal kingmesal removed the ? - Needs Triage Need team to review and classify label Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants