Ticket #9548 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Modify MergeRuns to work with algorithm history

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: major Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By: #9369
Blocking: Tester: Michael Wedel

Description

At the moment the MergeRuns algorithm takes a list of strings representing workspace names and merges them into a single workspace. Because the property is a list of strings, history is not recorded for input workspaces and the chain of execution is lost.

We can work around this by making fillHistory on algorithm virtual and implementing a specific version for MergeRuns that copies the history from the input workspaces to the output workspace.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Samuel Jackson

  • Status changed from assigned to inprogress

Refs #9548 Modify MergeRuns to fill history correctly.

Changeset: c95e703208a91f87ede0466eeadbbad7f1aea5df

comment:3 Changed 6 years ago by Samuel Jackson

  • Blocked By 9369 added

comment:4 Changed 6 years ago by Samuel Jackson

Refs #9548 Fix conflicting duplicate method declaration.

Changeset: 232d087130ec826818c00609c6ffb8a0ea27c31b

comment:5 Changed 6 years ago by Samuel Jackson

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

To Test

  • Check the system tests and unit tests are passing
  • Run MergeRuns and check that the history from all input workspaces gets copied to the output workspace.
    • Check that re-running the history produces the same workspace.

comment:6 Changed 6 years ago by Michael Wedel

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

comment:7 Changed 6 years ago by Michael Wedel

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9548_merge_runs_record_history'

Full changeset: 7f07deecad235ab6d8662339570caa1ea1c86a74

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10391

Note: See TracTickets for help on using tickets.