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

Update per_v_transform_reduce_incoming|outgoing_e to support edge masking #4085

Merged

Conversation

seunghwak
Copy link
Contributor

@seunghwak seunghwak commented Jan 10, 2024

per_v_transform_reduce_(incoming|outgoing_e) now supports edge masking.

@BradReesWork BradReesWork added this to the 24.02 milestone Jan 23, 2024
@seunghwak seunghwak marked this pull request as ready for review January 24, 2024 23:00
@seunghwak seunghwak requested a review from a team as a code owner January 24, 2024 23:00
@seunghwak seunghwak self-assigned this Jan 24, 2024
@seunghwak seunghwak added feature request New feature or request non-breaking Non-breaking change labels Jan 24, 2024
@ChuckHastings
Copy link
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit ff76a38 into rapidsai:branch-24.02 Jan 25, 2024
97 checks passed
rapids-bot bot pushed a commit that referenced this pull request Feb 9, 2024
… support (when edge masking is not in use) (#4119)

This addresses performance regression due to the added edge masking support when edge masking is disabled.

We addressed the issue for `per_v_transform_reduce_incoming|outgoing_e` in #4085

Using a similar approach, we address the issue for `transform_e`, `transform_reduce_e`, and `detail::nbr_intersection` in this PR.

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

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

URL: #4119
@seunghwak seunghwak deleted the fea_per_v_transform_reduce_edge_mask branch May 22, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuGraph feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants