Wrong logic was used
authorMark Abraham <mark.j.abraham@gmail.com>
Mon, 21 Sep 2020 14:21:14 +0000 (16:21 +0200)
committerArtem Zhmurov <zhmurov@gmail.com>
Mon, 21 Sep 2020 15:26:36 +0000 (15:26 +0000)
Log files in gitlab gave the warning message for twin-cutoff, but we
are not setting these env variables.

src/gromacs/nbnxm/nbnxm_gpu_data_mgmt.cpp

index 3276b9ece171db06f35c28c94a73435b83892b61..3fd85ae473c54b0cb36b086cc598a6a52acb9400 100644 (file)
@@ -108,8 +108,8 @@ int nbnxn_gpu_pick_ewald_kernel_type(const interaction_const_t& ic)
                                             || (getenv("GMX_OCL_NB_ANA_EWALD") != nullptr);
     const bool forceTabulatedEwaldLegacy = (getenv("GMX_CUDA_NB_TAB_EWALD") != nullptr)
                                            || (getenv("GMX_OCL_NB_TAB_EWALD") != nullptr);
-    const bool forceTwinCutoffEwaldLegacy = (getenv("GMX_CUDA_NB_EWALD_TWINCUT") == nullptr)
-                                            || (getenv("GMX_OCL_NB_EWALD_TWINCUT") == nullptr);
+    const bool forceTwinCutoffEwaldLegacy = (getenv("GMX_CUDA_NB_EWALD_TWINCUT") != nullptr)
+                                            || (getenv("GMX_OCL_NB_EWALD_TWINCUT") != nullptr);
 
     printEnviromnentVariableDeprecationMessage(forceAnalyticalEwaldLegacy, "NB_ANA_EWALD");
     printEnviromnentVariableDeprecationMessage(forceTabulatedEwaldLegacy, "NB_TAB_EWALD");