Ticket #8792 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Create algorithm to convert old SNS ROI file to mask

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: major Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

An algorithm needs to be written that will convert an old style SNS reduction ROI file into a Mantid mask.

Attachments

NewerGeometryFullROI.dat​ (55.4 KB) - added by Michael Reuter 7 years ago.
BASIS ROI file
BASIS_Mask.png​ (54.5 KB) - added by Michael Reuter 7 years ago.
BASIS Mask

Change History

comment:1 Changed 7 years ago by Michael Reuter

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Michael Reuter

Refs #8792. Algorithm does conversion.

Changeset: 42d899b1bb16ae174d29789d64257db7dc16596e

comment:3 Changed 7 years ago by Michael Reuter

Refs #8792. Adding mask file saving and some docs.

Changeset: 4f8a6c2e20bc99442ef765e02302989054a4c3f8

comment:4 Changed 7 years ago by Michael Reuter

Refs #8792. Docs and conformance.

Changeset: b3a520ab6e8f9bbde71875bb6fb5d24bd5e4702f

Changed 7 years ago by Michael Reuter

BASIS ROI file

Changed 7 years ago by Michael Reuter

BASIS Mask

comment:5 Changed 7 years ago by Michael Reuter

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

The branch can be found ​here.

To test: Run the ConvertSnsRoiFileToMask algorithm on the attached ROI file. The resulting workspace mask should look like the attached picture when the instrument view is in spherical y mode. Check to make sure that a file (by) default is saved as long a you have a Mantid default save directory set. Otherwise, set a save directory via the entry box.

comment:6 Changed 7 years ago by Andrei Savici

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

comment:7 Changed 7 years ago by Michael Reuter

Refs #8792. Removing bank offset.

Changeset: f79beb393546f125f33babaef8bcc42e9c4f70e9

comment:8 Changed 7 years ago by Andrei Savici

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8792_sns_roi_to_mask'

Full changeset: ​1d274f256eb89e37950a76948fe2c69bc4504a2f

comment:9 Changed 7 years ago by Andrei Savici

This is a temporary algorithm, just to get the BASIS people happy. It will be removed soon. There is no unit test

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9636

Note: See TracTickets for help on using tickets.