[Carpet] nPoints assertion

Ian Hinder hinder at gravity.psu.edu
Thu Mar 20 01:17:48 CET 2008


Erik Schnetter wrote:
> On Mar 19, 2008, at 13:05:16, Ian Hinder wrote:
>> Erik Schnetter wrote:
>>> On Mar 17, 2008, at 16:45:42, Ian Hinder wrote:
>>>
>>>> Hi,
>>>>
>>>> There is an assertion in Recompose.cc (line 1137)
>>>>
>>>>     assert (mynpoints.at(n) > 0);
>>>>
>>>> which aborts my run when I run at low resolution on 16 processors.
>>>> Erik, you suggested this check might not be necessary, and indeed
>>>> things
>>>> work fine with the assertion removed.  OK to commit a patch or should I
>>>> do more specific testing?
>>>
>>>
>>> OK to commit.  More specific testing would be difficult, since this
>>> assertion failure will only catches illegal cases which are not supposed
>>> to reach this point in code anyway.

OK - I've committed the patch.

-- 
Ian Hinder
hinder at gravity.psu.edu
http://www.gravity.psu.edu/~hinder


More information about the developers mailing list