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/> >
