Ticket #3959 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Make VATES views handle time updates

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 2.0
Component: VATES Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

The VATES views as a group should be able to furnish the information for time step updates to the main level widget. Currently, the main level widget determines this information.

Change History

comment:1 Changed 9 years ago by Michael Reuter

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Michael Reuter

In [15326]:

Adding animation control update to time control widget. This refs #3959.

comment:3 Changed 9 years ago by Michael Reuter

Adding another slot call. This refs #3959.

Changeset: af36457e072cf2c72aef7e0b71ae0bc5621c35ef

comment:4 Changed 9 years ago by Michael Reuter

Adding another slot call. This refs #3959.

Changeset: af36457e072cf2c72aef7e0b71ae0bc5621c35ef

comment:5 Changed 9 years ago by Michael Reuter

First step in moving time step updates. This refs #3959.

This is the first step in taking out the time step checks and updates from the main view widget. This step handles the initial checks for both standalone and plugin modes. The move allowed some unneeded code in the main view widget to be removed.

Changeset: 70737e1b3828dce8057a99abec72b7c1a347a3e2

comment:6 Changed 9 years ago by Michael Reuter

Finishing move of time handling to view base class. This refs #3959.

All of the time step handling for the animation controls is now in the view base class. I added a mode for updating the number of time steps in the pipeline object when the rebinner is called.

Changeset: e75bb74e14399cbb2df4aa8ef040ac811464a31d

comment:7 Changed 9 years ago by Michael Reuter

Refs #3959. Using more correct group discovery.

Changeset: f219a412f798441d7fc7db8da2541b00fc4cfeb2

comment:8 Changed 9 years ago by Michael Reuter

First step in moving time step updates. This refs #3959.

This is the first step in taking out the time step checks and updates from the main view widget. This step handles the initial checks for both standalone and plugin modes. The move allowed some unneeded code in the main view widget to be removed.

Changeset: 70737e1b3828dce8057a99abec72b7c1a347a3e2

comment:9 Changed 9 years ago by Michael Reuter

Finishing move of time handling to view base class. This refs #3959.

All of the time step handling for the animation controls is now in the view base class. I added a mode for updating the number of time steps in the pipeline object when the rebinner is called.

Changeset: e75bb74e14399cbb2df4aa8ef040ac811464a31d

comment:10 Changed 9 years ago by Michael Reuter

Refs #3959. Using more correct group discovery.

Changeset: f219a412f798441d7fc7db8da2541b00fc4cfeb2

comment:11 Changed 9 years ago by Michael Reuter

Refs #3959. Switching to first time step.

Changeset: 704a3b4f89d63885551a4012d20ddf565fb45985

comment:12 Changed 9 years ago by Michael Reuter

Refs #3959. Removing duplicated code.

Changeset: 4e7f56333b3852a517ae8a66119afa046bce1f01

comment:13 Changed 9 years ago by Michael Reuter

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

The best way to test is open a 4D data set (like fe_demo_30.sqw) in either the standalone or MantidPlot simple interface. Using the animation controls, play the "time" slices to see the number. Then apply the rebinner and change the number of time steps. Replay the movie and the number of slices shown should change.

comment:14 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:15 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

Following test scenario. Does what it's supposed to do during animation. Time values/indexes seem exactly right as read off the animation control after rebinning with various n-bins in the mapped t-dimension.

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4806

Note: See TracTickets for help on using tickets.