SlicedGeometry3D crash when volume is rotated by 45 degrees
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	espak
	Jun 22 2017, 5:43 PM

Description

Applications can crash because of a bug in mitk::PlaneGeometry and mitk::SlicedGeometry. The crash happens when reinitialising the renderers along a given orientation (sagittal, coronal or axial).

The crash happens because an invalid transformation matrix is created during the initialisation. This is because of the code I added to determine which axes is closest to the individual world orientations. In extreme cases when the volume is rotated exactly by 45 degrees, the same axis will be mapped to different orientations, and some axis will not be mapped to any orientations.

It is probably not common to have images which are rotated exactly by 45 degrees around some world axes. We discovered this because we have an affine transformation plugin that allows the user to apply any transformation on a volume interactively.

I will send a PR shortly.

Related Objects
Search...

		Status	Assigned	Task
		Resolved	kislinsk	T30214 Merge a few external GitHub pull requests
		Resolved	None	T22990 SlicedGeometry3D crash when volume is rotated by 45 degrees

Event Timeline

espak created this task.Jun 22 2017, 5:43 PM

https://github.com/MITK/MITK/pull/202

All the geometry tests of MITK pass on Linux, although probably there is not a test for this special case.

We still need to test it with our affine transform plugin.

espak added a subscriber: sparks.Jun 22 2017, 5:59 PM

espak updated the task description. (Show Details)Jun 22 2017, 6:03 PM

What's the result? :)

Our tests failed. :-(

We are working on it...

I have to allow an epsilon difference for rounding error, but it is not enough, it still crashes when the volume is rotated around two world axes by 45 degrees. I get a matrix that I thought is invalid but it is not.

I sent a new PR and closed the previous one.

https://github.com/MITK/MITK/pull/203

It is tested and works well for all the special cases.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs

kislinsk removed kislinsk as the assignee of this task.May 26 2020, 12:05 PM

kislinsk removed a subscriber: kislinsk.

kislinsk changed the task status from Wontfix to Resolved.Oct 4 2023, 9:14 AM

kislinsk added a parent task: T30214: Merge a few external GitHub pull requests.

kislinsk mentioned this in rMITKcf129df95b7f: Merge remote-tracking branch 'NifTK/T22990-map-axes-to-orientations' into….Oct 4 2023, 9:16 AM

kislinsk mentioned this in T30214: Merge a few external GitHub pull requests.Oct 4 2023, 9:17 AM

kislinsk mentioned this in T30248: 2023 Week 46 (Mid November).Nov 13 2023, 4:23 PM