Ticket #2791 (closed: invalid)
SANS: Set the moderator to sample distance to that in the nexus/raw file
| Reported by: | Steve Williams | Owned by: | Gesner Passos |
|---|---|---|---|
| Priority: | major | Milestone: | Release 3.0 |
| Component: | SANS | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Anders Markvardsen |
Description (last modified by Anders Markvardsen) (diff)
Could we use the moderator to sample distance stored in the nxs file, as an alternative to the one in the idf ? This would bring Mantid in line with the historic practise in COLETTE. Would need to have a command in the mask file to activate this, else we would have some horrible backward compatibility issues. e.g. SAMPLEDIST/NXS or /RAW and SAMPLEDIST/IDF [default] SAMPLE/OFFSET would apply in all cases, though the value for this should hopefully be zero for /NXS if the instrument scientist has fed the correct value into the nxs file via SECI.
This distance should be reported in the Geometry tab.
The sample to detector distance shouldn't be changed by this setting!
Change History
comment:2 Changed 9 years ago by Anders Markvardsen
- Owner changed from Steve Williams to Anders Markvardsen
- Status changed from new to assigned
comment:3 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:4 Changed 9 years ago by Anders Markvardsen
- Summary changed from Set the moderator to sample distance to that in the nexus/raw file to SANS: Set the moderator to sample distance to that in the nexus/raw file
comment:5 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:6 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:7 Changed 8 years ago by Anders Markvardsen
Steve writes:
Ok, in the NeXus file, the moderator-sample distance is stored in
\raw_data_1\instrument\moderator\distance
And appears as a negative figure (the run I’m looking at, SANS2D 11585, it’s -19.334 m).
I assume this needs to have any SAMPLE/OFFSET value in the User file applied to it...
For comparison, the SANS2D IDF has a value of 19.281 m.
comment:8 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.2 to Release 2.3
Moved at the end of release 2.2
comment:9 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:10 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:11 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5 to Release 2.6
comment:14 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:15 Changed 7 years ago by Anders Markvardsen
- Owner changed from Anders Markvardsen to Gesner Passos
- Component changed from Framework to SANS
- Description modified (diff)
comment:16 Changed 7 years ago by Gesner Passos
- Status changed from new to inprogress
Last talk to Stephen, it was decided that the correct approach will be:
- get the information from IDF
- override it using the SAMPLE/OFFSET
do not care about the sample distance in the nexus file.
Check that this is what it is doing. If it is correct, put this ticket as invalid.
comment:17 Changed 7 years ago by Nick Draper
- Milestone changed from Backlog to Release 3.0
moved to Release 3.0 as these all seem to be active
comment:18 Changed 7 years ago by Gesner Passos
- Status changed from inprogress to verify
- Resolution set to invalid
This is the current behaviour. The SAMPLE/OFFSET changes the SAMPLE_Z_CORR. It can be overrided by the ISISGUI, through the Geometry Tab, Z-offset. But this is allowed, as by default, the ISIS GUI is allowed to override any option in the MaskFile.
comment:19 Changed 7 years ago by Anders Markvardsen
- Status changed from verify to verifying
- Tester set to Anders Markvardsen
comment:20 Changed 7 years ago by Anders Markvardsen
- Status changed from verifying to closed
As stated this is the current behaviour
comment:21 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3638
"New" tickets moved at the code freeze of iteration 29