Ticket #3501 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Algorithm to save python script from workspace's history

Reported by: Peter Peterson Owned by: Peter Parker
Priority: critical Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

Create an algorithm that takes a workspace and an output file name and writes the python script for generating the workspace. This is already present in the gui by "Show History" and "Generate Script" then "To File".

Attachments

3501.png​ (79.8 KB) - added by Nick Draper 9 years ago.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 Changed 9 years ago by Martyn Gigg

  • Owner changed from Anyone to Peter Parker

comment:3 Changed 9 years ago by Peter Parker

  • Status changed from assigned to accepted

comment:4 Changed 9 years ago by Peter Parker

In [14689]:

Refs #3501 - Added algorithm for generating a Python script from a workspace history.

comment:5 Changed 9 years ago by Peter Parker

In [14690]:

Refs #3501 - Fixed several warnings and an error that appeared on the non-MS compilers that occurred as a result of the previous commit.

comment:6 Changed 9 years ago by Peter Parker

In [14697]:

Refs #3501:

"Copy to clipboard" and "Save to file" (WS history Python script) functionality of UI now utilises the GeneratePythonScript algorithm.

Made alteration to the constructor of AlgorithmHistoryWindow, and changed how it is called in MantidUI accordingly.

Changed heading for script output.

comment:7 Changed 9 years ago by Martyn Gigg

In [14699]:

Fix Linux builds. Refs #3501

comment:8 Changed 9 years ago by Martyn Gigg

In [14700]:

Fix Linux builds. Refs #3501

comment:9 Changed 9 years ago by Peter Parker

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

comment:10 Changed 9 years ago by Nick Draper

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

comment:11 Changed 9 years ago by Nick Draper

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This does not handle branched history records, look at the example image I've attached.

Changed 9 years ago by Nick Draper

comment:12 Changed 9 years ago by Nick Draper

  • Priority changed from minor to critical

No the code is changed it is critical we get this working

comment:13 Changed 9 years ago by Peter Parker

  • Status changed from reopened to accepted

comment:14 Changed 9 years ago by Peter Parker

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

Have checked a previous build and branched history records were not implemented before the changes. Raised enhancement #3766 to cover this functionality.

comment:15 Changed 9 years ago by Nick Draper

  • Status changed from verify to verifying

comment:16 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

Passed with the limitation is does not handle branched records

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4348

Note: See TracTickets for help on using tickets.