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

Refactor ElasticQ Conversions #8707

Closed
mantid-builder opened this issue Sep 2, 2013 · 1 comment
Closed

Refactor ElasticQ Conversions #8707

mantid-builder opened this issue Sep 2, 2013 · 1 comment
Assignees
@DanNixon
Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic
Milestone
Release 3.4

Comments

@mantid-builder
Copy link
Collaborator

Original Reporter: Samuel Jackson

There's a new conversion function in UnitConversions which has been added as part of ticket http://trac.mantidproject.org/mantid/ticket/7729 to convert to ElasticQ. An example of its use can be seen at around lines 1124-1131 in MantidUI.cpp.

It would be good if other places in the project could use a similar method of conversion (such as in ConvertSpectrumAxs) to prevent code duplication.
@mantid-builder
Copy link
Collaborator Author

This issue was originally trac ticket 7862

@mantid-builder mantid-builder added the Indirect/Inelastic Issues and pull requests related to indirect or inelastic label Jun 3, 2015
@mantid-builder mantid-builder added this to the Release 3.4 milestone Jun 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DanNixon DanNixon

Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic
Projects
None yet
Milestone
Release 3.4
Development

No branches or pull requests
2 participants
@mantid-builder @DanNixon