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

Add RemoveDuplicatedEdges transform #6709

Merged
merged 9 commits into from
Feb 15, 2023
Merged

Conversation

JakubPietrakIntel
Copy link
Contributor

Removes duplicate edged to the given homogeneous or heterogeneous graph. It will change the original order of dataset by concatenating one of duplicated edges at the end of the dataset. It can be used to clean-up a known repeated self-connecting edges issue in ogbn-products. Reference to ogbn-products Leaderboard: here

Moved this to separate PR from #6699

@codecov
Copy link

codecov bot commented Feb 15, 2023

Codecov Report

Merging #6709 (6f8b0ba) into master (844fc10) will increase coverage by 0.01%.
The diff coverage is 97.22%.

❗ Current head 6f8b0ba differs from pull request most recent head 7b1c3ea. Consider uploading reports for the commit 7b1c3ea to get more accurate results

@@            Coverage Diff             @@
##           master    #6709      +/-   ##
==========================================
+ Coverage   89.05%   89.06%   +0.01%     
==========================================
  Files         423      424       +1     
  Lines       22967    22998      +31     
==========================================
+ Hits        20453    20483      +30     
- Misses       2514     2515       +1     
Impacted Files Coverage Δ
...ch_geometric/transforms/remove_duplicated_edges.py 94.73% <94.73%> (ø)
torch_geometric/nn/models/linkx.py 100.00% <100.00%> (ø)
torch_geometric/nn/models/mlp.py 95.18% <100.00%> (ø)
torch_geometric/transforms/__init__.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rusty1s rusty1s changed the title Add RemoveDuplicateEdges transform Add RemoveDuplicatedEdges transform Feb 15, 2023
@rusty1s rusty1s enabled auto-merge (squash) February 15, 2023 15:05
@rusty1s rusty1s merged commit bca9647 into master Feb 15, 2023
@rusty1s rusty1s deleted the duplicate_edges_transform branch February 15, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants