-
Notifications
You must be signed in to change notification settings - Fork 310
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
Fix louvain_communities
, PropertyGraph
, cudf column.full
, dgl 2.1.0 CI failures
#4215
Conversation
louvain_communities
CI failurelouvain_communities
and PropertyGraph
CI failures
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, and thanks for fixing the PG test.
This brings up a question though: before NX 3.3 is released, should this recent change be updated to move the new max_level
arg to the end of the nx.louvain_communities
signature to not break other users possibly using position-only args?
/merge |
louvain_communities
and PropertyGraph
CI failureslouvain_communities
, PropertyGraph
, and cudf column.full
CI failures
/merge |
louvain_communities
, PropertyGraph
, and cudf column.full
CI failureslouvain_communities
, PropertyGraph
, cudf column.full
, dgl 2.1.0 CI failures
5cfea53
into
rapidsai:branch-24.04
Thanks all! 🙏 |
cudf.core.column.as_column
instead of the deprecatedcudf.core.column.full
API