Ticket #6272 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Auto Update on SliceViewer

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By: #6249
Blocking: #6392 Tester: Stuart Campbell

Description (last modified by Owen Arnold) (diff)

Output from design session in #6249. Would be good to have an auto-update feature similar to that used in ParaView, so that changes in rebinning are automatically applied to a changed view (without having to press the refresh button.)

While we are at it. Another simple change that would be nice, would be to make the 'Apply' button on the SliceViewer look more like the one in ParaView. This might help users go between the SliceViewer and the VSI.

Attachments

Screen Shot 2013-01-30 at 15.05.05.png (53.0 KB) - added by Owen Arnold 8 years ago.
Auto Rebin Circled

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

  • Description modified (diff)

comment:3 Changed 8 years ago by Owen Arnold

refs #6272 Auto refresh introduced.

An extra button and an extra menu option has been added for this. There are two points that I want to address and check. Firstly, we currently have a lock mode, which is a kind of auto update, but only for when the sliders are moved. Its probably confusing to have two forms of autoupdate, so we should probably roll them into the single lock mode. This should be easy to do. Secondly, if we do not opt for this, it might be best to make the auto refesh icon look something like the current refresh icon, to save confusion.

Changeset: 8b3c8b359d25212a0b437c86c182b2c16f1d63cb

comment:4 Changed 8 years ago by Owen Arnold

refs #6272 Auto refresh introduced.

An extra button and an extra menu option has been added for this. There are two points that I want to address and check. Firstly, we currently have a lock mode, which is a kind of auto update, but only for when the sliders are moved. Its probably confusing to have two forms of autoupdate, so we should probably roll them into the single lock mode. This should be easy to do. Secondly, if we do not opt for this, it might be best to make the auto refesh icon look something like the current refresh icon, to save confusion.

Changeset: 8b3c8b359d25212a0b437c86c182b2c16f1d63cb

comment:5 Changed 8 years ago by Owen Arnold

Tester. I'm marking this ticket as done, but I think we'll want to revisit the functionality around auto-updating and re-binning on the SliceViewer at some date to make it simpler. For future reference:

  • The Rebin button rebins only the current slice point within the viewing extents.
  • The refresh button has to be invoked every time the view is panned or zoomed.
  • The auto rebin button automatically applies the refresh.
  • The lock button when unlock auto rebins to the current slice point (as sliders are moved)

comment:6 Changed 8 years ago by Owen Arnold

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:7 Changed 8 years ago by Owen Arnold

  • Blocking 6392 added

comment:8 Changed 8 years ago by Owen Arnold

I created ticket #6392 to take a greater look into this area for the future.

comment:9 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:10 Changed 8 years ago by Stuart Campbell

I think that it works as expected - using the menu item is straight forward and obvious, but I found the use via the icons slightly confusing.

I couldn't actually see a autorebin button ? The icon file that was committed looks to be the one for overlaying a peaks workspace (from the tooltip text)

Changed 8 years ago by Owen Arnold

Auto Rebin Circled

comment:11 Changed 8 years ago by Owen Arnold

refs #6272. Wrong tooltip message fixed.

Changeset: cb727d15640f4263a9e19289814f1cf2574012fa

comment:12 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

I managed to get some random crashes when toggling auto rebin on/off using the icon - but only on the Mac. It's not 100% reproducible so we should just look at it next release.

comment:13 Changed 8 years ago by Owen Arnold

refs #6272. Wrong tooltip message fixed.

Changeset: cb727d15640f4263a9e19289814f1cf2574012fa

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7118

Note: See TracTickets for help on using tickets.