Skip to content

Fix bulk_solvent::k_mask_and_k_overall_grid_search#692

Open
sizmailov wants to merge 1 commit intocctbx:masterfrom
sizmailov:fix-k_mask_and_k_overall_grid_search
Open

Fix bulk_solvent::k_mask_and_k_overall_grid_search#692
sizmailov wants to merge 1 commit intocctbx:masterfrom
sizmailov:fix-k_mask_and_k_overall_grid_search

Conversation

@sizmailov
Copy link
Copy Markdown

Two rather small changes, probably it's fine to keep in one PR.

  • Add missing selection argument to scale()
  • Initialize k_overall_best / r_best to match k_mask_best = 0.0 case

Closes #654

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.

[BUG] Missing "selection" argument in bulk_solvent::k_mask_and_k_overall_grid_search

1 participant