HomePhabricator
Diffusion MITK a456c97939bf

Deleted unnecassary code, rewrote verify statistics, fixed accuracy of test…

Description

Deleted unnecassary code, rewrote verify statistics, fixed accuracy of test cases

Details

Provenance
kraeuterAuthored on Feb 26 2019, 8:44 AM
kraeuterPushed on Feb 26 2019, 8:44 AM
Parents
rMITK5fc1c2f37287: Removed debugging checks
Branches
Unknown
Tags
Unknown

Event Timeline

sieht gut aus, vielen dank :)
Lass uns die Anmerkungen am besten kurz besprechen

/Modules/ImageStatistics/Testing/mitkImageStatisticsCalculatorTest.cpp
39–50

Wenn möglich, TestCase* umbenennen in funktion mit sprechenden Namen

59

Bitte Task anlegen für 2D Statistik Test

93

immer mitk::Image::ConstPointer benutzen

138–144

CPPUNIT_ASSERT_THROW() nur um GetStatistics()

157

Hier wird 10x genau das gleiche getestet (für dasselbe Bild). Braucht man eigentlich nur einmal.

160

auch hier wird 10x genau das gleiche getestet

278

hier brauchst du nicht auf 10 Nachkommastellen genau zu sein, da du ja später mit mitk::eps schaust, ob es im "Rahmen" ist.

375

braucht man das?

451

Alle TestCase* sind relativ repetitiv. Es ist nur ein Test, aber das kann man noch etwas vereinfachen (und dadurch Code kürzen), indem man eine Funktion GeneratePlanarPolygon(mitk::PlaneGeometry::Pointer geometry, std::vector<mitk::Point2D> points) implementiert

549

s.o.: genauigkeit mitk::eps=10^-5 (vermute ich), daher brauchst es es hier auch nicht so genau angeben.

553

Jeweils noch einen Kommentar in die Funktionen, dass du die "Grundwahrheit" mit einem Skript ausgerechnet hast.

776

Bitte irgendwo noch testen, ob gültige Statistiken für alle Timesteps erstellt werden (hoffentlich sind es nicht so viele Timesteps...). Die Werte selbst brauchst du nicht immer zu prüfen, aber ob ein gültiges Objekt rauskommt und keine Exception geworfen wird, würde ich testen.

864

CPPUNIT_ASSERT_NO_THROW um diese Zeile (das gilt natürlich auch für dieselbe Zeile in den anderen Funktionen ;) )

882

Bitte auch ein Test für ein 4D Bild und eine 3D Maske schreiben. Da gabs schon einige Bugs.

995

am sinnvollsten ist nur eine ComputeStatistics funktion. Dafür kannst du die andere Funktion in eine GeneratePlanarFigure funktion umbauen

1008–1015

das ist recht gefährlich hier. Hier werden potentiell einige exceptions abgefangen, die uns evtl interessieren. GetStatistics sollte eigentlich keine Exception werfen und dann nur aus gutem Grund und diese Gründe können wir checken (kein Input).

1040

mitk::ImageStatisticsContainer::RealType anstatt double (das ist jetzt kein Unterschied, aber wenn man sich mal dazu entscheidet, den Datentyp zu ändern, wird es zum Unterschied. Ist also Allgemeingültig dann)

1042

CPPUNIT_ASSERT_NO_THROW um diese Zeile

1065–1076

CPPUNIT_ASSERT_NO_THROW um diese Zeilen