You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
and use extraParams for the solver needs to be replaced with one where a single parameter structure is stripped of any non-solver specific parameters before. What is problemType specific is defined here
paramNames = getCobraSolverParamsOptionsForType(problemType)
but that changes over time, making it confusing what parameter is a solver, problem-type vs other type of parameter with the potential for the same parameter to be different in different parameter structures. Better to only have one parameter structure then strip that to the solver specific ones just before calling the solver. One strip function per solver.
The text was updated successfully, but these errors were encountered:
Passing non-solver specific parameters to a solver gives an error. But this approach to split parameter structures
[problemTypeParams, extraParams] = parseSolverParameters(problemType, varargin)
and use extraParams for the solver needs to be replaced with one where a single parameter structure is stripped of any non-solver specific parameters before. What is problemType specific is defined here
paramNames = getCobraSolverParamsOptionsForType(problemType)
but that changes over time, making it confusing what parameter is a solver, problem-type vs other type of parameter with the potential for the same parameter to be different in different parameter structures. Better to only have one parameter structure then strip that to the solver specific ones just before calling the solver. One strip function per solver.
The text was updated successfully, but these errors were encountered: