18:44 < zoq> interesting: armadillo_bits/gmm_diag_meat.hpp(1715): error C3016: 't': index variable in OpenMP 'for' statement must have signed integral type

18:44 < zoq> I guess, one option is to switch to another version, I guess it's already fixed.

18:46 < rcurtin> yeah, I was just looking at that

18:46 < rcurtin> I did a much more primitive solution for now... let me make sure that works, then I can refine it a bit

18:48 < rcurtin> it doesn't seem like it's fixed in newer versions of Armadillo though, so maybe I should file a bug report there

18:48 < zoq> definitely

18:48 < rcurtin> ha, can't open an issue on the Armadillo repo, I guess I'll just send Conrad an email

18:49 < zoq> ah, right or open a PR

18:50 < rcurtin> I'd rather ask first, in case there is a different way Conrad wants to fix it

18:56 < rcurtin> sent an email, we'll see what the response is

19:04 < zoq> Yeah, he is probably happy to fix it for the upcoming version.

19:34 < zoq> rcurtin: You manually update the download page right? Just noted https://github.com/mlpack/mlpack.org/blob/master/download.html#L34 does reference to 2.2.4. I can update that section if that helps, but I guess the page isn't automatically deployed?

19:35 < rcurtin> yeah, these things are not automatic...

19:35 < rcurtin> perhaps that is a little bit ugly but I have not taken the time to automate it :)

19:36 < rcurtin> I'll fix it real quick...

19:38 < zoq> okay, great :)