View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002869 | Slicer4 | Core: MRML | public | 2013-01-14 08:17 | 2013-03-19 13:15 |
Reporter | wangk | Assigned To | wangk | ||
Priority | normal | Severity | minor | Reproducibility | N/A |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.2.2-1 | ||||
Target Version | Slicer 4.3.0 | Fixed in Version | Slicer 4.3.0 | ||
Summary | 0002869: Direction Matrix not exposed in volume info widget | ||||
Description | Currently the direction matrix of a volume node is not displayed in the volume module and this can potentially create some confusion. It would be nice to expose it by enhancing the qMRMLVolumeInfoWidget class. See https://www.assembla.com/spaces/slicerrt/tickets/198#/activity/ticket: for additional comments on this issue. | ||||
Tags | No tags attached. | ||||
If needed, I could create and submit a patch. |
|
Kevin, if you could submit a working patch, that would be great! Or we we can try to do it ourselves - unless I'm missing something it should be just a few lines. |
|
Steve, I will try to create a patch then. As you said there should be just a few lines need to be added. |
|
Topic pushed to https://github.com/kevinwangcanada/Slicer/tree/2869-Direction-Matrix-not-exposed-in-volume-info-widget/ I used 3 ctkCoordinateWidgets to display the direction vectors for I, J, K. see the attached picture. If you think this takes too much space, I can try either a textbox that puts the 3 vectors into one line (separated by ;) or use a ctkCollapsibleGroupBox. |
|
2013-01-18 09:54
|
|
I have also found a minor issue with the ctkVolumeInfoWidget code. when a volume is loaded and then removed from the scene, the volume info widget does not get updated and it will still display the old info. see attached volume_info_not_updated.png picture. I have modified the code to fix that. Do I need to create a new ticket/topic/branch for that or maybe I can use the same topic again? |
|
2013-01-18 10:00
|
|
I believe you could use a ctkMatrixWidget instead, it will take less space. |
|
Thanks for the explanation. I am able to select all the cells in the matrix widget by changing the selectionMode to QAbstractItemView::MultipleSeletion, however, ctkMatrixWidget will need to be modified to enable the copy/paste functionality. Maybe we reconsider this feature later? |
|
Sure, this is definitely out of scope. |
|
No problem, I have pushed the updated topic to https://github.com/kevinwangcanada/Slicer/tree/2869-Direction-Matrix-not-exposed-in-volume-info-widget Thanks. |
|
Fixed in r21711 |
|
thanks. I have closed the issue. |
|
It most likely generated the following issue 0003010 |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2013-01-14 08:17 | wangk | New Issue | |
2013-01-14 08:17 | wangk | Status | new => assigned |
2013-01-14 08:17 | wangk | Assigned To | => alexy |
2013-01-14 08:49 | wangk | Note Added: 0007662 | |
2013-01-14 09:02 | pieper | Note Added: 0007663 | |
2013-01-14 11:23 | wangk | Note Added: 0007666 | |
2013-01-18 09:54 | wangk | Note Added: 0007711 | |
2013-01-18 09:54 | wangk | File Added: newvolumeinfowidget.png | |
2013-01-18 09:59 | wangk | Note Added: 0007712 | |
2013-01-18 10:00 | wangk | File Added: volume_info_not_updated.png | |
2013-01-23 08:33 | finetjul | Note Added: 0007722 | |
2013-01-23 08:34 | finetjul | Note Edited: 0007722 | |
2013-01-23 10:59 | wangk | Note Added: 0007723 | |
2013-01-23 11:05 | finetjul | Note Added: 0007724 | |
2013-01-24 09:09 | wangk | Note Added: 0007731 | |
2013-02-15 06:14 | jcfr | Target Version | => Slicer 4.3.0 |
2013-02-15 06:42 | jcfr | Note Added: 0007946 | |
2013-02-15 06:42 | jcfr | Status | assigned => resolved |
2013-02-15 06:42 | jcfr | Fixed in Version | => Slicer 4.3.0 |
2013-02-15 06:42 | jcfr | Resolution | open => fixed |
2013-03-11 06:18 | wangk | Note Added: 0008106 | |
2013-03-11 06:18 | wangk | Status | resolved => closed |
2013-03-19 12:40 | finetjul | Relationship added | has duplicate 0002083 |
2013-03-19 12:41 | finetjul | Relationship added | related to 0003010 |
2013-03-19 12:42 | finetjul | Note Added: 0008171 | |
2013-03-19 13:15 | jcfr | Assigned To | alexy => wangk |