Ticket #11133 (closed: fixed)
Fix warning on FuryFit
| Reported by: | Dan Nixon | Owned by: | Dan Nixon |
|---|---|---|---|
| Priority: | major | Milestone: | Release 3.4 |
| Component: | Indirect Inelastic | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Martyn Gigg |
Description (last modified by Dan Nixon) (diff)
Currently the following warning is given by FuryFit (can also be seen on system tests):
CompositeFunction-[Warning] Numeric derivatives should be used when non-constant ties defined
Change History
comment:5 Changed 6 years ago by Dan Nixon
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #253.
comment:6 Changed 6 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:8 Changed 6 years ago by Martyn Gigg
It looks like there is now a merge conflict with master that needs resolving. It is most likely from #227 I guess
comment:9 Changed 6 years ago by Dan Nixon
Merge branch 'master' into 11133_fix_fury_fit_warnings
Conflicts:
Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py
Refs #11133
Changeset: 8be14547a4068b8a99831974fe7d01613d3f59d1
comment:10 Changed 6 years ago by Dan Nixon
Jenkins, retest this please.
comment:11 Changed 6 years ago by Martyn Gigg
This has removed the warnings and not affected the results but the error on OS X is still present.
comment:12 Changed 6 years ago by Martyn Gigg
- Status changed from verifying to closed
Merge pull request #253 from mantidproject/11133_fix_fury_fit_warnings
Fix warning on FuryFit regarding numerical derivatives
Full changeset: 5c266ce84915c003ce1bbefcabf65f4ba1449bf8
comment:13 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 11972
Add NumDeriv=true for composite functions within MultiDOmainFunction
Refs #11133
Changeset: 9f9d3325a75ff296367fcd017a438f37566163a2