optimistix icon indicating copy to clipboard operation
optimistix copied to clipboard

Check last point when using best so far minimiser

Open ColCarroll opened this issue 5 months ago • 1 comments

Fixes #33

There is some unnecessary looking lines

        best_f, best_aux = fn(state.best_y, args)
        best_loss = self._to_loss(state.best_y, best_f)

where I would expect to just use state.best_loss, but the test doesn't pass without it!

ColCarroll avatar Jan 06 '24 15:01 ColCarroll