Skip to content

Explainer minor improvements and bug fixes #10154

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Explainer minor improvements and bug fixes #10154

wants to merge 14 commits into from

Conversation

zechengz
Copy link
Member

@zechengz zechengz commented Mar 31, 2025 •
edited
Loading

  • Minor improvements for explainer and bug fixes for examples
  • Hack to fix the graphmask explainer with model such as GAT which has 3D tensor (with multiple heads)

@zechengz zechengz self-assigned this Mar 31, 2025
@zechengz zechengz changed the title Explainer minor improvements Explainer minor improvements and bug fixes Apr 2, 2025
@zechengz zechengz marked this pull request as draft April 2, 2025 16:37
@zechengz zechengz marked this pull request as ready for review April 11, 2025 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wsad1 wsad1 Awaiting requested review from wsad1 wsad1 is a code owner

@RexYing RexYing Awaiting requested review from RexYing RexYing is a code owner

@RBendias RBendias Awaiting requested review from RBendias RBendias is a code owner

@dufourc1 dufourc1 Awaiting requested review from dufourc1 dufourc1 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@zechengz zechengz

Labels
example explain
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zechengz @puririshi98