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

Fix device bug in get_degree_histogram #7830

Merged
merged 3 commits into from
Aug 2, 2023

Conversation

DamianSzwichtenberg
Copy link
Member

This PR fixes the following error:
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, xpu:0 and cpu!
deg_histogram now inherits the device type from edge_index.

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #7830 (b31595a) into master (0c0d9c1) will decrease coverage by 0.63%.
Report is 3 commits behind head on master.
The diff coverage is 100.00%.

❗ Current head b31595a differs from pull request most recent head 05be2b5. Consider uploading reports for the commit 05be2b5 to get more accurate results

@@            Coverage Diff             @@
##           master    #7830      +/-   ##
==========================================
- Coverage   90.44%   89.81%   -0.63%     
==========================================
  Files         455      455              
  Lines       26334    26332       -2     
==========================================
- Hits        23818    23651     -167     
- Misses       2516     2681     +165     
Files Changed Coverage Δ
torch_geometric/nn/conv/pna_conv.py 100.00% <100.00%> (ø)

... and 30 files with indirect coverage changes

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

@rusty1s rusty1s changed the title Fix bug in get_degree_histogram Fix device bug in get_degree_histogram Aug 2, 2023
@rusty1s rusty1s merged commit 20362ee into pyg-team:master Aug 2, 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