Ulm-IQO / qudi

(Legacy project) A modular laboratory experiment management suite, predecessor to qudi-core.
GNU General Public License v3.0
165 stars 140 forks source link

[core_pyside2] startup fix #670

Closed kay-jahnke closed 3 years ago

kay-jahnke commented 3 years ago

Description

Actually starting the modules listed under "startup" in the config.

Motivation and Context

The startup parameter was loaded in the config, but the modules were not started.

How Has This Been Tested?

Tested with custom config on Windows 10 21H1 for the following cases:

Types of changes

Checklist:

Neverhorst commented 3 years ago

I mean in principle the code looks good. I'm unable to test it right now with an actual setup. However, I'm a bit concerned about loading a huge number of complex modules before starting the Qt event loop. It's probably irrelevant but one should test this with complex modules to be sure.

kay-jahnke commented 3 years ago

I actually had already though about the starting of the modules before the event loop, but I did not find a better place for it. Is there a hook which we could use that let's us start it after the event loop?