Ticket #6222 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

IPeaksWorkspace hasBeenIntegrated

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: minor Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description (last modified by Owen Arnold) (diff)

Single method for establishing whether a peaks workspace has been integrated.

Support feature for #5167 and #6211 where we need to know whether the peaks workspace to be visualised has been integrated.

Add a bool hasIntegratedPeaks() const method to the IPeaksWorkspace.

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

refs #6222 Add and implement query method.

Changeset: be63fb120a5cb0e200c49801f32143f6fe49b7b1

comment:3 Changed 8 years ago by Owen Arnold

refs #6222 Expose members to python and test.

Changeset: a81ad744c27fb48d4b69829377e323712d6be2b1

comment:4 Changed 8 years ago by Owen Arnold

  • Description modified (diff)

comment:5 Changed 8 years ago by Owen Arnold

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

Tester:

This feature is covered by both c++ and python unit tests. The following python snippet should demonstrate it working. 

ws = Load(Filename='CSP78173.raw')
peaks = CreatePeaksWorkspace(InstrumentWorkspace=ws[0],NumberOfPeaks=1)
print "Without integration", peaks.hasIntegratedPeaks()
peaks.mutableRun().addProperty('PeaksIntegrated', True, False) #Add the critical flag.
print "With integration", peaks.hasIntegratedPeaks()

comment:6 Changed 8 years ago by Owen Arnold

refs #6222 Add and implement query method.

Changeset: be63fb120a5cb0e200c49801f32143f6fe49b7b1

comment:7 Changed 8 years ago by Owen Arnold

refs #6222 Expose members to python and test.

Changeset: a81ad744c27fb48d4b69829377e323712d6be2b1

comment:8 Changed 8 years ago by Owen Arnold

refs #6222. Solve issue with saving/loading.

Save nexus does not support saving of properties with value of type bool. See PropertyNexus.cpp

Changeset: 2e7543151ae9cd2226872ab8f108c08031882273

comment:9 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:10 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

python snippet seems to work ok.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7068

Note: See TracTickets for help on using tickets.