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.

Sign up for GitHub

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

Jump to bottom

Enable edge masking in additional primitives #4126

Merged
merged 25 commits into from
Feb 23, 2024
Merged

Enable edge masking in additional primitives #4126

merged 25 commits into from
Feb 23, 2024

Conversation

seunghwak
Copy link
Contributor

@seunghwak seunghwak commented Jan 30, 2024 •
edited
Loading

List of the primitives updated in this PR.

  • compute_in_weight_sums()
  • compute_out_weight_sums()
  • detail::extract_transform_v_frontier_e()
  • extract_transform_e
  • extract_transform_v_frontier_outgoing_e
  • transform_reduce_v_frontier_outgoing_e_by_dst

List of primitives added in this PR

  • edge_partition_device_view_t::compute_number_of_edges_with_mask, compute_local_degrees_with_mask

@seunghwak seunghwak added feature request New feature or request non-breaking Non-breaking change labels Jan 30, 2024
@seunghwak seunghwak added this to the 24.04 milestone Jan 30, 2024
@seunghwak seunghwak self-assigned this Jan 30, 2024
@seunghwak seunghwak added the DO NOT MERGE Hold off on merging; see PR for details label Jan 30, 2024
@seunghwak
test edge masking in mg_per_v_pair_transfomr_dst_nbr_weighted_interse…
d6848f5 
…ction.cu
@seunghwak
bug fix
51efd36
@seunghwak
add const to functions that can be const
3b814a3
@seunghwak
update detail::extract_transform_v_frontier_e to support edge masking
8f48eb9
@seunghwak
replace uint32_t{0xffffffff} with raft::warp_full_mask()
df6b4b6
@seunghwak
update/performance tune detail::extract_transform_v_froniter_e with e…
f1ad089 
…dge masking
@seunghwak
mark fill_edge_src|dst_property.cuh as edge-masking ready
3492ee9
@seunghwak
add compute_number_of_edges_with_mask & compute_local_degree_with_mask
3036bc7
@seunghwak
add missing include
fc3b380
@seunghwak
update extract_transform_v_frontier_outgoing_e to support edge masking
217ef47
@seunghwak
update transform_reduce_v_frontier_outgoing_e_by_dst to support edge …
6a44b55 
…masking
@seunghwak
update extract_transform_e to support edge masking
60fd404
@seunghwak
Merge branch 'branch-24.04' of https://github.com/rapidsai/cugraph in…
c5f556a 
…to fea_prim_edge_masking
@seunghwak
clang-format
d44c868
@seunghwak
Merge branch 'branch-24.04' of https://github.com/rapidsai/cugraph in…
02de2b7 
…to fea_prim_edge_masking
@seunghwak
Merge branch 'branch-24.04' of https://github.com/rapidsai/cugraph in…
f18fdf8 
…to fea_prim_edge_masking
@seunghwak
resolve merge conflicts
1ddf922
@seunghwak
copyright year
0b40837
@seunghwak
clang-format
b71f656
@seunghwak seunghwak marked this pull request as ready for review February 9, 2024 22:31
@seunghwak seunghwak requested a review from a team as a code owner February 9, 2024 22:31
@seunghwak seunghwak removed the DO NOT MERGE Hold off on merging; see PR for details label Feb 9, 2024
@seunghwak seunghwak mentioned this pull request Feb 15, 2024
Merged
@seunghwak seunghwak mentioned this pull request Feb 21, 2024
Merged
@ChuckHastings
Copy link
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit 6b7c114 into rapidsai:branch-24.04 Feb 23, 2024
110 checks passed
rapids-bot bot pushed a commit that referenced this pull request Mar 13, 2024
@seunghwak
Enable edge masking in the remaining primitives (#4186)
120e5b8 
This PR will update all the remaining primitives to support edge masking.

This PR pulls updates from #4126 and better be reviewed/merged after PR #4126.

Authors:
  - Seunghwa Kang (https://github.com/seunghwak)

Approvers:
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Naim (https://github.com/naimnv)
  - Joseph Nke (https://github.com/jnke2016)

URL: #4186
@seunghwak seunghwak deleted the fea_prim_edge_masking branch May 22, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@naimnv naimnv naimnv approved these changes

@jnke2016 jnke2016 jnke2016 approved these changes

@ChuckHastings ChuckHastings ChuckHastings approved these changes

Assignees

@seunghwak seunghwak

Labels
cuGraph feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Milestone
24.04
Development

Successfully merging this pull request may close these issues.
4 participants
@seunghwak @ChuckHastings @jnke2016 @naimnv