Ticket #1762 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Improve scriptability of direct excitations work

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Iteration 29
Component: Mantid Keywords:
Cc: campbellsi@… Blocked By:
Blocking: Tester: Stuart Campbell

Description

Allow more scripting of the excitations work by chopping the script up into smaller chunks.

Also improve the detector testing interaction so that it doesn't repeatedly load data files to speed up the reduction.

Change History

comment:1 Changed 10 years ago by Stuart Campbell

  • Cc campbellsi@… added

comment:2 Changed 10 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Martyn Gigg

You should consider refactoring the code to use Matthieus framework

comment:3 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:4 Changed 10 years ago by Martyn Gigg

(In [8639]) Updated ConvertToMatrixWorkspace so that it doesn't do anything if it doesn't need to as in some circumstances wihin the inelastic scripts it was taking a couple of seconds doing a copy unnecessarily. Re #1762

comment:5 Changed 10 years ago by Martyn Gigg

(In [8673]) Minor fix for an error message that wasn't printing correctly. Re #1762"

comment:6 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:7 Changed 10 years ago by Martyn Gigg

(In [10039]) Refs #1762. Jon Taylor notice that the sun files option did not work correctly on the convert to energy GUI, fixed this and a correction to a minor bug that occurred if the empty instrument workspace was already present.

comment:8 Changed 10 years ago by Martyn Gigg

(In [10102]) Fix for diag script overwriting masks from successive diagnose calls. This will allow them to be combined. Re #1762

comment:9 Changed 10 years ago by Martyn Gigg

(In [10163]) Re #1591. Added 'Version' keyword to the mantid simple API calls. It defaults to -1 indicating the highest version.\nUpdated Ubuntu package list which was slightly wrong.\nMinor fix to sum load_run function in shared module. Refs #1762

comment:10 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:11 Changed 9 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:12 Changed 9 years ago by Martyn Gigg

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

I'm going to close this as the bulk of the work is done. There will be a new ticket for the reducer work.

comment:13 Changed 9 years ago by Stuart Campbell

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

comment:14 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to closed

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2609

Note: See TracTickets for help on using tickets.