-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Added the GraphMaskExplainer
algorithm
#6284
Conversation
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## master #6284 +/- ##
==========================================
+ Coverage 85.51% 85.57% +0.06%
==========================================
Files 403 404 +1
Lines 22024 22313 +289
==========================================
+ Hits 18834 19095 +261
- Misses 3190 3218 +28
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Hi @rusty1s , I am still seeking your attention on this. May I get some review comments on this PR, please?? Thank you... |
Yes, we discussed how to deal with this PR and have plans to move this to |
Yes, it will. Please move this PR to |
Hi @rusty1s, Any updates?? Can we do something to review and merge this PR ASAP?? |
GraphMaskExplainer
algorithm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to contrib
- feel free to extend and clean-up in a follow-up. There were some device issues I needed to fix.
This PR contains implementation of how to compute
layer-wise weights
for eachedge
in order to produceexplanations
fornode-level
,edge-level
, andgraph-level
tasks. Furthermore, this implementation is different from authors' original implementation and is fast and more memory efficient than theirs. Have addedTests
andExamples
of the proposed implementation in order to make the overall approach understandable.