Skip to content

fix restarting pmls in 2D#5641

Merged
PrometheusPi merged 1 commit intoComputationalRadiationPhysics:devfrom
pordyna:fixPMLExtentCheckForRestartingIn2D
Mar 4, 2026
Merged

fix restarting pmls in 2D#5641
PrometheusPi merged 1 commit intoComputationalRadiationPhysics:devfrom
pordyna:fixPMLExtentCheckForRestartingIn2D

Conversation

@pordyna
Copy link
Member

@pordyna pordyna commented Mar 3, 2026

1d02d93 introduced in PR #5508 breaks restarting from checkpoint in 2D simulations. This PR fixes it.

@pordyna pordyna requested a review from franzpoeschel March 3, 2026 16:17
Copy link
Contributor

@franzpoeschel franzpoeschel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you for the fix.
Note that this will probably conflict with #5634, which will probably remove this PML layout.

@pordyna pordyna requested a review from psychocoderHPC March 4, 2026 09:09
@PrometheusPi PrometheusPi merged commit 4e5228a into ComputationalRadiationPhysics:dev Mar 4, 2026
10 checks passed
@BeyondEspresso
Copy link
Member

As this PR fixes a problem that should cease to exist with #5634 , I will delete the whole hacky (!isDomainBranch)-branch in aforementioned PR, so that code maintenance is not necessary anymore.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants