X-Git-Url: https://www.ginac.de/ginac.git//ginac.git?p=ginac.git;a=blobdiff_plain;f=check%2Ftime_lw_H.cpp;h=44f886c7b9f12577dc39bfe39f712c55d6a96838;hp=a36e5f249f1fce0a629f885b2e7fd248c0811f46;hb=0a2231e31d5d0eb780ebb7ffb8d0628c78b74248;hpb=1f5afe37b4c079821de9810435fa9fa11e4ea17b

diff --git a/check/time_lw_H.cpp b/check/time_lw_H.cpp
index a36e5f24..44f886c7 100644
--- a/check/time_lw_H.cpp
+++ b/check/time_lw_H.cpp
@@ -26,29 +26,26 @@
 static unsigned test(unsigned n)
 {
 	matrix hilbert(n,n);
-	
+
 	for (unsigned r=0; r<n; ++r)
 		for (unsigned c=0; c<n; ++c)
 			hilbert.set(r,c,numeric(1,r+c+1));
 	ex det = hilbert.determinant();
-	
+
 	/*
 	   The closed form of the determinant of n x n Hilbert matrices is:
 	
-	     n-1   /                      n-1                 \
-	    ----- |                      -----                 |
-	     | |  | pow(factorial(r),2)   | |    hilbert(r,c)  |
-	     | |  |                       | |                  |   
-	    r = 0  \                     c = 0                /
+             n-1   /                   \
+            ----- | pow(factorial(r),3) |
+             | |  | ------------------- |
+             | |  |    factorial(r+n)   |
+            r = 0  \                   /
 	*/
-	
+
 	ex hilbdet = 1;
-	for (unsigned r=0; r<n; ++r) {
-		hilbdet *= pow(factorial(numeric(r)),2);
-		for (unsigned c=0; c<n; ++c)
-			hilbdet *= hilbert(r,c);
-	}
-	
+	for (unsigned r=0; r<n; ++r)
+		hilbdet *= pow(factorial(r),3)/(factorial(r+n));
+
 	if (det != hilbdet) {
 		clog << "determinant of " << n << "x" << n << " erroneously returned " << det << endl;
 		return 1;
@@ -56,16 +53,16 @@ static unsigned test(unsigned n)
 	return 0;
 }
 
-unsigned time_lw_H(void)
+unsigned time_lw_H()
 {
 	unsigned result = 0;
 	unsigned count = 0;
 	timer rolex;
 	double time = .0;
-	
+
 	cout << "timing Lewis-Wester test H (det of 80x80 Hilbert)" << flush;
 	clog << "-------Lewis-Wester test H (det of 80x80 Hilbert):" << endl;
-	
+
 	rolex.start();
 	// correct for very small times:
 	do {
@@ -73,7 +70,7 @@ unsigned time_lw_H(void)
 		++count;
 	} while ((time=rolex.read())<0.1 && !result);
 	cout << '.' << flush;
-	
+
 	if (!result) {
 		cout << " passed ";
 		clog << "(no output)" << endl;
@@ -81,6 +78,6 @@ unsigned time_lw_H(void)
 		cout << " failed ";
 	}
 	cout << int(1000*(time/count))*0.001 << 's' << endl;
-	
+
 	return result;
 }