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

Make edges in HeterohilousGraphDataset undirected #7065

Merged
merged 5 commits into from
Mar 29, 2023

Conversation

OlegPlatonov
Copy link
Contributor

Hi! The graphs here are meant to be undirected. Since PyG treats all graphs as directed, I've added a call to transforms.ToUndirected to double the edges.

@codecov
Copy link

codecov bot commented Mar 28, 2023

Codecov Report

Merging #7065 (21917db) into master (692a8ce) will not change coverage.
The diff coverage is n/a.

❗ Current head 21917db differs from pull request most recent head 93a71f0. Consider uploading reports for the commit 93a71f0 to get more accurate results

@@           Coverage Diff           @@
##           master    #7065   +/-   ##
=======================================
  Coverage   91.44%   91.44%           
=======================================
  Files         436      436           
  Lines       23830    23830           
=======================================
  Hits        21792    21792           
  Misses       2038     2038           

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

rusty1s and others added 3 commits March 29, 2023 09:17
Co-authored-by: Jintang Li <cnljt@outlook.com>
Co-authored-by: Jintang Li <cnljt@outlook.com>
@rusty1s rusty1s changed the title Convert heterophilous graphs to undirected Make edges in HeterohilousGraphDataset undirected Mar 29, 2023
@rusty1s rusty1s enabled auto-merge (squash) March 29, 2023 07:21
@rusty1s rusty1s merged commit f749baa into pyg-team:master Mar 29, 2023
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.

3 participants