[Users] Upper-face reflection in ReflectionSymmetry.

Erik Schnetter schnetter at cct.lsu.edu
Wed Sep 29 10:43:25 CDT 2010


On Wed, Sep 29, 2010 at 10:20 AM, Frank Loeffler <knarf at cct.lsu.edu> wrote:
> On Wed, Sep 29, 2010 at 03:56:31PM +0200, Eloisa Bentivegna wrote:
>> I propose the attached patch. In brief, since the loop runs over the
>> "source" points in descending order and the "destination" points in
>> ascending order, then the index offset between the two sets should
>> just be 2 (or 1 if one is staggering the origin), rather than twice
>> the ghost size as in the lower-face case.
>>
>> The change affects branches that are only executed upon
>> reflection_upper_*, so most people won't see a difference. Any
>> input?
>
> I believe the patch is correct. However, it needed some serious
> head-wrapping on my side, so that I would like to have someone else
> look at the patch as well.

Eloisa

Yes, your patch is correct. I used Mathematica to solve for ioff and
received the same expression as you. I have applied the patch.

-erik

-- 
Erik Schnetter <schnetter at cct.lsu.edu>   http://www.cct.lsu.edu/~eschnett/


More information about the Users mailing list