Ticket #6044 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Improve peak height -> amplitude guess in Voigt Function

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

The formula matching a peak height to the initial guess for amplitude is poor and could be improved.

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted
  • Milestone changed from Release 2.4 to Release 2.3

comment:2 Changed 8 years ago by Martyn Gigg

From a few tests it looks like amp=1.5*height is a good start.

comment:3 Changed 8 years ago by Martyn Gigg

Improve height guess for Voigt. Refs #6044

Moved to amplitude=1.5*height

Changeset: a11cb51106c06d79602752ec5b3a4a51202a715a

comment:4 Changed 8 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:5 Changed 8 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:6 Changed 8 years ago by Nick Draper

  • Status changed from verifying to closed

Tested with 3 different peaks - looks good

comment:7 Changed 8 years ago by Martyn Gigg

Improve height guess for Voigt. Refs #6044

Moved to amplitude=1.5*height

Changeset: a11cb51106c06d79602752ec5b3a4a51202a715a

comment:8 Changed 8 years ago by Martyn Gigg

Improve height guess for Voigt. Refs #6044

Moved to amplitude=1.5*height

Changeset: a11cb51106c06d79602752ec5b3a4a51202a715a

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6890

Note: See TracTickets for help on using tickets.