Ticket #5445 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

CalculateEfficiency uses from pixel identifier

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

CalculateEfficiency should be using WorkspaceIndexList rather than SpectraList when using MaskDetectors.

Change History

comment:1 Changed 8 years ago by Mathieu Doucet

  • Status changed from new to accepted

Note that this does not affect HFIR SANS because the workspace index and the spectrum index are the same for those geometries.

comment:2 Changed 8 years ago by Mathieu Doucet

Re #5445 Fix problem with pixel id

Changeset: 48692bbdeead2933decda88738dc7acd0229da57

comment:3 Changed 8 years ago by Mathieu Doucet

Re #5445 Revert back ref data change, which didn't need updating

Changeset: cc3742c270e1fc52ffd988890d0186335893f0cf

comment:4 Changed 8 years ago by Mathieu Doucet

Re #5445 Fix sensivity system test

Changeset: 51a7449a81c95a14837e787142e9cb145349ff39

comment:5 Changed 8 years ago by Mathieu Doucet

Re #5445 Separating pre-processed sensitivity test

Changeset: b5c47fd09087bc18de2a272b2aa11987ea160574

comment:6 Changed 8 years ago by Mathieu Doucet

Re #5445 Separating pre-processed sensitivity test

Changeset: 88b3856f640888391d6c1b2d959921a334198bfc

comment:7 Changed 8 years ago by Mathieu Doucet

Re #5445 Fix problem with pixel id

Changeset: 48692bbdeead2933decda88738dc7acd0229da57

comment:8 Changed 8 years ago by Mathieu Doucet

Re #5445 Fix problem with pixel id

Changeset: 48692bbdeead2933decda88738dc7acd0229da57

comment:9 Changed 8 years ago by Mathieu Doucet

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

To test: check that the affected line of code has been changed and that the systems tests still run.

comment:10 Changed 8 years ago by Mathieu Doucet

Re #5445 Fix problem with pixel id

Changeset: 48692bbdeead2933decda88738dc7acd0229da57

comment:11 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:12 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

The appropriate lines of code have been changed to handle the issue and the system tests run correctly.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6291

Note: See TracTickets for help on using tickets.