-
Notifications
You must be signed in to change notification settings - Fork 3
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
ForwardDiff.gradient fails for global map #17
Comments
After eliminating unnecessary type definitions in the function, and separating the constant parameters in the argument list, we are left with an error related to the matrix left divide and the sparse LU decomposition |
Note that ReverseDiff.jl, which is the better choice because of the large number of inputs (~2806) and small number of outputs (1), also has trouble with the sparse matrix left divide. One workaround is the ILUZero.jl package. When differentiating a function with a sparse matrix left divide given by ILUZero.jl, I get the following error. |
Got rid of adjoint call by defining a function to compute just the cost function value, but ForwardDiff.gradient fails at |
The global map relies upon an optimization that uses a explicitly supplied gradient. It would be nice to check that the gradient is accurate using a comprehensive method, such as that of ForwardDiff.jl or Zygote.jl. The TMI.jl is not sufficiently generic to be used with ForwardDiff. It appears that a ForwardDiff container for the control vector cannot be handled by the TMI.jl routine. Other issues related to initialization of any real number seem to be fixed.
The text was updated successfully, but these errors were encountered: