Ticket #4110 (closed: fixed)
Convert To Energy GUI cannot find correct instrument definition file
| Reported by: | Stuart Campbell | Owned by: | Stuart Campbell |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.0 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Andrei Savici |
Description
When you select BASIS on the convert to energy GUI it reports an error saying that it cannot find the IDF file : BASIS_silicon_111_Definition.xml.
The definition file should not include the reflection, and for IRIS, OSIRIS it does not.
Change History
comment:2 Changed 9 years ago by Stuart Campbell
Added additional method to get IDF. refs #4110
Allow user to get instrument definition file without specifying a date, in this case the current date will be used.
Changeset: eceeaea8f8c00aac8aeda8c0f4538fa99e2c96bf
comment:3 Changed 9 years ago by Stuart Campbell
Added additional method to get IDF. refs #4110
Allow user to get instrument definition file without specifying a date, in this case the current date will be used.
Changeset: eceeaea8f8c00aac8aeda8c0f4538fa99e2c96bf
comment:4 Changed 9 years ago by Stuart Campbell
refs #4110. Changed to use central routine to determine IDF name.
Remove (well commented out) the local routine for determining the IDF name. Now use the central version.
Changeset: 9bba16f4771106d4de99f2311d464b5e0250bafc
comment:5 Changed 9 years ago by Stuart Campbell
- Status changed from accepted to verify
- Resolution set to fixed
The problem was due to the routine looking for all $INSTRUMENT_*Parameter.xml files and then taking the last one, and replace "Parameter" with "Definition". This works as long as the reflection comes before "p" for "Parameter" in the alphabet - not so well for "Silicon"!
comment:6 Changed 9 years ago by Stuart Campbell
To Test:
Just select BASIS on the Convert to Energy GUI and it should work. If you turn on debug messages, then you can see it is loading the correct file.
comment:7 Changed 9 years ago by Andrei Savici
- Status changed from verify to verifying
- Tester set to Andrei Savici
