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

Better dynamic bracket size in tuneLearnFast() #11

Open
mfasiolo opened this issue May 31, 2016 · 1 comment
Open

Better dynamic bracket size in tuneLearnFast() #11

mfasiolo opened this issue May 31, 2016 · 1 comment

Comments

@mfasiolo
Copy link
Owner

The way this function shrinks the bracket size is not entirely satisfactory at the moment.
In particular, the methods often shrinks the bracket when fitting central quantiles, but the optimal
log(sigma) oscillates quite a lot when extreme quantiles are considered. Hence, using a narrow bracket
for extreme quantiles is inefficient.

In addition, the algorithm does not recognise immediately that the bracket is too narrow, because it
does not look at derivative at the limits of the bracket. What happens is that, when the bracket is too narrow, the algorithms slowly moves toward the one end of the bracket, wasting quite a lot of function evaluations in that way.

@mfasiolo
Copy link
Owner Author

Partially addressed here

af959f9

now inside Brent() we monitor if we are getting too close to the boundary and, if we are too close, we exit. tuneLearnFast will then enlarge and shift the bracket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant