Skip to content

GitLab

  • Menu
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 84
    • Issues 84
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 80
    • Merge requests 80
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • dumux-repositories
  • dumuxdumux
  • Issues
  • #943

Closed
Open
Created Oct 19, 2020 by Katharina Heck@heckMaintainer

[tests][co2] co2 tests material parameters

I noticed that currently the materialparameters in the co2 tests are set to:

// Same material parameters for every layer
materialParams_.setSwr(0.2);  
materialParams_.setSwr(0.05);

I guess this is a mistake and one of these should be Snr. In order for the tests to pass in !1607 (merged) I set the parameters to this: Swr = 0.05 Snr = 0.0

The question is if the parameters are in any way meaningful and a physical intention was behind them? If yes, we would need to change the reference solution. @holle Do you think it is necessary to change them? If not I will close this issue again

Edited Oct 19, 2020 by Katharina Heck
Assignee
Assign to
Time tracking