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

IqtFit algorithm #9750

Closed
mantid-builder opened this issue Feb 7, 2014 · 2 comments
Closed

IqtFit algorithm #9750

mantid-builder opened this issue Feb 7, 2014 · 2 comments
Assignees
Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic

Comments

@mantid-builder
Copy link
Collaborator

This issue was originally TRAC 8907

They both do very similar things, but FuryFitMult uses a multi domain function instead. We can combine most of the functionality and make the output of the parameters table more consistent.

Part of the issue with this is that multi domain functions cannot be converted to a function string from the C++ API. Roman has a ticket to fix this as part of #10094.

Ideally they should be converted to a single algorithm that has the option to do an individual or sequential fit.

@mantid-builder
Copy link
Collaborator Author

@NickDraper (2014-02-14T11:07:55):
bulk move to assigned at the into of the triage step

@mantid-builder mantid-builder added the Indirect/Inelastic Issues and pull requests related to indirect or inelastic label Jun 3, 2015
@DanNixon DanNixon assigned ElliotAOram and unassigned DanNixon Jul 30, 2015
@DanNixon DanNixon changed the title IDA FuryFit: Merge FuryFit and FuryFitMult IqtFit algorithm Jul 31, 2015
@ElliotAOram
Copy link
Contributor

This is being completed in #15094 and this issue can be closed once the changes are merged

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

No branches or pull requests

4 participants