Ticket #6987 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

SANS: Display Mask does not work properly

Reported by: Nick Draper Owned by: Gesner Passos
Priority: critical Milestone: Release 3.2
Component: SANS Keywords:
Cc: peter.parker@… Blocked By:
Blocking: Tester: Peter Parker

Description (last modified by Gesner Passos) (diff)

From Richard Heenan:

In Masking tab, Display Mask, still does not work properly ( this is in the list somewhere ….) e.g. try MASK122M with both detectors on run 14259 from cycle 12_2

Answer for Richard: Also Can the new show instrument pick up our masks and display them ?

Attachments

display_not_working.py​ (54.4 KB) - added by Gesner Passos 7 years ago.
File that hightlights the problem.

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Priority changed from major to critical

comment:2 Changed 7 years ago by Gesner Passos

  • Status changed from new to accepted

comment:3 Changed 7 years ago by Nick Draper

  • Status changed from accepted to assigned

comment:4 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:5 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to the Backlog after R2.6

comment:6 Changed 7 years ago by Gesner Passos

  • Description modified (diff)
  • Milestone changed from Backlog to Release 3.0

comment:7 Changed 7 years ago by Gesner Passos

  • Component changed from MantidPlot to SANS

comment:8 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 3.0 to Backlog

Moved out to backlog as not enough time in R3.0

comment:9 Changed 7 years ago by Gesner Passos

  • Cc peter.parker@… added
  • Milestone changed from Backlog to Release 3.2

Changed 7 years ago by Gesner Passos

File that hightlights the problem.

comment:10 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

Refs #6987. Correct presentation of mask for SANS2D

Find the correct spectra number for empty loaded workspaces.

Changeset: 2548742254fd981ea4c8c5392f7fbe72d3f08c01

comment:11 Changed 7 years ago by Gesner Passos

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

Tester:

*Open SANS interface, select SANS2D instrument, and load mask_display.txt.

  • Change to Masking tab, and press display mask.
  • Change again to Run Numbers tab, put 22048 inside the Scattering/Sample
  • Go back to Masking tab, the masks should look like the same.

If you do this in the current version, you will see the mask in the front detector mirrored.

comment:12 Changed 7 years ago by Peter Parker

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

comment:13 Changed 7 years ago by Peter Parker

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/6987_sans_display_mask'

Full changeset: ​380575d02997ff800a017e1f9bf84eeb9fb26f3f

comment:14 Changed 7 years ago by Peter Parker

The "mirrored" effect was no longer apparent.

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7833

Note: See TracTickets for help on using tickets.