Ticket #6938 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Change live data on step scan interface to start from beginning of run.

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By: #6661
Blocking: Tester: Michael Reuter

Description

This is a one line change so that we pull in data from the start of the run rather than 'now'. Requires 6661 to be merged into master first.

Change History

comment:1 Changed 7 years ago by Russell Taylor

  • Priority changed from blocker to major
  • Milestone changed from Release 2.5 to Release 2.6

SMS replay isn't going to make it into this release.

comment:2 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:3 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to backlog at the code freeze for R2.6

comment:4 Changed 7 years ago by Russell Taylor

  • Status changed from new to inprogress
  • Milestone changed from Backlog to Release 2.6

comment:5 Changed 7 years ago by Russell Taylor

  • Milestone changed from Release 2.6 to Release 3.0

On second thoughts I'm going to defer this until the live listener is more friendly when you are not in a run.

Hmm....seems like I can't change it back to 'new'. Nevermind.

comment:6 Changed 7 years ago by Russell Taylor

  • Status changed from inprogress to new

comment:7 Changed 7 years ago by Russell Taylor

  • Status changed from new to inprogress

Re #6938. Collect live data from the start of the current run.

Instead of just whenever the listener is started.

Changeset: cc85dcbc5b37a8a35b87085fb6a14bb16f0846d2

comment:8 Changed 7 years ago by Russell Taylor

Re #6938. Start initial read from start of run as well.

Changeset: a7a93a4a7d158e08a9778b24c779acf313444644

comment:9 Changed 7 years ago by Russell Taylor

Re #6938. Make sure that signals & observers are only hooked up once.

Changeset: 2352d0de87dd4b2ce03b3fc57aaf642671a02924

comment:10 Changed 7 years ago by Russell Taylor

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

To test:

This needs to be tested at the SNS. The branch is feature/6938_stepscan_from_start_of_live_run

  • Set your default instrument to HYSPEC or SEQUOIA.
  • Bring up the Step Scan Analysis GUI from the Interfaces menu.
  • Select the 'Use live stream' button
    • If the chosen instrument is not in a run, you will get an error message at this point (assuming that #7785 has not been completed). The only option is to try the other instrument and if you encounter the same situation there, to try again later.
  • When it becomes enabled, click the start button.
  • Unless you happen to do this right at the start of the run, the plot should show a relatively low number of counts (< 1000) when it first appears, then a lot more on the first refresh, and then only a 'few' more on the second refresh. This contrasts with the previous behaviour where the growth would be ~linear.

comment:11 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:12 Changed 7 years ago by Michael Reuter

This seems to be working correctly.

comment:13 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/6938_stepscan_from_start_of_live_run'

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7784

Note: See TracTickets for help on using tickets.