Skip to content

openmp: fix compilation issues#57

Merged
haraldkl merged 2 commits intomainfrom
fix/omp
Feb 24, 2026
Merged

openmp: fix compilation issues#57
haraldkl merged 2 commits intomainfrom
fix/omp

Conversation

@haraldkl
Copy link
Copy Markdown
Member

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 22, 2026

Waf Unit Tests

  2 files    2 suites   0s ⏱️
 62 tests  62 ✅ 0 💤 0 ❌
124 runs  124 ✅ 0 💤 0 ❌

Results for commit d58bef5.

♻️ This comment has been updated with latest results.

@haraldkl haraldkl marked this pull request as ready for review February 22, 2026 11:34
@github-actions
Copy link
Copy Markdown

System Tests

71 tests   67 ✅  4h 9m 17s ⏱️
71 suites   4 💤
71 files     0 ❌

Results for commit eda2dff.

MikeW097
MikeW097 previously approved these changes Feb 23, 2026
Copy link
Copy Markdown
Member

@MikeW097 MikeW097 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haraldkl Thanks for the quick fix! I have verified it under my tests, and it behaves as expected. The decomposition still results in disjoint blocks within one partition, which is consistent with your expectation.

@haraldkl haraldkl enabled auto-merge (squash) February 24, 2026 07:53
@haraldkl haraldkl merged commit 2dce710 into main Feb 24, 2026
2 checks passed
@haraldkl haraldkl deleted the fix/omp branch February 24, 2026 07:58
@haraldkl
Copy link
Copy Markdown
Member Author

Thanks @MikeW097 the fix has now been merged back into the main branch and 2dce710 should now be usable. An advantage of using OpenMP is then also that you increase the MPI partition sizes (by reducing the number of partitions).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants