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 support for customizable interaction graph in SchNet model #5919

Merged
merged 17 commits into from
Nov 9, 2022

Conversation

hatemhelal
Copy link
Contributor

@hatemhelal hatemhelal commented Nov 7, 2022

This introduces a new interaction_graph arg that defaults to radius_graph to allow callers to customize how interatomic edges are defined in the SchNet architecture.

This removes the support that was added for pre-computed edges in #5401 so also updated the example to reflect this change.

@codecov
Copy link

codecov bot commented Nov 7, 2022

Codecov Report

Merging #5919 (6a51807) into master (55bcfc4) will increase coverage by 0.02%.
The diff coverage is 94.11%.

❗ Current head 6a51807 differs from pull request most recent head 926058e. Consider uploading reports for the commit 926058e to get more accurate results

@@            Coverage Diff             @@
##           master    #5919      +/-   ##
==========================================
+ Coverage   84.41%   84.44%   +0.02%     
==========================================
  Files         351      349       -2     
  Lines       19524    19511      -13     
==========================================
- Hits        16482    16476       -6     
+ Misses       3042     3035       -7     
Impacted Files Coverage Δ
torch_geometric/nn/models/schnet.py 70.42% <94.11%> (+0.81%) ⬆️
torch_geometric/utils/torch_sparse_tensor.py
torch_geometric/utils/spmm.py

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

torch_geometric/nn/models/schnet.py Outdated Show resolved Hide resolved
torch_geometric/nn/models/schnet.py Show resolved Hide resolved
torch_geometric/nn/models/schnet.py Outdated Show resolved Hide resolved
@hatemhelal hatemhelal marked this pull request as ready for review November 9, 2022 09:59
@rusty1s rusty1s enabled auto-merge (squash) November 9, 2022 14:06
@rusty1s rusty1s merged commit ee080ce into pyg-team:master Nov 9, 2022
JakubPietrakIntel pushed a commit to JakubPietrakIntel/pytorch_geometric that referenced this pull request Nov 25, 2022
…-team#5919)

This introduces a new `interaction_graph` arg that defaults to
`radius_graph` to allow callers to customize how interatomic edges are
defined in the `SchNet` architecture.

This removes the support that was added for pre-computed edges in pyg-team#5401
so also updated the example to reflect this change.

Co-authored-by: rusty1s <matthias.fey@tu-dortmund.de>
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.

2 participants