Skip to content
This repository has been archived by the owner on Feb 18, 2020. It is now read-only.

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
dpfoose committed Feb 3, 2016
1 parent 661da76 commit b6bb539
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,12 @@ You can link ARPACK to LAPACK and OpenBLAS. Compile ARPACK and LAPACK as if you

You can use CMake to configure OpenBLAS for Visual Studio.

You can use CMake to configure Armadillo for Visual Studio, but you should modify your config.hpp file before compiling and installing. Make sure to explicitly link the libraries in the armadillo project.
You can use CMake to configure Armadillo for Visual Studio, but you should modify your config.hpp file before compiling and installing. Make sure to explicitly link the libraries in the armadillo project. Also make sure that ARMA_USE_BLAS, ARMA_USE_ARPACK, ARMA_USE_LAPACK, ARMA_64BIT_WORD and ARMA_USE_CXX11 are enabled (the last two are technically optional and MLPACK will build without them).

You can download the Boost libraries here: http://sourceforge.net/projects/boost/files/boost-binaries/1.60.0/

You will have to modify CMakeLists.txt to prevent an error declaring MSVC as non C++-11 compliant. The patched file is included in the repo.
You will have to modify any file containing OpenMP for loops, as MSVC 2015 supports OpenMP 2.5, which requires signed integer indices. GCC and Clang already support OpenMP 3.0 with unsigned indices. Replace "size_t" with "maxint_t".
For 2.0.0 only src/mlpack/methods/det/dt_utils.cpp needs this fix.

MLPACK should then build correctly.

0 comments on commit b6bb539

Please sign in to comment.