Page MenuHomePhabricator

Convert diffusion image to mitk image
Closed, ResolvedPublic

Description

The purpose of this bug is to prototype the conversion from mitk::DiffusionImage to mitk::Image as a division into two different classes should no longer be necessary. Additional information, which is currently stored as member variables, could be moved to a base data property list.

Event Timeline

User goch has pushed new remote branch:

bug-17928-prototype-diffusion-image-conversion

User cordes has pushed new remote branch:

bug-17928-DiffusionProperties

User cordes has pushed new remote branch:

bug-17928-diffusion-image-io

User goch has pushed new remote branch:

bug-17928-clean-mitk-diffusion-image-replacement-branch

User goch has pushed new remote branch:

bug-17928-replace-calls-to-use-mitk-image

User cordes has pushed new remote branch:

bug-17928-clean-mitk-diffusion-image-reader

Replaced all use of mitk::DiffusionImage in the QmitkPreprocessing view, which necessitated the introduction of a helper class managing the properties. Currently working on a personal branch which will be ported to an offical bug branch once most of the restructuring is done.

Project onto shell "AKC" and "Bi-Exponential" are broken, but they are already broken on the current master.

Replaced all occurences in any of the views. It does compile now, however not everything works yet. Extensive testing will be required.

There remains still replacing to be done in algorithms, miniapps and tests.

User goch has pushed new remote branch:

bug-17928-diffusion-image-migration

Changes in the test data can be commited to the MITK-Data branch:
bug-17928-migrate-test-data

User cordes has pushed new remote branch:

bug-17928-imagereconstructiontest

We might want to take a look at the printing of base data properties as provided by T16811

User goch has pushed new remote branch:

bug-17928-post-release-integration

Builds on OSX.
Reader tested: Images will load but they are not recognized as an dwi!

The new reader can no longer add any properties to the data node. We might have to switch from the ISDIFFUSIONWEIGHTEDIMAGEPROPERTYNAME attached to the data node to just checking whether the base data has a gradient property.

This might warrant more discussion with Sascha/the MITK group to define a general solution for images which do not differ based on properties.

User cordes has pushed new remote branch:

bug-17928-post-release-IO-integration

Core Flag:

Added /Core/Code/DataManagement/mitkNodePredicateDataProperty.h
This predicate enables filtering for a given property in the basedata propertylist instead of the datanode propertylist

User goch has pushed new remote branch:

bug-17928-post-release-integration-2015

[b13674]: Merge branch 'bug-17928-post-release-integration-2015'

Merged commits:

2015-01-07 19:12:54 Caspar Goch [dc8c32]
Remove unneeded files


2015-01-07 19:12:23 Caspar Goch [ee1743]
Remove diffusion imaging from other modules


2015-01-07 19:11:51 Caspar Goch [54bd06]
Switch plugins to new concept


2015-01-07 19:11:24 Caspar Goch [4a0237]
Switch mini apps to new concept


2015-01-07 19:10:48 Caspar Goch [5198f6]
Switch fibertracking to new concept


2015-01-07 19:10:34 Caspar Goch [43bac4]
Switch diffusion io to new concept


2015-01-07 19:10:05 Caspar Goch [d5a74f]
Switch diffusion core to new concept


2015-01-07 19:07:59 Caspar Goch [3d2a89]
Only apply the diffusion mime type if applicable


2015-01-07 19:04:32 Caspar Goch [49bb82]
Create properties containing dwi information

Create properties which contain the information which can not be contained
in an mitk image itself. Also add a helper class which manages this
information and provides convenience functions for accessing it

[061669]: Merge branch 'bug-17928-post-release-integration-2015'

Merged commits:

2015-01-08 15:25:01 Caspar Goch [11d2ef]
COMP: Catch exceptions if the file can not be opened for reading image information


2015-01-08 13:36:00 Caspar Goch [26a8c3]
COMP: Change file endings back to match existing files

[cc2286]: Merge branch 'bug-17928-post-release-integration-2015'

Merged commits:

2015-01-08 16:38:56 Caspar Goch [e7d04a]
COMP: Assume default measurementframe when missing

User goch has pushed new remote branch:

bug-17928-mimetypes-do-not-check-for-validity-writing