Ticket #9857 (closed: fixed)
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
Change History
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