roamanalytics / mittens

A fast implementation of GloVe, with optional retrofitting
Apache License 2.0
243 stars 31 forks source link

Make Mittens Deterministic #23

Open groceryheist opened 1 year ago

groceryheist commented 1 year ago

At present, mittens is not reproducible because of calls to np.random.seed(None) in the function for generating random matrices. This is a nuisance for testing or research reproducibility. I'm still not 100% sure I'm going to use mittens in my current project, but if I do, I will send a pull request to fix this.