Ticket #1621 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Run: implement a += operator function for appending two runs (for MergeRuns and Plus)

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: pf9@… Blocked By:
Blocking: Tester: Vickie Lynch

Description


Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Russell Taylor

See WorkspaceOpOverloads.cpp line 186 for the existing implementation of operator+= for adding two workspaces using the Plus algorithm.

comment:3 Changed 10 years ago by Janik Zikovsky

(In [6152]) Refs #1621: Implemented a += operator for: Run; PropertyManager; Property; PropertyWithValue; TimeSeriesProperty

comment:4 Changed 10 years ago by Janik Zikovsky

(In [6160]) Refs #1621: Runs are appended together in MergeRuns. Fixed a bug in SampleLogs if dates aren't all correct. Also removed an unused, slow, duplicate call to property->value().

comment:5 Changed 10 years ago by Janik Zikovsky

(In [6166]) Refs #1541: gd_prtn_chrge is showing up for me for event nexus files. Also refs #1621: fixed missing value for some sample logs.

comment:6 Changed 10 years ago by Janik Zikovsky

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

(In [6170]) Fixes #1621: += for Run and Plus

comment:7 Changed 10 years ago by Vickie Lynch

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

comment:8 Changed 10 years ago by Vickie Lynch

  • Status changed from verifying to closed

Plus and MergeRuns work for SNAP event data.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2468

Note: See TracTickets for help on using tickets.