Skip to content

Conversation

@aditya0by0
Copy link
Member

fixes #29

@aditya0by0 aditya0by0 self-assigned this Nov 22, 2025
aditya0by0 and others added 2 commits November 22, 2025 14:24
Updated the README to use a more concise format for model configuration options and hyperparameters.
@sfluegel05
Copy link
Contributor

Thanks for adding this. Could you add one or two sentences each for (a) what is an augmented GNN? (b) What do the three pooling variants do? (c) And what is the difference between GAT and GATv2 (with a reference to the paper). In addition, it would be nice to have an image that shows how a molecule is represented in the augmented GNN

1 similar comment
@sfluegel05
Copy link
Contributor

Thanks for adding this. Could you add one or two sentences each for (a) what is an augmented GNN? (b) What do the three pooling variants do? (c) And what is the difference between GAT and GATv2 (with a reference to the paper). In addition, it would be nice to have an image that shows how a molecule is represented in the augmented GNN

Expanded the section on augmented graphs to explain the use of artificial nodes representing functional groups. Added details on connection schemes and provided commands for model and data configuration.
Copy link
Contributor

@sfluegel05 sfluegel05 left a comment

Choose a reason for hiding this comment

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

This looks good to me. Thanks!

@sfluegel05 sfluegel05 merged commit 3edf151 into dev Dec 8, 2025
2 checks passed
@sfluegel05 sfluegel05 deleted the documentation branch December 8, 2025 14:20
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.

Sorting configs and documentation

3 participants