Dmitry Ulyanov

Hi, fixed a bug here https://github.com/DmitryUlyanov/Multicore-TSNE/commit/f5c5be16d0b4950ce3be3e2c393c96e0aa0015c5#diff-f8b3cce0b3183b4e02d913d9eb933c6eR210 Can you please do git pull and check if it solved the issue?

Hi, as far as I know quadtree is only suited for euclidean. You can get cosine for free by normalizing your data first. Other metrics are not supported.

Now it is possible to easily add new metrics (yet in C++ with recompilation). If you have any preferences I could implement some.

Hi @asanakoy, my statement from 25 June was incorrect. >If I just change a distance function when you instantiate VpTree from euclidean to cosine will it be sufficient? Now I...

Well, you still can get cosine similarity by normalizing your data to have unit norm.

>Could you please point out which line of code it is? https://github.com/DmitryUlyanov/Multicore-TSNE/blob/master/multicore_tsne/tsne.cpp#L242 https://github.com/DmitryUlyanov/Multicore-TSNE/blob/master/multicore_tsne/tsne.cpp#L313 Maybe there are more. >But this is not exactly the same as the cosine distance. I see,...

Hi, I've tested now with sklearn 0.19.1, the same parameters, as for this repo: ``` tsne = TSNE(n_components=2, random_state=0, n_iter = 1000, min_grad_norm=0, verbose=1000) ``` Still got very high running...

Yes, it is the same. Can you please run https://github.com/DmitryUlyanov/Multicore-TSNE/blob/master/MulticoreTSNE/examples/test_sklearn.py on your machine?

Hi, I never tried it for audio, should be very easy to implement though. Send me results if you try it.

Hello, there's nothing that prevents using longer inputs, it probably just requires changing several lines.