Barry Smith <[email protected]> writes:
>   Less code is, of course, always most appealing. Currently 
> KSPConvergedDefault() has
>
>  if (normtype == KSP_NORM_NONE) 
> SETERRQ(PetscObjectComm((PetscObject)ksp),PETSC_ERR_ARG_WRONGSTATE,"Use 
> KSPConvergedSkip() with KSPNormType of KSP_NORM_NONE");
>
> presumably this is so that a user won't mistakenly set the convergence test 
> after setting the norm to none and assume that the test will work? But does 
> this really provide anything of value?

I think it likely causes more confusion than it fixes.  And when do people 
explicitly set the convergence test to Default?

Attachment: signature.asc
Description: PGP signature

Reply via email to