Page MenuHomePhabricator
Create Task
Maniphest T25704

Difference between GUI and expected values MitkImageStatisticsCalculatorTests
Closed, ResolvedPublic
Actions

Description

Different results if one runs GUI and uses ImageStatisticsCalculator with the same input data

  • TestUS4DCylAxialPlanarFigureMaskStatistics_time1
MITK VersionEntropyKurtosisMPPmaxmean
2016.11 GUI6.17781.587115.45199115.45
2018.04 GUI6.17781.587115.45199115.45
Test (since f94aca1d1637, 2 years ago)6.21811.732121.12199121.18

Expected test values same than GUI (2016.11/2018.04 new image statistics plugin)

  • TestPic3DStatistics
  • TestPic3DAxialPlanarFigureMaskStatistics
  • TestPic3DImageMaskStatistics_label1
  • TestPic3DIgnorePixelValueMaskStatistics
  • TestUS4DCylStatistics_time1
  • TestUS4DCylIgnorePixelValueMaskStatistics_time1

Not testable

  • TestUS4DCylImageMaskStatistics_time1_label_1
    • because label can't be set in GUI

Related Objects
Search...

Event Timeline

hentsch triaged this task as Normal priority.Nov 30 2018, 4:23 PM
hentsch created this task.
floca added a project: Restricted Project.Nov 30 2018, 9:18 PM
floca raised the priority of this task from Normal to Needs Triage.Nov 30 2018, 9:20 PM
floca moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.

Unexplained differences make me nervous. We should at least discuss in the next stand up how critical it is and how we prioritize it. I have the guts feeling that it should get a higher priority.

hentsch renamed this task from Are MitkImageStatisticsCalculatorTests correct? to Difference between GUI and expected values MitkImageStatisticsCalculatorTests.Dec 4 2018, 3:46 PM
hentsch updated the task description. (Show Details)
floca triaged this task as Unbreak Now! priority.Dec 5 2018, 10:58 AM

For clarification of the reason

hentsch claimed this task.Dec 5 2018, 11:06 AM
hentsch updated the task description. (Show Details)Dec 5 2018, 11:57 AM
hentsch updated the task description. (Show Details)
hentsch updated the task description. (Show Details)Dec 5 2018, 2:12 PM
hentsch closed subtask T25740: MitkImageStatisticsCalculatorTests fail as Resolved.Dec 20 2018, 2:24 PM
hentsch removed hentsch as the assignee of this task.Jan 18 2019, 8:17 AM
hentsch added a subscriber: • kraeuter.Feb 5 2019, 2:48 PM
• kraeuter added a comment.Feb 7 2019, 10:40 AM

Looking at the code I didn´t see the results of the GUI differing from the expected values of the test, maybe you looked at the wrong timestep because your GUI data matches with timestep 0.
Setting the US4D label1 is possible right now, but the GUI fails at label2.
But there are still results not matching up:
TestPic3DImageMaskStatistics_label2´s expected results and GUI data don´t add up.

hentsch added a comment.EditedFeb 7 2019, 1:05 PM

I checked TestPic3DImageMaskStatistics_label2 and TestUS4DCylImageMaskStatistics_time1_label_2 via code changes (not possible via GUI). The values are consistent between test and GUI.
I can confirm that in TestUS4DCylAxialPlanarFigureMaskStatistics_time1 everything is the same too (see T24653)

floca added a comment.Feb 7 2019, 1:29 PM

@hentsch So does that mean it was false alarm or do I missinterpret something?

hentsch added a comment.Feb 7 2019, 1:37 PM

Only the old statistics plugin was affected. In the new plugin, all is ok.

floca added a comment.Feb 7 2019, 1:39 PM

Hail to the new statistics plugin! One more reason why your epic effort was well invested 😄

hentsch closed this task as Resolved.Feb 7 2019, 3:03 PM
hentsch claimed this task.
floca moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Feb 13 2019, 10:46 AM
MITK · MIC · IMSY · DKFZ