Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • dumux dumux
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 102
    • Issues 102
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 91
    • Merge requests 91
  • 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
  • !1301

[sequential 2p2c flash] fix flash

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Beatrix Becker requested to merge fix/sequential-flash into master Nov 19, 2018
  • Overview 2
  • Commits 7
  • Pipelines 0
  • Changes 9

Fix sequential flash:

  • correct and more general calculation of equilibrium constants (divided by wrong phase pressure before) => small changes compared to reference
  • generalize 2-phase check by using phase mass fraction instead of feed mass fraction (first component had to be main component of first phase before this change) => does not change solution compared to reference
  • cleanup, comment, add literature

This was done/checked for the 1p2c flash and the 2p2c saturation flash, too, and the reference solutions were corrected.

Edited Dec 05, 2018 by Beatrix Becker
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/sequential-flash