Skip to content
Snippets Groups Projects

Volume-of-Fluid: better tests and make it actually work

Merged Michael Kuron requested to merge vof into master

The scheme corresponds to calculating the overlap volume of a cell shifted by the flow velocity with its neighbor cell. The new test does that explicitly, but generates convoluted expressions which should not be used directly because they are so slow to evaluate. Thanks to this test, an incorrect sign in the implementation was found and fixed, as well as improper scaling of the fluxes.

@alexander.reinauer, this could be important to you because you are the only other person using volume-of-fluid advection.

Edited by Michael Kuron

Merge request reports

Pipeline #25210 passed

Pipeline passed for 3fe1aaf4 on vof

Test coverage 77.70% (0.02%) from 1 job

Merged by Markus HolzerMarkus Holzer 4 years ago (Jul 10, 2020 7:18pm UTC)

Loading

Pipeline #25216 passed

Pipeline passed for cfd770cf on master

Test coverage 77.70% (0.02%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading