Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SANS2D Tube Detectors - Masking Umbrella Ticket #9691

Closed
2 of 4 tasks
mantid-builder opened this issue Feb 3, 2014 · 2 comments
Closed
2 of 4 tasks

SANS2D Tube Detectors - Masking Umbrella Ticket #9691

mantid-builder opened this issue Feb 3, 2014 · 2 comments
Assignees
Labels
High Priority An issue or pull request that if not addressed is severe enough to postponse a release. SANS Issues and pull requests related to SANS

Comments

@mantid-builder
Copy link
Collaborator

sans_feature
This issue was originally TRAC 8847

Original Reporter: Gesner Passos

This ticket is blocked by :

This ticket is blocks : TRAC8921

Umbrella ticket used to cover all work to support masking the new SANS2D tube detectors.

Child of http://trac.mantidproject.org/mantid/ticket/8921, which covers all work necessary for the new layout of the instrument.

@mantid-builder
Copy link
Collaborator Author

@NickDraper (2014-02-14T10:59:49):
Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state


Gesner Passos (2014-03-04T16:27:33):
Richard asked to extend the Mask Syntax as described in http://trac.mantidproject.org/mantid/ticket/8972.

I have suggested Richard

Why not use the InstrumentView to generate the mask you want? 

In the instrumentView you are able to create mask using squares, circles. Using it in the cylindrical view, you will be able to get the correct position of the detectors inside the tube. You could them apply the mask to the workspace and point that specific workspace as the mask template. 

The masking process would be go through the workspace to process and apply mask to all the spectra that is masked inside the mask template workspace.

Besides, we could add some tools to allow us to construct the mask based on previous mask templates. 

The idea is to extend the Mask Syntax to use a MaskFile to define the spectra to mask. To complete this solution, an UNMASK tool should be provided by InstrumentView.


Gesner Passos (2014-03-04T16:28:51):
It is an open discussion up to now.


@NickDraper (2014-03-07T16:20:29):
Ownership handed over to Peter

@mantid-builder mantid-builder added High Priority An issue or pull request that if not addressed is severe enough to postponse a release. SANS Issues and pull requests related to SANS labels Jun 3, 2015
@Matthew-Andrew
Copy link
Contributor

It is likely that ROI and mask combined complete this ticket. I will double check with scientists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High Priority An issue or pull request that if not addressed is severe enough to postponse a release. SANS Issues and pull requests related to SANS
Projects
None yet
Development

No branches or pull requests

4 participants