Dylan Copeland

@vladotomov The usual usage, which is fine, is the following: for each timestep: hydro.ResetTimeStepEstimate(); // sets qdata.dt_est = infinity ode_solver->Step: call LagrangianHydroOperator::Mult UpdateQuadratureData solve for velocity, energy qdata_is_current = false;...

Yes, I agree not to change the code. At first it seemed like a simple fix to avoid potential bugs, but it is not so simple, and we don't necessarily...

@kevinhkhuynh can you please provide some sample runs for DMD? We have not put any ROM sample runs in laghos.cpp, because it would be so many lines. Instead, we keep...

There is some code to deal with this, not in the main MFEM or glvis library, see `ProlongToMaxOrder` for example in `miniapps/gslib/findpts.cpp`: https://github.com/mfem/mfem/blob/a96065ed1802ee95e7a33ee749b2ae4a6e4a6de6/miniapps/gslib/findpts.cpp#L46

We thought about it in the recent PR https://github.com/mfem/mfem/pull/2766 which added this code. This "experimental" function is for L2 or H1 spaces. It would need to be generalized for arbitrary...

Thank you @hsllo for the interesting question. Currently, libROM does not support deal.II applications, but we would gladly welcome the contribution of a PR that adds this support. Implementing this...

> Hello @dylan-copeland , Can I help you with this issue? Hi @samyakdarshan, if you would like to work on deal.II support, I will be glad to answer any questions...

@pranav-vempati Should `regression_tests/simulationLogFile` be in this PR?

@chldkdtn This PR looks fine, but the other DMD examples should also be changed in the same way.

Are there any examples in libROM, or just Laghos?