Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O ogs
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 72
    • Issues 72
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 29
    • Merge requests 29
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ogs
  • ogs
  • Merge requests
  • !3615

Merged
Created May 10, 2021 by wenqing@wenqingMaintainer2 of 3 tasks completed2/3 tasks

Bug fixing in EffectiveThermalConductivityPorosityMixing: Avoided the porosity calculation

  • Overview 4
  • Commits 3
  • Pipelines 2
  • Changes 5

In the existing implementation of EffectiveThermalConductivityPorosityMixing, the porosity is calculated in the member function of value, which repeats the computation performed in local assembler. Besides, the porosity property of PorosityFromMassBalance cannot be used because that it requires the historical porosity value, i.e. the value of the previous time step.

With the changes in this MR, the porosity, which has already been calculated in local assemblers, is directly passed to EffectiveThermalConductivityPorosityMixing via the variable array.

  1. Feature description was added to the changelog
  2. Tests covering your feature were added? Tested by the existing tests.
  3. Any new feature or behavior change was documented?
Edited Jul 14, 2021 by wenqing
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix_MPL