Ticket #4513 (closed: invalid)

Opened 9 years ago

Last modified 5 years ago

"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

RAW.FOR;5 (14.7 KB) - added by Peter Parker 9 years ago.
Forward Scattering Angles Routine
RAWB.FOR;8 (13.6 KB) - added by Peter Parker 9 years ago.
Backward Scattering Angles Routine
TFITD.FOR;74 (43.8 KB) - added by Peter Parker 9 years ago.
Tfit Routine

Change History

Changed 9 years ago by Peter Parker

Forward Scattering Angles Routine

Changed 9 years ago by Peter Parker

Backward Scattering Angles Routine

Changed 9 years ago by Peter Parker

Tfit Routine

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:

  1. 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.
  1. Check that periods used for monitor calculations can be different from the periods used for corresponding spectra.
  1. Is dividing through by a "Time" ws not equivalent to a Rebin?
  1. 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.
  1. 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

Last edited 9 years ago by Peter Parker (previous) (diff)

comment:2 Changed 9 years ago by Peter Parker

Discussion with Jerry.

  1. 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.
  1. Didn't bring up. Happy to just reproduce current functionality at this point.
  1. N/A. Still need to find this out.
  1. 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.
  1. 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:4 Changed 8 years ago by Peter Parker

  • Priority changed from major to critical

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

comment:9 Changed 8 years ago by Peter Parker

Refs #4513 - Further amendments to LoadEVSRaw

comment:10 Changed 8 years ago by Peter Parker

Refs #4513 - Further amendments to LoadEVSRaw

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:14 Changed 8 years ago by Martyn Gigg

  • Status changed from assigned to accepted

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:17 Changed 8 years ago by Nick Draper

  • Status changed from verifying to closed

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5360

Note: See TracTickets for help on using tickets.