Ticket #5542 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Change Phase angle range to (-pi:+pi) in Muon Oscillating Fit Functions

Reported by: Karl Palmen Owned by: Karl Palmen
Priority: major Milestone: Release 2.2
Component: Mantid Keywords: phase angle muon
Cc: anders.markvardsen@… Blocked By:
Blocking: Tester: Anders Markvardsen

Description

The muon oscillating fit functions presently put the phase angle parameter in the range 0 to 2pi. We have found out that the muon scientist prefer the range -pi to +pi.

This ticket is to change this range as described for all muon fitting functions that have a phase angle parameter.

Change History

comment:1 Changed 8 years ago by Karl Palmen

  • Owner set to Karl Palmen
  • Status changed from new to assigned

comment:2 Changed 8 years ago by Karl Palmen

Change needed in

Abragam.cpp

ExpDecayOsc.cpp

GausOsc.cpp

Last edited 8 years ago by Karl Palmen (previous) (diff)

comment:3 Changed 8 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:4 Changed 8 years ago by Karl Palmen

All three unit tests use are phase angle of 45 degrees and so do not need changing.

comment:5 Changed 8 years ago by Karl Palmen

Changed phase angle range as required by re #5542

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: b65765e8aead5b88637fb30c02606701ffe43052

comment:6 Changed 8 years ago by Karl Palmen

Tidy up code re #5542

Changeset: 946522d00f7fbc76e441549749d04e4a01e30f60

comment:7 Changed 8 years ago by Karl Palmen

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

comment:8 Changed 8 years ago by Karl Palmen

Changed phase angle range as required by re #5542

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: b65765e8aead5b88637fb30c02606701ffe43052

comment:9 Changed 8 years ago by Karl Palmen

Tidy up code re #5542

Changeset: 946522d00f7fbc76e441549749d04e4a01e30f60

comment:10 Changed 8 years ago by Karl Palmen

I suggest you execute CreateWorkspace with arguments such as

dataX = [-1,0,1,2,3,4,5,6,7,8,9,10,11]

dataY = [10,7,0,-7,-10,-7,0,7,10,7,0,-7]

dataE = [1,1,1,1,1,1,1,1,1,1,1,1]

Fit the function to it noting the phase angle parameter. Repeat with a constant added to the values of dataX to get both positive and negative phase angles. The phase angle parameter should be greater than minus pi, but no greater than plus pi.

comment:11 Changed 8 years ago by Jean Bilheux

  • Status changed from verify to verifying
  • Tester set to Jean Bilheux

comment:12 Changed 8 years ago by Jean Bilheux

  • Status changed from verifying to verify
  • Tester Jean Bilheux deleted

comment:13 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:14 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Tried fitting parts of a EMU data set to all three above mentioned fitting functions varying the starting values and in all cases phase between -pi and pi

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6389

Note: See TracTickets for help on using tickets.