View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002878Slicer4Core: GUIpublic2013-01-18 05:352017-06-07 23:27
Reporterpieper Assigned Topieper  
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product VersionSlicer 4.2.2-1 
Target VersionSlicer 4.3.0Fixed in VersionSlicer 4.3.0 
Summary0002878: slice locations are at pixel boundaries instead of pixel centers after SnapSliceOffsetToIJK
Description

The vtkMRMLSliceLogic::SnapSliceOffsetToIJK() method should move the slice plane (slice node offset) to be in the middle of background pixel. This is important for visualizing and editing volume data because when the slice plane is exactly at the boundary between two pixels then a rounding error can lead to different parts of the code selecting to view or operate on different slices.

Steps To Reproduce

1) Load MR-head
2) Switch to Volumes module and turn off interpolation
3) Turn on slice intersections
4) Zoom in close to see where the slice intersection lies with respect to the pixel boundaries
5) Enter the Editor and select an effect (e.g. Draw tool)

Note that with 4.2 the line stays exactly at the boundary of the pixels.

In 4.1 the line is properly adjusted to lie in the middle of the pixel.

Additional Information

This is a regression that needs to be investigated and fixed.

TagsNo tags attached.

Relationships

duplicate of 0002930 closedpieper labelmaps created in Slicer4 off by one slice 
related to 0002999 closedpieper Default slice position is not centered on the slice 

Activities

2013-01-18 05:36

 

Screen Shot 2013-01-18 at 10.33.36 AM.png (327,606 bytes)
pieper

pieper

2013-01-18 05:36

administrator   ~0007707

See attached screen shot showing the different behavior between 4.1 and 4.2.
pieper

pieper

2013-01-27 08:23

administrator   ~0007742

http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21640
pieper

pieper

2013-02-26 13:00

administrator   ~0008054

Hi Steve
It worked very well today so thanks a lot.
Celine

Hi Celine -

There is a bug in version 4.2 that can lead to something like what you
describe [1]. There is a fix in the nightly builds of slicer for the
issue

  • can you try one of those and let me know if it fixes the issue you are
    seeing?

-Steve

[1] http://na-mic.org/Bug/view.php?id=2878

On Mon, Feb 25, 2013 at 4:03 PM, <celine@nmr.mgh.harvard.edu> wrote:

Hi
My name is Celine and I am working on MS patient MRI. I use 3D slicer to
segment MS white matter lesions. I have a problem at some moment, only
on
the axial view, when I want to edit a segmentation on a label, it
applies
on the slice under the slice I am working on. So I close slicer and
restart, but sometimes it happens again. I use the editor tool.
Have anyone experienced such an issue?
Thanks for your help
Celine Louapre

slicer-users mailing list
slicer-users@bwh.harvard.edu
http://massmail.spl.harvard.edu/mailman/listinfo/slicer-users
To unsubscribe: send email to
slicer-users-request@massmail.spl.harvard.edu with unsubscribe as the
subject
pieper

pieper

2017-06-07 23:27

administrator   ~0014665

Fix committed to 2145-support-for-installing-extension-from-file branch.

Related Changesets

Slicer: 2145-support-for-installing-extension-from-file 5916e3d8

2013-01-27 13:22:17

pieper

Details Diff 		BUG: fix calculation for snapping to center of slice (fixes 0002878)

When performing pixel level operations, as with the Editor module,
or even when rendering multiple layers, the result can be ambiguous
when the slice plane is exactly at the border between two pixels due
to rounding differences accumulated through transforms. This can lead
to inconsistent rendering or operations being applied in a differnt
location than expected.

A regression from 4.1 had crept in by which the slice plane was
being consistently adjusted to be exactly on the pixel boundary,
leading to the potential issues. This fix ensures that the plane
is adjusted to the middle of the pixel.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21640 3bd1e089-480b-0410-8dfb-8563597acbee
mod - Libs/MRML/Logic/vtkMRMLSliceLogic.cxx Diff File

Issue History

Date Modified Username Field Change
2013-01-18 05:35 pieper New Issue
2013-01-18 05:35 pieper Status new => assigned
2013-01-18 05:35 pieper Assigned To => pieper
2013-01-18 05:36 pieper File Added: Screen Shot 2013-01-18 at 10.33.36 AM.png
2013-01-18 05:36 pieper Note Added: 0007707
2013-01-27 08:23 pieper Note Added: 0007742
2013-01-27 08:23 pieper Status assigned => resolved
2013-01-27 08:23 pieper Fixed in Version => Slicer 4.2.3
2013-01-27 08:23 pieper Resolution open => fixed
2013-02-12 09:40 jcfr Fixed in Version Slicer 4.2.3 => Slicer 4.3.0
2013-02-26 13:00 pieper Note Added: 0008054
2013-02-26 13:00 pieper Status resolved => closed
2013-03-07 05:39 pieper Relationship added duplicate of 0002930
2013-03-11 08:16 pieper Relationship added related to 0002999
2017-06-07 23:27 pieper Changeset attached => Slicer 2145-support-for-installing-extension-from-file 5916e3d8
2017-06-07 23:27 pieper Note Added: 0014665