An error occurred while fetching the assigned milestone of the selected merge_request.
ENH: cyclicAMI: add non-blocking for GAMG
Summary
- see #3052 (closed) : add polling support for processorGAMGInterfaceField
- extend cyclicAMI, cyclicACMI to add non-blocking support for cyclicA(C)MIGAMGInterfaceField
Resolved bugs (If applicable)
Risks
- processorGAMGInterfaceField fix is for adding polling and not risky
- cyclicAMI, cyclicACMI support has been tested little
Edited by Mattijs Janssens
Merge request reports
Activity
Filter activity
assigned to @andy
- Resolved by Mattijs Janssens
- Resolved by Mattijs Janssens
- Resolved by Mattijs Janssens
- Resolved by Mattijs Janssens
- tested tutorials/incompressible/pimpleFoam/laminar/mixerVesselAMI2D/mixerVesselAMI2D
- v2306-rc1 vs feature-nonblockingGAMG
- serial run is exactly the same
- parallel (using 4 processors scotch - causes no local contributions) slight differences between the two versions.
- this is because of the neighbourValue caching:
- mesh might have moved so the AMI addressing is different
- U_0 somehow (R&C?) triggers patchNeighbour field caching but gets overwritten with solved-for U. This does not reset the patchNeighbour cache but does reset the upToDate state. Hence the patchNeighbour cache is now out of date. For now disabling patchNeighbour caching.
- no problems in valgrind
- no problems with processor-agglomeration
- re-ran with 2 processors hierarchical (so a processor also has local contributions)
Edited by Mattijs Janssensadded 1 commit
- 2d26d4b2 - STYLE: cyclicAMI: add non-blocking for GAMG. See #3052 (closed)
- tested tutorials/incompressible/pimpleFoam/RAS/oscillatingInletACMI2D
- but switched off turbulence (since differs with localConsistency)
- v2306-rc1 vs feature-nonblockingGAMG
- serial run is exactly the same
- parallel (4 processors scotch) exactly the same
- no problem in valgrind
- no problem with processor-agglomeration
Edited by Mattijs Janssensmentioned in commit 3217726b
added 1 commit
- 3217726b - ENH: cyclicAMI: revert to not-caching patchNeighbourField. See !651 (merged).
mentioned in commit 2528e046
added 1 commit
- 2528e046 - ENH: cyclicAMI: caching patchNeighbourField. See !651 (merged).
- Resolved by Mattijs Janssens
- Resolved by Mattijs Janssens
- Resolved by Mattijs Janssens
mentioned in commit 91252f33
added 2 commits
- b48e3d2f - BUG: multiWorld: switch to inter-world communicator. Fixes #3053 (closed).
- 91252f33 - ENH: cyclicAMI: caching patchNeighbourField. See !651 (merged).
mentioned in commit 04654207
Please register or sign in to reply