AnotherFoxGuy/polyvox
1
0
Fork 0
Code Issues Pull Requests Activity

Fix LowPassFilter test

Browse Source 
The implicit default kernel size for the non-SAT version was 3 but the SAT
version was using an explicit kernel of 5 which caused a discrepancy.

Now just use a kernel of 3.
This commit is contained in:
Matt Williams
2012-11-22 22:23:07 +00:00
parent 0f838d54b6
commit 3e79e0bfaa
1 changed files with 11 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
tests/TestLowPassFilter.cpp
View File

@@ -58,7 +58,7 @@ void TestLowPassFilter::testExecute()
RawVolume<Density8> resultVolume(reg);
LowPassFilter< RawVolume<Density8>, RawVolume<Density8>, Density16 > lowPassfilter(&volData, reg, &resultVolume, reg, 5);
LowPassFilter< RawVolume<Density8>, RawVolume<Density8>, Density16 > lowPassfilter(&volData, reg, &resultVolume, reg, 3);
//Test the normal implementation
QBENCHMARK {
@@ -74,16 +74,17 @@ void TestLowPassFilter::testExecute()
QCOMPARE(resultVolume.getVoxelAt(7,7,7), Density8(4));
//Test the SAT implmentation
//FIXME - Shouldn't the results be the same as the normal case?
QBENCHMARK {
lowPassfilter.executeSAT();
}
QCOMPARE(resultVolume.getVoxelAt(0,0,0), Density8(4));
QCOMPARE(resultVolume.getVoxelAt(1,1,1), Density8(8));
QCOMPARE(resultVolume.getVoxelAt(2,2,2), Density8(19));
QCOMPARE(resultVolume.getVoxelAt(3,3,3), Density8(12));
QCOMPARE(resultVolume.getVoxelAt(4,4,4), Density8(19));
QCOMPARE(resultVolume.getVoxelAt(5,5,5), Density8(12));
QCOMPARE(resultVolume.getVoxelAt(6,6,6), Density8(8));
QCOMPARE(resultVolume.getVoxelAt(7,7,7), Density8(2));
QCOMPARE(resultVolume.getVoxelAt(1,1,1), Density8(21));
QCOMPARE(resultVolume.getVoxelAt(2,2,2), Density8(10));
QCOMPARE(resultVolume.getVoxelAt(3,3,3), Density8(21));
QCOMPARE(resultVolume.getVoxelAt(4,4,4), Density8(10));
QCOMPARE(resultVolume.getVoxelAt(5,5,5), Density8(21));
QCOMPARE(resultVolume.getVoxelAt(6,6,6), Density8(10));
QCOMPARE(resultVolume.getVoxelAt(7,7,7), Density8(4));
}
QTEST_MAIN(TestLowPassFilter)