Page MenuHomePhabricator

re-init does not work with images with flipped axes
Closed, InvalidPublic

Assigned To
Authored By
espak
Nov 15 2016, 8:34 PM
Referenced Files
F31787: Screenshot - 05.12.2016 - 17:13:28.png
Dec 5 2016, 5:25 PM
F31789: Screenshot - 05.12.2016 - 17:13:47.png
Dec 5 2016, 5:25 PM
F31662: test-image-2x3x4-2x3x4.nii.gz
Dec 5 2016, 12:13 PM
F31660: test-image-2x3x4-2x3x4-cor-ax-flipped.nii.gz
Dec 5 2016, 12:13 PM
F31661: test-image-2x4x3-2x4x3-cor-ax-swapped.nii.gz
Dec 5 2016, 12:13 PM
F31663: test-image-2x3x4-2x3x4-cor-flipped.nii.gz
Dec 5 2016, 12:13 PM
F31673: Screen Shot 2016-12-05 at 11.16.21.png
Dec 5 2016, 12:13 PM
F31664: test-image-2x3x4-2x3x4-cor-ax-swapped.nii.gz
Dec 5 2016, 12:13 PM

Description

After doing re-init on an image with flipped axial and coronal axes, the windows go blank.

Event Timeline

cannot reproduce.
awaiting example data file and exact specification of affected version.

Here are some:





Try the first one, it does not show up neither after global reinit, nor after reinit.

Commit:
8d4c322

Screenshot:

Screen Shot 2016-12-05 at 11.16.21.png (1×2 px, 255 KB)

Actually, this might fail also because the nifti reader in MITK (ITK) is broken. We are using a patched nifti reader.

I was testing this also in our application that has the fixed nifti reader and it has the MITK display and the images with flipped axes did not display. But that was using our MITK fork that still has 2015.05.2 + patches.

I will generate test images in nrrd format and test again.

kislinsk added a subscriber: kislinsk.

Great, thank you. I will really help us when you provide nrrd test data. I'll have a look at T22113 meanwhile.

We have a utility to generate 3D test images with check pattern with any rotation matrix but it cannot save nrrd because the nrrd writer service is in DiffusionIO and we disabled the diffusion stuff because we do not really need it. I am trying to enable it but it has so many dependencies (even in ITK and VTK) that it is not easy. By the way, would you guys accept a PR for moving the nrrd writer service to MitkIOExt?

Seems to work now in master, commit bfcf1f07676058b32bee39f295ea1019013943ec

Mr Miklos Espak, would you kindly reconfirm or refute this?

Is it the nifti image or it was converted to nrrd?
If nrrd, would you mind attaching it to this task if it is not too much of a hassle?

I get the same blank windows when opening the nifties with bfcf1f07676058b32bee39f295ea1019013943ec.
I did a superbuild and deleted the plugin cache.

I made a clean build on linux and mac of the latest master. On linux my test images are displayed correctly, both after re-init and global re-init. I tried images with flipped, permuted and rotated axes, they all looked fine.

On mac none of the images appear at all, not even regular ones. But this must be an unrelated issue.

Thank you for checking.

Just for the record. I closed PR160 and pushed an almost identical pull request for T22254.

PR172. (Branch name changed to refer to the new task and 'None' orientation handled in the same was as 'Axial'.)