Page MenuHomePhabricator
Create Task
Maniphest T18320

Tensor reconstruction b0 threshold preview broken
Closed, ResolvedPublic
Actions

Description

To reproduce:

  1. Open MITK Diffusion
  2. Open Tensor reconstruction
  3. Check Advanced checkbox
  4. Enter a b0 threshold

-> You should see in red a mask of which voxels are eliminated by that threshold (but you don't)

Related Objects

Event Timeline

• goch added projects: MITK, Bugzilla.Oct 22 2014, 9:40 AM
• kolbch added a subscriber: • kolbch.Nov 4 2014, 3:52 PM

When did this happen? There is no clue about a threshold preview in the code... so is this a feature request?

• goch added a subscriber: • goch.Nov 4 2014, 4:39 PM

This was remarked by Romy and Julia during their testing, as Romy was certain there used to such a functionality. I have not checked when it was available and Peter did not remember it. If we can not track down a version in which this worked change it to feature request and set target milestone to after next release.

I think I remember this as well, so you might want to talk to Andi if there used to be a general segmentation/threshold preview which has since been removed.

• goch added a comment.Nov 6 2014, 1:57 PM

Can not find where this was located, this would still be a nice feature though.

• webechr added a subscriber: • webechr.Nov 6 2014, 2:02 PM

I think i worked on something like this some time ago

T14639 . maybe it got lost in some Diffusion Release branch .. i'll have a look.

• webechr added a comment.Nov 7 2014, 11:11 AM

Referenced branch is working. will merge into master as soon as release flag is given.

• goch added a comment.Nov 7 2014, 11:24 AM

Does it work for the tensor reconstruction as well as the qball reconstruction?

• git-user added a subscriber: • git-user.Nov 7 2014, 7:51 PM

User webechr has pushed new remote branch:

bug-18320-RecoverLostPreview
• webechr added a comment.Nov 7 2014, 7:51 PM

Yes, we can! (... do both)

• webechr added a comment.Nov 7 2014, 7:51 PM

[ebac6e]: Merge branch 'bug-18320-RecoverLostPreview'

Merged commits:

2014-11-07 19:45:07 Christian Weber [739668]
use typedef for diffusion type

2014-11-06 13:57:56 Christian Weber [b8bc93]
Merge remote-tracking branch 'origin/bug-14639-TresholdPreviewRemovedOnPluginDeactivation' into bug-18320-RecoverLostPreview

2013-10-04 11:26:47 Thomas Van Bruggen [6f3428]
same for qball reconstruction. refresh wenn threshold preview is removed on deactivation

2013-10-02 16:39:54 Thomas Van Bruggen [d7f244]
set threshold preview to helper object again

2013-10-02 16:28:11 Thomas Van Bruggen [06f140]
remove threshold preview node when the plugin is deactivated

2013-10-02 15:33:46 Thomas Van Bruggen [47f603]
Merge branch 'bug-14639-ShowThresholdPreviewInTensorReconstruction_ITK4' into thresholdBug

2013-05-29 16:29:10 Christian Weber [e66e33]
Merge branch 'bug-14639-ShowThresholdPreviewInTensorReconstruction' into bug-14639-ShowThresholdPreviewInTensorReconstruction_ITK4

2013-03-06 17:04:52 Christian Weber [9e47be]
Added a Threshold preview for Tensor reconstruction

kislinsk edited projects, added MITK (2014-10); removed MITK.Aug 1 2016, 10:49 PM
MITK · MIC · IMSY · DKFZ