Ticket #3637 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Add a SaveMask algorithm

Reported by: Stuart Campbell Owned by: Karl Palmen
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: petersonpf@… Blocked By: #3622
Blocking: Tester: Stuart Campbell

Description

Should save a mask file from an input workspace.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:3 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 32 to Iteration 33

Moved to iteration 33 at iteration 32 code freeze

comment:4 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:5 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:6 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:7 Changed 8 years ago by Karl Palmen

I'm curious about the meaning of this request.

Is the mask to be saved to a file? If so, sould there not also be a LoadMask algorithm to load a mask into a workspace.

Or does the mask need to be incuded in the Nexus file when the workspace is saved to Nexus?

comment:8 Changed 8 years ago by Karl Palmen

I see LoadMask already exists. http://www.mantidproject.org/LoadMask

comment:9 Changed 8 years ago by Karl Palmen

  • Owner changed from Anyone to Karl Palmen
  • Milestone changed from Release 2.4 to Release 2.5

comment:10 Changed 8 years ago by Karl Palmen

A SaveMask algorithm already exists http://www.mantidproject.org/SaveMask

comment:11 Changed 8 years ago by Karl Palmen

A unit test exists for SaveMask.

comment:12 Changed 8 years ago by Karl Palmen

  • Milestone changed from Release 2.5 to Release 2.4

comment:13 Changed 8 years ago by Karl Palmen

To test just run SaveMask for a workspace with a mask not involving group detectors. Then LoadMask on the saved mask file and check the mask is loaded OK.

There is a separate ticket #5311 concerning grouped detectors.

comment:14 Changed 8 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:15 Changed 8 years ago by Karl Palmen

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

comment:16 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:17 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

A SaveMask does indeed exist.

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4484

Note: See TracTickets for help on using tickets.