Ticket #11561 (closed: fixed)
Add FABADA option to ConvFit and IqtFit
Reported by: | Dan Nixon | Owned by: | Dan Nixon |
---|---|---|---|
Priority: | major | Milestone: | Release 3.5 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | spencer.howells@… | Blocked By: | #11582, #11651 |
Blocking: | #11241, #11581, #11729 | Tester: | Raquel Alvarez |
Description
Add option to set minimizer to FABADA.
Will need additional options for:
- PDF and chain outputs
- MaxIteration
- ChainLength
Change History
comment:6 Changed 5 years ago by Dan Nixon
Default to a higher MaxIteration limit for FABADA
Refs #11561
Changeset: 6bfb9413035ec1176e44101f6373f7707313b71d
comment:8 Changed 5 years ago by Dan Nixon
Merge branch 'master' into 11561_convfit_furyfit_fabada
Refs #11561
Changeset: 39a33f16c6b4d70841de1e4278da583b92ee0dd2
comment:10 Changed 5 years ago by Dan Nixon
Add ability to use FABADA on seq fit
Refs #11561
Changeset: 7d384329356bfb8299f5baa7561d049c4a726590
comment:11 Changed 5 years ago by Dan Nixon
comment:13 Changed 5 years ago by Dan Nixon
Support FABADA for single fit on IqtFit
Refs #11561
Changeset: b45aa5050d426809db8f1f14c11c56e7fb290ffe
comment:14 Changed 5 years ago by Dan Nixon
comment:15 Changed 5 years ago by Dan Nixon
Merge branch 'master' into 11561_convfit_furyfit_fabada
Refs #11561
Changeset: 3a56fd53ecd2309d6ab1d231a67de060afeef7f8
comment:16 Changed 5 years ago by Dan Nixon
Merge branch '11651_plotpeakbylogvalue_minimizer_string' into 11561_convfit_furyfit_fabada
Refs #11561
Changeset: 2431adfe7470cbab459c00493644b4369c2607e9
comment:17 Changed 5 years ago by Dan Nixon
comment:18 Changed 5 years ago by Dan Nixon
comment:19 Changed 5 years ago by Dan Nixon
comment:23 Changed 5 years ago by Dan Nixon
Merge branch 'master' into 11561_convfit_furyfit_fabada
Refs #11561
Changeset: 24c185596f74dcdf731be83d53810d66260ea841
comment:24 Changed 5 years ago by Dan Nixon
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #768.
comment:25 Changed 5 years ago by Raquel Alvarez
- Status changed from verify to verifying
- Tester set to Raquel Alvarez
comment:26 Changed 5 years ago by Raquel Alvarez
You have to give very good starting parameters, otherwise Mantid crashes because of the minimizer (this has been reported in trac issue #11578(http://trac.mantidproject.org/mantid/ticket/11578)). I have been able to fit the datasets using Two Lorentzians plus a Linear background and using starting values from a previous fit without the FABADA minimizer.
Also, the corresponding entry in the release notes has been added: http://www.mantidproject.org/Release_Notes_3_5_Indirect_Inelastic
comment:27 Changed 5 years ago by Raquel Alvarez
- Status changed from verifying to closed
Merge pull request #768 from mantidproject/11561_convfit_furyfit_fabada
Add FABADA option to I(Q, t) fit and ConvFit
Full changeset: da36d79dc63ef37a010303b988e779c49ce314e4
comment:28 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 12399