Skip to content

Fix protein Conv1d dimensions and align CNN architecture with paper#21

Open
matija-marijan wants to merge 2 commits intothinng:masterfrom
matija-marijan:master
Open

Fix protein Conv1d dimensions and align CNN architecture with paper#21
matija-marijan wants to merge 2 commits intothinng:masterfrom
matija-marijan:master

Conversation

@matija-marijan
Copy link

This pull request addresses the issues explained in #20, #12, and #13.

Changes:

  • Apply Conv1d along protein sequence dimension instead of the embedding dimension (the protein embedding matrix is permuted, and the number of Conv1d input/output channels is updated)
  • Align the protein convolution network architecture with the paper (three consecutive Conv1d layers).

Notes:

  • The Conv1d dimension modification preserves sequential information in the protein embeddings.
  • The three-layer protein CNN uses ascending channel sizes, with ReLU activations after each layer, following the DeepDTA implementation.

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.

1 participant