Skip to content
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

Slow loading #135

Closed
jebej opened this issue Nov 22, 2020 · 1 comment
Closed

Slow loading #135

jebej opened this issue Nov 22, 2020 · 1 comment

Comments

@jebej
Copy link

jebej commented Nov 22, 2020

This package is slow to load due to its dependence on FiniteDiff and ForwardDiff. This is somewhat problematic given it should be a "lightweight" base package, especially if packages above it do not need the functionality.

julia> @time using NLSolversBase
  9.662709 seconds (14.06 M allocations: 740.722 MiB, 3.58% gc time)
@pkofod
Copy link
Member

pkofod commented Dec 17, 2020

Well, it's not meant to be light weight. It provides AD as one if its features for Optim and NLsolve and LsqFit. However, if you look at NLSolvers.jl which is the new backend code for Optim, you will see that I specifically left out AD for this reason. Then, Optim will include AD and NLSolvers.jl will not. So if you really need something cheap, you can use NLSolvers.jl

@pkofod pkofod closed this as completed Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants