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.

Sign up for GitHub

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

Jump to bottom

Regression test for MaskWorkspaceToCalFile #7050

Closed
OwenArnold opened this issue Nov 19, 2012 · 1 comment · Fixed by #13067
Closed

Regression test for MaskWorkspaceToCalFile #7050

OwenArnold opened this issue Nov 19, 2012 · 1 comment · Fixed by #13067
Assignees
@NickDraper
Labels
Diffraction Issues and pull requests related to diffraction

Comments

@OwenArnold
Copy link
Contributor

This issue was originally TRAC 6204

Fix in http://trac.mantidproject.org/mantid/ticket/6124 is not supported by a regression test to prove that it now works. Should add a regression test to the existing MaskWorkspaceToCalFileTest.py.
@OwenArnold
Copy link
Contributor Author

@NickDraper (2013-04-29T09:49:53):
Moved to r2.6 at the end of r2.5

@NickDraper (2013-07-26T13:55:00):
Moved to backlog at the code freeze for R2.6

@NickDraper (2014-02-02T15:34:47):
Moved to backlog at the end of Release 3.1

@NickDraper (2014-02-14T11:04:58):
Bulk move to assigned at the introduction of the triage step

@OwenArnold OwenArnold added the Diffraction Issues and pull requests related to diffraction label Jun 3, 2015
NickDraper added a commit that referenced this issue Jul 9, 2015
@NickDraper
added grouped unit test, and soed up all unit tests by 5 times
37059b0 
By moving from laoding files to CreateSampleWorkspace
25-30sec -> 5.2sec

re #7050
@NickDraper NickDraper mentioned this issue Jul 9, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NickDraper NickDraper

Labels
Diffraction Issues and pull requests related to diffraction
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added grouped unit test, and sped up all unit tests by 5 times mantidproject/mantid
2 participants
@NickDraper @OwenArnold