IntervalOptimisation.jl icon indicating copy to clipboard operation
IntervalOptimisation.jl copied to clipboard

Add `unify` keyword and default if to `true`

Open Suavesito-Olimpiada opened this issue 3 years ago • 6 comments

fix #48

Suavesito-Olimpiada avatar May 27 '21 18:05 Suavesito-Olimpiada

Still to add tests and discuss the proper algorithm to unify boxes.

Suavesito-Olimpiada avatar May 27 '21 18:05 Suavesito-Olimpiada

Out of curiosity, do you know how much computational overhead this introduces? Just wondering which one would be a better default value.

Here a simple MWE that might be good for benchmarking

julia> f(x) = x^2 - 2x + 1
f (generic function with 1 method)

julia> minval, minimisers = minimise(f, 0..2, tol=1e-6)

this example should return 2990 minimisers, so it might be a good small but not too small example for benchmarking

lucaferranti avatar May 27 '21 21:05 lucaferranti

Ah, you answered my questions while I was writing that 😄 Amazing! sorry for the noise

lucaferranti avatar May 27 '21 21:05 lucaferranti

Don't worry, happy to help. :D

Suavesito-Olimpiada avatar May 27 '21 21:05 Suavesito-Olimpiada

At a first glance this looks great, thanks! Could you please add some tests?

dpsanders avatar May 28 '21 00:05 dpsanders

Sure, I'll add test. I just want to know what do you think about the "problem" of using union-find that I described in the comment in the original issue.

Suavesito-Olimpiada avatar May 28 '21 05:05 Suavesito-Olimpiada