idaholab / moose

Multiphysics Object Oriented Simulation Environment
https://www.mooseframework.org
GNU Lesser General Public License v2.1
1.77k stars 1.05k forks source link

keep_solution_during_restore should be able to also keep the auxiliary system #19078

Closed vincentlaboure closed 1 week ago

vincentlaboure commented 3 years ago

Bug Description

For some complicated setups, keep_solution_during_restore can fail, which seems to be related to the auxiliary system not being restored.

Steps to Reproduce

This simple sabertooth example can reproduce the issue.

Impact

This can make the computation of a steady state solution painfully and unnecessarily slow.

Tag @fdkong @jortensi @nicolasmartin3

GiudGiud commented 1 year ago

Tagging myself. This will be part of the checkpoint rework effort.

YaqiWang commented 1 year ago

@loganharbour can this be closed with the latest update in checkpoint restart?

loganharbour commented 1 year ago

I think so - we now only load what we can, so it shouldn't error. That sounds satisfied, right?

We still have the outstanding issue (if desired) of loading data into other names/vectors (i.e., non direct mapping) but I don't think this issue pertains to that.

loganharbour commented 1 year ago

I think so - we now only load what we can, so it shouldn't error. That sounds satisfied, right?

We still have the outstanding issue (if desired) of loading data into other names/vectors (i.e., non direct mapping) but I don't think this issue pertains to that.

vincentlaboure commented 11 months ago

can this be closed with the latest update in checkpoint restart?

I am not sure to understand how this is related to checkpoint restart. Is the claim that that the recent checkpoint work solved this issue?

loganharbour commented 11 months ago

I might need more of an explanation on this one. I actually think that they're not related now

GiudGiud commented 1 month ago

I ll take a look over the next few weeks. There's a VTB model with a workaround but I would be down not to have that