Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • W waLBerla
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 40
    • Issues 40
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • waLBerla
  • waLBerla
  • Merge requests
  • !493

Update !487 and avoid implicit OpenMP parallelization in mesh::BoundarySetup

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Christoph Schwarzmeier requested to merge partially-revert-mr-487 into master Nov 07, 2021
  • Overview 2
  • Commits 2
  • Pipelines 2
  • Changes 1

This merge request updates !487 (merged) where manual loops where exchanged with WALBERLA_FOR_ALL_CELLS macros. Since the macros are implicitly parallelized when using OpenMP, function setDomainCells() does not work as expected anymore. The content of vector domainCells will change when OpenMP parallelization is used such that its size can not be evaluated correctly in the later course of the algorithm. Thus OpenMP is turned off in the macro

This is the reason for the failure of the ComplexGeometry benchmark on the current master (see job 672374).

Edited Nov 08, 2021 by Markus Holzer
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: partially-revert-mr-487