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

IDR calibration show pre and post smooth #171

Merged
merged 4 commits into from Feb 16, 2015

Conversation

DanNixon
Copy link
Member

@DanNixon DanNixon commented Feb 5, 2015

This is Trac ticket: http://trac.mantidproject.org/mantid/ticket/10513

To test run IDR > Calibration using IRS26173.raw checking "Create RES file", "Smooth RES" and "Plot Result", you should get a plot for the calibration workspace and resolution showing the pre and post smooth spectra on top of each other.

Disabling "Smooth RES" should only give the post smooth workspace plot.

@DanNixon DanNixon added Indirect/Inelastic Issues and pull requests related to indirect or inelastic Inelastic labels Feb 5, 2015
@DanNixon DanNixon modified the milestone: Release 3.4 Feb 5, 2015
@DanNixon
Copy link
Member Author

This also does not save the correct workspace when smooth is enabled.

…ooth

Conflicts:
	Code/Mantid/MantidQt/CustomInterfaces/src/Indirect/IndirectCalibration.cpp

Refs #10513
@DanNixon
Copy link
Member Author

Jenkins, retest this please.

@martyngigg martyngigg self-assigned this Feb 16, 2015
@martyngigg
Copy link
Member

I've played around with the tab and as far as I can tell it is doing the correct thing.

martyngigg added a commit that referenced this pull request Feb 16, 2015
…show_pre_post_smooth

IDR calibration show pre and post smooth
@martyngigg martyngigg merged commit fbe8a6f into master Feb 16, 2015
@martyngigg martyngigg deleted the feature/10513_calibration_show_pre_post_smooth branch February 16, 2015 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants