Ticket #8003 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

DetectorDotDat File wrong specification

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Jay Rainey

Description

It seems that the Number_of_user_table_parameters is wrong in Mantid.

The number of columns is always 5 + Number_of_user_table_parameters.

While, we are using Number_of_user_table_parameters as Number of columns.

Attachments

detector.dat (460 bytes) - added by Gesner Passos 7 years ago.
detector_con_phi.dat (556 bytes) - added by Gesner Passos 7 years ago.
detector_complete.dat (1.8 KB) - added by Gesner Passos 7 years ago.

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Gesner Passos

Correct the interpretation of DetectorDotDatFile

The Number of columns in a detector.dat file is 5 plus the number of user table parameters. This commit correct this information.

re #8003

Changeset: 7c2ec2688a3a860cf06344210dc0e154792f2505

Changed 7 years ago by Gesner Passos

Changed 7 years ago by Gesner Passos

Changed 7 years ago by Gesner Passos

comment:3 Changed 7 years ago by Gesner Passos

Tester:

The three dotdat files are valid ones. This script should work for all of them.

ws = Load('MAP17186.raw')
ModifyDetectorDotDatFile(ws,'detector.dat','/tmp/test1.dat')
ModifyDetectorDotDatFile(ws,'detector_con_phi.dat','/tmp/test2.dat')
ModifyDetectorDotDatFile(ws,'detector_complete.dat','/tmp/test3.dat')

comment:4 Changed 7 years ago by Gesner Passos

Correct the incorrect assumption of detectordotdat file

Although present almost always, according to the control group, the phi column is the first of the User optional column. So, we can not suppose it is there.

re #8003

Changeset: 3350f9ad6cffcdd58fb2115e1d97d93a7598fe8e

comment:5 Changed 7 years ago by Gesner Passos

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

comment:6 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying
  • Tester set to Jay Rainey

comment:7 Changed 7 years ago by Gesner Passos

Correct the interpretation of DetectorDotDatFile

The Number of columns in a detector.dat file is 5 plus the number of user table parameters. This commit correct this information.

re #8003

Changeset: 7259e7be498b25cf2582e962af150ebe6b08a854

comment:8 Changed 7 years ago by Gesner Passos

Correct the incorrect assumption of detectordotdat file

Although present almost always, according to the control group, the phi column is the first of the User optional column. So, we can not suppose it is there.

re #8003

Changeset: 515fc40c000a4b62f2f2e528bff1336190932cd0

comment:9 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

It seems that Wenduo Zhou has tested this ticket and passed it (pushed to master) without accepting it for verification. I also tested it with the provided and script and it passes.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8848

Note: See TracTickets for help on using tickets.