-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
TSNE has no fit
#4133
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
Comments
IMHO no: we are breaking the API contract here. |
I was wondering about this too there is no way to put new data through the existing model ie transform / predict. So although I can create very nice clusters with T-SNE, I cannot use the model to embed new data and use those values for prediction. Looking in more depth at the algorithm and code it may not be possible to embed new values with an existing model. Could someone confirm?? |
It's not intended to be inductive, i.e. generalise to new samples. But On 27 January 2015 at 15:13, 8evers notifications@github.com wrote:
|
I thought that the new Barn-Hut solver could to that. |
I confirm that this is fixed in #4025 that is still under review (I plan on finishing my pass on the code that today). The semantics of |
The
TSNE
class has nofit
method. This issue is basically asking whether this is ok.I thought having a
fit
was an API requirement.Other manifold methods store the result of
fit_transform
in a member and simply returnself
infit
.The text was updated successfully, but these errors were encountered: