-
Notifications
You must be signed in to change notification settings - Fork 21
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
fixed modifcation parsing #389
base: master
Are you sure you want to change the base?
Conversation
csbrasnett
commented
Oct 8, 2024
- -mods flag now only read once for all modifications
- patch all molecule termini as proteins, so any modification in addition to these ones are also applied.
- add more modifications from vermouth to martini3 forcefield
- change logging level and fix tests as per change in record level #385
- -mods flag now only read once for all modifications - patch all molecule termini as proteins, so any modification in addition to these ones are also applied. - add more modifications from vermouth to martini3 forcefield - change logging level and fix tests as per marrink-lab#385
- -mods flag now only read once for all modifications - patch all molecule termini as proteins, so any modification in addition to these ones are also applied. - add more modifications from vermouth to martini3 forcefield - change logging level and fix tests as per marrink-lab#385
# Conflicts: # polyply/src/apply_modifications.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but that one itp test case I would suggest to include; also could we have one test case where more than just one modification is added like terminal plus histidine?
;[ edges ] | ||
;BB SC1 | ||
;SC1 SC2 | ||
;SC1 SC3 | ||
;SC2 SC3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these commented out on purpose?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. For some reason modifications.ff can't be read if they're included, and excluding them doesn't seem to affect them being applied correctly 🤷♂️
polyply/src/apply_modifications.py
Outdated
LOGGER.warning("meta_molecule has come from itp. Will not attempt to modify.") | ||
LOGGER.info("meta_molecule has come from itp. Will not attempt to modify.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
having a test for this one would be ideal
Re: the ignoring if coming from an itp test, two quick questions:
|
- input -seq HIS:5 - with two modifications, HIS1:HIS-HD HIS1:NH2-ter, so the first residue is modified twice, both in its terminal type, and in its histidine variant.
- input -seq HIS:5 - with two modifications, HIS1:HIS-HD HIS1:NH2-ter, so the first residue is modified twice, both in its terminal type, and in its histidine variant.