set_perplexity doesn't respect symmetrize parameter

pavlin-policar / openTSNE

Extensible, parallel implementations of t-SNE

https://opentsne.rtfd.io

BSD 3-Clause "New" or "Revised" License

1.44k stars 158 forks source link

set_perplexity doesn't respect symmetrize parameter #184

Closed pavlin-policar closed 2 years ago

pavlin-policar commented 3 years ago

Potential bug in PerplexityBasedNN and MultiscaleMixture --- in both cases, the set_perplexity(...) / set_perplexities(...) functions compute the P matrix with symmetrize=True, irrespective of how the symmetrize flag was set during initialization.

See https://github.com/pavlin-policar/openTSNE/issues/183#issue-890192358