Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix box refinement with restart #1184

Merged
merged 9 commits into from
Jul 2, 2024
Merged

Fix box refinement with restart #1184

merged 9 commits into from
Jul 2, 2024

Conversation

hepap
Copy link
Collaborator

@hepap hepap commented Jul 2, 2024

Description

Box refinement was applied when restarting simulations, changing the restarting triangulation and leading to undesirable behavior.

Solution

A boolean stating if the simulation restarts is now checked before applying the box refinement in box_refine_mesh(const bool &restart).

Testing

A new test simulate the lpbf benchmark case using the extra-extra-coarse mesh with restart and box refinement. Temperature statistics and mass conservation monitoring are the outputted metrics (to track both the change in HT and NS-VOF solvers).

Documentation

No documentation needs to be modify.

@hepap hepap requested review from blaisb and oguevremont July 2, 2024 15:13
@hepap hepap mentioned this pull request Jul 2, 2024
11 tasks
@blaisb blaisb merged commit 64885a4 into master Jul 2, 2024
8 checks passed
@blaisb blaisb deleted the fix_box_ref_with_restart branch July 2, 2024 20:35
M-Badri pushed a commit to M-Badri/lethe that referenced this pull request Sep 29, 2024
Description
Box refinement was applied when restarting simulations, changing the restarting triangulation and leading to undesirable behavior.

Solution
A boolean stating if the simulation restarts is now checked before applying the box refinement in box_refine_mesh(const bool &restart).

Testing
A new test simulate the lpbf benchmark case using the extra-extra-coarse mesh with restart and box refinement. Temperature statistics and mass conservation monitoring are the outputted metrics (to track both the change in HT and NS-VOF solvers).

Documentation
No documentation needs to be modify.

Former-commit-id: 64885a4
blaisb pushed a commit that referenced this pull request Sep 30, 2024
Description
Box refinement was applied when restarting simulations, changing the restarting triangulation and leading to undesirable behavior.

Solution
A boolean stating if the simulation restarts is now checked before applying the box refinement in box_refine_mesh(const bool &restart).

Testing
A new test simulate the lpbf benchmark case using the extra-extra-coarse mesh with restart and box refinement. Temperature statistics and mass conservation monitoring are the outputted metrics (to track both the change in HT and NS-VOF solvers).

Documentation
No documentation needs to be modify.

Former-commit-id: 64885a4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants