Enable Linux build: updated CMakeLists and YAML parser

Author: Ubuntu

CMakeLists.txt

@@ -175,19 +175,8 @@ endif()
 
 # ── HDF5 Integration ──
 # Prefer config package at provided root; avoid environment/registry
-if(DEFINED HDF5_DIR AND NOT DEFINED HDF5_ROOT)
-  set(HDF5_ROOT "${HDF5_DIR}")
-endif()
-set(HDF5_ROOT "${HDF5_ROOT}" CACHE PATH "HDF5 root directory")
-if(DEFINED HDF5_ROOT)
-  # Prepend so our supplied root wins
-  if(CMAKE_PREFIX_PATH)
-    set(CMAKE_PREFIX_PATH "${HDF5_ROOT};${CMAKE_PREFIX_PATH}")
-  else()
-    set(CMAKE_PREFIX_PATH "${HDF5_ROOT}")
-  endif()
-endif()
-find_package(HDF5 CONFIG REQUIRED COMPONENTS CXX)
+
+find_package(HDF5 REQUIRED COMPONENTS CXX)
 
 # ---- Eigen: support both CONFIG and module modes ----
 find_package(Eigen3 QUIET CONFIG)
@@ -619,9 +608,14 @@ if(HC_INSTALL_DEV_KIT)
     )
 endif()
 
-# ── Debug Symbols in Release Mode ──
-# Keep debug symbols in Release builds for better profiling and crash analysis
-set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /Zi /O2")
+if(MSVC)
+    # Windows: Enable debug symbols in Release builds with MSVC
+    set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /Zi /O2")
+else()
+    # Non-MSVC (Linux, macOS): Use -g for debug symbols
+    set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -g -O2")
+endif()
+
 
 # ──────────────────────────────────────────────────────────────────────────────
 # 7.1 Runtime installer and ZIP packaging for releases
@@ -750,12 +744,15 @@ if(HC_INSTALL_DEV_DEMOS)
 			FILES_MATCHING PATTERN "demo_*.exe")
 endif()
 
+
 # MSVC runtime DLLs and ZIP packaging via CPack
-include(InstallRequiredSystemLibraries)
-set(CPACK_GENERATOR "ZIP")
-set(CPACK_PACKAGE_NAME "HydroChrono")
-set(CPACK_COMPONENTS_ALL runtime python-tests dev-demos)
-include(CPack)
+ include(InstallRequiredSystemLibraries)
+ set(CPACK_GENERATOR "ZIP")
+ set(CPACK_PACKAGE_NAME "HydroChrono")
+ set(CPACK_COMPONENTS_ALL runtime python-tests dev-demos)
+ include(CPack)
+
+
 
 # Optionally include Python runtime DLL if Chrono::Parsers depends on it
 find_package(Python3 QUIET COMPONENTS Interpreter)
@@ -768,4 +765,4 @@ if(Python3_Interpreter_FOUND OR Python3_FOUND)
             install(FILES "${_PY_DLL_PATH}" DESTINATION bin COMPONENT runtime)
         endif()
     endif()
-endif()
+endif()

src/hydro_yaml_parser.cpp

@@ -388,7 +388,7 @@ YAMLHydroData ReadHydroYAML(const std::string& hydro_file_path) {
                             t += step;
                         }
                         if (inclusive) {
-                            if (std::fabs((data.waves.period_values.empty() ? start : data.waves.period_values.back()) - stop) > eps) {
+                            if (std::abs((data.waves.period_values.empty() ? start : data.waves.period_values.back()) - stop) > eps) {
                                 // add last if not already close to stop and within tolerance by stepping once more would overshoot but inclusive means include stop
                                 data.waves.period_values.push_back(stop);
                             } else {
@@ -458,4 +458,4 @@ YAMLHydroData ReadHydroYAML(const std::string& hydro_file_path) {
     }
 
     return data;
-}
+}