Page MenuHomePhabricator

Default view of 3D object is wrong
Closed, ResolvedPublic

Description

Clicking the reset view button in rendering window involves a 3D view of the patient, who lies on his stomach in the transversal view

Event Timeline

I'm resetting the needs_core_mod flag until there is a change request (then I'll get a new mail notification)

We are not able to decide, if the current default 3D view should be changed or not.
Ivo, what do you think? Is there a reason why the default 3D view is rotated regarding the upper left view?

We compared the default 3D view with Slicer and Osirix:

-Slicer shows the 3D volume by default from a coronal view.
-Osirix shows a volume rendered image by default from a sagital view.

So, both programs are not really comparable to MITK's transversal view for the 3D window and it doesnt seem to exist a norm.

What should we do?

(In reply to comment #2)

We are not able to decide, if the current default 3D view should be changed or
not.

What do you mean by "default 3D view"? The default initializiation in the CoreApp/ExtApp does show the patient lying on his back, doesn't it?

I do not know what the "reset view button" actually does. Perhaps the problem is related to the one described in T3402, see esp T3402 comment #5:

The default setting of "rotated" for "viewDirection==Transversal" is "rotated==true" - otherwise the patient would lie on his stomach in the transversal view (for "viewDirection==sagittal" or "coronal" the setting "rotated==false" results in the expected behavior). The origin of this behavior is the definition of the dicom coordinate system.

Ivo, what do you think? Is there a reason why the default 3D view is rotated
regarding the upper left view?

We compared the default 3D view with Slicer and Osirix:

-Slicer shows the 3D volume by default from a coronal view.
-Osirix shows a volume rendered image by default from a sagital view.

So, both programs are not really comparable to MITK's transversal view for the
3D window and it doesnt seem to exist a norm.

The patient should definitely lie on his back!

What should we do?

Probably change what the "reset view button" is doing - does it call a SliceNavigationController? If so, may only need to change the way how it calls it. Can you post the name of the file with the implementation.
You probably do NOT need to change something within the Geometry classes.

(In reply to comment #3)
The comment did not fit to the question: the question refers to the 3D view, the comment to the transversal view. Thus, ignore comment #3.

[SVN revision 22551]
FIX (#3472): changed the default 3d view to an anterior camera view, as it is in other viewers (e.g. Slicer) and as we discussed it with Ivo Wolf.