Ticket #6812 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

GeneratePythonScript is broken for Load algorithm

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: blocker Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

After fixing #5885 it appears that the unit tests didn't pick up that GeneratePythonScript doesn't work with properties that don't exist on the standard algorithm call. This is now broken in master and needs fixing!

Change History

comment:1 Changed 8 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:2 Changed 8 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:3 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:4 Changed 8 years ago by Martyn Gigg

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

Branch: bugfix/6812_fix_generatepythonscript

Tester: Certain algorithms, Load & Fit, have some properties that get dynamically. Check that setting these extra properties, i.e Precount for Load with an event nexus file or SpectrumMax for a raw file allows a python script to be generated.

comment:5 Changed 8 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:6 Changed 8 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:7 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:8 Changed 8 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:9 Changed 8 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:10 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

The system test is passing now (https://builds.sns.gov/job/ornl_test_rhel6_develop/) and script generation worked for both a raw & event nexus file.

comment:11 Changed 8 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:12 Changed 8 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:13 Changed 7 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:14 Changed 7 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:15 Changed 7 years ago by Martyn Gigg

Fix GeneratePythonScript for Load algorithm Refs #6812

It does not currently handle properties that don't exist when the algorithm is initialized very well.

Changeset: 73d6288bc527c9f344de92a83a93153d317a4c52

comment:16 Changed 7 years ago by Martyn Gigg

Put back file removal. Refs #6812

Changeset: 673de16aad90c2eba0c0b6f64519f883690dfb8a

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7658

Note: See TracTickets for help on using tickets.