Wrong comment in the class DGL_NeuralFP #68

Closed
opened 2025-10-14 17:43:12 -06:00 by navan · 0 comments
Owner

Originally created by @Yuhan-Fei on 6/20/2021

Hi @kexinhuang12345 ,
Thank you for the great works for the DTI predictions. We found you comment the class DGL_NeuralFP as GAT in encoders.py (LINE:#336. The comment said ## adapted from 2fbf5fd6ac/python/dgllife/model/model_zoo/gat_predictor.py) . However, the DGL_NeuralFP is a different model apparently. Due to GAT is a very popular model for the small molecules, will you update the GAT for the Deeppurpose recently ?
Best

*Originally created by @Yuhan-Fei on 6/20/2021* Hi @kexinhuang12345 , Thank you for the great works for the DTI predictions. We found you comment the class DGL_NeuralFP as GAT in encoders.py (LINE:#336. The comment said ## adapted from https://github.com/awslabs/dgl-lifesci/blob/2fbf5fd6aca92675b709b6f1c3bc3c6ad5434e96/python/dgllife/model/model_zoo/gat_predictor.py) . However, the DGL_NeuralFP is a different model apparently. Due to GAT is a very popular model for the small molecules, will you update the GAT for the Deeppurpose recently ? Best
navan closed this issue 2025-10-14 17:43:13 -06:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github/DeepPurpose#68
No description provided.