Ticket #9857 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

MaskBTP fails when choppers are added to IDF.

Reported by: Ross Whitfield Owned by: Andrei Savici
Priority: minor Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Ross Whitfield

Description

Should use getComponentByName('A row') instead of index. If you add chopper with a lower index, the index of the bank is wrong.

Attachments

CORELLI_Definition.xml (106.0 KB) - added by Ross Whitfield 6 years ago.

Change History

Changed 6 years ago by Ross Whitfield

comment:1 Changed 6 years ago by Andrei Savici

MaskBTP is looking for some components. Refs #9857

Changeset: 6a42efa8e4d79f503db112d009360097e7560b12

comment:2 Changed 6 years ago by Andrei Savici

  • Status changed from new to assigned

To test: On a current mantid release, LoadEmptyInstrument with the IDF attached to this ticket. When running MaskBTP, on this workspace (no instrument should be selected), on bank 2 for example, it should complain about ObjComponent not supporting indexing. With this commit, that issue is fixed. It now allows for the same if you add components to ARCS and SEQUOIA

comment:3 Changed 6 years ago by Andrei Savici

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

comment:4 Changed 6 years ago by Ross Whitfield

  • Status changed from verify to verifying
  • Tester set to Ross Whitfield

comment:5 Changed 6 years ago by Ross Whitfield

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9857_MaskBTP_components'

Full changeset: 7ff65dd1c4a0d564e08836711a6e10e8e9d594e2

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10699

Note: See TracTickets for help on using tickets.