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

nx-cugraph: check networkx version #4571

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

eriknw
Copy link
Contributor

@eriknw eriknw commented Jul 30, 2024

This is to address the feedback from @bdice in #4531 (comment), which I was unable to do in that PR.

I know checking version strings is playing it fast-and-loose, but I believe we are abundantly safe to do so here, as networkx releases have been, and are expected to be, slow and predictable. Also, we do not have packaging as a runtime dependency (it is a test dependency for finer control).

So, even better than using packaging.version.parse, this PR performs a sanity check on the networkx version. I put it in both nx_cugraph.__init__ and _nx_cugraph.__init__ to give us obvious breadcrumbs to discover and follow.

@eriknw eriknw requested a review from a team as a code owner July 30, 2024 21:30
@rlratzel rlratzel added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 31, 2024
@rlratzel
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit d77a2d6 into rapidsai:branch-24.08 Jul 31, 2024
130 of 131 checks passed
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 nx-cugraph python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants