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

Rebin2D performance test #10718

Closed
OwenArnold opened this issue Jul 11, 2014 · 1 comment
Closed

Rebin2D performance test #10718

OwenArnold opened this issue Jul 11, 2014 · 1 comment
Assignees
@NickDraper
Milestone
Release 3.3

Comments

@OwenArnold
Copy link
Contributor

The Rebin2D performance test is reporting a slight increase in processing time. I notice that the implementation of the test for performance https://github.com/mantidproject/mantid/blob/master/Code/Mantid/Framework/Algorithms/test/Rebin2DTest.h includes the setup steps for the workspace to rebin. This time should not be tracked in this test. Make the input workspace as part of the test object construction instead of within the test method for test_On_Large_Workspace()

Keywords: Student
@OwenArnold
Copy link
Contributor Author

This issue was originally trac ticket 9876

@OwenArnold OwenArnold added this to the Release 3.3 milestone Jun 3, 2015
wdzhou added a commit that referenced this issue Nov 26, 2015
@wdzhou
Refs #10718. Added the support on parameter to smooth background.
804e12a 
And cleaned up the outputs as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NickDraper NickDraper

Labels
None yet
Projects
None yet
Milestone
Release 3.3
Development

No branches or pull requests
2 participants
@NickDraper @OwenArnold