JuliaStats / KernelDensity.jl

Kernel density estimators for Julia
Other
175 stars 40 forks source link

Fix .minimum -> Optim.minimizer. #40

Closed pkofod closed 7 years ago

ararslan commented 7 years ago

Does this require bumping the minimum Optim version to 0.7.0 for the tests to pass on 0.6?

pkofod commented 7 years ago

No, I believe this Optim.minimizer way of doing it has been in Optim since v0.5.0. That symbol/field-error is new to me. I am currently unaware of what is causing that, but I'll see if I can put out a v0.7.1 that doesn't suffer from this asap.

pkofod commented 7 years ago

Can you restart tests? V0.6 should pass now