Closed benegee closed 4 weeks ago
This checklist is meant to assist creators of PRs (to let them know what reviewers will typically look for) and reviewers (to guide them in a structured review process). Items do not need to be checked explicitly for a PR to be eligible for merging.
NEWS.md
.Created with :heart: by the Trixi.jl community.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.13%. Comparing base (
8665300
) to head (7b11b29
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Can we have a CI test for this?
I am still unhappy with this fix. While new_p4est
allows to just allocate user data, load_p4est
requires to also provide the data. So in the end I am only saving the data to have memory allocated upon restart.
I also do not know if there was a good reason for not saving user data in the first place.
The solution is to use p4est_reset_data
!
https://github.com/cburstedde/p4est/issues/308
So with this, is it possible to restart a simulation and refine the initial condition?
So with this, is it possible to restart a simulation and refine the initial condition?
Just checked with this example here and it seems to work.
Just checked with this example here and it seems to work.
Cool, thanks for picking this up!
So with this, is it possible to restart a simulation and refine the initial condition?
Just checked with this example here and it seems to work.
Do we have a test for this? Or is it covered by the tests we already have?
Do we have a test for this? Or is it covered by the tests we already have?
So the newly introduced test does exactly this. So in some sense we "test for crash".
Thanks for your feedback!
Following @JoshuaLampert 's comment above and https://github.com/trixi-framework/Trixi.jl/pull/1384#issuecomment-1505878188 I removed ode_default_options()
in elixir_advection_{extended,restart,restart_amr}.jl
for p4est_2d_dgsem
and tree_2d_dgsem
.
In the new elixir_advection_restart_amr.jl
for p4est_2d_dgsem
and tree_2d_dgsem
I set adapt_initial_condition = true
. This still tests the restarting capability in general.
Thanks! However, some test tolerances are not satisfied right now: https://github.com/trixi-framework/Trixi.jl/actions/runs/9398803634/job/25884924262?pr=1915#step:7:2874
Thanks! However, some test tolerances are not satisfied right now: https://github.com/trixi-framework/Trixi.jl/actions/runs/9398803634/job/25884924262?pr=1915#step:7:2874
Yes! I wanted to update them, but it seems I also introduced some issue in the MPI test. I'm on it.
It seems to work now!
This fixes restating when using AMR. Resolves #1914