From dd1de7b60b07aca73801d91b96b00cbf3693194b Mon Sep 17 00:00:00 2001
From: Davide Branduardi <davide.branduardi@gmail.com>
Date: Mon, 25 May 2015 10:54:38 +0200
Subject: [PATCH] bug in 1d and multivariate
removed condition that seemed to create very large support
in sum hills calculation and resulting into a very large computational
time. Credits to Carolyn Phillips from the mailing list.
---
src/tools/KernelFunctions.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/tools/KernelFunctions.cpp b/src/tools/KernelFunctions.cpp
index 7a851f8da..3126970d3 100644
--- a/src/tools/KernelFunctions.cpp
+++ b/src/tools/KernelFunctions.cpp
@@ -120,7 +120,7 @@ void KernelFunctions::setData( const std::vector<double>& at, const std::vector<
center.resize( at.size() ); for(unsigned i=0;i<at.size();++i) center[i]=at[i];
width.resize( sig.size() ); for(unsigned i=0;i<sig.size();++i) width[i]=sig[i];
diagonal=false;
- if (multivariate==false || at.size()==1 ) diagonal=true;
+ if (multivariate==false ) diagonal=true;
// Setup the kernel type
if(type=="GAUSSIAN" || type=="gaussian"){
--
GitLab