-
Notifications
You must be signed in to change notification settings - Fork 8
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
re-starts at test time #49
Comments
ps. Rix requested it by phone today, and rightly so. |
andycasey
added a commit
that referenced
this issue
Feb 14, 2016
andycasey
added a commit
that referenced
this issue
Feb 14, 2016
|
andycasey
added a commit
that referenced
this issue
Feb 15, 2016
Happy!! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now we do a single optimization at test time. This can't be right, since the optimization is non-convex. Also, Anna Ho finds that for LAMOST there are wrong answers caused by local minima. We should define a set of K initializations and start from all K, and choose the best answer (best in a likelihood sense). This, unfortunately, is high priority...
The text was updated successfully, but these errors were encountered: