Ticket #2157 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Provide equivalent of COLETTE DISPLAY TIME diagnostic

Reported by: Nick Draper Owned by: Sofia Antony
Priority: critical Milestone: Iteration 27
Component: Mantid Keywords:
Cc: stephen.king@… Blocked By:
Blocking: Tester: Steve Williams

Description (last modified by Nick Draper) (diff)

see #2155

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Priority changed from major to critical
  • Description modified (diff)

comment:2 Changed 10 years ago by Sofia Antony

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Sofia Antony

(In [9322]) re#2157 -Time Integral button now use SumSpectra algorithm and plots the first spectrum

comment:4 Changed 10 years ago by Sofia Antony

(In [9393]) re#2157 - added time channel and pixel masking functionality SNAS to diagnostics tab

comment:5 Changed 10 years ago by Sofia Antony

(In [9399]) re#2157 - some improvements to sans diagnostics tab

comment:6 Changed 10 years ago by Sofia Antony

(In [9433]) re#2157 - some improvements to sans diagnostics tab

comment:7 Changed 10 years ago by Sofia Antony

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

comment:8 Changed 10 years ago by Steve Williams

  • Status changed from verify to verifying
  • Tester set to Steve Williams

comment:9 Changed 10 years ago by Steve Williams

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The time diagnostic button needs to be specific for the rear and front detector and exclude monitors. Note in SANS runs before SANS2D00000684 the detector banks and monitors were stored in different places in the workspace.

comment:10 Changed 10 years ago by Steve Williams

Don't handle the changes to the detector bank and monitor positions before run SANS2D00000684 these will be dealt with in ticket #2435.

comment:11 Changed 10 years ago by Steve Williams

Passing the correct spectra to SumSpectra will be enough. No change is needed to the IncludeMonitors property this can remain at the default value.

comment:12 Changed 10 years ago by Sofia Antony

(In [9535]) re#2157 - fix for passing spectra to SumSpectra.Also implemented the #2155 suggestions by Steve and re factored the code.

comment:13 Changed 10 years ago by Sofia Antony

  • Status changed from reopened to accepted

comment:14 Changed 10 years ago by Sofia Antony

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

comment:15 Changed 10 years ago by Steve Williams

  • Status changed from verify to verifying

comment:16 Changed 10 years ago by Steve Williams

  • Status changed from verifying to closed

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3004

Note: See TracTickets for help on using tickets.