Ticket #11133 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

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:1 Changed 6 years ago by Dan Nixon

Add NumDeriv=true for composite functions within MultiDOmainFunction

Refs #11133

Changeset: 9f9d3325a75ff296367fcd017a438f37566163a2

comment:2 Changed 6 years ago by Dan Nixon

  • Status changed from new to assigned

comment:3 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

comment:4 Changed 6 years ago by Dan Nixon

  • Description modified (diff)

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:7 Changed 6 years ago by Dan Nixon

Jenkins, retest this please.

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

Note: See TracTickets for help on using tickets.