Code review comment for lp:~fluidity-core/fluidity/fix-lagr-detectors

Revision history for this message
Rhodri Davies (rhodri-davies) wrote :

Hi Michael,

I can confirm that the error is fixed on my local system, whilst detectors_parallel_adaptivity also passes locally. Sadly the buildbot branch is still red for the detectors_parallel_adaptivity case, however, which I think is a tolerance issue. Are you happy to reduce the final_location_error tolerance slightly (if this is indeed the issue on buildbot of course)? Once this test passes on buildbot I think this merge is good to go.

Rhod

« Back to merge proposal