On Thu, Jun 24, 2010 at 01:23:50AM -0400, Ian Hinder wrote:
In r80, a patch was applied to "improve parallelisation by openmp" which changed to using offset coordinates directly in the twopunctures loop, and so removed the first loop which modified the coordinates, but didn't remove the later loop to put them back.
It seems that we agree that the later loop should also be removed now. If no one objects I will commit this later today.
Maybe we should look into running couple of iterations of a full BBH simulation in a test suite? The only problem is memory usage. This can be solved with symmetries, but then that restricts the cases you can test.
You are right. Didn't we think about a special place for large testsuites once?
PS: I tracked this down using "git svn", which allows you to treat an SVN repository as a git repository, so I could use the "git bisect" command.
Newer versions of the subversion-tools contain a similar command: svn-bisect.
Frank