View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001381Slicer4(No Category)public2011-08-30 11:142012-07-28 10:13
Reporterpieper Assigned Toalexy  
PrioritylowSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.0.1Fixed in VersionSlicer 4.0.0 
Summary0001381: Fibers reset to tubes after tractography
Description

Create fiducial tractography, change from tubes to lines, then change fiducial: mode goes back to tubes.

TagsNo tags attached.

Activities

alexy

alexy

2011-09-02 14:13

developer   ~0002890

I don't think it's a bug. The module says "tubes" and the display says "lines visible, tubes invisible", who is right? If we always use the display setting, then how to deal with the display setting "lines visible, tubes visible"?
pieper

pieper

2011-09-02 14:43

administrator   ~0002893

I didn't see that menu - so I thought it was a bug. But it's definitely a logical issue that should be sorted out.

1) One possibility would be to warn the user if the display node has a different setting than the fiducial seeding module is set to; then it could offer to use one or the other.

2) Another option is to remove the option from the fiducial seeding and replace it with an instance of the fiber bundle display options widget.

3) Another option would be to remove this menu and option altogether so there is only one place to change fiber bundle display properties - the fiducial seeding module could have a button that takes you directly to the fiber bundle module with the current fiber bundle selected

4) Another would be to implement the Edit Properties... menu item in slicer4 to work like the one in slicer3, so that users could get to the right module in a consistent way throughout the program.

Long term I would like to see option 4. Nearer term I think option 3 is okay.

Realistically this bug is lower priority than other outstanding issues (like, annotations) so it could stay as-is for now I would say.
demian

demian

2011-09-02 15:08

developer   ~0002894

Last edited: 2011-09-02 15:09

I agree with Steve (Pieper). I think that option 2) would be great (specially if the instance is on a collapsed widget), to reduce the amount of module switching while working. In any case option (4) is a good one to have implemented for slicer in general.

In any case, I agree that it is a low priority one.
alexy

alexy

2011-09-05 04:45

developer   ~0002904

Option 2) stills leave the question of how fibers are created initially, as tubes or lines. Perhaps in the module it should say "Create initially as: Lines/Tubes" and this setting should be only applied to a newly created tracts?
pieper

pieper

2011-09-05 04:55

administrator   ~0002905

That would work for me - it's the simplest too.
demian

demian

2011-09-05 06:05

developer   ~0002906

That would work for me too.
alexy

alexy

2011-09-05 06:21

developer   ~0002907

Fixed as described in my previous comment.
At revision: 17951
jcfr

jcfr

2012-07-27 15:36

administrator   ~0005310

Dear issue reporter,

Good news :) Slicer developers SOLVED the problem you reported - YOU now need to VERIFY and CLOSE this issue.

Issue History

Date Modified Username Field Change
2011-08-30 11:14 pieper New Issue
2011-08-30 11:14 pieper Status new => assigned
2011-08-30 11:14 pieper Assigned To => alexy
2011-08-30 11:36 demian Tag Attached: Diffusion
2011-08-31 07:13 demian Category GUI => Diffusion
2011-09-02 14:13 alexy Note Added: 0002890
2011-09-02 14:14 alexy Status assigned => feedback
2011-09-02 14:43 pieper Note Added: 0002893
2011-09-02 15:08 demian Note Added: 0002894
2011-09-02 15:09 demian Note Edited: 0002894
2011-09-02 15:11 demian Priority normal => low
2011-09-03 05:14 alexy Status feedback => assigned
2011-09-03 05:14 alexy Target Version Slicer 4.0 RSNA => Slicer 4.0 post RSNA
2011-09-05 04:45 alexy Note Added: 0002904
2011-09-05 04:55 pieper Note Added: 0002905
2011-09-05 06:05 demian Note Added: 0002906
2011-09-05 06:21 alexy Note Added: 0002907
2011-09-05 06:21 alexy Status assigned => resolved
2011-09-05 06:21 alexy Resolution open => fixed
2011-09-15 19:52 finetjul Fixed in Version => Slicer 4.0 RSNA
2012-07-27 15:36 jcfr Note Added: 0005310
2012-07-28 10:13 pieper Status resolved => closed
2013-01-27 12:27 jcfr Tag Detached: Diffusion
2016-02-29 13:56 jcfr Category Module Diffusion => (No Category)