Skip to content

Add a new graph primitive to filter edges (extract_if_e)#1938

Merged
rapids-bot[bot] merged 18 commits intorapidsai:branch-21.12from
seunghwak:fea_copy_fi_e
Nov 16, 2021
Merged

Add a new graph primitive to filter edges (extract_if_e)#1938
rapids-bot[bot] merged 18 commits intorapidsai:branch-21.12from
seunghwak:fea_copy_fi_e

Conversation

@seunghwak
Copy link
Contributor

@seunghwak seunghwak commented Nov 9, 2021 •
edited
Loading

Close #1911

  • Implement the new primitive extract_if_e to extract an edge list (to construct a subgraph) by filtering the edges with e_op
  • Implement test suites.

@seunghwak seunghwak added feature request New feature or request 2 - In Progress non-breaking Non-breaking change labels Nov 9, 2021
@seunghwak seunghwak added this to the 21.12 milestone Nov 9, 2021
@seunghwak seunghwak requested a review from a team as a code owner November 9, 2021 19:58
@seunghwak seunghwak self-assigned this Nov 9, 2021
@seunghwak seunghwak changed the title [WIP][skip-ci] Add a new graph primitive to filter edges (copy_if_e) [WIP][skip-ci] Add a new graph primitive to filter edges (extract_if_e) Nov 11, 2021
@seunghwak seunghwak requested a review from a team as a code owner November 12, 2021 21:47
@seunghwak seunghwak changed the title [WIP][skip-ci] Add a new graph primitive to filter edges (extract_if_e) Add a new graph primitive to filter edges (extract_if_e) Nov 15, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 15, 2021 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.12@5c8a5b7). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.12    #1938   +/-   ##
===============================================
  Coverage                ?   70.28%           
===============================================
  Files                   ?      143           
  Lines                   ?     8922           
  Branches                ?        0           
===============================================
  Hits                    ?     6271           
  Misses                  ?     2651           
  Partials                ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c8a5b7...911cba2. Read the comment docs.

@BradReesWork
Copy link
Member

BradReesWork commented Nov 16, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 8096c53 into rapidsai:branch-21.12 Nov 16, 2021
@seunghwak seunghwak deleted the fea_copy_fi_e branch December 2, 2021 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChuckHastings ChuckHastings ChuckHastings approved these changes

+1 more reviewer

@kaatish kaatish kaatish approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@seunghwak seunghwak

Labels

feature request New feature or request non-breaking Non-breaking change

Projects

None yet

Milestone

21.12

Development

Successfully merging this pull request may close these issues.

[FEA] MNMG edge filtering by source & destination properties.

6 participants

@seunghwak @codecov-commenter @BradReesWork @kaatish @ChuckHastings @divyegala