Fixed empty task bug on console app

rainman110 commented 3 months ago

Description

This commit switches back the executor to the coreexecutor. This change is required, since the batch app has no event loop and thus cannot handle running tasks in threads correctly.

Thanks at @real-ct-ac for digging into it in depth.

Fixed #1233

Note: In a second commit, I removed many annoying warnings (or changed the logging level).

How Has This Been Tested?

This bug first revealed on python calculators, which crashed or failed to run.

I tested these changes with

Empty tasks
Tasks with Python calculators
On Windows and Linux

This change seems to restore the old functionality.

Checklist:

[ ] A test for the new functionality was added (if applicable).
[ ] All tests run without failure.
[ ] The changelog has been extended, if this MR contains important changes from the users's point of view.
[ ] The new code complies with the GTlab's style guide.
[ ] New interface methods / functions are exported via EXPORT. Non-interface functions are NOT exported.
[ ] The number of code quality warnings is not increasing.

jensschmeink commented 3 months ago

@rainman110 There still seems to be a strange problem with the GUI tests. They still fail after multiple restarts

rainman110 commented 3 months ago

@rainman110 There still seems to be a strange problem with the GUI tests. They still fail after multiple restarts

Nothing should have changed. I suspect, that the VM is currently slow and that we have a timing problem.

dlr-gtlab / gtlab-core