Ticket #3618 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SANS: Don't use monitor 4 to cal neutron background level

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: critical Milestone: Release 2.0
Component: Python Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

Contrary to the other SANS2D monitor the new-ish monitor 4 contain counts for all times and no part of this monitor can be used to calculate background neutron count level.

Currently either all or none of the monitors are used for this.

Add user command along the lines of

BACK/M4/OFF

To switch this off for monitor 4.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:2 Changed 9 years ago by Anders Markvardsen

  • Summary changed from ISIS SANS: Don't use monitor 4 to cal neutron background level to SANS: Don't use monitor 4 to cal neutron background level

comment:3 Changed 9 years ago by Anders Markvardsen

  • Priority changed from major to critical

comment:4 Changed 9 years ago by Anders Markvardsen

Implemented ISIS SANS BACK/M4/OFF. Re #3618

Changeset: 6d5631705b375735166596adbf22c7ef91c4de47

comment:5 Changed 9 years ago by Anders Markvardsen

  • Status changed from new to accepted

To test e.g. add BACK/M1/OFF to systemtests\Data\LOQ\MASK.094AA after BACK/MON/TIMES 31000 39000 and run systemtest LOQTransFitWorkspace2D, which should then fail

comment:6 Changed 9 years ago by Anders Markvardsen

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

comment:7 Changed 9 years ago by Anders Markvardsen

Implemented ISIS SANS BACK/M4/OFF. Re #3618

Changeset: 6d5631705b375735166596adbf22c7ef91c4de47

comment:8 Changed 9 years ago by Michael Reuter

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

comment:9 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

Adding the requested line to the specified file does make the test crash.

comment:10 Changed 9 years ago by Michael Reuter

That should be test fail not crash!

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4465

Note: See TracTickets for help on using tickets.