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
In some cases, e.g. where the gradient calculation via adjoint states requires a smaller step size, this might affect the function value.
To make it consistent, the calculation of fval_p and fval_m should use the same sensi_orders as fval. However, for difficult problems that might introduce some overhead in computation.
The text was updated successfully, but these errors were encountered:
The finite difference calculation in
check_grad
usessensi_orders=(0, 1)
(or higher) forfval
andsensi_orders=(0,)
forfval_p
andfval_m
.pyPESTO/pypesto/objective/base.py
Lines 563 to 566 in b628397
In some cases, e.g. where the gradient calculation via adjoint states requires a smaller step size, this might affect the function value.
To make it consistent, the calculation of
fval_p
andfval_m
should use the samesensi_orders
asfval
. However, for difficult problems that might introduce some overhead in computation.The text was updated successfully, but these errors were encountered: