Ticket #9839 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

History needs to clear properties before filling

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: critical Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

In the fillAlgorithmHistory method in AlgorithmHistory we set the add all of the property histories from an algorithm to the object. The container should be cleared first so we don't end up with properties from another algorithm.

Attachments

test_history.py (703 bytes) - added by Samuel Jackson 6 years ago.

Change History

comment:1 Changed 6 years ago by Samuel Jackson

Refs #9839 Clear the list of properties before setting them.

Changeset: e627969ce11c679b843d667458e75dc2d2c9db88

comment:2 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:3 Changed 6 years ago by Samuel Jackson

To Tester

This is a small change that needs to be treated with caution. Follow all the instructions below to check the common and edge cases for history generation and check they all work.

First check that you can reproduce the original issue:

  • Open Indirect > Convert to Energy.
  • Choose IRS21360.raw from the auto test data as the run to use.
  • Click run energy transfer
  • Open the history
  • Generate a script from the history
  • Run the script.
  • It should fail on SaveNexusProcessed because its arguments are duplicated.

Now apply the changes and repeat the above. The script should now work. Workspace generated by the script should match the original workspace. Some other things should also be checked. I've provided a script to help with this. The special case algorithms we need to check are:

  • MergeRuns
  • SaveNexus
  • SaveNexusProcessed

Run the script and check that you can reproduce the same workspace from the history. The file used is in the system tests data.

Last edited 6 years ago by Samuel Jackson (previous) (diff)

Changed 6 years ago by Samuel Jackson

comment:4 Changed 6 years ago by Samuel Jackson

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

comment:5 Changed 6 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:6 Changed 6 years ago by Nick Draper

  • Status changed from verifying to verify
  • Tester Nick Draper deleted

Other things came up

comment:7 Changed 6 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:8 Changed 6 years ago by Nick Draper

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/9839_history_clear_props'

Full changeset: d395af54269a048f449e98b747511fb569c8ca83

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10681

Note: See TracTickets for help on using tickets.