Ticket #10749 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Add the option for SofQW3 back to S(Q, w) tab

Reported by: Dan Nixon Owned by: Dan Nixon
Priority: blocker Milestone: Release 3.3
Component: Indirect Inelastic Keywords:
Cc: Blocked By:
Blocking: Tester: Lottie Greenwood

Description

There is an issue with SofQW2 at large energy windows that is not an issue with SofQW3.

For now the option to choose between the two should be present.

Change History

comment:1 Changed 6 years ago by Dan Nixon

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Revert "Only have SofQW2 on Sqw tab"

This reverts commit f16cd217ad819a92d085dc18a6773ff4b60837e7.

Refs #10749

Changeset: 8cd667a8527f9b6787944d43293eaed8c008fd1e

comment:3 Changed 6 years ago by Dan Nixon

To test:

  • Open Indirect > Data Reduction > S(Q, w)
  • Load input file: irs26176_graphite002_red.nxs
  • Q binning: 0.6, 0.1, 1.8
  • Energy binning: -0.5, 0.05, 0.5
  • Run with both Rebin Types, check correct algorithm was used in Results Log and/or workspace history

comment:4 Changed 6 years ago by Dan Nixon

  • Summary changed from Add the option for SofQW2 back to S(Q, w) tab to Add the option for SofQW3 back to S(Q, w) tab

comment:5 Changed 6 years ago by Dan Nixon

Sqw should be a choice between 2 and 3

Refs #10749

Changeset: d4de111650333790caacde87e2a2053aae1ef112

comment:6 Changed 6 years ago by Dan Nixon

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

comment:7 Changed 6 years ago by Lottie Greenwood

  • Status changed from verify to verifying
  • Tester set to Lottie Greenwood

comment:8 Changed 6 years ago by Lottie Greenwood

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/10749_add_sofqw3_back_to_sqw_tab'

Full changeset: ​e75b1de3e8e658195eb17939b6c99145cbdcde23

comment:9 Changed 6 years ago by Lottie Greenwood

Did find trivial issue re: graph plotting (#10757) this might be a problem with the way graphs work in Mantid throughout, though.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11591

Note: See TracTickets for help on using tickets.