Ticket #7870 (closed: fixed)
[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: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