Merge pull request #303 from HASEL-UZH/feature/295-issues-with-experi…

…ence-sampling Cancel all jobs, reduce interval for testing
HASEL-UZH · Mar 19, 2024 · 077ddf8 · 077ddf8
2 parents 80786bc + 6e041a1
commit 077ddf8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/electron/electron/main/services/trackers/ExperienceSamplingTracker.ts b/src/electron/electron/main/services/trackers/ExperienceSamplingTracker.ts
@@ -59,7 +59,8 @@ export class ExperienceSamplingTracker implements Tracker {
   }
 
   public stop(): void {
-    this.checkIfExperienceSamplingIsDueJob.cancel();
+    this.checkIfExperienceSamplingIsDueJob?.cancel();
+    this.forcedExperienceSamplingJob?.cancel();
     this.isRunning = false;
   }
 

diff --git a/src/electron/shared/study.config.ts b/src/electron/shared/study.config.ts
@@ -34,8 +34,8 @@ const studyConfig: StudyConfiguration = {
         ['not at all productive', 'moderately productive', 'very productive'],
         ['not well', 'moderately well', 'very well']
       ],
-      // 3 hours
-      intervalInMs: 1000 * 60 * 60 * 3,
+      // TODO: Change back to 3 hours
+      intervalInMs: 1000 * 60 * 60 * 1,
       // 10% randomization, so the interval will be between 2.7 and 3.3 hours
       samplingRandomization: 0.1
     }