Ticket #1903 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Adjust Diagnose* algorithms to output MaskedWS rather than an output list

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: critical Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

The various Diagnose algorithms for excitations should not ouput a list of the bad detectors (the long list causes problems).

See #1902 for structure

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Martyn Gigg to Andrei Savici

comment:2 Changed 10 years ago by Andrei Savici

  • Status changed from assigned to accepted

comment:3 Changed 10 years ago by Martyn Gigg

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

comment:4 Changed 10 years ago by Martyn Gigg

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:5 Changed 10 years ago by Martyn Gigg

  • Owner changed from Andrei Savici to Martyn Gigg
  • Status changed from reopened to assigned

comment:6 Changed 10 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:7 Changed 10 years ago by Martyn Gigg

(In [7962]) Add a method on MatrixWorkspace to mask a single spectra as I found I was replicating code in algorithms. Re #1903

comment:8 Changed 10 years ago by Martyn Gigg

(In [7976]) Updated detector diagnostic script to work with new masking functionality. Re #1903

comment:9 Changed 10 years ago by Martyn Gigg

(In [8014]) A few updates to the diagnostics.py script and better handling of temporary workspaces in binary operations in Python. Re #1903

comment:10 Changed 10 years ago by Martyn Gigg

(In [8061]) Minor fix for detector diagnostic script. Re #1903

comment:11 Changed 10 years ago by Martyn Gigg

(In [8084]) Diagnostic algorithms now output a mask workspace rather than a list of spectra. Scripts have been update to handle this. Re #1903 Re #1904. Also killed a pervasive warning in MDGeometry.

comment:12 Changed 10 years ago by Martyn Gigg

(In [8085]) Revert an accidental commit to a file. Re #1903

comment:13 Changed 10 years ago by Martyn Gigg

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

comment:14 Changed 10 years ago by Michael Whitty

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

comment:15 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

seems to work. creates output workspace which conforms to requirements detailed for ExtractMasking.

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2750

Note: See TracTickets for help on using tickets.