You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, there is a significant amount of duplicate code in the implementation of the GAT and GCN saliency maps. It would be useful to refactor this code before landing the GraphSage interpretability PR.
User Story
As a developer, I would like the code implementing saliency maps to clear, concise and modular to make it easier to implement new saliency maps.
Done Checklist
Produced code for required functionality
Tests written and coverage checked
Code review performed
Documentation on Google Docs (if applicable)
Documentation in repo
Version number reflects new status
CHANGELOG.md updated
Team demo
The text was updated successfully, but these errors were encountered:
Description
Currently, there is a significant amount of duplicate code in the implementation of the GAT and GCN saliency maps. It would be useful to refactor this code before landing the GraphSage interpretability PR.
User Story
As a developer, I would like the code implementing saliency maps to clear, concise and modular to make it easier to implement new saliency maps.
Done Checklist
The text was updated successfully, but these errors were encountered: