Ticket #8049 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

SANS2D: Bug in SANSadd2.py

Reported by: Martyn Gigg Owned by: Gesner Passos
Priority: major Milestone: Release 3.0
Component: SANS Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description (last modified by Martyn Gigg) (diff)

From Sarah

Whilst adding quite a few files together (13 in total) using the GUI, Mantid gave the error message attached. If we add the files a couple at a time with the GUI then the error message does not appear and also using Richard's adding script allowed us to add all 13 files together in one go without any error (and the added files all looked as expected).

Attachments

Mantid_add_error.png (279.0 KB) - added by Martyn Gigg 7 years ago.

Change History

Changed 7 years ago by Martyn Gigg

comment:1 Changed 7 years ago by Martyn Gigg

  • Description modified (diff)

comment:2 Changed 7 years ago by Martyn Gigg

I believe the error is on line 89 of SANSadd2.py where

dataY(i)

should be

dataY(0)

comment:3 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

Correct SANSadd2 script

Correct the way it gets the binning from the monitors.

re #8049

Changeset: 2c554c00f4c64695c668c89bf2b97f0702310bd6

comment:4 Changed 7 years ago by Gesner Passos

To make easier, put this line in the SANS2d adding interface:

20801, 20802, 20803, 20804, 20805, 20806, 20807, 20808, 20809, 20810, 20811, 20812, 20813

comment:5 Changed 7 years ago by Gesner Passos

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

Tester:

open Sans isis, go to add runs. And paste the string above. Click on sum. It should sum them up and create an output correctly.

comment:6 Changed 7 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:7 Changed 7 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8049_sans2d_add_many_files'

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8894

Note: See TracTickets for help on using tickets.