Remove AnalysisNeighborhood::setUseBoundingBox()
authorTeemu Murtola <teemu.murtola@gmail.com>
Tue, 13 Jan 2015 18:48:41 +0000 (20:48 +0200)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Mon, 19 Jan 2015 05:12:13 +0000 (06:12 +0100)
commit0cb539272e13bf71f6b99551fc8585243f2d587f
tree8ef574974bafe32ba26eb23ffafad3401d9e146b
parentd594ca5d60df3b5d540cc1e4c1c91fd3c1f3c57d
Remove AnalysisNeighborhood::setUseBoundingBox()

Originally, this was added to provide more control for the caller about
how the search is done, with main user probably being more comprehensive
tests.  This method was possible because at that time, all the
coordinates were clamped to the edges of the grid when determining the
cell, so it was at most a bit inefficient if some positions were outside
the grid.

But now, there is no clamping, so this method no longer works correctly:
the grid must cover all the reference positions (i.e., be at least the
size of the bounding box), so that the reference positions really are
within the grid cell to which they belong.  Otherwise, when the grid
cells to search are pruned based on their distance from the test
position, some pairs could be missed.

Change-Id: I94ae4081f456c08ff89779ef20b91be0ca061d96
src/gromacs/selection/nbsearch.cpp
src/gromacs/selection/nbsearch.h