Ticket #1956 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

FilterByLogValue errors

Reported by: Peter Peterson Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

Two things

  • The wording of the happy yellow box in the dialog is backwards from what the function does (minor)
  • The spectra to detector map disappears when modifying a workspace in place.

For the second item it can be reproduced by:

  1. LoadSNSEventNeXus PG3_1879_event.nxs
  2. FilterByLogValue inplace
  3. ConvertUnits to dspacing

This last step generates the error: 

ConvertUnits: SpectraDetectorMap has not been populated.
Error in execution of algorithm ConvertUnits:
ConvertUnits: error (see log)

A workaround for this error (for now) is to filter into a new workspace and rename back on top of the original.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

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

(In [7621]) Fixes #1956: Test added for filtering by log value in place; passes. See also ticket #1957.

comment:3 Changed 10 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:4 Changed 10 years ago by Peter Peterson

  • Status changed from verifying to closed

This works as advertised.

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2803

Note: See TracTickets for help on using tickets.