Ticket #2578 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

HRPD Parameters file

Reported by: Nick Draper Owned by: Anders Markvardsen
Priority: critical Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description

From Aziz (therefore I'm pretty sure this is HRPD):

Playing with the new _parameters.xml file I finally understood why guesses were computed wrong : There is no need for inserting a conversion factor in the “Fullprof formulaes” : these can be literally used as they are. However, since the formulae parametrize a peak profile in TOF, the proper behaviour is obtained by setting all results units in <TOF> as follows:

<component-link name="bank_bsk" >
  <parameter name="BackToBackExponential:S" type="fitting">
    <formula eq="sqrt(7.429*centre^4+132.188*centre^2)" unit="dSpacing" result-unit="TOF" />
  </parameter> 
  <parameter name="BackToBackExponential:A" type="fitting">
    <formula eq=" 0.086851/centre" unit="dSpacing" result-unit="TOF" /> <fixed />
  </parameter>   
  <parameter name="BackToBackExponential:B" type="fitting">
    <formula eq=" 0.005047/(centre^4)+0.023347" unit="dSpacing" result-unit="TOF" /> <fixed />
  </parameter>  
</component-link>

Change History

comment:1 Changed 10 years ago by Martyn Gigg

  • Owner changed from Martyn Gigg to Anders Markvardsen
  • Status changed from new to assigned

Anders said he didn't understand why this change should actually make any difference given the factor that was removed should have been exactly the same as setting the result to dSpacing.

comment:2 Changed 10 years ago by Anders Markvardsen

(In [10463]) Bring BackToBackExponentialTest back into live. re #2578.

comment:3 Changed 10 years ago by Anders Markvardsen

(In [10591]) Updated HRPD parameter file.

Tested that it works for the 1st bank in TOF and dSpacing and for focussed data for all 3 banks in dspacing.

Taken out temporary TOF-dSpacing constants once put in as a temporary measure while code for param units was developed.

re #2578

comment:4 Changed 10 years ago by Anders Markvardsen

(In [10596]) Fix broken test.

re #2578

comment:5 Changed 10 years ago by Anders Markvardsen

  • Status changed from assigned to accepted

comment:6 Changed 9 years ago by Anders Markvardsen

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

comment:7 Changed 9 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:8 Changed 9 years ago by Roman Tolchenov

  • Status changed from verifying to closed

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3425

Note: See TracTickets for help on using tickets.