Ticket #5074 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

MedianDetectorTest changes the InputWorkspace

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: critical Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

Run this script 

LoadEventNexus(Filename=r'/home/8oz/data/TOPAZ_5625_event.nxs',OutputWorkspace='TOPAZ_5625')
Rebin(InputWorkspace='TOPAZ_5625',OutputWorkspace='TOPAZ_5625_1bin',Params='0,20000,20000',PreserveEvents='0')
MedianDetectorTest(InputWorkspace='TOPAZ_5625_1bin',OutputWorkspace='TOPAZ_5625_1bin_tested',HighThreshold='2')

The TOPAZ_5625, and TOPAZ_5625_1bin end up looking grayed out (seems their pixels got masked or something). These were INPUT workspaces so they should not have been modified.

Perhaps the parameter map was copied by reference/pointer instead of cloned???

Change History

comment:1 Changed 8 years ago by Andrei Savici

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.1 to Release 2.2

Moved at end of release 2.1

comment:3 Changed 8 years ago by Janik Zikovsky

  • Owner changed from Andrei Savici to Janik Zikovsky

comment:4 Changed 8 years ago by Janik Zikovsky

  • Milestone changed from Release 2.2 to Release 2.1

comment:5 Changed 8 years ago by Janik Zikovsky

Refs #5074: fix for shared parameter maps when copying a workspace

affected MedianDetectorTest in SOME cases

Changeset: 36e0dfd694ff1b1ee9647d21e066766e9d6bf4ac

comment:6 Changed 8 years ago by Janik Zikovsky

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

comment:7 Changed 8 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:8 Changed 8 years ago by Andrei Savici

  • Status changed from verifying to closed

comment:9 Changed 8 years ago by Janik Zikovsky

Refs #5074: fix for shared parameter maps when copying a workspace

affected MedianDetectorTest in SOME cases

Changeset: 36e0dfd694ff1b1ee9647d21e066766e9d6bf4ac

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5920

Note: See TracTickets for help on using tickets.