Orange to points partition OASIS decomposition #1361
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
orange to points partition
Pull Request Summary
Change the orange to points partition OASIS decomposition for regular and curvilinear grids.
Description
The current OASIS decomposition for regular and curvilinear grids is 'orange', described by a global offset and a local extent. The local extent in the case of WAVEWATCHIII is always equal to 1. In this situation, it is more natural to describe the OASIS decomposition as a points partition. Furthermore, the orange partition has a limitation of 200 segments, while the points partition can have any number of segments. The code was reviewed to remove unnecessary variables and to deallocate vectors once they are no longer needed.
There is no change of answers expected, as the OASIS decomposition will be the same but described in a different way. Please refer to the OASIS manual for details on OASIS partitioning.
Lables that could be added: enhancement
Issue(s) addressed
Commit Message
Change the orange to points partition OASIS decomposition
Check list
Testing
Changes are fully covered by the ww3_tp2.14 regression tests, as the changes proposed only affect OASIS coupling.
Regression tests have been run on the Met Office XC40 machine using the gnu compiler.
No changes in the regression tests are expected.