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

NCA should allow an arbitrary optimizer #249

Closed
rcurtin opened this Issue Dec 29, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@rcurtin
Member

rcurtin commented Dec 29, 2014

Reported by rcurtin on 11 Jun 42841984 04:13 UTC
Previously (before #254) NCA used the L-BFGS optimizer, and now it uses the SGD optimizer. Realistically we should allow the user the option of whichever optimizer they like; there are situations where one optimizer may turn out to be better than others. After all, what NCA really is is just an objective function anyway.

@rcurtin rcurtin self-assigned this Dec 29, 2014

@rcurtin rcurtin closed this Dec 29, 2014

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin Dec 30, 2014

Member

Commented by rcurtin on 23 Sep 42924777 16:16 UTC
r13966 adapts the NCA class to allow an arbitrary optimizer, and r13968 extends the main executable to accept these options. The functionality from before with SGD is preserved.

Member

rcurtin commented Dec 30, 2014

Commented by rcurtin on 23 Sep 42924777 16:16 UTC
r13966 adapts the NCA class to allow an arbitrary optimizer, and r13968 extends the main executable to accept these options. The functionality from before with SGD is preserved.

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