Ticket #7870 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[IDA/C2E] Change interfaces to use IDF/IPF values

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

Description

The are a number of parameters defined in the IDF/IPFs for the indirect instruments which could be used as default starting values at various points in the interface. Update the GUI so to use these values, but still fall back on the old default value for backwards compatibility.

Change History

comment:1 Changed 7 years ago by Samuel Jackson

  • Blocked By 4841 added

comment:2 Changed 7 years ago by Samuel Jackson

  • Status changed from new to inprogress

Refs #7870 Elwin now loads sample log from IPF

Changeset: de9e93273f86159faaa4b62e04abfde4d364cb28

comment:3 Changed 7 years ago by Samuel Jackson

Refs #7870 Calibration tab now uses IPF values.

Changeset: 20f917191602a0c2493d8276578f28b1bb78bf8f

comment:4 Changed 7 years ago by Samuel Jackson

Spoke with Spencer this morning and these are the only two areas that currently need data loaded from the IPF files.

To Tester

There are two things to test. First check that Elwin interface is loading the correct value for the log name. This should be initially blank. Selecting a file should retrieve the log name from the plotted workspaces instrument or default to "sample" if unavailable. Most of the instruments have this value set to "sample" anyway, but for testing purposes I would suggest changing the property in the of the IPF files (e.g. IRIS_Parameters.xml) and running a fresh reduction using the C2E interface, then loading this in Elwin.

The second thing to test is plotting a energy in the calibration tab of the C2E interface. The rebinning low and high values should be set to +/-10*resolution. The background start and end should also get set to -9*res and -8*res respectively. Otherwise they should default to the same values as before this ticket.

comment:5 Changed 7 years ago by Samuel Jackson

  • Status changed from inprogress 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/7870_IDA_and_C2E_IDF_values'

Full changeset: bb452d70c023aa9853a6dc350d88b54a52b69b42

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8715

Note: See TracTickets for help on using tickets.