Ticket #3883 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Algorithms are each initialized 20 times during MantidPlot startup.

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Parker

Description

This seems to be pretty slow. Examine if it can be avoided.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Priority changed from major to minor
  • Status changed from new to accepted

comment:2 Changed 9 years ago by Russell Taylor

These all come out of the current PythonAPI. A brand new python interface is being developed, so hopefully this will just go away when we move completely over to that.

comment:3 Changed 9 years ago by Janik Zikovsky

In [14992]:

Refs #3883: Changes to python startup: algorithms' initialize method is now called 8 times (each) instead of 20 times each.

comment:4 Changed 9 years ago by Janik Zikovsky

In [14993]:

Refs #3883: Now down to 2 calls to init() per algorithm.

comment:5 Changed 9 years ago by Janik Zikovsky

In [14994]:

Refs #3883: This seems to work better for python algos

comment:6 Changed 9 years ago by Janik Zikovsky

In [14995]:

Refs #3883: OK, putting it back. Should be back at 8 inits per algo.

comment:7 Changed 9 years ago by Janik Zikovsky

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

comment:8 Changed 9 years ago by Peter Parker

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

comment:9 Changed 9 years ago by Peter Parker

I only seem to see init called 6 times per algo (not 8).

Janik - is this a problem or is it simply a case of fewer the better?

Last edited 9 years ago by Peter Parker (previous) (diff)

comment:10 Changed 9 years ago by Janik Zikovsky

The fewer the better, I would say. The Python API has changed since this ticket so it is not surprising the number of initializations is different.

comment:11 Changed 9 years ago by Peter Parker

Roger that. All looks good to me then.

comment:12 Changed 9 years ago by Peter Parker

  • Status changed from verifying to closed

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4730

Note: See TracTickets for help on using tickets.