Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 104
    • Issues 104
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 88
    • Merge requests 88
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositoriesdumux-repositories
  • dumuxdumux
  • Merge requests
  • !1226

fix/wallDistance

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Ned Coltman requested to merge fix/wallDistance into master Sep 28, 2018
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 9

Noticed this bug just recently. In cases where there is a concave domain, the wall distance was overestimated.

|
|<...........>x
|           / ^
|_________ /  :
         |/   :
         |    :
         |    :
         |____v_______

previously, the wall distance for the location x would be the nearest distance to a wall along any axis ( .. or : line). The proper wall distance should be the distance to a wall, regardless which axis this wall is on (/ line).

Not sure how efficient this method is, but it works and does not decrease the speed of the simulations considerably.

Edited Sep 28, 2018 by Ned Coltman
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/wallDistance