Closed thesofakillers closed 6 months ago
As raised in #1515, the args field of EvalSpec is optional. Therefore it is possible for evals with no args to exist. Here args is None.
args
EvalSpec
None
However, currently our arg overriding code mistakingly does not support this API, since it assumes args is not None.
This PR addresses the issue with an if statement. Fixes #1515
As raised in #1515, the
args
field ofEvalSpec
is optional. Therefore it is possible for evals with no args to exist. Hereargs
isNone
.However, currently our arg overriding code mistakingly does not support this API, since it assumes
args
is notNone
.This PR addresses the issue with an if statement. Fixes #1515