Ticket #3637 (closed: fixed)
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: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: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