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

Add a Region Of Interest mode to the Instrument view #13374

Merged
merged 4 commits into from
Aug 18, 2015

Conversation

NickDraper
Copy link
Contributor

This adds a region of interest (ROI) mode to the instrument window.

This gathers together functionality from other modes, and add an intersection based view for applying ROIs.

This also changes the Mask/Group tab name to "draw" and the instrument tree tab name to "instrument".

fixes #11824

Documentation

The page http://www.mantidproject.org/MantidPlot:_Instrument_View has been updated

release notes

http://www.mantidproject.org/ReleaseNotes_3_5_UI_Changes#Instrument_View

To Test

  1. Load up some data
  2. Play around with adding masks, ROI's and groups, check the interplay between them.
  3. Test the options under the Save or apply and save button in each mode.

@NickDraper NickDraper added this to the Release 3.5 milestone Aug 13, 2015
@NickDraper NickDraper changed the title Add a region of Interest mode to the instrument view Add a Region Of Interest mode to the Instrument view Aug 13, 2015
@mantid-roman mantid-roman self-assigned this Aug 17, 2015
@mantid-roman
Copy link
Contributor

  • When I apply and save ROI to a workspace (or file) the region on the view is masked out instead.
  • When the save file dialog opens the cursor changes to "wait".
  • I saved a ROI from a LOQ workspace to a file. Then I loaded it and masked the workspace with it. There were detectors I didn't select.

@NickDraper
Copy link
Contributor Author

Thanks I’ll take another look

Regards,
Nick Draper

From: Roman Tolchenov [mailto:notifications@github.com]
Sent: 17 August 2015 10:11
To: mantidproject/mantid
Cc: Draper, Nick (-,RAL,ISIS)
Subject: Re: [mantid] Add a Region Of Interest mode to the Instrument view (#13374)

  • When I apply and save ROI to a workspace (or file) the region on the view is masked out instead.
  • When the save file dialog opens the cursor changes to "wait".
  • I saved a ROI from a LOQ workspace to a file. Then I loaded it and masked the workspace with it. There were detectors I didn't select.


Reply to this email directly or view it on GitHubhttps://github.com//pull/13374#issuecomment-131739715.

•	When I apply and save ROI to a workspace (or file) the region on the view is masked out instead.
•	When the save file dialog opens the cursor changes to "wait".

Final issue was present before (so was the 2nd) and needs a new issue creating

re #11824
@NickDraper
Copy link
Contributor Author

The third issue we agreed after discussing it appears to be due to a mismatch of the instrument definintons used for LoadMask and Load for the dataset.

@mantid-roman
Copy link
Contributor

The windows test failure seems unrelated.

mantid-roman added a commit that referenced this pull request Aug 18, 2015
…I_to_new_workspace

Add a Region Of Interest mode to the Instrument view
@mantid-roman mantid-roman merged commit d516c16 into master Aug 18, 2015
@mantid-roman mantid-roman deleted the feature/11824_Extract_ROI_to_new_workspace branch August 18, 2015 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extract ROI into a new workspace
2 participants