Ticket #6987 (closed: fixed)
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
Change History
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: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
- Attachment display_not_working.py added
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