Ticket #4513 (closed: invalid)
"Vesuvio" Instrument Support
Reported by: | Peter Parker | Owned by: | Martyn Gigg |
---|---|---|---|
Priority: | critical | Milestone: | Release 2.4 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Nick Draper |
Description
- Decipher "raw", "rawb" and "tfit" programs. (Fortran code files attached.)
- Discuss further with Jerry Mayers.
- Make decision on best way to implement within Mantid. (Via UI, scripts, etc.)
- Make changes.
Attachments
Change History
comment:1 Changed 9 years ago by Peter Parker
Refs #4513 - Initial Raw and RawB translation.
Added a python algorithm which is the first attempt at translating the Raw and RawB Fortran code. Note:
- Backward foil periods are currently assumed to always be the same for each run. Check that this is the case. If not, check that they are always the same for detectors in the same module.
- Check that periods used for monitor calculations can be different from the periods used for corresponding spectra.
- Is dividing through by a "Time" ws not equivalent to a Rebin?
- Backward results do not currently match RawB output. Check that RAWB.FOR;8 is actually the same code that has been compiled to create the RawB program currently on the VMS systems.
- Not sure if runs with less / more than 6 periods should be supported. If so, what are the period schemes in those cases?
Will now pass the script and pose questions to Jerry. Will also make a start dechipering TFitD.
Changeset: 1b91d6be07d253586701cce3b89d45f079c3f1dc
comment:2 Changed 9 years ago by Peter Parker
Discussion with Jerry.
- This is a correct assumption to make. More clearly: detectors that are part of the same module share the same foil out, thin foil and thick foil periods. There are three modules, and each are in a different foil "state" in any given period.
- Didn't bring up. Happy to just reproduce current functionality at this point.
- N/A. Still need to find this out.
- RAWB.FOR;8 is definately the file used. The problem likely has something to do with the fact that, at some point in the converted Python script, the labelling of foil periods seems to be mixed up so that foil out should be thick foil.
- Jerry has said that runs with less / more than 6 periods are not going to be supported any time soon.
comment:3 Changed 8 years ago by Peter Parker
Current script does not support only a single run file. (SumWsList throws error.)
comment:5 Changed 8 years ago by Peter Parker
Refs #4513 - Minor bug fix of current Vesuvio code.
Changeset: 65ba1ffa7fff8bbe56f4796f00c88f225607f3d7
comment:6 Changed 8 years ago by Peter Parker
Refs #4513 - Minor bug fix of current Vesuvio code.
Changeset: 65ba1ffa7fff8bbe56f4796f00c88f225607f3d7
comment:7 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:8 Changed 8 years ago by Peter Parker
Refs #4513 - Further amendments to LoadEVSRaw
- Remove Beta property, replaced with a value of 0.28 in code.
- Swap around defs of thick and out periods. Need to look into this.
- Comments. Changeset: 359bd1c139ef193fc2f6b56874dbd270979911ec
comment:9 Changed 8 years ago by Peter Parker
Refs #4513 - Further amendments to LoadEVSRaw
- Remove Beta property, replaced with a value of 0.28 in code.
- Swap around defs of thick and out periods. Need to look into this.
- Comments. Changeset: 359bd1c139ef193fc2f6b56874dbd270979911ec
comment:10 Changed 8 years ago by Peter Parker
Refs #4513 - Further amendments to LoadEVSRaw
- Remove Beta property, replaced with a value of 0.28 in code.
- Swap around defs of thick and out periods. Need to look into this.
- Comments. Changeset: 359bd1c139ef193fc2f6b56874dbd270979911ec
comment:11 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:12 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:13 Changed 8 years ago by Nick Draper
- Status changed from new to assigned
- Owner changed from Peter Parker to Martyn Gigg
comment:15 Changed 8 years ago by Martyn Gigg
- Status changed from accepted to verify
- Resolution set to invalid
Closing to break this down in to smaller tickets. First is #6316
comment:16 Changed 8 years ago by Nick Draper
- Status changed from verify to verifying
- Tester set to Nick Draper
Agreed
comment:18 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5360
Forward Scattering Angles Routine