Mantid Algorithms inside PythonAlgorithms cause crash

[Michael Whitty]

Running a python algorithm that contains main mantid algorithms causes a crash around ln 258 of algorithm.cpp example offending python algorithm attached

[Michael Whitty]

Offending algortihms work when run from inside a Python script, but not when using the Algorithm dialog from algorithms pane. Calling the algorithm dialog function from a python script works, also.

Problem with how the algorithm pane calls the algorithms?

[Nick Draper]

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

[Nick Draper]

Bulk move of tickets at the end of iteration 27

[Nick Draper]

Bulk move of tickets at the end of iteration 28

[Martyn Gigg]

I think this is because the main original Python algorithm falls of the stack of managed algorithms when the internal ones are run as managed algorithms.

[Martyn Gigg]

(In [11401]) Ensure that when algorithms are run using the simple API as part of a Python algorithm that they are unmanaged. If they are not then they can cause the main algorithm to be deleted before it is actaully finished./unmanaged_alg.py Refs #1459

[Martyn Gigg]

Test case:

from MantidFramework import PythonAlgorithm, mtd, Direction
from mantidsimple import *

class SubAlgTest(PythonAlgorithm):
    
    def PyInit(self):
        self.declareProperty('Count', 1, Direction=Direction.Input)

    def PyExec(self):
        n_algs = self.getProperty('Count')
        print 'Number of algs',n_algs
        
        name = 'test_'
        dataX = [i for i in range(11)]
        dataY = [i for i in range(10)]
        dataE = [i for i in range(10)]

        for i in range(n_algs):
            CreateWorkspace(name + str(i), dataX, dataY, dataE, 1)            


mantid.registerPyAlgorithm(SubAlgTest())

Use a number that is higher than the algorithms.retained property.

[Martyn Gigg]

(In [11404]) Fix the build and tests. Re #1459

[Martyn Gigg]

I hope this is fixed now.

[Peter Peterson]

The tests run successfully.

[Stuart Campbell]

This ticket has been transferred to github issue ​2306