-
Notifications
You must be signed in to change notification settings - Fork 288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SlepcEigenSolver clear function resetting the default solver type which may prevents calling solve function properly #3341
Comments
@roystgnr suggested to remove the call to |
The call to I disagree with your point 1. above since the original purpose of the Solver class' This PR reminds me that we kind of lost track of #3208 as well. |
This:
in
void SlepcEigenSolver<T>::clear ()
function is giving me a trouble.I've called
void set_eigensolver_type (const EigenSolverType est)
to set the desired the solver type and need to callEigenSystem::solve ()
(that calls one of the solve functions inSlepcEigenSolver
multiple times). But because thatclear
function resets the default solver type and is called by all the solve functions, I cannot make the second solve call to use the same non-default solver type I set previously. Because of this, I have to use a different way to set the solver type with SLEPc command-line option-eps_type
, which defeats the purpose of havingset_eigensolver_type
function.I propose to remove that line because,
solve
functions but not options set by users to the solver in my opinion;_position_of_spectrum
, etc., which should be cleared as well if we think we should reset the solver type inclear
;clearSolveData
and called by solve functions, but I think it is unnecessary. Users can reset the solver type manually if they want.I've tried removing that line and ran all MOOSE tests. All tests past. I will push up a PR to let civet check if there are any implications.
The text was updated successfully, but these errors were encountered: