Ticket #5922 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Implement a Voigt function as single function

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

Description

This avoids using a FFT to do the convolution.

Change History

comment:2 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:3 Changed 8 years ago by Martyn Gigg

Implement a Voigt function. Refs #5922

Changeset: 3f21881db7c806721c0d0ba0b35e572f0e9668e7

comment:4 Changed 8 years ago by Martyn Gigg

Use explicit new rather than make_shared to fix test. Refs #5922

Changeset: 5cd83f02ee8499fd32d4f98cd335d3617530f672

comment:5 Changed 8 years ago by Martyn Gigg

Fix for windows to find the correct function. Refs #5922

It seems to have trouble if the value is not explictly a double.

Changeset: 6fc2f5370a381a3d1f4ea2c71b9056b164e77a69

comment:6 Changed 8 years ago by Martyn Gigg

Register Voigt into function factory. Refs #5922

Changeset: d9b1cff143739dad97d35a5b026007dcea90e0c4

comment:7 Changed 8 years ago by Martyn Gigg

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

comment:8 Changed 8 years ago by Nick Draper

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

comment:9 Changed 8 years ago by Nick Draper

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This should be implemented as a peak function rather than a general function. Then it will interact with the GUI better. I suggest using amp = height/2 and half of the HWHM for each lorentz and Gauss functions.

comment:10 Changed 8 years ago by Martyn Gigg

  • Priority changed from critical to blocker

comment:11 Changed 8 years ago by Martyn Gigg

  • Status changed from reopened to accepted

comment:12 Changed 8 years ago by Martyn Gigg

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

comment:13 Changed 8 years ago by Nick Draper

  • Status changed from verify to verifying

comment:14 Changed 8 years ago by Nick Draper

  • Status changed from verifying to closed

Amplitude guess from the interface is not good #6044

Otherwise passed

comment:15 Changed 8 years ago by Martyn Gigg

Implement a Voigt function. Refs #5922

Changeset: 3f21881db7c806721c0d0ba0b35e572f0e9668e7

comment:16 Changed 8 years ago by Martyn Gigg

Use explicit new rather than make_shared to fix test. Refs #5922

Changeset: 5cd83f02ee8499fd32d4f98cd335d3617530f672

comment:17 Changed 8 years ago by Martyn Gigg

Fix for windows to find the correct function. Refs #5922

It seems to have trouble if the value is not explictly a double.

Changeset: 6fc2f5370a381a3d1f4ea2c71b9056b164e77a69

comment:18 Changed 8 years ago by Martyn Gigg

Register Voigt into function factory. Refs #5922

Changeset: d9b1cff143739dad97d35a5b026007dcea90e0c4

comment:19 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6768

Note: See TracTickets for help on using tickets.