Remove 'mutable' member from `UMAP`

[weefuzzy]

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.