Update CMakeLists.txt for required packages and optimize compiler fla…

…gs; add debug output in normalise_vector function

CMakeLists.txt

@@ -23,19 +23,19 @@ if(UNIX AND NOT APPLE)
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS")
     set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,--as-needed")
 endif()
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -ffast-math -funroll-loops -m64")
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -O1 -ffast-math -funroll-loops -m64")
 
 # Set paths for non standard lib/ and include/ locations
 if(DEFINED ENV{CONDA_PREFIX})
     include_directories($ENV{CONDA_PREFIX}/include)
     link_directories($ENV{CONDA_PREFIX}/lib)
     link_directories($ENV{CONDA_PREFIX}/lib/intel64)
 else()
-  find_package(OpenMP)
+    find_package(OpenMP REQUIRED)  # Ensures OpenMP is found
 endif()
 
 # Add libraries
-find_package(ZLIB)
+find_package(ZLIB REQUIRED)
 find_package(Python COMPONENTS Interpreter Development REQUIRED)
 find_package(pybind11 CONFIG REQUIRED)
 include_directories(${CMAKE_CURRENT_SOURCE_DIR}/vendor)

src/vector_norm.hpp

@@ -15,8 +15,11 @@ inline void normalise_vector(std::vector<T> &vec, const bool check_positive,
 #pragma omp parallel for schedule(static) reduction(+: sum) reduction(&: all_positive) num_threads(n_threads)
   for (uint64_t it = 0; it < vec.size(); ++it) {
     sum += vec[it];
+    std::cout << vec[it] << std::endl;
     all_positive &= vec[it] >= 0;
   }
+  std::cout << "Positive: " << all_positive << std::endl;
+  std::cout << "Sum: " << sum << std::endl;
 
   if (check_positive && !all_positive) {
     throw std::runtime_error("Probability vector has negative entries");

src/wtsne.hpp

@@ -1,6 +1,5 @@
 #pragma once
 
-#include <wchar.h>
 #include <algorithm>
 #include <array>
 #include <chrono>