Wake DynamicExperimentThread task runner up when state changes

[simantics/platform.git] / bundles / org.simantics.simulator.toolkit / src / org / simantics / simulator / toolkit / DynamicExperimentThread.java

diff --git a/bundles/org.simantics.simulator.toolkit/src/org/simantics/simulator/toolkit/DynamicExperimentThread.java b/bundles/org.simantics.simulator.toolkit/src/org/simantics/simulator/toolkit/DynamicExperimentThread.java
index 66cec38a46b62664c0a5037af4d24060378730b2..bff9fe751b666455b8c9655ad3b59e93fe807c98 100644 (file)
--- a/bundles/org.simantics.simulator.toolkit/src/org/simantics/simulator/toolkit/DynamicExperimentThread.java
+++ b/bundles/org.simantics.simulator.toolkit/src/org/simantics/simulator/toolkit/DynamicExperimentThread.java
@@ -125,7 +125,7 @@ abstract public class DynamicExperimentThread extends Thread {
                                runTasks();
                                taskTime += System.nanoTime() - asd2;
 
-                               System.err.println(" st = " + 1e-9*stepTime + " tt = " + 1e-9*taskTime);
+                               //System.err.println(" st = " + 1e-9*stepTime + " tt = " + 1e-9*taskTime);
 
                                while(obtainedRealtimeRatio > desiredRealtimeRatio) {
                                        int ran = runTasks();
@@ -138,10 +138,12 @@ abstract public class DynamicExperimentThread extends Thread {
 
                                                if(deltaNs > 0) {
                                                        synchronized(tasks) {
-                                                               try {
-                                                                       tasks.wait(deltaMs, deltaNsRem);
-                                                               } catch (InterruptedException e) {
-                                                                       e.printStackTrace();
+                                                               if (inState(StandardExperimentStates.Running.class)) {
+                                                                       try {
+                                                                               tasks.wait(deltaMs, deltaNsRem);
+                                                                       } catch (InterruptedException e) {
+                                                                               LOGGER.warn("Dynamic experiment thread '" + this.getName() + "' (" + this.getClass().getName() + ") interrupted", e);
+                                                                       }
                                                                }
                                                        }
                                                }
@@ -151,15 +153,14 @@ abstract public class DynamicExperimentThread extends Thread {
 
                        } else {
 
-                               while(!inState(StandardExperimentStates.Running.class) && inActiveState()) {
-
-                                       synchronized(tasks) {
+                               synchronized(tasks) {
+                                       while(!inState(StandardExperimentStates.Running.class) && inActiveState()) {
                                                int ran = runTasks();
                                                if(ran == 0) {
                                                        try {
                                                                tasks.wait(Integer.MAX_VALUE);
                                                        } catch (InterruptedException e) {
-                                                               e.printStackTrace();
+                                                               LOGGER.warn("Dynamic experiment thread '" + this.getName() + "' (" + this.getClass().getName() + ") interrupted", e);
                                                        }
                                                }
                                        }
@@ -189,7 +190,7 @@ abstract public class DynamicExperimentThread extends Thread {
        };
 
        public int runTasks() {
-               ArrayList<Runnable> todo = new ArrayList<>();
+               ArrayList<Runnable> todo = new ArrayList<>(tasks.size());
                synchronized(tasks) {
                        todo.addAll(tasks);
                        tasks.clear();
@@ -272,6 +273,9 @@ abstract public class DynamicExperimentThread extends Thread {
        public void changeState(ExperimentState state) {
                this.state = state;
                fireStateChanged(state);
+               synchronized (tasks) {
+                       tasks.notifyAll();
+               }
        }
 
        public void addListener(DynamicExperimentThreadListener listener) {