Fix visited flag

[roigcarlo]

That should be an OR

[roigcarlo]

Also we should improve the tests for this, its alarming that this were not catch :S

[philbucher]

Eieiei this should solve the problem right?
Basically you were again accumulating multiple times the values like before right?

[roigcarlo]

Actually it was worst than before :P.
Was a corner case before, with the and and or swaped it turned it quite more frequent

[mpentek]

For me it works, seems to solve the problem in #7109. Perhaps @AndreasWinterstein can also say if this solves it for him as well, but I am quite confident it should.

[philbucher]

@roigcarlo should we port this also to the release?

[roigcarlo]

Well release is done, we should add it to the next one (probably 8.0.1) with a couple of other important bugfixes. I can make a release once this is merged.

[philbucher]

Well release is done, we should add it to the next one (probably 8.0.1) with a couple of other important bugfixes. I can make a release once this is merged.

ok makes sense

[philbucher]

@roigcarlo can you add a small test? I.e. it should give the same results in OpenMP and MPI, this way I think we are on the safe side in the future

[mpentek]

Thanks!

[roigcarlo]

Will add the test next week

[loumalouomega]

Will add the test next week

There is a OMP test in #7062