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

Disconnected GNN: Catalyst-Adsorbate separator #44

Open
wants to merge 134 commits into
base: main
Choose a base branch
from

Conversation

AlvaroC-ML
Copy link
Collaborator

@AlvaroC-ML AlvaroC-ML commented Jun 13, 2023

By adding --is_disconnected as a parameter, a data transform is activated that deletes the edges between catalyst and adsorbate vertices.

@AlvaroC-ML AlvaroC-ML changed the title [WIP] Implementing disconnected gnn Edge deleter Jun 14, 2023
@AlvaroC-ML AlvaroC-ML changed the title Edge deleter Catalyst-Adsorbate separator Jun 14, 2023
@AlvaroC-ML AlvaroC-ML changed the title Catalyst-Adsorbate separator [WIP] Catalyst-Adsorbate separator Jun 14, 2023
@AlvaroC-ML AlvaroC-ML changed the title [WIP] Catalyst-Adsorbate separator Catalyst-Adsorbate separator Jun 14, 2023
…in yaml files. Also modified some yaml experiment files.
…oint for every (catalyst, adsorbate, cell) tuple.
…s get concatenated, and then passed through a 2-layer MLP to return a single value. Testing still needed.
…model doesn't use it, and we start writing the independent model.
@vict0rsch vict0rsch changed the title Catalyst-Adsorbate separator Disconnected GNN: Catalyst-Adsorbate separator Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants