Ticket #5602 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Add monitor count to the single crystal peak object.

Reported by: Dennis Mikkelson Owned by: Dennis Mikkelson
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: petersonpf@…, mikkelsonr@… Blocked By:
Blocking: Tester: Vickie Lynch

Description

The integrated monitor count is needed if ANVRED is to be used to process an ISAW peaks file written by Mantid. This will require adding the monitor count to the Peak object and adding methods to set/get it from python and when reading and writing ISAW peaks files.

Change History

comment:1 Changed 8 years ago by Dennis Mikkelson

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Dennis Mikkelson

RE 5602 Added monitor count to Peaks object

Added monitor count to IPeak.h, IPeak.cpp, Peak.h, Peak.cpp. LoadIsawPeaks and SaveIsawPeaks also now handle the monitor count. refs #5602

Changeset: f30c5a46aa33b670ab9f71249daa2ae343584fce

comment:3 Changed 8 years ago by Dennis Mikkelson

RE 5602 Added get/setMonitorCount to "old" python API

refs #5602

Changeset: d838712d4b2dd4e905b50b9bf791ab7390a25b45

comment:4 Changed 8 years ago by Dennis Mikkelson

RE 5602 Added monitor count to Peaks object

Added monitor count to IPeak.h, IPeak.cpp, Peak.h, Peak.cpp. LoadIsawPeaks and SaveIsawPeaks also now handle the monitor count. refs #5602

Changeset: f30c5a46aa33b670ab9f71249daa2ae343584fce

comment:5 Changed 8 years ago by Dennis Mikkelson

RE 5602 Added get/setMonitorCount to "old" python API

refs #5602

Changeset: d838712d4b2dd4e905b50b9bf791ab7390a25b45

comment:6 Changed 8 years ago by Dennis Mikkelson

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

comment:7 Changed 8 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:8 Changed 8 years ago by Vickie Lynch

  • Status changed from verifying to closed

setMonitorCount and getMonitor count work from python. If I loadIsawPeaks, the MonitorCount is correct from the file. If I create a new peaks workspace using FindPeaksMD, the MonitorCount is 0. Should this be a new ticket?

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6448

Note: See TracTickets for help on using tickets.