lmcinnes / umap

Uniform Manifold Approximation and Projection
BSD 3-Clause "New" or "Revised" License
7.39k stars 803 forks source link

Setting a random state still leads to stochastic results #1080

Open dannywhuang opened 9 months ago

dannywhuang commented 9 months ago

When setting a random state, results should be reproducible, as explained in the documentation: https://umap-learn.readthedocs.io/en/latest/reproducibility.html

Setting a random state should mean that parallelization is disabled, however, this is not the case. Only a warning is given, but no where self.n_jobs gets set to 1. Line 1943: warn(f"n_jobs value {self.n_jobs} overridden to 1 by setting random_state. Use no seed for parallelism.")