devtools can no longer build uwot on Windows

[jlmelville]

devtools can no longer build uwot on Windows

[jlmelville]

Ok I bought myself some time by removing a gradient instantiation

[LTLA]

I don't recall seeing that many templates in your C++ code. Certainly it's not as bad as anything using Eigen or any of those header-only libraries floating around on CRAN. Surely they would be hit harder by template-related limitations.

[jlmelville]

It's a mingw problem it seems. Running devtools::load_all under Linux doesn't give a problem.

Good point about Eigen. Maybe it's because src/r_uwot.cpp basically generates all combinations of the templated classes: r_uwot.o and uwot.dll end up at > 32MB in size.

[LTLA]

If major surgery on uwot's C++ code is required... maybe you might consider whether umappp might be a good place to put that effort? It might be possible to achieve the dream of consolidating all the C++ code floating around into a central library that can be re-used elsewhere (e.g., LTLA/umappp#5).

Happy to give you maintainer access, I pretty much copied uwot's code anyway, though I did strip out TBB in favor of std::thread and/or OpenMP. I don't think I used much templating either.

Anyway, just a thought, no pressure.

[jlmelville]

Moving over to umappp is a good idea (I also moved to only using std::thread a while ago). But I probably can't make the change for uwot without breaking backwards compatibility or adding a lot of stuff that no-one would want in umappp. So I will have to work out how to split that up.

In the current code base there are some templated things that only get called once an epoch and which can be made pointer-based, which might buy me some time.