Deleted unnecassary code, rewrote verify statistics, fixed accuracy of test cases
Description
Description
Details
Details
- Provenance
• kraeuter Authored on Feb 26 2019, 8:44 AM • kraeuter Pushed on Feb 26 2019, 8:44 AM - Parents
- rMITK5fc1c2f37287: Removed debugging checks
- Branches
- Loading...
- Tags
Event Timeline
Comment Actions
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 | |