Ticket #7427 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Update Indirect Bayes routines to output GroupWorkspace with all spectra

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.6
Component: Indirect Inelastic Keywords:
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description (last modified by Martyn Gigg) (diff)

From Spencer:

Bayes Changes

QLRun

  1. Results WS now has name *_Result and is a GroupWorkspace with a WS for each spectrum which has histograms

for Data, Calc.1 & Diff.1 (for 1 peak), Calc.2 & Diff.2 (for 2 peaks). ie format like Fury/Conv fit.

  1. routine QLPlotQL & QLPLotQSe modified to take new Result WS format

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Owner set to Martyn Gigg
  • Status changed from new to accepted

comment:2 Changed 7 years ago by Martyn Gigg

Update IndirectBayes to use a WorkspaceGroup to hold all spectra.

Refs #7427

Changeset: 61c141d46479de9920cef46c37e5440b5b8dbffc

comment:3 Changed 7 years ago by Martyn Gigg

Allow for single sample input in QLrun. Refs #7427

If there is only a single sample input then you just get back a single workspace rather than a group.

Changeset: b0a70d0afa6d9b0b57a85f93d014d57f2961c359

comment:4 Changed 7 years ago by Martyn Gigg

Update Bayes system tests following workspace structure changes.

Refs #7427

Changeset: ebc346ba9e2cd6b6b5dd9f7054e28f788ae69941

comment:5 Changed 7 years ago by Martyn Gigg

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

Branch: feature/7427_update_indirect_bayes in BOTH code & systemtest repos. Both need merging during testing.

Tester: This can only be tested on Windows at the moment. The ISISIndirectBayes system tests should be running and passing. These scripts give you an idea of how to run them. It only affects the ones whose name starts with Q. For a single workspace input you should get a single workspace back but for many sample run inputs (use a list of loaded workspaces) you should get a WorkspaceGroup.

comment:6 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:7 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to verify
  • Tester Gesner Passos deleted

sorry, I will not have time to build it in windows. I read that information just now.

comment:8 Changed 7 years ago by Martyn Gigg

  • Description modified (diff)

testing trac

comment:9 Changed 7 years ago by Roman Tolchenov

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

comment:10 Changed 7 years ago by Roman Tolchenov

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/feature/7427_update_indirect_bayes'

comment:11 Changed 7 years ago by Roman Tolchenov

Merge remote-tracking branch 'origin/feature/7427_update_indirect_bayes'

comment:12 Changed 7 years ago by Nick Draper

  • Component changed from Framework to Indirect Inelastic

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8273

Note: See TracTickets for help on using tickets.