Skip to content

Added support for heterogeneous graphs in GNNExplainer #10158

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
Merged
XinweiHe merged 15 commits into from
Apr 6, 2025
Merged

Added support for heterogeneous graphs in GNNExplainer #10158

XinweiHe merged 15 commits into from
Apr 6, 2025

Conversation

@XinweiHe
Copy link

@XinweiHe XinweiHe commented Apr 2, 2025 •
edited
Loading

Fixes #9112

@XinweiHe XinweiHe force-pushed the xinwei_support_gnn_explainer_hetero_v1 branch from 9c17868 to 49dfc7a Compare April 2, 2025 04:42
@XinweiHe XinweiHe force-pushed the xinwei_support_gnn_explainer_hetero_v1 branch from 8f81845 to 6312cc8 Compare April 2, 2025 04:54
@XinweiHe XinweiHe self-assigned this Apr 2, 2025
@XinweiHe XinweiHe force-pushed the xinwei_support_gnn_explainer_hetero_v1 branch from a2640db to 96ca2ba Compare April 2, 2025 04:58
wsad1
wsad1 approved these changes Apr 2, 2025
View reviewed changes
Copy link
Member

@wsad1 wsad1 left a comment

Choose a reason for hiding this comment

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

Nice work.
Ok to merge once we add more tests.

akihironitta
akihironitta reviewed Apr 3, 2025
View reviewed changes
Copy link
Member

@akihironitta akihironitta left a comment

Choose a reason for hiding this comment

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

I think, ideally, we can create three new internal classes, (one for base class, and one for each of homogenous and heterogeneous cases), and write these actual implementations in these two subclasses. This isn't blocking given the timeline though :)

@XinweiHe XinweiHe force-pushed the xinwei_support_gnn_explainer_hetero_v1 branch from 0ada1bf to 3b97c80 Compare April 4, 2025 07:30
@XinweiHe XinweiHe force-pushed the xinwei_support_gnn_explainer_hetero_v1 branch from 1d48e49 to 31a8476 Compare April 4, 2025 08:07
@XinweiHe
Copy link
Author

XinweiHe commented Apr 4, 2025

I think, ideally, we can create three new internal classes, (one for base class, and one for each of homogenous and heterogeneous cases), and write these actual implementations in these two subclasses.

will do in a follow-up.

@XinweiHe XinweiHe requested a review from akihironitta April 4, 2025 08:26
akihironitta
akihironitta approved these changes Apr 4, 2025
View reviewed changes
@akihironitta akihironitta changed the title Support hetero data for GNN Explainer Support hetero data in GNNExplainer Apr 4, 2025
@XinweiHe XinweiHe merged commit e7050f1 into master Apr 6, 2025
16 checks passed
@XinweiHe XinweiHe deleted the xinwei_support_gnn_explainer_hetero_v1 branch April 6, 2025 20:09
@XinweiHe XinweiHe changed the title Support hetero data in GNNExplainer Added support for heterogeneous graphs in GNNExplainer May 9, 2025
chrisn-pik pushed a commit to chrisn-pik/pytorch_geometric that referenced this pull request Jun 30, 2025
@pre-commit-ci @akihironitta @chrisn-pik
Support hetero data in GNNExplainer (pyg-team#10158)
9ee066c 
Fixes pyg-team#9112

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Akihiro Nitta <[email protected]>
@rutujagurav
Copy link

rutujagurav commented Jul 10, 2025

In what version of pyg is/would be the GNNExplainer support for hetero graphs available?

@benstear
Copy link

benstear commented Oct 25, 2025

Im also curious what version of pyg the GNNExplainer for heteroData will be available.

@akihironitta
Copy link
Member

akihironitta commented Oct 27, 2025

@rutujagurav @benstear The change in this PR is included in 2.7.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wsad1 wsad1 wsad1 approved these changes

@akihironitta akihironitta akihironitta approved these changes

@RexYing RexYing Awaiting requested review from RexYing

@RBendias RBendias Awaiting requested review from RBendias

@dufourc1 dufourc1 Awaiting requested review from dufourc1

Assignees

@XinweiHe XinweiHe

Labels

explain

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Roadmap] Heterogeneous Graphs Explainability Support

6 participants

@XinweiHe @rutujagurav @benstear @akihironitta @wsad1