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

Explain: Updates for heterogeneous graphs and CaptumExplainer #6487

Merged
merged 10 commits into from
Jan 22, 2023

Conversation

wsad1
Copy link
Member

@wsad1 wsad1 commented Jan 21, 2023

Add example for heterogenous graphs and captum.

@rusty1s rusty1s changed the title Explain: docs update. Explain: Updates for heterogeneous graphs and CaptumExplainer Jan 22, 2023
@rusty1s rusty1s marked this pull request as ready for review January 22, 2023 08:51
@codecov
Copy link

codecov bot commented Jan 22, 2023

Codecov Report

Merging #6487 (8489412) into master (3478f86) will not change coverage.
The diff coverage is n/a.

❗ Current head 8489412 differs from pull request most recent head 1e18a6e. Consider uploading reports for the commit 1e18a6e to get more accurate results

@@           Coverage Diff           @@
##           master    #6487   +/-   ##
=======================================
  Coverage   85.62%   85.62%           
=======================================
  Files         405      405           
  Lines       22383    22383           
=======================================
  Hits        19166    19166           
  Misses       3217     3217           
Impacted Files Coverage Δ
torch_geometric/explain/explanation.py 94.49% <ø> (ø)
torch_geometric/nn/models/captum.py 100.00% <ø> (ø)

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

Copy link
Member

@rusty1s rusty1s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool.

@rusty1s rusty1s enabled auto-merge (squash) January 22, 2023 09:27
@rusty1s rusty1s merged commit 6d6f69e into master Jan 22, 2023
@rusty1s rusty1s deleted the wsad1-explain-doc branch January 22, 2023 09:30
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