Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Ktruss implementation #4059
Ktruss implementation #4059
Changes from 91 commits
fad7a83
6afb913
b68381f
b5ac46e
cc284e4
1635f87
fa61f9b
87f45be
74d24be
0656dd9
dfbc33a
e6f6784
c9c3a2b
e827457
06d4f77
4b4fb46
7f25cfc
a7d0fff
becf133
5d3ed2a
e8d0ccc
737f438
00789bf
0b45356
49f46d0
7cfe60a
c81655d
43f8485
8ff673a
7b0d132
99c973f
2f3a4db
d805440
66ec45f
246ec16
0286934
98b09cf
e354565
531d8c9
0ea3de0
61d238c
edfabea
4dffcf5
d2694b7
1130354
66a2310
f2dd1f3
209335f
17a7013
e61d1b8
1a64e0e
0355693
28275b3
c362f45
55876aa
873199f
b1bdf25
12b1c35
ca20811
990a8e9
78562a2
4f8159c
bc7f181
2cfd553
f0fce6f
979f715
d71e5b0
d605fef
ce2e9f9
6217057
67e64ce
56fc131
caf3dde
2077dc3
1b4c580
05ed540
e824eb3
5e95bb4
9a97490
0851b3e
4126d70
e517abe
085f4ea
8694f31
ccf1b6d
9cc76bd
585c052
8fb2b5b
4be2cf6
3ad4c20
d6fd0bf
31c8902
eab194a
d2d6999
5817b3f
8c347c1
874897e
42c8ed8
94fa611
f30b4ff
7cda92e
a3f83ef
3e03ef3
d23a3ad
3ce4e31
264a7a2
a678340
3426fd8
dbf220e
5768ed3
3187183
9c2c19d
7c2f4c4
ee1d8c2
813aa83
71b89a5
bfca100
cdf328b
c966eba
e926fae
0704eba
4954e24
49433a7
7858b3e
302eaa2
bd8ac94
b470713
f7a43ec
7b80a8d
68b97bd
8287244
701d87f
22527be
2ea7ed2
9bab048
a9cdb98
8297ae6
406a317
a0a2d63
288251b
20597f9
a87f07a
ecbc2ce
e6e2166
3593436
fbc5681
a4fed55
2100e30
b66bd1a
048b408
3e03255
8f75316
24a6f76
f95fc13
3f6928d
56d5010
21b7b84
59546b7
b3ba407
5dba8b3
8dae624
b5f0d3a
c9ada76
ce8e594
9b51e5b
65024ea
1d8c5d9
aa327f9
4549f10
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Better match
k_core
, we should takestd::optional<edge_property_view_t<edge_t, weight_t const*>> edge_weight_view
and optionally return weights as well.If this takes time to implement, you can add
CUGRAPH_EXPECTS(edge_weights.has_value() == false, "unimplemented.");
and address this in a follow-up PR.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.
You are replacing the call to legacy ktruss with the call to primitive-based ktruss. That's great in that it saves python work (although we should still plan MG python work for 24.06). There's already issues for that work, we can update those issues.
However, if we do that, we should rename this file to
k_truss.cpp
since it will no longer be the legacy implementation.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.
Now we should start addressing all the issues we should address before merging this PR.
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.
delete dead code.
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.
Check pre-commit (as mentioned above). I believe these should now be in
""
and before the other headers.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.
Is this FIXME still relevant?
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.
Should be in
""
.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.
Don't forget to address this.