Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • O ogs
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 129
    • Issues 129
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 40
    • Merge requests 40
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ogsogs
  • ogs
  • Merge requests
  • !4055

Fixing a bug in Process::setInitialConditions under PETSc

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged wenqing requested to merge wenqing/ogs:fix_petsc into master Mar 23, 2022
  • Overview 4
  • Commits 3
  • Pipelines 2
  • Changes 3

In Process::setInitialConditions, MathLib::LinAlg::setLocalAccessibleVector(x) is missed call before x is used for the computation the initial secondary variables. The bug leads to incorrect initial secondary variable assignment for some cases when PETSc is used.

This bug fixing fixes the issue addressed in #3217 (closed). As a verification, the example in #3217 (closed) is added as a benchmark. The new PETSc related benchmark use the existing reference file. The following figures show the comparison of the results solved by Eigen and PETSc linear solver, respectively: TH1

This MR closes #3217 (closed).

  1. Feature description was added to the changelog
  2. Tests covering your feature were added?
  3. Any new feature or behavior change was documented?
Edited Mar 23, 2022 by wenqing
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix_petsc