MRtrix3 / mrtrix3

MRtrix3 provides a set of tools to perform various advanced diffusion MRI analyses, including constrained spherical deconvolution (CSD), probabilistic tractography, track-density imaging, and apparent fibre density
http://www.mrtrix.org
Mozilla Public License 2.0
291 stars 179 forks source link

Alert user when switching between images with lock to grid on. #452

Open draffelt opened 8 years ago

draffelt commented 8 years ago

To further an email discussion with @jdtournier, and an in-house discussion with @thijsdhollander and @Lestropie, it would be nice to alert users when they switch between two images that have grids with different orientations wrt to scanner coordinates.

A lot of users don't appreciate the difference between scanner space and image space, and that the FOV can be differently rotated with respect to scanner space for different images acquired in the same session. Furthermore I've seen users that do understand this get caught up on why two images of different modalities are not aligned after pre-processing when opening in mrview, because they did not realise lock to axis was on (or what this button even does).

We don't want to have lock-to-axis off by default, since it would require full load of images that are rendered off axis (wrt to the first image). And as @Lestropie pointed out, users may also just want to see the image as it was acquired, and not with respect to another image that is currently loaded.

So, we have been discussing the option of somehow alerting the user that when switching between two images, they are not in the same space.

Options so far include:

jdtournier commented 8 years ago

I was originally in favour of the axes render, just as a little glyph in the corner somewhere (although we're running out of corners...), but it's true that for subtle changes, that wouldn't be noticeable. Besides, the orientation labels at the edges of the display already serve that purpose, and clearly aren't visible enough for people to notice (maybe we could change the colour of those labels to something more obvious?). A full blown pop-up would be too disruptive, but how about a little one liner at the top of the display with a temporary message in a red font, rendered the same way as the header info, etc? Just have it pop up for 3 seconds with the message "locking to image axes" or something...?

Lestropie commented 5 years ago

One idea I had in this context that I thought I'd write down:

If we had a small window displaying the axis orientations (which could potentially have some kind of temporary visual emphasis upon main image change if lock to axes is on), it could show x-y-z scanner space directions as RGB lines, and i-j-k image axis directions as CMY lines. Potential weakness is that at low resolution & only subtlely tilted acquisition FoVs it may not be possible to distinguish them, though maybe there are solutions for that, e.g. display the first axis set thicker than the second & don't use a depth buffer.

Lestropie commented 5 years ago

Bumping due to workshop attendee confusion.