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 HeteroJK class #9380

Merged
merged 6 commits into from
Jun 6, 2024
Merged

add HeteroJK class #9380

merged 6 commits into from
Jun 6, 2024

Conversation

m-atalla
Copy link
Contributor

Hi,

this PR implements #9355, please do let me know if any additional changes or documentation is needed.
Thank you

@github-actions github-actions bot added the nn label May 31, 2024
@m-atalla m-atalla changed the title add HeteroJK add HeteroJK module May 31, 2024
@m-atalla m-atalla changed the title add HeteroJK module add HeteroJK class May 31, 2024
Copy link

codecov bot commented Jun 6, 2024

Codecov Report

Attention: Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.86%. Comparing base (476e768) to head (3866ae5).

Current head 3866ae5 differs from pull request most recent head 4d5ae36

Please upload reports for the commit 4d5ae36 to get more accurate results.

Files Patch % Lines
torch_geometric/nn/models/jumping_knowledge.py 88.88% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9380      +/-   ##
==========================================
- Coverage   87.99%   87.86%   -0.13%     
==========================================
  Files         473      473              
  Lines       30820    30830      +10     
==========================================
- Hits        27120    27090      -30     
- Misses       3700     3740      +40     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rusty1s rusty1s merged commit bb131e8 into pyg-team:master Jun 6, 2024
16 checks passed
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.

Jumping Knowledge support for Heterogeneous Graph Features
2 participants