OK, makes sense. I didn't know monitors were called before the step. Thanks, On Thu, Feb 18, 2021 at 7:25 AM Matthew Knepley <[email protected]> wrote:

Stefano is right. Changes in convergence should probably go in TSPostStep.

Thanks,

Matt

On Thu, Feb 18, 2021 at 7:22 AM Stefano Zampini <[email protected]> wrote:

...

Mark

monitors are not supposed to change the TS. You can think at monitors being 'const' methods of the TS. Also, TSMonitor is called at the beginning of each step , see here https://gitlab.com/petsc/petsc/-/blob/master/src/ts/interface/ts.c#L4169

Il giorno gio 18 feb 2021 alle ore 15:16 Mark Adams <[email protected]> ha scritto:

...

TSSetConvergedReason(ts,TS_CONVERGED_USER);

does not seem to stop the iteration in a user monitor function. I have verified that it works from a post step method. Is this intentional?

Mark

-- Stefano

-- What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead. -- Norbert Wiener

https://www.cse.buffalo.edu/~knepley/ <http://www.cse.buffalo.edu/~knepley/>