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

descriptor_type = "2D" #4

Open
kimsu55 opened this issue Oct 4, 2024 · 1 comment
Open

descriptor_type = "2D" #4

kimsu55 opened this issue Oct 4, 2024 · 1 comment

Comments

@kimsu55
Copy link

kimsu55 commented Oct 4, 2024

Thank you for your excellent work - it's benefited my work a lot!

According to "scripts/config-example.yaml", descriptor_type = "2D" and feature_type = "rdkit" are used together.
Are they compatible?

The default model "GIN" for the "2D" descriptor only uses the atomic number as input to embed it into the 128-dimensional representation specified in "muben/model/gin/gin.py -> self.emb = nn.Embedding(max_atomic_num, d_hidden)"

Could you please clarify it?

@Yinghao-Li
Copy link
Owner

Hi,

You are right. When the 2D descriptors are selected, the feature_type argument will be ignored and its value does not matter. You can safely remove the line if you find it confusing.

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

No branches or pull requests

2 participants