Remove 'mutable' member from `UMAP`
weefuzzy opened this issue · 0 comments
weefuzzy commented
UMAP's embedding is currently marked as mutable in order to avoid code duplication between const and non-const invocations of optimizeLayout. Using mutable without an accompanying lock is asking for grief in a multithreaded world. Refactor to git rid of the need for mutable.