Ticket #6114 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Bug in instrument view

Reported by: Peter Peterson Owned by: Wenduo Zhou
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description (last modified by Peter Peterson) (diff)

Antonio DosSantos pointed this one out. 

Hi Pete,
 
Here is the screen shot of what I was telling you about.

I am attaching the code that did it, it resorts to a set of pre-saved masked files 
in one of the IPTS’s of the SNAP area.
 
The code is not very neat, so let me know if you have any questions.
 
Antonio

Attachments

image003.jpg​ (51.3 KB) - added by Peter Peterson 8 years ago.
Bug shown in instrument view
apply_masks.py​ (4.7 KB) - added by Peter Peterson 8 years ago.
Python script to generate bug

Change History

Changed 8 years ago by Peter Peterson

Bug shown in instrument view

Changed 8 years ago by Peter Peterson

Python script to generate bug

comment:1 Changed 8 years ago by Peter Peterson

  • Owner set to Wenduo Zhou
  • Status changed from new to assigned
  • Description modified (diff)

comment:2 Changed 8 years ago by Wenduo Zhou

  • Status changed from assigned to accepted

comment:3 Changed 8 years ago by Wenduo Zhou

The instrument view was messed up by algorithm "SumNeighbours".

comment:4 Changed 8 years ago by Wenduo Zhou

SumNeighbours() is just a wrap on SmoothNeighbours(). In the case to deal with Workspace2D, the current implementation fails to write the correct detector IDs to each spectrum.

comment:5 Changed 8 years ago by Wenduo Zhou

Fixed the instrument view. Refs #6114.

Changeset: 78c8b4032127b65ec67c90875db9fa835c85f4b7

comment:6 Changed 8 years ago by Wenduo Zhou

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

comment:7 Changed 8 years ago by Wenduo Zhou

Fixed the instrument view. Refs #6114.

Changeset: da7204ee18d00e43ca6c2c0dfbf10a174a0c2004

comment:8 Changed 8 years ago by Michael Reuter

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

comment:9 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks to have been fixed.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6960

Note: See TracTickets for help on using tickets.