Fix inplace optimization changing init (in some cases)

pavlin-policar / openTSNE

Extensible, parallel implementations of t-SNE

https://opentsne.rtfd.io

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

1.42k stars 157 forks source link

Fix inplace optimization changing init (in some cases) #221

Closed pavlin-policar closed 1 year ago

pavlin-policar commented 1 year ago

Issue

In some forms of usage, the initialization was changed, which is incorrect.

Description of changes

Copy the intialization when constructing TSNEEmbedding and PartialTSNEEmbedding objects. This only happens once, so the memory impact should be minimal.

Includes

[X] Code changes
[X] Tests
[ ] Documentation