Ticket #4574 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

S(Q, w) Gives Incorrect Results

Reported by: Peter Parker Owned by: Peter Parker
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: nick.draper@…, campbellsi@… Blocked By:
Blocking: Tester: Stuart Campbell

Description

Franz Demmel has reported that the S(Q, w) CTE interface tab is giving wrong results. E.g. OSI95111, Graphite 002.

Attachments

Output Plot.png (186.2 KB) - added by Peter Parker 9 years ago.

Change History

Changed 9 years ago by Peter Parker

comment:1 Changed 9 years ago by Peter Parker

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Peter Parker

Refs #4574 - Change to from SofQW to SofQW2.

Changeset: 8e4fcaa20390ed2e563e4a2e4fe825c39307e97d

comment:3 Changed 9 years ago by Stuart Campbell

  • Cc nick.draper@… added

Can we not have a option to select which rebinning method to use ?

comment:4 Changed 9 years ago by Peter Parker

  • Cc campbellsi@… added

comment:5 Changed 9 years ago by Peter Parker

Refs #4574 - Added option to choose SofQW or SofQW2.

A combobox is now available for the user to specify whether to use the old or new algorithm.

I also took the opportunity to make a minor improvement to the layout of the tab.

Changeset: 9f51bc9ebd6ece92d1a8b749aed43fbec94db7c5

comment:6 Changed 9 years ago by Peter Parker

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

Franz is now happy with the results. Combo box added for those who want to still use the old version.

comment:7 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:8 Changed 9 years ago by Stuart Campbell

Made workspace list in S(Q,w) tab bigger. refs #4574

Increased the min size of the QGroupBox from 100 to 150 in order to make the widgets for workspace selection readable.

Changeset: 0d06db471e22afea5c9b3d6dcaf37a1dcaf2c757

comment:9 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to closed

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5421

Note: See TracTickets for help on using tickets.