View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0004410||Slicer4||Core: Segmentation||public||2017-08-07 13:49||2018-07-21 14:47|
|Product Version||Slicer 4.6.2|
|Target Version||Fixed in Version||Slicer 4.9.0|
|Summary||0004410: Changing master volume in SegmentEditor does not update edit-able FOV|
Steps to reproduce:
|Tags||No tags attached.|
Observed in 4.7.0-2017-08-01 r26187 on Mac.
I think it's a "feature", or a lack of a feature if you will that changing the master doesn't affect the geometry, and the first master is still used.
As Andras wrote in an email correspondence (including you Andrey) a few months ago:
This will be a part of a major overhaul.
The context was somewhat different in that discussion you mentioned, and in any case I think it is good to document this in the bug tracker.
It it's a "feature", then it's a counter-intuitive one! IMHO, this is not obvious to the user. It probably would be helpful to do a quick check whether master volume FOV overlaps with the segmentation geometry, and if it doesn't - warn the user. Otherwise all that the user can see is that there is no effect in trying to paint on the slice, and it is not obvious what is the reason to this. Anyway, this is my perspective as a user.
I completely agree. I didn't want to defend the implementation, I just stated that it's a shortcoming we're aware of and will need to be addressed for sure.
Makes sense, thank you!
Segmentation labelmap geometry can be specified in a new widget. Hopefully it addresses this issue
Changes integrated in https://github.com/Slicer/Slicer/commit/6a8012e30952ff7e990956f11a892349387cd974
Andrey, please assess whether this is a good solution for the issue. Thanks!
|2017-08-07 13:49||fedorov||New Issue|
|2017-08-07 13:49||fedorov||Status||new => assigned|
|2017-08-07 13:49||fedorov||Assigned To||=> pinter|
|2017-08-07 13:49||fedorov||Note Added: 0015029|
|2017-08-07 14:15||pinter||Note Added: 0015030|
|2017-08-07 14:22||fedorov||Note Added: 0015031|
|2017-08-07 14:33||pinter||Note Added: 0015032|
|2017-08-07 14:44||fedorov||Note Added: 0015033|
|2018-06-14 15:55||pinter||Relationship added||related to 0004308|
|2018-06-20 18:52||pinter||Status||assigned => resolved|
|2018-06-20 18:52||pinter||Resolution||open => fixed|
|2018-06-20 18:52||pinter||Fixed in Version||=> Slicer 4.9.0|
|2018-06-20 18:52||pinter||Note Added: 0015905|
|2018-07-21 14:47||pinter||Note Added: 0015926|