Ticket #4321 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SANS: extra check of PixelAdj in Q1D & Qxy

Reported by: Anders Markvardsen Owned by: Karl Palmen
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

check in Q1D and Qxy that if the input parameter PixelAdj contains numbers less than or equal to zero for none masked pixels a warning/error is thrown to match SK comment: we used 0 for the efficiency of the monitor spectra in the flat cell and -1 for unused/irrelevant spectra.

Change History

comment:1 Changed 9 years ago by Anders Markvardsen

  • Status changed from new to assigned
  • Owner changed from Anders Markvardsen to Karl Palmen
  • Priority changed from critical to major

Also add unit test to Q1D2Test. Add method createInputWorkspacesMask which load the WS created by createInputWorkspaces() and mask some of its pixels. Then create new test where a) create pixelAdj WS as a chop of WS from createInputWorkspaces(). Then run Q1D where a) chopped WS have zeros for masked pixels - should pass b) where chopped WS have zeros for pixels which are not masked - should fail c) where chopeed WS have all positive values - should pass.

comment:2 Changed 9 years ago by Michael Reuter

Refs #4321. Updates to tests.

Changeset: 58fba8c3cc7fff1936c1301bf3d1369ef98dc752

comment:3 Changed 9 years ago by Michael Reuter

Checked into wrong ticket. Sorry.

comment:4 Changed 9 years ago by Michael Reuter

Refs #4321. Updates to tests.

Changeset: 58fba8c3cc7fff1936c1301bf3d1369ef98dc752

comment:5 Changed 9 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:6 Changed 9 years ago by Karl Palmen

Correct a comment re #4321

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: d64301bb410351272934842bd616389c18a7cc4f

comment:7 Changed 9 years ago by Karl Palmen

Check bounding box first, so that error can be thrown re #4321

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 2787a8c9c2c6c67993e769d01463c80f8c9c16b4

comment:8 Changed 9 years ago by Karl Palmen

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

comment:9 Changed 9 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:10 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Implemented and unit tested

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5168

Note: See TracTickets for help on using tickets.