From 69f14edd8080cdb0fc84979dcc443d981266520d Mon Sep 17 00:00:00 2001
From: patriceac <48073125+patriceac@users.noreply.github.com>
Date: Mon, 28 Nov 2022 01:19:31 -0800
Subject: [PATCH] Tweak the seed behavior

Update the seed *before* starting the processing, so interrupting the processing retains the seed being used for the batch being currently processed.

The idea behind that is that if I like the gen I'm currently seeing and want to build on top of it, I can create a new task with the same seed without having to wait for the current task to complete.
---
 ui/media/js/main.js | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/ui/media/js/main.js b/ui/media/js/main.js
index 2811a50c..fe4caacb 100644
--- a/ui/media/js/main.js
+++ b/ui/media/js/main.js
@@ -695,6 +695,12 @@ async function checkTasks() {
 
     const genSeeds = Boolean(typeof task.reqBody.seed !== 'number' || (task.reqBody.seed === task.seed && task.numOutputsTotal > 1))
     const startSeed = task.reqBody.seed || task.seed
+    
+    // Update the seed *before* starting the processing so it's retained if user stops the task
+    if (randomSeedField.checked) {
+        seedField.value = task.seed
+    }
+    
     for (let i = 0; i < task.batchCount; i++) {
         let newTask = task
         if (task.batchCount > 1) {
@@ -741,10 +747,6 @@ async function checkTasks() {
         }
     }
 
-    if (randomSeedField.checked) {
-        seedField.value = task.seed
-    }
-
     currentTask = null
 
     if (typeof requestIdleCallback === 'function') {
@@ -1201,7 +1203,7 @@ async function getModels() {
 function checkRandomSeed() {
     if (randomSeedField.checked) {
         seedField.disabled = true
-        seedField.value = "0"
+        //seedField.value = "0" // This causes the seed to be lost if the user changes their mind after toggling the checkbox
     } else {
         seedField.disabled = false
     }