<html><head><meta http-equiv="content-type" content="text/html; charset=us-ascii"></head><body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div><br></div> That makes incrementally adding new options to an existing object difficult, since each call to setFromOptions() screws up previous calls.<div><br></div><div>We already have<div><br></div><div>-ksp_monitor_cancel</div><div>-ksp_converged_reason_view_cancel</div><div><br></div><div><br><div><br><blockquote type="cite"><div>On Jan 22, 2023, at 12:45 PM, Matthew Knepley <knepley@gmail.com> wrote:</div><br class="Apple-interchange-newline"><div><div dir="ltr"><div dir="ltr">On Sat, Jan 21, 2023 at 4:34 PM Barry Smith <<a href="mailto:bsmith@petsc.dev">bsmith@petsc.dev</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
If I run KSPSetFromOptions() on an options database that contains -ksp_view and then push another options database that does not contain -ksp_view and call KSPSetFromOptions() again it undoes the previous setting of that viewer. So -ksp_view is not used.<br>
<br>
I think this is the wrong model, it should not turn off the previously set view just because it is not being set in the new call, since it was previously set in the KSP. I fear some PETSc function will need another argument to properly handle this; like another flag to PetscOptionsGetViewer().<br>
<br>
Note this is for much more than KSP.<br>
<br>
Thoughts?<br></blockquote><div><br></div><div>I think it must get rid of it. Otherwise, it would stay forever since there would be no way to get rid of it.</div><div><br></div><div> Matt </div></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div><div><br></div><div><a href="http://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br></div></div></div></div></div></div></div></div>
</div></blockquote></div><br></div></div></body></html>