Ticket #5060 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SANS: implement DET/CORR/FRONT/RADIUS x

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

Purpose allow user command of type : DET/CORR/FRONT/RADIUS x

Steve King writes:

I would like to propose that Anders rationalise the syntax on the DET command.

DET/CORR FRONT RADIUS x or DET/CORR FRONT SIDE y are truly horrible!

The proposal is that text parameters on this command become qualifiers (in keeping with COLETTE and other commands in our Mantid scripts); ie, DET/CORR/FRONT/RADIUS x, and so on.

Attachments

mask_temp.txt (1.8 KB) - added by Anders Markvardsen 9 years ago.
isis sanss User file
directM1_9672_6m_08Aug11.DAT (8.0 KB) - added by Anders Markvardsen 9 years ago.
used by user file

Change History

comment:1 Changed 9 years ago by Anders Markvardsen

Richards write:

Well DET/CORR have been “different” to the rest all along (as below), so if we allow the proposed DET/CORR/FRONT/RADIUS x, the old versions will have to keep on working too, as we are not going to go back and edit all the existing mask files.

So allow both formats

comment:2 Changed 9 years ago by Anders Markvardsen

SANS command: allow DET/CORR/FRONT/RADIUS x. re #5060

Changeset: d591459e40339d759b069881011fc4548df3a2a8

Changed 9 years ago by Anders Markvardsen

isis sanss User file

Changed 9 years ago by Anders Markvardsen

used by user file

comment:3 Changed 9 years ago by Anders Markvardsen

  • Status changed from new to accepted

To test open attached mask file in isis sans gui. If no error messages it works

comment:4 Changed 9 years ago by Anders Markvardsen

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:5 Changed 8 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:6 Changed 8 years ago by Karl Palmen

  • Status changed from verifying to closed

Command added to userfile used by a system test.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5906

Note: See TracTickets for help on using tickets.