Comment 1 for bug 1035351

Revision history for this message
Lawrence Mitchell (wence) wrote : Re: [Bug 1035351] [NEW] Some parallel adaptive longtests fail after hybrid-assemble was merged into trunk.

On 10/08/12 16:15, Christian Jacobs wrote:
> Public bug reported:

...

> After changing the following line in assemble/Momentum_DG.F90:
> assemble_element = .not.dg.or.element_neighbour_owned(U, ele)
> to:
> assemble_element = .not.dg.or.element_owned(U, ele)

I was convinced at the time that element_neighbour_owned(U, ele) would
return .true. if ele was owned. i.e. it's a superset of
element_owned(U, ele). However, it's possible that for some corner
cases this may not be true.

Does it work if you change the test to:

.not.dg .or. (element_owned(U, ele) .or. element_neighbour_owned(U, ele))

?

Lawrence