-
Notifications
You must be signed in to change notification settings - Fork 44
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
Updated Ligand Motifs Code #59
Open
toadlover
wants to merge
53
commits into
main
Choose a base branch
from
updated_ligand_discovery_search_code_ag_may_2024
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+18,723
−441
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… option to print out matrices
…uplicated from a motifs list
…ector of indices instead of a single value
…o remove usage of unused parameters
…used in other apps on poses generated in runtime
Adjusting use of downstream score functions
…e in optional use of atrrep function after high res dock
…tifs and record residues in the pose that yield motifs
…ct motifs from a placed ligand and determine if they look like real motifs
…s to chunks of motif library if we only want to look at motifs with specific residues and atoms
… reflect added functionality (space fill method, collecting motifs off of good-placing ligands, protocol verbosity)
toadlover
added
90 standard tests
ready_for_review
This PR is ready to be reviewed and merged.
and removed
90 standard tests
labels
May 9, 2024
…score function in high res dock need to beautify and add option for motifs::highresdock_with_whole_score_fxn
…that I do not remember why I had them,removed commented code for better presentation
…hen adjust score terms
…ange ddg scores otherwise)
…to allow customization of rounding error tolerance when calculating sin and cos
…for users to define tolerance Users can now use option run:sin_cos_range_tolerance to use tolerance values that are different from 0.001. Additionally, sin_cos_range can now allow any value pass with the use of run:keep_beyond_sin_cos_range being set to true.
Reasoning is that library levels requirement makes it impossible for numeric code to call for basic (which is where options is).
…is() Increased tolerance to better account for rounding errors
…de of the native range A warning is thrown that the value is out of range, and the value is then set to the corresponding range bound, just like is done if using BOINC.
…breaks the function in benchmark tests for pyrosetta (works in other builds) I adjusted sin_cos_range anyway to always return -1 or 1 if the value is outside the range with tolerance
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The code I am contributing here are updates to improve the functionality of the ligand motif code. This includes improvements to the following protocols:
motifs/LigandDiscoverySearch - Created a method for scoring placed ligands based on how much of a user-defined binding pocket is filled, improved use of score functions, collection and analysis of motifs collected off of placed ligands to confirm contacts against residues of interest
motifs/IdentifyLigandMotifs - Moved usage of process_for_motifs from private to public so that the function to collect motifs can be used on poses in runtime (and not exclusively from imported files). The process_for_motifs function is now used in LigandDiscoverySearch.
Added new options in options_rosetta.py under motifs for use in LigandDiscoverySearch.
Added public app remove_duplicate_motifs.cc, which removed motifs that are considered duplicates based on similar distance and angle of atoms involved. This helps improve runtimes of protocols that use lists of motifs by not considering the use of motifs that are effectively very similar/same. I have added an integration test as well.