Skip to content

post_process and simulation disagree on the Lagrangian void-fraction index position for bubbles_lagrange + mhd/elasticity/cont_damage #1565

Description

@sbryngelson

Symptom: post_process misreads simulation restart/output layouts for bubbles_lagrange combined with mhd, elasticity/stress, or cont_damage — the void-fraction field lands at a different index than post expects. No checker forbids the combinations.

Mechanism: on master, post folds beta into sys_size mid-build, BEFORE the mhd/elasticity/cont_damage blocks (src/post_process/m_global_parameters.fpp:480-483 vs :485+), while simulation writes beta at sys_size + 1 after the complete system (src/common/m_mpi_common.fpp:115-126).

Introduced: the original lagrange PR #749 was consistent; the divergence came with the MPI refactor #831 (8e94d0a5f, 2025-06-13, @wilfonba), which moved post's beta into sys_size ahead of the pre-existing feature blocks. Tagging @wilfonba for context.

Fix: #1552's unified equation-index builder (one s_initialize_eqn_idx shared by all three executables) makes the layouts agree by construction.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions