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

Draft: Add pKa and hydrophobicity to amino acid #66

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

aukeheerdink
Copy link
Contributor

In regard to issue #65

Made a small beginning to implement this issue but @douweschulte mentioned that you (@di-hardt) might have some code laying around that is similar to what I'm trying to do. Do you have something by any chance? Or some ideas on how to implement this? @douweschulte is already thinking about how to mold this into a trait (with IdentifiedPeptides).

My current idea is to make a separate module with these static tables and the pka function would work as a table lookup function basically. I would implement something similar for the hydrophobicity.

@douweschulte douweschulte marked this pull request as draft February 11, 2025 13:27
@douweschulte
Copy link
Member

The trait proposed here allows for implementing the pKa basic behaviour for any pKa source and for any AminoAcid type. This should be generic enough for it to work for most use cases. I have some questions on how this should handle modifications, right now it takes a list of SimpleModification (so no ambiguous or cross-linked stuff), which to my mind indicates side chain modifications which would mean that terminal modifications are not taken into account properly.

@di-hardt I remember something about triplet or even quadruplet lookup tables for pKa values, should we think about some structure to allow that?

Then some more things that need to be done are more documentation, placing this into a separate file, and the implementation for the peptide_pKa. Additionally I am not overly happy about how all the names turned out, the normal casing pKa does not look best in all places. Any thoughts on this would be appreciated.

@douweschulte douweschulte linked an issue Feb 11, 2025 that may be closed by this pull request
@douweschulte douweschulte linked an issue Feb 13, 2025 that may be closed by this pull request
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.

Add the pKa and hydrophobicity index to aminoacid_properties Use trait IsAminoAcid
2 participants