Ticket #4226 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Import PythonAlgorithms into new framework

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

Should be just a case of moving the loading code over now that everything else is in place

Change History

comment:1 Changed 9 years ago by Martyn Gigg

Refs #4226. Start exporting ConfigService before Python algorithms

Changeset: 9b97d3e1f1b39ae7c2f8eef71f287c9e8596dac1

comment:2 Changed 9 years ago by Martyn Gigg

Refs #4226. Add plugin loader for Python framework.

Adds a generic plugin (module) loader for importing additional python modules at start up

Changeset: 1907ead6b64e65fa9a77d1d55a33991977930334

comment:3 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:4 Changed 9 years ago by Martyn Gigg

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

comment:5 Changed 9 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:6 Changed 9 years ago by Peter Peterson

  • Status changed from verifying to reopened
  • Resolution fixed deleted

On RHEL6 in MantidPlot during startup there are a bunch of errors. An example one:

Error: Importing module "FindCurrentLiveNexusFile" failed". Python argument types in
    FrameworkManager._getPropertyOrder(MantidPyFramework, IAlgorithm)
did not match C++ signature:
    _getPropertyOrder(Mantid::PythonAPI::FrameworkManagerProxy {lvalue}, Mantid::API::IAlgorithm const*)

comment:7 Changed 9 years ago by Martyn Gigg

  • Status changed from reopened to accepted

comment:8 Changed 9 years ago by Martyn Gigg

Committed to the wrong ticket and had already pushed when I found out. Here is the message:

Refs #4426. Fix for RHEL6 so that PythonAlgorithms load

This should hopefully fix the later Ubuntu (>=11.04) problems as well but I have no test instance for that.

Changeset: b5e02194abff59dd376bf2a604dc2253af19a60b

comment:9 Changed 9 years ago by Martyn Gigg

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

comment:10 Changed 9 years ago by Martyn Gigg

Refs #4226. Fix windows build.

Changeset: b7587f10296cb317e6fee4826299fe924f4a7a86

comment:11 Changed 9 years ago by Peter Peterson

  • Status changed from verify to verifying

comment:12 Changed 9 years ago by Peter Peterson

  • Status changed from verifying to closed

It works now.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5073

Note: See TracTickets for help on using tickets.