Ticket #3208 (closed: fixed)
ISIS Convert to energy transfer interface should have field to enter psi if save_nxspe option is selected
| Reported by: | Alex Buts | Owned by: | Andrei Savici |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.2 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Michael Reuter |
Description
nxspe file format has field to store psi (the crystal rotation wrt the reference frame). This is important parameter, which is currently used by all software, accepting nxspe. (in ISIS)
save_nxspe algorithm have field to enter psi value, but when a user uses convert_to_energy interface and asks to save data as nxspe, there are no place to enter psi and default value (0) will be stored in resulting nxspe, making the data analysis incorrect, based on resulting nxspe incorrect unless this value is stored in nxspe using an hdf-editor.
The option to provide psi has to be available when user selects "save nxspe" file.
Change History
comment:2 Changed 9 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Martyn Gigg
comment:4 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 30 to Iteration 31
Bulk move of tickets to iteration 31 at the iteration 30 code freeze
comment:5 Changed 9 years ago by Andrei Savici
Changed a keyword for saveNXSPE Refs #3208
Convert to energy transfer interface works for MARI11001.raw. Do not put anything in the SEMotor. Anything you put in offset will be stored as psi in the nxspe.
Changeset: fabd7dc9b224470af899808c8f207282aaa74dc0
comment:6 Changed 9 years ago by Andrei Savici
Changed a keyword for saveNXSPE Refs #3208
Convert to energy transfer interface works for MARI11001.raw. Do not put anything in the SEMotor. Anything you put in offset will be stored as psi in the nxspe.
Changeset: fabd7dc9b224470af899808c8f207282aaa74dc0
comment:7 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:8 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:11 Changed 8 years ago by Andrei Savici
- Status changed from accepted to verify
- Resolution set to fixed
comment:12 Changed 8 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:13 Changed 8 years ago by Michael Reuter
- Status changed from verifying to closed
The value entered into the offset entry box does appear in the psi field in the NXSPE.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4055
"New" tickets moved at the code freeze of iteration 29