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

revert margin to positive, rescore immediately after refining #214

Merged
merged 4 commits into from
Jun 16, 2023

Conversation

diogomart
Copy link
Member

No description provided.

@diogomart
Copy link
Member Author

Using 34267 ligands, here's the number of results for which the ranking of predicted free energies is in ascending order:

version, rigid_only, with_flex
 v1.2.3,      34267,     34267
 v1.2.4,      29872,     25613
this PR,      34267,     34267

Also plotting INTRA vs UNBOUND for the top pose, from this PR, with and without flex side chains
dock-fix211-9bffe99-bin-9poses-1cpu-flex-withfinal
dock-fix211-9bffe99-bin-9poses-1cpu-withfinal

src/lib/vina.cpp Outdated
@@ -1009,8 +1014,7 @@ void Vina::global_search(const int exhaustiveness, const int n_poses, const doub
// Clean up by putting back the best pose in model
m_model.set(poses[0].c);
} else {
std::cerr << "WARNING: Could not find any conformations completely within the search space.\n";
std::cerr << "WARNING: Check that it is large enough for all movable atoms, including those in the flexible side chains.\n";
std::cerr << "WARNING: Zero poses in output container after global search. Something went wrong.\n";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor optimization of the warning needed :-)

@diogomart
Copy link
Member Author

Thanks @atillack ;)

@diogomart diogomart merged commit 726844f into ccsb-scripps:develop Jun 16, 2023
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.

2 participants