Ticket #7837 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[C2E] Energy Transfer: Output to AddSampleLog

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: major Milestone: Release 3.0
Component: Indirect Inelastic Keywords:
Cc: spencer.howells@… Blocked By:
Blocking: Tester: Jay Rainey

Description

Add options & parameters to the Sample Log when running an energy transfer with indirect instruments. The following skeleton code for this was outlined by Spencer:

AddSampleLog(Workspace=outWS, LogName='Calib File', LogType='String', LogText=< Calib File option value>)
If < Calib File option value> is true
AddSampleLog(Workspace=outWS, LogName='Calib File name', LogType='String', LogText=< Calib File name>)

AddSampleLog(Workspace=outWS, LogName='Detailed Balance', LogType='String', LogText=<DB option value>)
If <DB option value> is true
AddSampleLog(Workspace=outWS, LogName='Detailed Balance Temperature', LogType='String', LogText=<temperature value>)

AddSampleLog(Workspace=outWS, LogName='Scale', LogType='String', LogText=< Scale option value>)
If < Scale option value> is true
AddSampleLog(Workspace=outWS, LogName='Scale factor', LogType='String', LogText=< Scale factor value>)

Change History

comment:1 Changed 7 years ago by Samuel Jackson

  • Status changed from new to inprogress

Refs #7837 add logs to indirect energy transfer.

Changeset: 22fd4f62c07586380c67899983f47cf753e69636

comment:2 Changed 7 years ago by Samuel Jackson

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

To Tester

Check the following sample logs have been added to the workspaces produced when running convert to energy with an indirect instrument:

  • calib_file - boolean
  • calib_file_name - string - only present if calib_file is true
  • detailed_balance - boolean
  • detailed_balance_temp - number - only present if detailed_balance is true
  • scale - boolean
  • scale_factor - number - only present if scale is true

It should be obvious from the interface what the property values are and you should check that the values match. Further information on running a reduction can be found here: http://www.mantidproject.org/Indirect:ConvertToEnergy

If multiple files are used then the logs should be present in each of the resulting workspaces.

comment:3 Changed 7 years ago by Samuel Jackson

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:4 Changed 7 years ago by Samuel Jackson

  • Milestone changed from Backlog to Release 3.0

comment:5 Changed 7 years ago by Samuel Jackson

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

comment:6 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying
  • Tester set to Jay Rainey

comment:7 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7837_Energy_Transfer_logs'

Full changeset: 45193ba70725dd3ae6f498983042692cc01ae1af

comment:8 Changed 7 years ago by Jay Rainey

The three requested options appear in the "Sample log" table as expected.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8682

Note: See TracTickets for help on using tickets.