metric = "cosine" bug

[ONeillMB1]

Thanks for developing this!

The umap function appears to have a bug when the 'metric = "cosine"' option is invoked. I get the following error:

Error in search_nn_func(index_file, X, k, search_k, grain_size = grain_size, :
vector::_M_range_insert

However, if I use 'metric = "manhattan" or leave it to the default it works just fine.

Best.

[jlmelville]

Thank you for reporting this! Fixed in the latest version (version 0.0.0.9002).

I was incorrectly checking for a 'Cosine' class, when Annoy uses 'Angular'. Compounding this, I failed to add an else clause to catch the 'impossible' situation of an unknown option (as recommended by all basic programming advice books ever). Shame on me.